- 1
- 2
foreach ($period_range as $k=>$v)
{?><option value="<?=$k?>"<?=($k==$period)?' selected':''?>><?=$v?></option><?}
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+155
foreach ($period_range as $k=>$v)
{?><option value="<?=$k?>"<?=($k==$period)?' selected':''?>><?=$v?></option><?}
Накодил как-то...
dev6alexander 11.01.2012 14:40 # −1
Vasiliy 11.01.2012 14:47 # −2
alex228 11.01.2012 16:23 # 0
nobody 11.01.2012 18:34 # −6
alex228 11.01.2012 18:38 # 0
nobody 11.01.2012 19:12 # −3
{ echo '<option value="'.$k.'"'.($k==$period ? ' selected' : '').'>'.$v.'</option>'; }
Не ? Ну или ...
foreach ($period_range as $k=>$v)
{
if($k==$period)
$selected = ' selected';
else
$selected = '';
echo '<option value="'.$k.'"'.$selected.'>'.$v.'</option>';
}
Vasiliy 11.01.2012 19:17 # +2
alex228 11.01.2012 21:41 # +1
так что, видимо тролль здесь вы
dev6alexander 12.01.2012 08:36 # 0
zima 11.01.2012 19:40 # 0
При условии, что $k является числом, один из лучших вариантов, т.к. нет блока кода и вызов меньшего числа операторов.
istem 11.01.2012 21:09 # +2
zima 11.01.2012 21:13 # 0
parker 12.01.2012 09:52 # 0
<?
while (list ($k, $v) = each($period_range)) {
?><option value="<?=$k ?>"<?
if($k == $period) {
?> selected="selected"<?
}
?>><?=$v ?></option><?
}
?>
Тут: $out .= "<option value=$k" . (($k==$period)?' selected':'') . ">$v</option>";
нагружаете пхп для того чтобы разобрать строку и выделить переменные + 1 переменная out + пассивная конструкция if без then