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

    • тока щас заметил двойные скобки у циклов :)))
      Ответить
    • Похоже на выхлоп генератора.
      Ну или китайца.
      Ответить
      • вполне себе русский программер :) но видимо или трудолюбивая девочка или вообще школота, не продвинувшаяся в алгоритмах дальше таких линейных схем....
        Ответить
    • $global_city = ($_GET['city'] == '' ? $_POST['city_select']: $_GET['city']);

      Универсальный фэйл, если GET не передан - завалится. Ну или плюнет ворнинг, не помню уже.
      Ответить
      • нотис.
        по факту там был гет запрос, но не приходил $_GET['city'], он попадал true и еще нотис валился от $_POST['city_select']
        Ответить
    • О чем думал говнокодер когда писал вот это: $v == $global_city ? "" : ""
      Ответить
    • циклы развернуло распидорасило.
      этож надо оптгруппы руками делать, да еще и без массивов.
      Ответить

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