PHP|変数を使ったfunctionと繰り返しについて

Question

PHPを勉強中です。
こちらで何度か質問したり色々参考にしてディレクトリ内の最新画像とテキストを表示させるものを作ったのですが、現時点では3回同じようなことを書いてます。これをforeachやwhileを使って3回繰り返すプログラムに書き変えたいのですがどのように書けばいいでしょうか。

やりたいこと
コードの簡略化

functionに変数使っていいのか、複数のforeachを使えるのか、そもそもこの変数の使い方って合っているのかどうか、検索してもなかなか思うような答えにつなげることができませんでした。もし違う書き方があれば教えてもらえると助かります。
※変数名や階層は少し変えてるのでどこか間違ってたらすみません

//テキスト
$text1 = file_get_contents("../text1.txt");//テキスト表示
$text2 = file_get_contents("../text2.txt");
$text3 = file_get_contents("../text3.txt");

//画像
function display_img($path){//「display_img」関数定義
$path = "img1/";
$dir = __DIR__ .$path;

$filesarray = array();//ディレクトリを開いて中身をソート
	if ($handle = opendir($dir)) {
		while (false !== ($file = readdir($handle))) {
			if ($file != "." && $file != "..") {
				$filesarray[filemtime( $dir.$file )] = $file;
			}
		}
		closedir($handle);
	}
	ksort($filesarray);
	if ($handle = opendir($dir)) { //画像表示
		foreach($filesarray as $file){
			global $landscapeimg;
			$landscapeimg = "<img src='".$dir.$file."' width='100%' />";
		}
		closedir($handle);
	}
}

<div>
	<?php
		display_img($dir,$path);//display_img実行
		echo $displayimg;//img1最新画像を表示 ?>
	<p><?php echo h($text1);//text1表示 ?></p>
</div>
<div>
	<?php
		$path = "img2/";//パスを変更
		$dir = __DIR__ .$path;
		display_img($dir,$path)//display_img実行;
	?>
	<?php echo $displayimg;//img2最新画像を表示 ?>
	<p><?php echo h($text2);//text2表示 ?></p>
</div>
<div>
	<?php
		$path = "img3/";//パスを変更
		$dir = __DIR__ .$path;
		display_img($dir,$path)//display_img実行;
	?>
	<?php echo $displayimg;//img3最新画像を表示 ?>
	<p><?php echo h($text3);//text3表示 ?></p>
</div>

Answer 1

foreachなのであれば配列でファイル名のリストを持てばできますし、
whileよりもループカウンタ持ってるforの方が良いでしょう。

※ヒントプログラムなので参考に自身でやってみてください。

forでの例

for($i=1;$i<=3;$i++){
  $text = file_get_contents("../text{$i}.txt");
  $path = "img{$i}/";
  echo <<<EOF
<div id="area{$i}">
text:{$text}<br />
path:{$path}
</div>
EOF;
}

• for
• ヒアドキュメント構文

display_img()に$path渡されていますが、その関数冒頭でimg1にpathを上書きしているので引数の意味がなくなってます。

Comments

[hati3693]

参考にしてやってみたところうまくいきました！
<?php
for($i=1;$i<=3;$i++){
$text = file_get_contents("../text/text{$i}.txt");
$path = "img{$i}/";
$dir = "../{$path}";
display_img($dir,$path);
echo <<<EOF
<div>//繰り返し
$displayimg//画像
{$text}<br>//テキスト
</div>
EOF;
}//繰り返しEND
?>
forの使い道がいまいちわかっていなかったのですが、すごく便利だと知りました。
大変参考になりました！ありがとうございます。

[m.ts10806]

なんでもかんでもforではなく適材適所使い分けてくださいね。
どれでも同じようにはできますので。