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

    +142

    1. 1
    2. 2
    3. 3
    4. 4
    <tr>
                     <td valign="top" align="center"> <?=($image = $gallery->setId($line['id'])->getImaget("s")) ? "<a href='news/$line[id]'><img src='$image[src]' alt='$line[name]' width='50' title='$line[name]' style='float: left; border: 0px; margin-right: 10px;'></a>" : ""?></td>
                     <td valign="top"><a href="<?=$line['link'];?>"><?=$line['name'];?></a><br /><?=$line['announce'];?></td>
                  </tr>

    Скопировал как нашел

    Запостил: kindofbear, 12 Мая 2012

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

    • ->getImaget
      Картиньетка
      Ответить
      • Ну там есть функция просто getImage... а тут видимо подразумевалось получение thumbnail
        Ответить
    • Никого не смущает вывод с тернарным оператором, где второй веткой является пустая строка?
      Ок, буду теперь так делать :)
      Ответить
      • А как ещё, если это в echo завёрнуто? Зато функциональненько!
        Предложите заменить банальным ифом?
        Ответить
        • Да, банальным if. Чтобы было читабельно
          Ответить
          • Тогда придётся отказаться от клёвых скобочек <?= ?>, писать слова «if» и «echo». А что делать, если программист неграмотный и букв не знает?

            Можно и без echo, вложив <?= ?> в if, но это будет ещё длиннее.
            Ответить
            • или хотя бы так
              <? if ($image = $gallery->setId($line['id'])->getImaget("s")) { ?>
              <a href="news/<?=$line['id']?>">
              <img src="<?=$image['src']?>" alt="<?=$line['name']?>" width="50" title="<?=$line['name']?>" style="float: left; border: 0px; margin-right: 10px;">
              </a>
              <? } ?>
              Ответить
      • Без второй ветки похапе это не выполнит то ли до 5.2.0, то ли до 5.3.0.
        Ответить
    • .Твой вброс упал.
      Ответить
    • Ужас то какой, как теперь жить) Кстати код реальный, из сайта, где заказали сменить дизайн
      Ответить
    • Код прошел фейсконтроль, если ты не можешь написать или использовать нормальный шаблонизатор, чтобы отделить серверный код от клиентского, то это лучшая альтернатива.
      Ответить
      • окай. давайте еще условия типа
        if (empty($var)) { } else { echo 'Переменная не пуста!'; }
        я не спорю, просто меняю шаблоны)
        Ответить
    • Не особо говно. Хотя использовал бы тэмплейтор - было бы красивее.
      Ответить

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