Grarnik
@Grarnik
web-мастер

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

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

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>';
  • Вопрос задан
  • 104 просмотра
Пригласить эксперта
Ответы на вопрос 2
@Eugeny1987
Работаю с HostCMS
примерно как-то так
<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>
Ответ написан
gromdron
@gromdron
Bitrix developer
Если я все правильно понял и нигде не ошибся, то можно попробовать и так:

<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>
Ответ написан
Ваш ответ на вопрос

Войдите, чтобы написать ответ

Войти через TM ID
Похожие вопросы
от 90 000 до 150 000 руб.
Reklama Guru Санкт-Петербург
от 50 000 до 60 000 руб.
Sveak Барнаул
от 40 000 руб.
15 авг. 2018, в 23:38
300 руб./за проект
15 авг. 2018, в 22:39
60000 руб./за проект
15 авг. 2018, в 21:18
30000 руб./за проект