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

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
  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 scorcher, (Updated )

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

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

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

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

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

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

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

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

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

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

  • ava Lure Of Chaos # Ссылка на оригинал 0

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

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

Помни, guest, за тобой могут следить!

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

    ↑
    8
    ↓