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

    +79

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    private void getCurrentBranch();
public TDictBranch getCurrentBrach();
public TDictBranch2 getCurrentBrach2();
public TDictBranch2 getCurrentBrach2(Date lookupDate);
public String getCurrentBrachId();
public String getCurrentBrunchCode();
public static String getCurrentBranchCharId();

    некоторые методы из класса конфигурации приложения

    Запостил: borka, 20 Июля 2010

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

    • ava Мистер Хэнки 20.07.2010 13:25 # +7

      >>Branch
      >>Brach
      >>Brunch

      Т_Т
      Ответить

      • ava borka 20.07.2010 14:03 # 0

        а еще один из этих методов внезапно ничего не возвращает, а другой вдруг объявлен как static, что наводит на мысль а почему бы и всем остальным не быть static?
        Ответить

        • ava Lure Of Chaos 20.07.2010 14:08 # 0

          и желательно private. что бы никто не видел такого позора
          Ответить

    • ava absolut 20.07.2010 13:39 # +3

      Drink, Drank, Drunk ... изучаем неправильные глаголы :)
      Ответить

    • ava Lure Of Chaos 20.07.2010 14:01 # 0

      и плюс чехарда с модификаторами. то есть getCurrentBranch приватный, а getCurrentBranchCharId публичный статический. аффтару посоветовал бы убиться веником
      Ответить

      • ava borka 20.07.2010 14:06 # +2

        судя по эээ... почерку, это несколько разных аффтаров
        Ответить

        • ava Lure Of Chaos 20.07.2010 14:07 # +1

          тогда, блин, они друг другу братья по разуму
          Ответить

        • ava absolut 21.07.2010 07:47 # +3

          >это несколько разных аффтаров
          Так вот она какая эффективная командная разработка
          Ответить

          • ava borka 23.07.2010 06:30 # 0

            проект очень бородатый, прошел через руки многих аффтаров
            Ответить

            • ava Altravert 27.07.2010 07:52 # 0

              Не оправдание. Велосипедить в рамках одного класса -- это ничто не оправдает.
              Ответить

              • ava absolut 29.07.2010 19:46 # 0

                Обычно последнему в пищевой цепочке приходится это всё причесывать.
                Ответить

    • ava Lure Of Chaos 20.07.2010 14:11 # +1

      второй взгляд: по TDictBranch видно, что создатель (или, как подсказывают, создатели) 5 минут как с паскаля слез, на жабку
      Ответить

      • ava Lure Of Chaos 20.07.2010 14:12 # +1

        или делфей, но маловероятно, раз с модификаторами так туго
        Ответить

    • ava bober_maniac 20.07.2010 14:22 # 0

      Почему функция getCurrentBranch() возвращает void? Это же не просто ужас, а ужас-ужас!
      Ответить

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