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

    +165

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    foreach ($user as $field => $value) {
    if ($field == 'username') {
        $user[$field] = md5($value);
    }
}

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

    Запостил: mr-lekafe, 18 Ноября 2014

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

    • ava someone 18.11.2014 19:22 # +2

      Антипаттерн for-if.

      http://blogs.msdn.com/b/oldnewthing/archive/2011/12/27/10251210.aspx
      Ответить

      • ava 3.14159265 18.11.2014 20:04 # +1

        Это не антипаттерн, это говнопаттерн. Как и бульщиты например.
        Антипаттерном я бы назвал скорее спорное решение, в котором зачастую сознательно отказываются от паттернов, допустим пишут свищ вместо создания фабрики.
        Или любовь Тараса к structам с публичными полями. Само по себе это не есть плохо.
        Ответить

        • ava Анонимус 18.11.2014 22:03 # +1

          >> Как и бульщиты например.
          А что за говнопатерн "бульщит"?
          Ответить

          • ava bormand 18.11.2014 23:11 # +1 

            // typical boolshit
if (b == true)
   return false;
return true;
            И прочая ерунда от незнания логических операций.
            Ответить

            • ava Анонимус 18.11.2014 23:14 # +2

              Я когда-то на phpclub видал такой вопрос: "if ($var) выполняет код, если $var истино. А как мне выполнить его если $var ложно?"
              Ответить

      • ava Xom94ok 18.11.2014 22:37 # +1

        Подглядел у коллег подобное в форме свитча.
        for(int i = 0; i < 11; i++) switch(i) {
case 0: //...
case 1: //...
//...
case 11: //...
}
        Ладно сами антипаттерны показывать, лечить-то такое чем?
        Ответить

        • ava bormand 18.11.2014 23:13 # +1

          Зато тут можно произвольно прыгать между блоками, если менять i перед бряком :)
          Ответить

          • ava kipar 19.11.2014 00:07 # +1

            Это же паттерн Интерпретатор!
            а, ну да, программы же нет. Не подходит.
            Ответить

            • ava bormand 19.11.2014 06:22 # +1

              > программы же нет
              Мы ее здесь просто не видим. Она может быть в каком-то массиве.
              Ответить

        • ava niggerzilla 19.11.2014 13:01 # 0

          Газом. Из доступного сейчас на рынке - Uragan D2
          Ответить

        • ava bot 19.11.2014 21:36 # 0

          > лечить-то такое чем?
          Наверное, isset($user['username']); ну или array_key_exists('username', $user);
          Ответить

    • ava inkanus-gray 18.11.2014 21:15 # +2

      А вдруг холостой цикл нужен для того, чтобы сокрыть время поиска элемента массива?
      Ответить

      • ava wvxvw 18.11.2014 21:49 # 0

        Тогда почему нет break?
        Ответить

      • ava bormand 18.11.2014 23:16 # +2

        Он не холостой! В строке 2 сравнение на "==", поэтому этот цикл может поменять несколько значений.

        http://ideone.com/LA18Uj
        Ответить

        • ava inkanus-gray 19.11.2014 00:40 # 0

          Слона-то я и не заметил... Тогда это смешнее в разы!
          Ответить

        • ava 1024-- 19.11.2014 03:02 # 0

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

          • ava bormand 19.11.2014 06:41 # +1

            true == "хуй", а false == ""
            0 == "хуй" и 0 == "",
            но 0 != true

            Твой пых. Твои правила.
            Ответить

            • ava Анонимус 19.11.2014 16:59 # +5

              Рассказывают (на самом деле нет) что в новой версии PHP будет оператор ==== который вводит строгую типизацию и будет кидать exception в случае если операнды разных типов.
              Ответить

        • ava Анонимус 19.11.2014 23:22 # 0

          Можно еще иметь гомогенные ключи в ассоциативном массиве и не стрелять себе в ногу
          Ответить

    • ava barbarbar 27.11.2014 15:17 # 0

      Либо я недооцениваю предсказуемость тупизны, либо запостивший сие - пи**обол.
      Ответить

      • ava Анонимус 27.11.2014 15:18 # 0

        Всмысле Вы не видите в этом коде никаких проблем?)
        Ответить

        • ava barbarbar 27.11.2014 15:29 # 0

          Я вижу проблему, но мне видится пара вариантов.
          1) Раньше были и другие условия в цикле. Либо предполагается, что в скором времени они добавятся. Вобщем, пример явно подправленный или вырван из контекста.
          2) Программист действительно больной дебил.
          Ответить

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