Говнокод

<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.05.2012 (Updated 25.03.2018)

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

Lure Of Chaos 14 лет ago # 0

->getImaget
Картиньетка
Ответить
- kindofbear 14 лет ago # 0 ⇈
  
  Ну там есть функция просто getImage... а тут видимо подразумевалось получение thumbnail
  Ответить
kindofbear 14 лет ago # 0

Никого не смущает вывод с тернарным оператором, где второй веткой является пустая строка?
Ок, буду теперь так делать 🙂
Ответить
- inkanus-gray 14 лет ago # 0 ⇈
  
  А как ещё, если это в echo завёрнуто? Зато функциональненько!
  Предложите заменить банальным ифом?
  Ответить
  - kindofbear 14 лет ago # 0 ⇈
    
    Да, банальным if. Чтобы было читабельно
    Ответить
    - inkanus-gray 14 лет ago # 0 ⇈
      
      Тогда придётся отказаться от клёвых скобочек <?= ?>, писать слова «if» и «echo». А что делать, если программист неграмотный и букв не знает?
      
      Можно и без echo, вложив <?= ?> в if, но это будет ещё длиннее.
      Ответить
      - kindofbear 14 лет ago # 0 ⇈
        
        или хотя бы так
        
        <? 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> <? } ?>
        
        Ответить
- 7ion 14 лет ago # 0 ⇈
  
  Без второй ветки похапе это не выполнит то ли до 5.2.0, то ли до 5.3.0.
  Ответить
  - kindofbear 14 лет ago # 0 ⇈
    
    Спасибо, дарагой, поэтому я и говорю про if
    Ответить
HaskellGovno 14 лет ago # 0

.Твой вброс упал.
Ответить
kindofbear 14 лет ago # 0

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

Код прошел фейсконтроль, если ты не можешь написать или использовать нормальный шаблонизатор, чтобы отделить серверный код от клиентского, то это лучшая альтернатива.
Ответить
- kindofbear 14 лет ago # 0 ⇈
  
  окай. давайте еще условия типа
  if (empty($var)) { } else { echo 'Переменная не пуста!'; }
  я не спорю, просто меняю шаблоны)
  Ответить
Anderson 14 лет ago # 0

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