"PHP" / Говнокод #2267 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
  7. 7
  8. 8
  9. 9
  10. 10
  11. 11
  12. 12
  13. 13
  14. 14
  15. 15
if($type=="amulet") {$w=60;$h=20;}
        else if($type=="sergi") {$w=60;$h=20;}
            else if($type=="sword") {$w=60;$h=60;}
                else if($type=="axe") {$w=60;$h=60;}
                    else if($type=="fail") {$w=60;$h=60;}
                        else if($type=="knife") {$w=60;$h=60;}
                            else if($type=="spear") {$w=60;$h=60;}
                                else if($type=="armour") {$w=60;$h=80;}
                                    else if($type=="ring1") {$w=20;$h=20;}
                                        else if($type=="ring2") {$w=20;$h=20;}
                                            else if($type=="ring3") {$w=20;$h=20;}
                                                else if($type=="helmet") {$w=60;$h=60;}
                                                    else if($type=="perchi") {$w=60;$h=40;}
                                                        else if($type=="shield") {$w=60;$h=60;}
                                                            else if($type=="boots") {$w=60;$h=40;}

про elseif не в курсе

Запостил: dicos dicos, (Updated )

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

  • ava whiskeysam # Ссылка на оригинал 0

    брррр... ну хоть более менее читабельно)
    p.s. switch - отстой.
    Ответить

  • ava apple-way # Ссылка на оригинал 0

    Между прочим, что if ... else, что if ... elseif, что switch ... case - особой разницы нет. Так что в этом случае глупая придирка.
    А вот то что надо было бы использовать AND для совмещения условий для которых одинаковая инициализация переменных - это да!
    Сократилось бы до 5 строчек.
    Ответить

    • ava guest # Ссылка на оригинал 0 ⇈

      Я, конечно, ПХП знаю хреноватенько, но тем не менее разве switch-break не работает как OR?

      switch ($type){
      "amulet":
      "sergi":
      $w=60;$h=20;
      break;
      //........
      }

      Ну конечно это все можно прооптимизировать получше...
      Ответить

    • ava guest # Ссылка на оригинал 0 ⇈

      Вообще-то, elseif а уж тем более switch используется для улучшения читабельности:
      - гораздо легче воспринимается колонка switch, чем длинная диагональ if..elseif, особенного для такого кода

      не так ли?)
      Ответить

  • ava guest # Ссылка на оригинал 0

    ну я в таких целях просто n-мерный массив использую - так проще изменить данные потом
    Ответить

  • ava guest # Ссылка на оригинал 0

    почитал комменты...
    я у вас проэкты заказывать небуду
    Ответить

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

Переведи на "PHP", guest!

    А не использовать ли нам bbcode?

    ↑
    8
    ↓