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

    +77

    1. 1
    if (!employees.equals("") && employees != null) {

    Запостил: gorsash, 23 Января 2012

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

    • если условия поменять местами то код из дебильного превратится просто в дерьмовый
      во-первых StingUtils
      во-вторых если автор не уверен -- пустая у него строка или нулл то это как-бы тоже намекает
      Ответить
      • список работников, хранящийся в строке, тоже доставляет
        Ответить
        • Нет, список не хранится в строке. Список id работников передаётся в виде параметра запроса
          Ответить
          • непидарские фреймворки умеют превращать такое в массив
            даже если не умеют -- надо такое написать
            как только у тебя есть строка с именем "работники" где через зопятую что-то лежит -- так сразу вон из джавы в ПХПшный быдлостан
            Ответить
            • Не у меня, а у тех, кто передо мной писал. А JSONом кошерно передавать и десериализовать явой? И какие например фреймворки превращают в массив? Struts умеет?
              Ответить
              • play!
                Ответить
              • 1) да, кошегно.
                2) да, умеет
                3) струтс гавно, лучше spring mvc

                понимаете, отсутствие массивов в POST и GET это заморочка HTTP и следовательно -- знание очень низкоуровневое.
                А имя поля "users" или что там у вас -- это знание более высокоуровневое.
                они должны жить на разных этажах.

                превращение строки в коллекцю или массив это задача такого же плана как сборка мусора или проверка типов в ран-тайме.

                а "users" это понятие уровня бизнес-логики.

                их надо разделять.

                фрейм-ворк должен отдавать бизнес-логике (контроллерам) коллекции и массивы а не строки с запятыми
                Ответить
    • Наверное, код написан в Израиле или у арабов. Компилировать с флагом -rtl (Right-to-Left) - тогда выражения будут вычисляться справа налево.

      (☺)
      Ответить
    • показать все, что скрытоvanished
      Ответить

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