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

    • ava Lure Of Chaos 12.05.2012 19:12 # +4

      ->getImaget
      Картиньетка
      Ответить

      • ava kindofbear 13.05.2012 02:10 # 0

        Ну там есть функция просто getImage... а тут видимо подразумевалось получение thumbnail
        Ответить

    • ava kindofbear 13.05.2012 02:13 # −1

      Никого не смущает вывод с тернарным оператором, где второй веткой является пустая строка?
      Ок, буду теперь так делать :)
      Ответить

      • ava inkanus-gray 13.05.2012 11:16 # 0

        А как ещё, если это в echo завёрнуто? Зато функциональненько!
        Предложите заменить банальным ифом?
        Ответить

        • ava kindofbear 13.05.2012 11:49 # −1

          Да, банальным if. Чтобы было читабельно
          Ответить

          • ava inkanus-gray 13.05.2012 17:12 # 0

            Тогда придётся отказаться от клёвых скобочек <?= ?>, писать слова «if» и «echo». А что делать, если программист неграмотный и букв не знает?

            Можно и без echo, вложив <?= ?> в if, но это будет ещё длиннее.
            Ответить

            • ava kindofbear 14.05.2012 09:44 # −3

              или хотя бы так
              <? 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>
<? } ?>
              Ответить

      • ava 7ion 13.05.2012 18:22 # +1

        Без второй ветки похапе это не выполнит то ли до 5.2.0, то ли до 5.3.0.
        Ответить

    • ava HaskellGovno 13.05.2012 11:31 # −2

      .Твой вброс упал.
      Ответить

    • ava kindofbear 13.05.2012 11:50 # 0

      Ужас то какой, как теперь жить) Кстати код реальный, из сайта, где заказали сменить дизайн
      Ответить

    • ava oooZinka 14.05.2012 15:19 # 0

      Код прошел фейсконтроль, если ты не можешь написать или использовать нормальный шаблонизатор, чтобы отделить серверный код от клиентского, то это лучшая альтернатива.
      Ответить

      • ava kindofbear 14.05.2012 17:12 # −1

        окай. давайте еще условия типа
        if (empty($var)) { } else { echo 'Переменная не пуста!'; }
        я не спорю, просто меняю шаблоны)
        Ответить

    • ava Anderson 14.05.2012 18:10 # +1

      Не особо говно. Хотя использовал бы тэмплейтор - было бы красивее.
      Ответить

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