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

    +148

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    $type = isset($config['type']) ? $config['type'] : 'default';
switch($type) {
    default:
        $blockClass = 'adminhtml/widget_grid_massaction_item_additional_default';
        break;
}

    Magento core

    Запостил: crook, 18 Июля 2013

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

    • ava kostoprav 18.07.2013 11:03 # +4

      На мой взгляд, если это просто вырвано из коммита - не говнокод, просто в процессе разработки...
      Ответить

    • ava clgs 18.07.2013 14:34 # +1

      Всегда поражало наличие "break;" в кейсе "default:" непосредственно перед закрывающейся скобкой
      Ответить

      • ava TarasB 18.07.2013 14:38 # +6

        Всегда поражало наличие "break;" в кейсе
        Ответить

        • ava inkanus-gray 18.07.2013 14:56 # 0

          Кстати, бывают ситуации, когда свитч-кейс внутри цикла и нужно брякнуть цикл. Как это сделать в Си и в PHP, ведь там брякается не цикл, а свитч-кейс?
          Ответить

          • ava defecate-plusplus 18.07.2013 15:02 # +3

            goto.sex
            Ответить

          • ava clgs 18.07.2013 15:10 # +4

            PHP: break 2; (http://www.php.net/manual/ru/control-structures.break.php)
            Ответить

            • ava roman-kashitsyn 18.07.2013 15:18 # +6

              > break $num;
              о_О
              Это ж какой трэш можно на PHP написать до версии 5.4!
              Ответить

              • ava inkanus-gray 18.07.2013 17:20 # +3

                Например, такой: 
                <?php function Arrangement10($k) {
  $m = 10; $n = 1; 
  do {
    do {
      do {
        do {
          do {
            do {
              do {
                do {
                  do {
                    do {
                        break $m - $k;
                    } while (false);
                    $n *= $m; $m--;
                  } while (false);
                  $n *= $m; $m--;
                } while (false);
                $n *= $m; $m--;
              } while (false);
              $n *= $m; $m--;
            } while (false);
            $n *= $m; $m--;
          } while (false);
          $n *= $m; $m--;
        } while (false);
        $n *= $m; $m--;
      } while (false);
      $n *= $m; $m--;
    } while (false);
    $n *= $m; $m--;
  } while (false);
  return $n;
}

echo "Размещение из 10 по 3 равно 10*9*8 = " . strval(Arrangement10(3)) . PHP_EOL;
                P.S. На Ideone на 5.4 перешли, сволочи!
                Ответить

        • ava heleg 22.07.2013 22:57 # 0

          а как же иначе? объясните быдлокодеру
          Ответить

          • ava bormand 23.07.2013 06:44 # +3

            Эта претензия не к тем кто пользуется break'ом в switch'е ;)

            Эта претензия к тому извращенцу, который придумал такую ебанутую конструкцию как сишный свич. Натуральный костыль, запиленный на скорую руку из одного блока и меток потому что автору было лень немного подумать... Сколько программ реально пользуются "проскоком" с одной ветки на другую? Да их практически нет... Ну кроме дафф девайса, и тех, которым приходится выполнять одинаковые действия на несколько констант (а это опять ебанутость свича, не позволяющая просто перечислить константы через запятую, как в паскале). И если бы автор свича немного пошевелил мозгами - то break писать бы не приходилось. Было бы как-то так:
            switch (x) {
case 1,2..5:
    std::cout << "1 or 2..5" << std::endl;
case some::enum::value:
    std::cout << "some::enum::value" << std::endl;
case default:
    std::cout << "default" << std::endl;
}
            Но теперь уже никто ничего не изменит ;( И эта конструкция, к сожалению, останется навсегда.
            Ответить

            • ava anonimb84a2f6fd141 23.07.2013 08:05 # 0

              Я видел одну. Что-то такое:

              switch(i)
case 1:
    asd;
case 2:
    asd2;
    if (i==1)
        break;
    asd3;
    break;

              Но это достойно отдельного говнокода.
              Ответить

      • ava inkanus-gray 18.07.2013 15:18 # +3

        А наличие "break;" в других кейсах непосредственно перед закрывающейся скобкой никогда не поражало? Подсказка: "default:" может быть не последним.
        Ответить

        • ava clgs 18.07.2013 15:32 # −3

          Поржало)
          Подсказка: "непосредственно перед закрывающейся скобкой"
          Ответить

          • ava inkanus-gray 18.07.2013 15:49 # −1 

            $a = 'Apple';
 switch ($a) {
     default:
         echo '$a is not an orange<br>';
         break;
     case 'Orange':
         echo '$a is an orange';
         break;
 }
            А тут?
            Ответить

            • ava clgs 18.07.2013 16:00 # +1

              А тут "break; }". У меня же было написано "наличие "break;" в кейсе "default:" перед скобкой"
              Ответить

              • ava inkanus-gray 18.07.2013 16:56 # +2

                О том и говорю, что ключевой момент здесь — перед скобкой. У знающих толк в извращениях "default:" может быть и не последним...
                Ответить

      • ava blackhearted 18.07.2013 16:51 # +1

        Рефакторинг и, внезапно, твой default уже не последний и break; не перед скобкой.
        Ответить

        • ava clgs 18.07.2013 17:11 # +2

          Ну если только рефакторинг будет делать говнокодер ;)
          Ответить

        • ava scriptin 18.07.2013 20:28 # +3

          Рефакторинг предполагает улучшение кода, вы же говорите про обфускацию.
          Ответить

        • ava bormand 18.07.2013 20:34 # −1

          Приведите, пожалуйста, вменяемый пример, в котором вам понадобилось добавлять ветки ниже дефолтной.
          Ответить

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