もっとシンプルに書きたいのですが・・・

以下の内容をもっとシンプルに書きたいのですが思い浮かびません。
どなたかアドバイスお願い致します。

if(!(empty($test1))){

if(!(empty($test2))){

if(!(empty($test3))){
echo '<img src="'.$test1.'"><img src="'.$test2.'"><img src="'.$test3.'"><br>';
}
else{
echo '<img src="'.$test1.'"><img src="'.$test2.'"><br>';
}

}else{
echo '<img src="'.$test1.'"><br>';
}

}else{
echo '';
}

No.1 ベストアンサー 回答者： yambejp 回答日時： 2009/12/22 17:59

たとえばこんな感じで？

<?
$str='';
$str=!empty($test3)?'<img src="'.$test3.'">':'';
$str=!empty($test2)?'<img src="'.$test2.'">'.$str:'';
$str=!empty($test1)?'<img src="'.$test1.'">'.$str:'';
print $str;
?>

この回答へのお礼

どうもありがとうございました。

お礼日時：2009/12/22 23:51

No.4 回答者： hogehoge78 回答日時： 2009/12/22 20:15

yambejpさん、ご指摘/修正ありがとうございます。

要件を完全に見落としてました。

質問者さんごめんなさい。

この回答へのお礼

とんでもないです。
こちらこそ他のやり方をおしえてくださってありがとうございました。

お礼日時：2009/12/22 23:50

No.3 回答者： yambejp 回答日時： 2009/12/22 19:38

あ、<br>がいるんですね・・・

#２さんは若干フローが違うのでは？
$test1が空なら、$test2や$test3にデータがはいっていても
表示したくないんですよね？
であればbreakしてやらないといけないでしょう

<?php
$result = '';
foreach(array($test1, $test2, $test3) as $t){
if(!empty($t)) $result .= sprintf('<img src="%s">', $t);
else break;
}
if(!empty($result)) $result .= '<br />';
echo $result;
?>

No.2 回答者： hogehoge78 回答日時： 2009/12/22 19:07

<?php

$result = '';
foreach(array($test1, $test2, $test3) as $t){
if(!empty($t)) $result .= sprintf('<img src="%s">', $t);
}
if(!empty($result)) $result .= '<br />';
echo $result;
?>

こんな書き方も。