Как можно оптимизировать код?

Question

[Rasul Gudantov]

Здравствуйте. Написал небольшой скриптик и он уж совсем смотрится убого. Можете посоветовать какими способами можно код сделать более читабельным?

echo '<div class="currency"><ul>';

for ($i=0; $i < $count_obj ; $i++) { 

	if($obj[$i]['percent_change_24h'] > 0) {
		$color_change = '#4ac06a';
		$plus = '+';
	} else {
		$color_change = '#ff8d8d';
		$plus = '';
	}
	echo '<li>';
	echo '<i class="cc '.$obj[$i]["symbol"].' iconsi" title="'.$obj[$i]["name"].'"></i>';
	echo '<div class="block_coin">';
	echo '<span class="coin_name">'.$obj[$i]["name"].'</span><span style="color:'.$color_change.';" class="coin_price">'.$obj[$i]["price_usd"].'$</span>';
	echo '<span style="color:'.$color_change.';" class="coin_change">(';
	echo $plus.$obj[$i]["percent_change_24h"];
	echo '%)</span></div></li>';
}
echo '</ul></div>';

Answer 1

[Андрей Николаев]

Если я все правильно понял и нигде не ошибся, то можно попробовать и так:

<div class="currency">
  <ul>
    <? foreach ($obj as $element):?>
      <? $color = ($element['percent_change_24h'] > 0) '#4ac06a' : '#ff8d8d'; ?>
      <? $plus = ($element['percent_change_24h'] > 0) '+' : ''; ?>
      <li>
        <i class="cc <?=$element["symbol"];?> iconsi" title="<?=$element["name"];?>"></i>
        <div class="block_coin">
          <span class="coin_name"><?=$element["name"];?></span>
          <span style="color: '<?=$color;?>';" class="coin_price"><?=$element['price_usd'];?></span>
          <span style="color: '<?=$color;?>';" class="coin_change">(<?=$plus;?><?=$element["percent_change_24h"];?>%)</span>
        </div>
      </li>
    <? endforeach; ?>
  </ul>
</div>

Comments

[xzdshr]

Этот вариант получше верхнего будет. Осталось только отступы согласно PSR привести и слишком длинные строки со span разбить на отдельные строки с самими тегами и их содержимым. Тогда читабельность повысится еще больше.
Касательно использования шорт тегов, имхо на усмотрение топикстартера.

[Андрей Николаев]

xzdshr, да тут все очень субьективно. Например при проектировании контроллеров/моделей (т.е. непосредственно там где код), стараюсь выдерживать ровные строки, а вот во вьюхах понять и принять этого не могу.

По факту можно и еще лучше сделать:

1) Объявить css-класс change24h для всего тега
При появлении этого класса уже красить все остальное в нужные цвета
Таким образом мы избавимся от inline-стилей

2) Плюсик и скобки тоже можно выводить через css (раз у нас есть класс change24h, значит на нужный тег можно подвесить :before и :after селекторы и добавить уже нужное)

Тогда вообще html-часть превратиться в нечто вида:

<div class="currency">
  <ul>
    <? foreach ($obj as $element):?>
      <li <?=($element['percent_change_24h'] > 0)? 'change24h' : '';?>>
        <i class="cc <?=$element["symbol"];?> iconsi" title="<?=$element["name"];?>"></i>
        <div class="block_coin">
          <span class="coin_name"><?=$element["name"];?></span>
          <span class="coin_price"><?=$element['price_usd'];?></span>
          <span class="coin_change"><?=$element["percent_change_24h"];?></span>
        </div>
      </li>
    <? endforeach; ?>
  </ul>
</div>

Answer 2

[Евгений]

примерно как-то так

<div class="currency">
	<ul>
		<?php
		for ( $i = 0; $i < $count_obj; $i++ ) {

			if ( $obj[ $i ][ 'percent_change_24h' ] > 0 ) {
				$color_change = '#4ac06a';
				$plus = '+';
			} else {
				$color_change = '#ff8d8d';
				$plus = '';
			}
			?>
			<li><i class="cc <?=$obj[$i][" symbol "]?> iconsi" title="<?=$obj[$i][" name "]?>"></i>
				<div class="block_coin">
					<span class="coin_name">
						<?=$obj[$i]["name"]?>
					</span>
					<span style="color:<?=$color_change?>;" class="coin_price">
						<?=$obj[$i]["price_usd"]?>$</span>
					<span style="color:<?=$color_change?>;" class="coin_change">(<?=$plus.$obj[$i]["percent_change_24h"]?>%)</span>
				</div>
			</li>
		<?php
		}
		?>
	</ul>
</div>

Comments

[Rasul Gudantov]

Спасибо! Более приятный вид)