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

    +77

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

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

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

    • ava guest 24.01.2012 00:42 # 0

      если условия поменять местами то код из дебильного превратится просто в дерьмовый
      во-первых StingUtils
      во-вторых если автор не уверен -- пустая у него строка или нулл то это как-бы тоже намекает
      Ответить

      • ava roman-kashitsyn 24.01.2012 12:06 # 0

        список работников, хранящийся в строке, тоже доставляет
        Ответить

        • ava gorsash 24.01.2012 17:56 # 0

          Нет, список не хранится в строке. Список id работников передаётся в виде параметра запроса
          Ответить

          • ava guest 24.01.2012 20:02 # 0

            непидарские фреймворки умеют превращать такое в массив
            даже если не умеют -- надо такое написать
            как только у тебя есть строка с именем "работники" где через зопятую что-то лежит -- так сразу вон из джавы в ПХПшный быдлостан
            Ответить

            • ava gorsash 24.01.2012 20:35 # 0

              Не у меня, а у тех, кто передо мной писал. А JSONом кошерно передавать и десериализовать явой? И какие например фреймворки превращают в массив? Struts умеет?
              Ответить

              • ava roman-kashitsyn 24.01.2012 21:27 # 0

                play!
                Ответить

              • ava guest 24.01.2012 22:17 # +2

                1) да, кошегно.
                2) да, умеет
                3) струтс гавно, лучше spring mvc

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

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

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

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

                фрейм-ворк должен отдавать бизнес-логике (контроллерам) коллекции и массивы а не строки с запятыми
                Ответить

    • ava Steve_Brown 25.01.2012 09:45 # +4

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

      (☺)
      Ответить

    • ava guest8 09.04.2019 12:34 # −999

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

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