次の配列のコードをもっと簡潔に書きたいのですが、どなたかご教授願えないでしょうか。

Question

はじめまして。

for文を用いて以下のようにコードを打ったのですが、簡潔にしようとするとエラーが発生します。どなたかご指摘お願いできますでしょうか。
よろしくお願いします。

該当のソースコード

 <?php
$pdata["code"]["A0001"][0] = "A0001";
$pdata["code"]["A0001"][1] = "白菜（1玉）";
$pdata["code"]["A0001"][2]= 298;
$pdata["code"]["K0001"][0] = "K0001";
$pdata["code"]["K0001"][1] = "いわし（5尾）";
$pdata["code"]["K0001"][2] = 258;
$pdata["code"]["A0002"][0] = "A0002";
$pdata["code"]["A0002"][1] = "九条葱（1パック）";
$pdata["code"]["A0002"][2] = 258;
$pdata["code"]["A0003"][0] = "A0003";
$pdata["code"]["A0003"][1] = "サツマイモ（1袋）";
$pdata["code"]["A0003"][2] = 180;
$pdata["code"]["K0002"][0] = "K0002";
$pdata["code"]["K0002"][1] = "きびなご（１皿）";
$pdata["code"]["K0002"][2] = 180;

echo "<table>";
echo "
<tr>
<td>code</td>
<td>product</td>
<td>price</td>
</tr>
";

for($i=0;$i<count($pdata);$i++){

echo "
<tr>
<td>".$pdata["code"]["A0001"][0]."</td>
<td>".$pdata["code"]["A0001"][1]."</td>
<td>".$pdata["code"]["A0001"][2]."</td>
</tr>
";
}
"</table>";

for($i=0;$i<count($pdata);$i++){

echo "
<tr>
<td>".$pdata["code"]["K0001"][0]."</td>
<td>".$pdata["code"]["K0001"][1]."</td>
<td>".$pdata["code"]["K0001"][2]."</td>
</tr>
";
}
"</table>";

for($i=0;$i<count($pdata);$i++){

echo "
<tr>
<td>".$pdata["code"]["A0002"][0]."</td>
<td>".$pdata["code"]["A0002"][1]."</td>
<td>".$pdata["code"]["A0002"][2]."</td>
</tr>
";
}
"</table>";

for($i=0;$i<count($pdata);$i++){

echo "
<tr>
<td>".$pdata["code"]["A0003"][0]."</td>
<td>".$pdata["code"]["A0003"][1]."</td>
<td>".$pdata["code"]["A0003"][2]."</td>
</tr>
";
}
"</table>";

for($i=0;$i<count($pdata);$i++){

echo "
<tr>
<td>".$pdata["code"]["K0002"][0]."</td>
<td>".$pdata["code"]["K0002"][1]."</td>
<td>".$pdata["code"]["K0002"][2]."</td>
</tr>
";
}
"</table>";
?>

Comments

[mather]

「簡潔にしようとする」とはどう簡潔にしたいと考えているのか書いてください。「エラーが発生します」のエラーを具体的に提示してください。

[motuo]

手元で動かすと動いてしまいました。何をしようとして、どんなエラーが出たのでしょうか？エラーが出たコードとメッセージをセットにして質問すると回答が得られやすいと思いますが…

[m.ts10806]

同じ人？　https://teratail.com/questions/128825 内容とタイミングが露骨過ぎます。

[退会済みユーザー]

せっかく丁寧に回答をつけても、退会してしまうんですね？

[ttyp03]

普通に動きますなあ。動きの良し悪しは見てないけど、少なくともエラーにはならない。

[imprezza]

なんとも…こう…配列の持ち方にモヤモヤしたものを感じるんですよねぇ…後で自分で消化独習してみよう

Answer 1

スマートな回答が既に出ているところで敢えて回答をつけます。
理解の助けになるように。

<?php

$pdata["code"]["A0001"][0] = "A0001";
$pdata["code"]["A0001"][1] = "白菜（1玉）";
$pdata["code"]["A0001"][2]= 298;
$pdata["code"]["K0001"][0] = "K0001";
$pdata["code"]["K0001"][1] = "いわし（5尾）";
$pdata["code"]["K0001"][2] = 258;
$pdata["code"]["A0002"][0] = "A0002";
$pdata["code"]["A0002"][1] = "九条葱（1パック）";
$pdata["code"]["A0002"][2] = 258;
$pdata["code"]["A0003"][0] = "A0003";
$pdata["code"]["A0003"][1] = "サツマイモ（1袋）";
$pdata["code"]["A0003"][2] = 180;
$pdata["code"]["K0002"][0] = "K0002";
$pdata["code"]["K0002"][1] = "きびなご（１皿）";
$pdata["code"]["K0002"][2] = 180;

echo count($pdata) . PHP_EOL;
echo count($pdata["code"]) . PHP_EOL;
foreach ($pdata["code"] as $arr) {
    echo $arr[0] . PHP_EOL;
    echo $arr[1] . PHP_EOL;
    echo $arr[2] . PHP_EOL;
}

《実行結果》

$pdataのキーはcodeしかないので、count($pdata)は1になります。
$pdata["code"]のキーは定義した分あるので、count($pdata["code"])はこの場合5になります。

for()ループよりもforeach()ループのほうが、
カウンター変数を管理しなくて良く、
配列を順になぞって処理を行うのにラクです。

複数行出力するecho文はヒアドキュメント構文を使ったほうが見やすいです。
以上を踏まえて、

<?php

$pdata["code"]["A0001"][0] = "A0001";
$pdata["code"]["A0001"][1] = "白菜（1玉）";
$pdata["code"]["A0001"][2]= 298;
$pdata["code"]["K0001"][0] = "K0001";
$pdata["code"]["K0001"][1] = "いわし（5尾）";
$pdata["code"]["K0001"][2] = 258;
$pdata["code"]["A0002"][0] = "A0002";
$pdata["code"]["A0002"][1] = "九条葱（1パック）";
$pdata["code"]["A0002"][2] = 258;
$pdata["code"]["A0003"][0] = "A0003";
$pdata["code"]["A0003"][1] = "サツマイモ（1袋）";
$pdata["code"]["A0003"][2] = 180;
$pdata["code"]["K0002"][0] = "K0002";
$pdata["code"]["K0002"][1] = "きびなご（１皿）";
$pdata["code"]["K0002"][2] = 180;

echo <<<EOT
<table>

EOT;
foreach ($pdata["code"] as $arr) {
    echo <<<EOT
<tr>
  <td>${arr[0]}</td>
  <td>${arr[1]}</td>
  <td>${arr[2]}</td>
</tr>

EOT;
}
echo <<<EOT
</table>

EOT;

《実行結果》

また、phpからの離脱を組み合わせることも出来ます。

<?php

$pdata["code"]["A0001"][0] = "A0001";
$pdata["code"]["A0001"][1] = "白菜（1玉）";
$pdata["code"]["A0001"][2]= 298;
$pdata["code"]["K0001"][0] = "K0001";
$pdata["code"]["K0001"][1] = "いわし（5尾）";
$pdata["code"]["K0001"][2] = 258;
$pdata["code"]["A0002"][0] = "A0002";
$pdata["code"]["A0002"][1] = "九条葱（1パック）";
$pdata["code"]["A0002"][2] = 258;
$pdata["code"]["A0003"][0] = "A0003";
$pdata["code"]["A0003"][1] = "サツマイモ（1袋）";
$pdata["code"]["A0003"][2] = 180;
$pdata["code"]["K0002"][0] = "K0002";
$pdata["code"]["K0002"][1] = "きびなご（１皿）";
$pdata["code"]["K0002"][2] = 180;

?>
<table>
<?php
foreach ($pdata["code"] as $arr) {
    echo <<<EOT
<tr>
  <td>${arr[0]}</td>
  <td>${arr[1]}</td>
  <td>${arr[2]}</td>
</tr>

EOT;
}
?>
</table>

「</tr>」と「EOT;」との間に空の行があるのは、
それがないと改行コードがつかなくて、「</tr>」と次の「<tr>」が同じ行に出力されてしまうからです。

加えて、単純に0,1,2の順に出力するだけであれば、
それこそfor()ループを使うこともできますが、
たかだか３件だしコードが増えて見た目のシンプルさが損なわれるから今回はなしで。

Answer 2

たとえばこう？

$pdata["code"]["A0001"][0] = "A0001";
$pdata["code"]["A0001"][1] = "白菜（1玉）";
$pdata["code"]["A0001"][2]= 298;
$pdata["code"]["K0001"][0] = "K0001";
$pdata["code"]["K0001"][1] = "いわし（5尾）";
$pdata["code"]["K0001"][2] = 258;
$pdata["code"]["A0002"][0] = "A0002";
$pdata["code"]["A0002"][1] = "九条葱（1パック）";
$pdata["code"]["A0002"][2] = 258;
$pdata["code"]["A0003"][0] = "A0003";
$pdata["code"]["A0003"][1] = "サツマイモ（1袋）";
$pdata["code"]["A0003"][2] = 180;
$pdata["code"]["K0002"][0] = "K0002";
$pdata["code"]["K0002"][1] = "きびなご（１皿）";
$pdata["code"]["K0002"][2] = 180;

echo <<<eof
<table>
<tr>
<td>code</td>
<td>product</td>
<td>price</td>
</tr>

eof;
foreach($pdata["code"] as $row){
echo <<<eof
<tr>
  <td>{$row[0]}</td>
  <td>{$row[1]}</td>
  <td>{$row[2]}</td>
</tr>

eof;
};
echo "</table>";