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

• guest 16 лет ago # 0

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

  Ответить

• nur 16 лет ago # 0

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

  Ответить

• Abductio 16 лет ago # 0

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

  Ответить

  • guest 16 лет ago # 0 ⇈

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

    Ответить

    • Abductio 16 лет ago # 0 ⇈

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

      Ответить

• guest 16 лет ago # 0

  $is_use_newstyle = $newstyle == 1;
  😉

  Ответить

  • guest 16 лет ago # 0 ⇈

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

    Ответить

  • leprosus 16 лет ago # 0 ⇈

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

    Ответить

    • guest 16 лет ago # 0 ⇈

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

      Ответить

      • leprosus 16 лет ago # 0 ⇈

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

        Ответить

        • guest 16 лет ago # 0 ⇈

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

          Ответить

          • leprosus 16 лет ago # 0 ⇈

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

            Ответить

            • shureg 16 лет ago # 0 ⇈

              Единственное место где используеться:
              

              if ($is_new_style){
              ...
              }

              Ответить

              • leprosus 16 лет ago # 0 ⇈

                Ну, уважаемый shureg (он же guest), теперь всё встало на свои места. 🙂
                В таком контексте этот код можно назвать малограмотным, но далеко не говнокодом, так говнокодиком.
                
                Скорее этот пост похож на обиды на автора кода!
                
                Полистайте другие разделы PHP-говнокода, найдёте действительно стоющее...

                Ответить

• guest 16 лет ago # 0

  Само по себе не говонокод

  Ответить