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

    +155

    1. 1
    2. 2
    foreach ($period_range as $k=>$v) 
	{?><option value="<?=$k?>"<?=($k==$period)?' selected':''?>><?=$v?></option><?}

    Накодил как-то...

    Запостил: zima, 11 Января 2012

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

    • ava dev6alexander 11.01.2012 14:40 # −1

      Ничего необычного, обычная мешанина на php. Юзай тимлейтеры чтоли...
      Ответить

      • ava Vasiliy 11.01.2012 14:47 # −2

        ну это в полне может быть кусоком представления. Хотя можно и какой нибуть класс написать для построения хтмл типа html::select($k,$htmlOptions); Но я не вижу приимуществ.
        Ответить

    • ava alex228 11.01.2012 16:23 # 0

      вполне нормальный код.
      Ответить

    • ava nobody 11.01.2012 18:34 # −6

      показать все, что скрытокакой в жопу нормальный код, сегодня что день троля что ли ?
      Ответить

      • ava alex228 11.01.2012 18:38 # 0

        ну так предложите свой более нормальный аналог
        Ответить

        • ava nobody 11.01.2012 19:12 # −3

          foreach ($period_range as $k=>$v)
          { 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>';
          }
          Ответить

          • ava Vasiliy 11.01.2012 19:17 # +2

            в сабже вариант лучше хотябы потому что быстрее и читабельней.
            Ответить

          • ava alex228 11.01.2012 21:41 # +1

            о мля.... вместо того, что разместил автор, стоило бы поместить ваш вариант. потому что он ужасен. во-первых потому что слишком большой, во-вторых потому что вы html-теги выводите с помощью echo, что нагружает интерпретатор и весьма не читабельно. может быть для такого маленько куска кода это и не так страшно, но вот когда будет более-менее объемная форма и куча другого кода, будет тихий ужас.
            так что, видимо тролль здесь вы
            Ответить

          • ava dev6alexander 12.01.2012 08:36 # 0

            это называеться - поменять шило на мыло. Другими словами одно говно на другое
            Ответить

    • ava zima 11.01.2012 19:40 # 0 

      $out = '';
foreach ($period_range as $k=>$v) 
	$out .= "<option value=$k" . (($k==$period)?' selected':'') . ">$v</option>";
echo $out;


      При условии, что $k является числом, один из лучших вариантов, т.к. нет блока кода и вызов меньшего числа операторов.
      Ответить

    • ava istem 11.01.2012 21:09 # +2

      Объявляется конкурс на лучшее исполнение конструкции тега "select" в php...
      Ответить

      • ava zima 11.01.2012 21:13 # 0

        А это идея. Я предложил свой вариант.
        Ответить

        • ava parker 12.01.2012 09:52 # 0

          Самый первый способ самый лучший из предложенных, можно добавить немного косметики, например заменить foreach чтобы не работать не с копией массива, ну и убрать пассивную конструкцию сравнения if?:
          <?
          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
          Ответить

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