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

    +170.7

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    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, 11 Декабря 2009

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

    • ava guest 11.12.2009 18:40 # +11

      Тут нужно было быть в курсе про switch
      Ответить

    • ava whiskeysam 11.12.2009 20:10 # −9

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

    • ava apple-way 11.12.2009 21:26 # +4

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

      • ava guest 11.12.2009 21:32 # +1

        Ток не AND, а OR
        Ответить

      • ava Oleg_quadro 12.12.2009 02:42 # +1

        Согласен, здесь if elseif от switch`а мало чем отличиться
        Ответить

      • ava guest 13.12.2009 15:26 # +1

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

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

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

      • ava guest 17.12.2009 20:44 # +2

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

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

    • ava guest 12.12.2009 12:45 # +4

      массив надо юзать и не епать мозг
      Ответить

    • ava Alice 12.12.2009 16:13 # 0

      Это китайский код.
      Ответить

    • ava guest 13.12.2009 17:00 # −2

      тут должны были быть классы, чукчи
      Ответить

    • ava guest 13.12.2009 19:38 # +3

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

    • ava guest 14.12.2009 14:23 # 0

      эх, ролевики ))
      Ответить

    • ava guest 14.12.2009 17:58 # 0

      Сам так писал когда-то давно
      Ответить

    • ava HyperGeek 23.12.2009 15:26 # +2

      Может это ASCII-арт?
      Ответить

    • ava guest 23.12.2009 20:41 # 0

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

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