Говнокод

"PHP" / Говнокод #140

echo “<select id=\”seltel[".$i."]\”>”;
switch ($data2["PHONE"][$i]["type"]) {
case “Факс”:
echo “<option selected>Факс”;
echo “<option>Офисный телефон”;
echo “<option>Колцентр”;
echo “<option>Телефон”;
echo “<option>Мобильный телефон”;
break;
case “Офисный телефон”:
echo “<option>Факс”;
echo “<option selected>Офисный телефон”;
echo “<option>Колцентр”;
echo “<option>Телефон”;
echo “<option>Мобильный телефон”;
break;
case “Колцентр”:
echo “<option>Факс”;
echo “<option>Офисный телефон”;
echo “<option selected>Колцентр”;
echo “<option>Телефон”;
echo “<option>Мобильный телефон”;
break;
case “Телефон”:
echo “<option>Факс”;
echo “<option>Офисный телефон”;
echo “<option>Колцентр”;
echo “<option selected>Телефон”;
echo “<option>Мобильный телефон”;
break;
case “Мобильный телефон”:
echo “<option>Факс”;
echo “<option>Офисный телефон”;
echo “<option>Колцентр”;
echo “<option>Телефон”;
echo “<option selected>Мобильный телефон”;
break;
};
echo “</select><br/>”;

вот как надо создавать выпадающие списки!

Запостил: guest, 12.12.2008 (Updated 24.03.2018)

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

guest 18 лет ago # 0

страйко:
бггг, атас!
Ответить
guest 18 лет ago # 0

Dr.Freez:
страйко пади сам такие конструкции юзает)
Ответить
guest 18 лет ago # 0

M-K:
Тут надо поменьше echo и побольше нормального HTML? Или что-то еще не так?
Ответить
guest 18 лет ago # 0

страйко:
[quote=M-K]
Тут надо поменьше echo и побольше нормального HTML? Или что-то еще не так?
[/quote]
тут достаточно задать список значений выпадающего списка и уже выбирать к какому из них делать selected.
Ответить
guest 18 лет ago # 0

Батько:
а я бы так и сделал =]
Ответить
guest 18 лет ago # 0

chiro:
Ярко))
Ответить
guest 17 лет ago # 0

Говноодмин:
#5 Батько, поздравляем, вы можете высылать на свои работы
Ответить
guest 17 лет ago # 0

крабик в гробике:
Них-чево в программировании не понимаю(быдлокодер), я бы сделал так:
[code=php]
/* Автор не знает, чем отличаются одинарные кавычки от двойных(знаков дюйма), переписано под одинарные: строки записаные в одинарные кавычки + конкатенация через точку — поднимает производительность. */
$phone_type = array('телефон','Факс','Офисный' 'телефон','Колцентр','Телефон','Мобильны й телефон'); /* Ага, одно с большой буквы, другое — с маленькой. Татьяныч бы за такое убил */

switch ($data2["PHONE"][$i]["type"]) {
case 'Факс':
$list_selected = 1;
break;
case 'Офисный телефон':
$list_selected = 2;
break;
case 'Колцентр':
$list_selected = 3;
break;
case 'Телефон':
$list_selected = 4;
break;
case 'Мобильный телефон':
$list_selected = 5;
break;
} /* после фигурной скобки точка-с-запятой не ставится, или я вообще забыл, как_это_было_на_самом_деле?.. */

echo '<select id="seltel['.$i.']">'; /* Видимо, автора задолбали переменные, которые он наплодил, и он ничего не знает про область видимости, и его фантазия в придумывании переменных закончилась. Ну и он всю форму засовывает в ячейки массива */

foreach($phone_type as $list_item) {
$i = 0; $i++;
$option_selected = '';
if($i === $list_selected) $option_selected = ' selected="selected"';
echo '<option'.$option_selected.'>'.$list_ite m.'</option>';
}
echo '</select><br />';
[/c
Ответить
guest 17 лет ago # 0
крабик в гробике:
$i = 0;
```
$i = 0;
```
там ещё есть такой указатель в пхп, чтоб переменная присваивалась только один раз, я его забыл
Ответить
guest 17 лет ago # 0

крабик в гробике:
А вообще эпичный говнокод, ржалъ!!
не, всё-таки я неправ, там в последнем цикле надо поставить вначале $i=1; а в конце цикла ставить инкремент.
Ответить

guest 17 лет ago # 0

крабик в гробике:
Них-чево в программировании не понимаю(быдлокодер), я бы сделал так:

/* Автор не знает, чем отличаются одинарные кавычки от двойных(знаков дюйма), переписано под одинарные: строки записаные в одинарные кавычки + конкатенация через точку - поднимает производительность. */
$phone_type = array('Факс','Офисный','Колцентр','Телефон','Мобильный телефон');

switch ($data2["PHONE"][$i]["type"]) {
case 'Факс':
$list_selected = 1;
break;
case 'Офисный телефон':
$list_selected = 2;
break;
case 'Колцентр':
$list_selected = 3;
break;
case 'Телефон':
$list_selected = 4;
break;
case 'Мобильный телефон':
$list_selected = 5;
break;
} /* после фигурной скобки точка-с-запятой
не ставится, или я вообще забыл, как_это_было_на_самом_деле?.. */

echo '<select id="seltel['.$i.']">'; /*
Видимо, автора задолбали переменные, которые он наплодил, и он ничего не знает про область видимости, и его фантазия в придумывании переменных закончилась. Ну и он всю форму засовывает в ячейки массива */

foreach($phone_type as $list_item) {
    $i = 1;
    $option_selected = '';
    if($i === $list_selected)
    $option_selected = 
    ' selected="selected"';
    echo '<option'.$option_selected.'>'
    .$list_item.'</option>';
    $i++;
}
echo '</select><br />';

Извините за быдлокод, изучал ПХП поверхностно, высшей математики не знаю.

Там

Ответить

zim 14 лет ago # 0 ⇈

>высшей математики не знаю.
Она ведь просто необходима, чтобы написать выпадающий список!
Ответить

guest 17 лет ago # 0
```
$phone_type = array('Факс','Офисный','Колцентр','Телефон','Мобильный телефон');
echo '<select id="seltel['.$i.']">'; 
foreach($phone_type as $list_item) {
    $option_selected =($data2["PHONE"][$i]["type"] == $list_item)?  ' selected="selected"': '';
    echo "<option $option_selected>$list_item</option>";// можно и в одну строчку
}
echo '</select><br />';
```
mapron
Ответить
- guest 15 лет ago # 0 ⇈
  
  $phone_type = array('Факс','Офисный','Колцентр','Телеф он','Мобильный телефон');
  $option_selected[$data2["PHONE"][$i]["type"]]='selected';
  foreach($phone_type as $list_item) $options.='<option '.$option_selected[$list_item].'>'.$list_item.'</option>';
  echo '<select id="seltel['.$i.']">'.$options.'</select>';
  Ответить
Pauchok-Anaynckiy 16 лет ago # 0

Сцуко, не перевелись еще вундеркинды.
Сий ГК неоправданно обделен в оценках (+81.3 на момент данного коммента вместе с моим голосом), как минимум раз в 10. Это же жемчужина среди всех говнокодов!
Ответить

guest 14 лет ago # 0

$contacts = array('Факс','Офисный','Колцентр','Телефон','Мобильный телефон');
$out =  '<select id=”seltel['.$i.']”>';
foreach($contacts as $contact) {
    $selected = $contact==$data2["PHONE"][$i]["type"] ? ' selected="selected"' : '';
    $out .= '<option'. $selected.'>'.$contact.'</options>';
}
$out .= '</select>';
echo $out;

Ответить

bot 14 лет ago # 0 ⇈

Иди на хуй придурок, там, видимо, не в одном месте эта дрянь:

function buildSelect( $i, $ar=array(), $sel=null ) {
  $out = array();
  foreach($ar as $v) 
      $out[] = "<option" . (($sel==$v)? ' selected':'') . ">$v</options>";

return "<select id=\"seltel[$i]\">" . implode('', $out) . "</select>";
}
// ...
$contacts = array('Факс','Офисный','Калцентр','Телефон','Мобильный телефон');
echo buildSelect( $i, $contacts, $data2["PHONE"][$i]["type"]);

Ответить

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

Я, guest, находясь в здравом уме и твердой памяти, торжественно заявляю:

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