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

    +153

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    9. 9
    if(count($colors)>1){
    if($i==2){
        $col = $colors[1];
    }else{
        $col = $colors[0];
    }
}else{
    $col=$colors[0];
}

    Вот где не надо - тьма тернарных операторов, а где их логичнее всего ожидать - вот такой огород из IF/ELSE

    Запостил: Khvorostin, 13 Октября 2014

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

    • ava eandr67 13.10.2014 13:06 # +6

      Тут даже тернарного оператора не нужно:

      $col=$colors[(int)(count($colors)>1&&$i==2)];
      Ответить

      • ava Khvorostin 13.10.2014 13:39 # 0

        Круть!
        Ответить

      • ava tirinox 13.10.2014 15:14 # +2

        ВыPerlился!
        Ответить

        • ava eandr67 13.10.2014 15:19 # +1

          В каждом углу perl мерещится?
          Ответить

          • ava tirinox 13.10.2014 15:25 # +2

            Нет, только в тех местах где много странных символов в рандомном порядке и среди них есть $.
            Ответить

            • ava 1024-- 13.10.2014 16:00 # 0

              > много странных символов в рандомном порядке и среди них есть $
              https://www.nyse.com/quote/index/NYA
              Ответить

            • ava inkanus-gray 13.10.2014 16:43 # +3

              Ну почему сразу Perl?

              Google closure compiler может сделать «перл» даже из ECMAScript'а, причём совершенно автоматически.
              Ответить

      • ava Lokich 13.10.2014 15:36 # +2

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

      • ava roman-kashitsyn 13.10.2014 16:50 # +2

        Меня смущает волшебное число 2, хотелось бы услышать мнение топикстартера по этому поводу.

        Моя догадка - тут неявно предполагается, что цветов всегда не больше двух, причём индексируются они с единицы.
        Если это действительно так, то при добавлении нового цвета всё посыпется. Поэтому правильный (с моей точки зрения) вариант:
        $numColors = count($colors)
$colorIndex = (0 < $i && $i <= $n) ? ($i - 1) : 0;
$selectedColor = colors[$colorIndex];
        Ответить

    • ava Vindicar 13.10.2014 16:31 # 0

      Зато читаемо.
      Ответить

    • ava Soul_re@ver 13.10.2014 17:30 # +2

      Это всё потому что тернарный оператор в ПХП — говно. Вот с ним и не связываются.
      Ответить

      • ava bormand 13.10.2014 17:34 # +1

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

        • ava inkanus-gray 13.10.2014 19:49 # 0

          А на Перле можно написать говно даже в тех случаях, когда в выражении тернарник единственный.
          http://govnokod.ru/16479#comment242321
          Ответить

      • ava tirinox 13.10.2014 18:58 # 0

        Отсыплю тебе скобок, бро. (((())))
        Ответить

    • ava inkanus-gray 13.10.2014 23:02 # 0

      Выхлоп конпелятора замыканий (с параметрами по умолчанию для js):
      $col=1<count($colors)?2==$i?$colors[1]:$colors[0]:$colors[0];
      Ответить

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