冗長なソースをコンパクトにしたい

質問

冗長なソースをコンパクトにしたい

お世話になります。
CakePHP＋MySqlでサイトを構築しているのですが、以下のソースが冗長なので、もう少しコンパクトにできないものかと悩んでいます。

このソースは「hogehoge」テーブルにあるデータ（0～100までの数値）を各列ごとに書き出しているものです。
出力イメージはこのようになります（テーブルもほぼこんな感じです）。
　　　　jan　feb　mar　～
項目A　　1　28　20　～
項目B　58　78　40　～
項目C　～
……

また、たんにデータを出力するだけでなく、値によってセルの背景色を変えたかったので、値を1つずつチェックし、それに応じたCSSクラス（pat1～）を変数$colorに代入しています。
<table>
<?php
for($i = 0;$i < count($data);$i++){
$arr = $data[$i]['hogehoge（テーブル名）'];
echo "<tr><td>{$arr['hoge（項目名）']}</td>";
　if($arr['jan'] < 10 ){
　　$color = "pat1";
　}elseif(9 < $arr['jan'] && $arr['jan'] < 20 ){
　　$color = "pat2";
　}elseif(19 < $arr['jan'] && $arr['jan'] < 30 ){
　　$color = "pat3";
　}else{
　　$color = "pat4";
　}
　echo "<td class =".$color.">{$arr['jan']}</td>";

　if($arr['feb'] < 10 ){
　　$color = "pat1";
　}elseif(9 < $arr['feb'] && $arr['feb'] < 20 ){
　　$color = "pat2";
　}elseif(19 < $arr['feb'] && $arr['feb'] < 30 ){
　　$color = "pat3";
　}else{
　　$color = "pat4";
　}
　echo "<td class =".$color.">{$arr['feb']}</td>";

　if($arr['mar'] < 10 ){
　　$color = "pat1";
　}elseif(9 < $arr['mar'] && $arr['mar'] < 20 ){
　　$color = "pat2";
　}elseif(19 < $arr['mar'] && $arr['mar'] < 30 ){
　　$color = "pat3";
　}else{
　　$color = "pat4";
　}
　echo "<td class =".$color.">{$arr['mar']}</td>";

　※これが12月（dec）まで延々と続きます……
}
?>
</table>

私の頭とスキルではコンパクトにできなかったので、どなたかお知恵を拝借できますでしょうか。

そもそもこの方法が正しいのかどうかも分かりません。もし、もっとスマートなやり方をご存じの方はぜひご教授ください。

よろしくお願いいたします。

回答No.2

php504 · ベストアンサー

こうですかね

$month = array('jan', 'feb', 'mar', 'apr', 'may', 'jun', 'jul', 'aug', 'sep', 'oct', 'nov', 'dec');
for ($k = 0; $k < 12; $k++)
{
if($arr[ $month[$k] ] < 10 ){
$color = "pat1";
}elseif($arr[ $month[$k] ] < 20 ){
$color = "pat2";
}elseif($arr[ $month[$k] ] < 30 ){
$color = "pat3";
}else{
$color = "pat4";
}
echo "<td class =".$color.">{$arr[ $month[$k] ]}</td>";
}

お礼

ご回答ありがとうございます。
おかげさまで小さくなりました。

見れば「なるほど」と理解できるんですが、
いざ自分で組むとなるとなかなかできなくて困ったものです。

回答No.1

cotae_bb

こんにちは

PHPをやったことがほとんどないため、正しい方法というものはわかりませんが、
月名を配列にいれておいて、それをループで回して処理をすれば、
単純計算で1/12のソース量になるのでは？

お礼

早速のご回答ありがとうございます。
おかげさまで小さくなりました。

見れば「なるほど」と理解できるんですが、
いざ自分で組むとなるとなかなかできなくて困ったものです。