1. PHP / Говнокод #2855

    +149.2

    1. 1
    $is_use_newstyle = ($newstyle == 1) ? 1 : 0;

    Запостил: shureg, 24 Марта 2010

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

    • ava guest 24.03.2010 11:56 # +2.8

      Что здесь не так?
      Ответить

    • ava nur 24.03.2010 12:15 # +0.6

      нормальный код, ничего говнистого не замеченно
      Ответить

    • ava Abductio 24.03.2010 12:17 # +4

      Как то все эти детские говнокоды уже надоели
      Ответить

      • ava guest 24.03.2010 19:31 # +1

        а что такое взрослоый говнокод? его от взрослого ума пишут?)
        Ответить

        • ava Abductio 24.03.2010 20:46 # +0.8

          Что более сложное, в плане архитектуры и логики. А не примеры ошибок детей которые вчера книгу по php читали.
          Ответить

    • ava guest 24.03.2010 12:32 # −3.4

      $is_use_newstyle = $newstyle == 1;
      ;-)
      Ответить

      • ava guest 24.03.2010 13:01 # −4.2

        менее наглядный вариант
        Ответить

      • ava leprosus 24.03.2010 13:18 # +1.8

        Авто преобразование типов приведёт твой пример к Boolean значению, а автору кода, возможно, нужен был чистый Int
        Очень редко, но бывают неопределённые для интерпритатора ситуации, когда PHP автоматические преобразования типов делают дрянную работу...

        Это не говнокод!
        Ответить

        • ava guest 24.03.2010 15:48 # −1.4

          Ключевое слово 'Очень редко'. Вероятость написания такого по тупости намного выше чем написания по необходимости. Кроме того название переменной is_use_newstyle указывает на bool.
          Ответить

          • ava leprosus 24.03.2010 15:54 # +2.4

            Ну ты не знаешь, где эта переменная будет использоваться...
            Возможно, в SQL или REST и т.д.
            А ключевое слово говорит, что необходимо это "Очень редко" учитывать.
            Этим и отличается грамотный код от безграмотного!
            Ответить

            • ava guest 24.03.2010 16:20 # −2.2

              Тут все упирается в контекст, которого никто кроме оп'а не видел. Если это происходит перед сериализацие/запросом - тогда никаких проблем. Если это посередине логики - тогда это говно.
              Ответить

              • ava leprosus 24.03.2010 16:24 # 0

                Тогда может попросим автора этого поста поделиться всем кодом, чтоб уж наверняка судить, а то из-за одной строчки кода тут баталий в комментариях уже на 5 часов.
                Ответить

                • ava shureg 24.03.2010 16:36 # 0

                  Единственное место где используеться: 
                  if ($is_new_style){
...
}
                  Ответить

                  • ava leprosus 24.03.2010 16:52 # +2.6

                    Ну, уважаемый shureg (он же guest), теперь всё встало на свои места. :)
                    В таком контексте этот код можно назвать малограмотным, но далеко не говнокодом, так говнокодиком.

                    Скорее этот пост похож на обиды на автора кода!

                    Полистайте другие разделы PHP-говнокода, найдёте действительно стоющее...
                    Ответить

    • ava guest 25.03.2010 09:08 # −1.2

      Само по себе не говонокод
      Ответить

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