PHPで似たコードを繰り返す場合の簡潔な書き方？？

質問

ご質問失礼します。

PHPでデータベースのデータを下記のようなコードを使い表示しています。
※質問内容に該当する一部のみ記載させていただいています

・
・
・
・
・
if(preg_match("/新規/", $row['status1'])){
$table .= $row['status1']."　新規です";
} else {
if(preg_match("/更新/", $row['status1'])){
$table .= $row['status1']."　更新です";
} else {
if(preg_match("/削除/", $row['status1'])){
$table .= $row['status1']."　削除です";
} else {
$table .= "いません";
}
}
}

if(preg_match("/新規/", $row['status2'])){
$table .= $row['status2']."　新規です";
} else {
if(preg_match("/更新/", $row['status2'])){
$table .= $row['status2']."　更新です";
} else {
if(preg_match("/削除/", $row['status2'])){
$table .= $row['status2']."　削除です";
} else {
$table .= "いません";
}
}
}

if(preg_match("/新規/", $row['status3'])){
$table .= $row['status3']."　新規です";
} else {
if(preg_match("/更新/", $row['status3'])){
$table .= $row['status3']."　更新です";
} else {
・
・
・
・
・
・

status1から、5つあればstatus5まで、似たようなPHPを繰り返し書いて動作させています。

このような場合、簡潔に書く方法などありますでしょうか。

ご存知の方いらっしゃいましたらお時間のある際にお教え下さいませんでしょうか。

また、簡潔に書く方法がない場合でも無いとお教えいただければ幸いです(○ ｀人´ ○)

回答No.2

mpro-gram · ベストアンサー

pregの正規表現用の文字列と出力文字とが対応してるなら、それらを配列に入れて、さらにチェックされる側も複数なので、２重ループで回せば？
固定文字の有無を調べてるのなら、pregより、strpos() の方が速いですけど
<?php
$needles = array('新規','更新','削除');
for( $i=1; $i<=5; $i++ ){
$haystack = $row[ 'status'.$i ];
$chk = 0;
foreach( $needles as $nee ){
if( strpos($haystack, $nee) !== false ){
$table .= $haystack.' '. $nee . 'です。';
$chk ++;
break;
}
}
if( $chk<1 ){
$table .= $haystack.' いずれでもありません。';
}
}

お礼

ご回答ありがとうございます！！

>>　for( $i=1; $i<=5; $i++ ){

for文というのでしょうか、使いこなせてなくてこのような使い方すればよいのですね・・・

勉強になりました、無事できました！！

回答No.4

JaneDue

Ano3 間違いすみません。 $row は status1～5だけとは限りませんでしたね。
foreach($row ～ )の部分はANo2さんのように forにして番号を付加したり、あるいは別の配列を用意して　
foreach( array($row['status1'],$row['status2'] ～) 　と必要な部分のみ回るようにする必要があります。

お礼

追記もありがとうございます参考になりました！！

回答No.3

JaneDue

検索語と、それぞれに対応させる語句を配列にしておいて、後方参照で取り出す方法もあります。

$texts = array("新規"=>"　新規です","更新"=>"　更新です","削除"=>"　削除です");
$table = '';
foreach($row as $val){
$table .= preg_match("/新規|更新|削除/",$val,$match) ? $val.$texts[$match[0]]: "いません";
}
echo $table;

? : については三項演算子を参照下さい。
http://php.net/manual/ja/language.operators.comparison.php
また、if/else でやる場合は、if(){}～else{ if(){}} と入れ子にしなくても elseif があります。
http://jp2.php.net/manual/ja/control-structures.elseif.php

お礼

ご回答ありがとうございます！！

すっごくまとまるのですね・・・参考になります！！

また　elseif　についてなのですが、こちらも補足ありがとうございました！！

他所にも多量に　if(){}～else{ if(){}}　使ってましたので
早速活用することができました、ありがとうございます！！

回答No.1

piraippi

単純statusを受けとって、「新規です」とか「更新です」とかを返すファンクションをつくればいいのではないですか？

お礼

お早いご回答ありがとうございます！！

やはり私は、単純なものをダラダラ書いてたんですね・・・

functionですね、これは勉強不足でした、活用してみます！！