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

    +161

    1. 1
    $period = in_array($bullRegionId,array_keys($periods)) ? $periods[$bullRegionId]['text'] : $periods['default']['text'];

    Интересна сама проверка на существование ключа, не правда ли? :-)
    Код мой. Коллега учуял, что от него несколько плохо пахнет и предложил запостить сюда))

    Запостил: Nemoden, 09 Сентября 2011

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

    • ava jokz 09.09.2011 02:43 # 0

      ну не так уж чтобы и гавно, вся оптимизация сведется к следующему

      $period = isset($periods[$bullRegionId]['text'])? $periods[$bullRegionId]['text'] : $periods['default']['text'];
      Ответить

      • ava Nemoden 09.09.2011 02:47 # 0 

        isset($periods[$bullRegionId]['text'])

        выдаст E_NOTICE: Undefined index, если ключа $bullRegionId не существует.

        Для проверки ключа массива есть array_key_exists, которая еще и быстрей отработает, чем isset
        array_key_exists($bullRegionId, $periods) ? 
           $periods[$bullRegionId]['text'] : $periods['default']['text'];


        Ну а вообще, не жуткий говнокод, конечно, но хуже у нас в проекте нету. Извините.
        Ответить

        • ava jokz 09.09.2011 03:13 # 0

          опечатался, извиняйте. конечно же не

          isset($periods[$bullRegionId]['text'])

          а просто

          isset($periods[$bullRegionId])
          Ответить

        • ava Snark 09.09.2011 03:25 # +1

          >>> но хуже у нас в проекте нету. Извините.
          Наглая ложь! Найдем еще, и много! ; )
          Ответить

          • ava Nemoden 09.09.2011 03:54 # 0

            Чуть меньше миллиона строк кода... Ну да, наверное найдем еще чего-нибудь :)
            Ответить

        • ava sectus 09.09.2011 04:17 # +1

          > isset($periods[$bullRegionId]['text'])
          > выдаст E_NOTICE: Undefined index, если ключа $bullRegionId не существует.

          Ничего не путаете?
          Ответить

    • ava FAKYOUINTIRNEAT 09.09.2011 03:13 # 0 

      $period = $periods[$bullRegionId]['text'] or $period = $periods['default']['text']

      ?
      Ответить

      • ava Nemoden 09.09.2011 03:18 # 0

        Будет тот же самый E_NOTICE :)
        Ответить

        • ava jokz 09.09.2011 03:21 # −1

          error_reporting(0);

          разруливает
          Ответить

          • ava Nemoden 09.09.2011 03:40 # +2

            У нас ошибки всех уровней логгируются в файл. Моё мнение по этому поводу: никаких ошибок никакого уровня быть не должно.
            Это в идеале :) На деле "не смертельные" ошибки у нас есть, конечно.
            Ответить

            • ava jokz 09.09.2011 10:58 # 0

              у меня такоеже мнение, а ошибки если и есть то логируются в базу) но в идеале, ни одного нотиса быть не должно.
              Ответить

        • ava FAKYOUINTIRNEAT 09.09.2011 09:49 # 0

          хм, я проверял, перед тем как запостить и никакой ошибки не было:
          http://codepad.org/2v0cCi3P
          Если что, я плохо знаком с PHP
          Ответить

          • ava Nemoden 09.09.2011 13:19 # 0

            Ну codepad, конечно, E_NOTICE подавляет. Я думаю, что он все подавляет, кроме E_ERROR и E_CORE_ERROR
            Ответить

    • ava roman-kashitsyn 09.09.2011 09:26 # 0

      А если так:
      $regionId = in_array($bullRegionId, array_keys($periods)) ? $bullRegionId : 'default';
$period = $periods[$regionId]['text'];
      ?
      Ответить

      • ava Nemoden 09.09.2011 13:21 # 0

        Так можно, но сам прикол в проверке существования ключа :) Намек на говнокод был именно в этом ;)
        Ответить

        • ava roman-kashitsyn 09.09.2011 13:28 # 0

          Да, я уже понял. В любом случае, повторяющиеся куски кода - плохо, поэтому оптимальным вариантом считаю следующий:
          $regionId = isset($periods[$bullRegionId]) ? $bullRegionId : 'default';
$period = $periods[$regionId]['text'];
          Ответить

    • ava Lure Of Chaos 09.09.2011 09:41 # 0

      быки и коровы? ))
      Ответить

      • ava Nemoden 10.09.2011 07:44 # 0

        bull - наше сокращение от bulletin. Я когда начинал работать в компании привыкнуть сначала не мог, а теперь норм :)
        Ответить

    • ava TarasB 09.09.2011 21:02 # 0 

      $period = $periods[in_array($bullRegionId,array_keys($periods)) ? $bullRegionId:'default']['text'];
      Ответить

      • ava Nemoden 10.09.2011 07:39 # 0

        Ахахаха :) Да! Это можно еще раз запостить на говнокод, но уже синтетика :-(
        Ответить

    • ava guest8 09.04.2019 11:28 # −999

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

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