1. Java / Говнокод #7512

    +84

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30
    31. 31
    32. 32
    33. 33
    34. 34
    35. 35
    36. 36
    37. 37
    38. 38
    39. 39
    40. 40
    41. 41
    42. 42
    43. 43
    44. 44
    45. 45
    46. 46
    47. 47
    48. 48
    49. 49
    50. 50
    51. 51
    52. 52
    53. 53
    54. 54
    55. 55
    56. 56
    57. 57
    58. 58
    59. 59
    60. 60
    61. 61
    62. 62
    63. 63
    64. 64
    65. 65
    66. 66
    67. 67
    68. 68
    69. 69
    70. 70
    71. 71
    72. 72
    73. 73
    74. 74
    75. 75
    76. 76
    77. 77
    search = 0;
try {
	search = search + wdContext.currentSearchElement().getLifnr().length();
} catch (Exception ex) {
//	ex.printStackTrace();
}
try {
	search = search + wdContext.currentSearchElement().getBahns().length();
} catch (Exception ex) {
//	ex.printStackTrace();
}
try {
	search = search + wdContext.currentSearchElement().getINN().length();
} catch (Exception ex) {
//	ex.printStackTrace();
}
try {
	search = search + wdContext.currentSearchElement().getKPP().length();
} catch (Exception ex) {
//	ex.printStackTrace();
}
try {
	search = search + wdContext.currentSearchElement().getKtokk().length();
} catch (Exception ex) {
//	ex.printStackTrace();
}
try {
	search = search + wdContext.currentSearchElement().getLand1().length();
} catch (Exception ex) {
//	ex.printStackTrace();
}
try {
	search = search + wdContext.currentSearchElement().getBanks().length();
} catch (Exception e) {
	// TODO: handle exception
}
try {
	search = search + wdContext.currentSearchElement().getBankl().length();
} catch (Exception ex) {
	// TODO: handle exception
}
try {
	search = search + wdContext.currentSearchElement().getBankn().length();
} catch (Exception ex) {
//	TODO: handle exception
}
try {
	search = search + wdContext.currentSearchElement().getLocco().length();
} catch (Exception ex) {
//	ex.printStackTrace();
}
try {
	//	search = search + wdContext.currentSearchElement().getLoevm().length();
	search = search + wdContext.currentSearchElement().getName().length();
} catch (Exception ex) {
//	ex.printStackTrace();
}
try {
	search = search + wdContext.currentSearchElement().getOKPO().length();
} catch (Exception ex) {
//	ex.printStackTrace();
}
try {
	search = search + wdContext.currentSearchElement().getOrt01().length();
} catch (Exception ex) {
//	ex.printStackTrace();
}
try {
	search = search + wdContext.currentSearchElement().getSortl().length();
} catch (Exception ex) {
//	ex.printStackTrace();
}
if (search == 0)
	wdComponentAPI.getMessageManager().reportException("Введите критерии поиска", true);
else {
	//Далее идет поиск
}

    Вот такой вот была забабахана проверка входных параметров перед вызовом функции поиска (мною добавлен только последний комментарий, все остальное оригинальное)

    Запостил: foGa, 11 Августа 2011

    Комментарии (13) RSS

    • ava foGa 11.08.2011 09:40 # 0

      Забыл добавить, переменная search не какая-то там локальная переменная внутри метода, а параметр класса (естественно безо всяких методов set/get).
      Ответить

      • ava tir 11.08.2011 10:05 # 0

        И как ты это отрефакторил?
        Ответить

        • ava foGa 11.08.2011 10:18 # 0

          Я вообще эти проверки убрал, т.к. они уже есть в вызываемом модуле, да и сам экран поиска пришлось значительно расширить (критерии поиска задаются теперь не по одному, а по диапазону или списку значений). Единственное, что проверяю, так это wdContext.currentSearchElement() не был null
          Ответить

          • ava tir 11.08.2011 10:20 # −2

            понятно =) вообще надо ввести правило: запостил ГК - покажи как надо делать правильно! думаю станет еще веселее =)
            Ответить

    • ava Lure Of Chaos 11.08.2011 10:23 # 0

      новое слово в копипасте!
      Ответить

      • ava foGa 11.08.2011 10:26 # 0

        какой-то копипаст некопипастный, где то
        // TODO: handle exception
        а где-то
        // ex.printStackTrace();
        Ответить

    • ava absolut 11.08.2011 10:30 # 0

      интересно, а какой сакральный смысл в складывании длин в этом бухгалтерском убожестве?
      Ответить

      • ava roman-kashitsyn 11.08.2011 10:39 # 0

        Убедиться, что хотя бы в одно поле что-то ввели. Видимо, вводить пробелы не запрещается, хотя ХЗ, что там в геттерах (может, trim()).
        Ответить

        • ava foGa 11.08.2011 10:45 # 0

          никаких trim, геттеры и сеттеры автоматом сгенерированы и тупо возвращают/присваивают значение.
          Ответить

        • ava absolut 11.08.2011 20:00 # 0

          т.е., видимо там, где "//Далее идет поиск" снова вызываются все эти методы, но уже без .length()
          Ответить

          • ava foGa 12.08.2011 01:01 # 0

            Кэп, а я тебя сразу не признал.
            Да, так и есть, но там уже все вызовы в один try-catch блок вставлены.
            Ответить

    Добавить комментарий