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

    +52

    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
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    <?
....
//башкортостан
$city_select_arr = "<OPTGROUP LABEL=\"Республика Башкортостан\">";
foreach($city_select_data as $k=>$v) {
{
$global_city = ($_GET['city'] == '' ? $_POST['city_select']: $_GET['city']);
$city_select_arr .= "<option value=\"".$v."".( $v == $global_city ? " selected" : "" ) . "\">".$v."</option>";
}
}
$city_select_arr .= "</OPTGROUP>";
//пермский край
$city_select_arr .= "<OPTGROUP LABEL=\"Пермский край\">";
foreach($city_select2_data as $k=>$v) {
{
$global_city = ($_GET['city'] == '' ? $_POST['city_select']: $_GET['city']);
$city_select_arr .= "<option value=\"".$v."".( $v == $global_city ? "" : "" ) . "\">".$v."</option>";
}
}
$city_select_arr .= "</OPTGROUP>";

.... и так еще циклов 10-15

    обратите внимание на переменную $global_city - ей в каждой итерации каждого цикла присваивают условное значение, которое в пределах этого файла точно не меняется :)
    ну и сам набор циклов можно было бы конечно систематизировать.

    Запостил: scorcher, 19 Августа 2012

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

    • ava scorcher 19.08.2012 21:12 # 0

      тока щас заметил двойные скобки у циклов :)))
      Ответить

    • ava Vindicar 19.08.2012 21:28 # +1

      Похоже на выхлоп генератора.
      Ну или китайца.
      Ответить

      • ava scorcher 19.08.2012 21:36 # 0

        вполне себе русский программер :) но видимо или трудолюбивая девочка или вообще школота, не продвинувшаяся в алгоритмах дальше таких линейных схем....
        Ответить

    • ava Vindicar 20.08.2012 09:16 # 0

      $global_city = ($_GET['city'] == '' ? $_POST['city_select']: $_GET['city']);

      Универсальный фэйл, если GET не передан - завалится. Ну или плюнет ворнинг, не помню уже.
      Ответить

      • ava scorcher 20.08.2012 10:38 # 0

        нотис.
        по факту там был гет запрос, но не приходил $_GET['city'], он попадал true и еще нотис валился от $_POST['city_select']
        Ответить

    • ava shadoff 21.08.2012 06:05 # +2

      О чем думал говнокодер когда писал вот это: $v == $global_city ? "" : ""
      Ответить

    • ava Lure Of Chaos 21.08.2012 06:50 # 0

      циклы развернуло распидорасило.
      этож надо оптгруппы руками делать, да еще и без массивов.
      Ответить

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