セレクトメニューでファイルの一部分を表示させたい

質問

こんばんは、初めまして。初心者です。
今回、セレクトメニューの部分にファイル（allcategory)の中の
Ccategoryという値を入れたいのですが、
さっぱり分からなくって。
他の部分は、別のファイル（allpage)のファイルの値が入ります。
また、for($i=0;$lines[$i];$i++) の部分で
カテゴリー名が入らないのです。

どうやってもうまくいかないので教えてください。
よろしくお願いします。
（たくさんスクリプトを入れてすみません。）
function contents() {
$id = $_GET[id];
$lines = @file($this->allpage);
$lines = @file($this->allcategory);

for($i=0;$lines[$i];$i++) {
list($Pid,$Ptitle,$Psetsu) = explode("<>",$lines[$i]);
if($Pid == $id) {
$ePid = $Pid;
$ePtitle = $Ptitle;
$ePsetsu = $Psetsu;}
for($i=0;$lines[$i];$i++) {
list($Cid,$Ccategory) = explode("<>",$lines[$i]);
if($Pid == $id) {
$eCid = $Cid;
$eCcategory = $Ccategory;
}
if($lines[0]) {
$contents .= "<tr bgcolor=\"#cccccc\"><td>カテゴリー名</td><td>ID</td><td>ページ名</td><td>簡易説明</td><td>処理</td></tr>\n";
$contents .= "</table>\n";
}
$contents .= "<br />\n";
$contents .= "<br />\n";
if($id) {
$pt1 = "編集";
$e_lines = file("$this->pagedir$ePid.cgi");
for($e=0;$e_lines[$e];$e++) {
$ev .= "$e_lines[$e]";
}
$ev = str_replace("&","&amp;",$ev);
$e_lines = file("$this->allcategory");
for($e=0;$e_lines[$e];$e++) {
$cv .= "$e_category[$e]";
}
$cv = str_replace("&","&amp;",$ev);

$contents .= "<tr bgcolor=\"#cccccc\"><td width=\"15%\">カテゴリーの選択</td><td bgcolor=\"#ffffff\"><SELECT name=\"Ccategory\">\n";
for($i=0;$i<count($lines);$i++){
$contents .= "<OPTION
value=\"$i\">\"$Ccategory[$i]\"</OPTION>}\n";
$contents .= "</SELECT></td></tr>\n";

回答No.1

utano_m · ベストアンサー

全体のソースがないのでなんともいえませんが
for文をネストしていますがfor文の判定式に同じ$iという
変数を使うのはまずいです。
最初のfor文で$iが増加して1になっても次のfor文で0に($i=0という式があるため)なると思われます。
まずはそこを修正してはいかがでしょう？

蛇足ですが
$contents .= "<tr bgcolor=\"#cccccc\"><td>カテゴリー名</td><td>ID</td><td>ページ名</td><td>簡易説明</td><td>処理</td></tr>\n";
$contents .= "</table>\n";
とするよりは

$contents .= <<<HERE
　<tr bgcolor="#cccccc">
　　<td>カテゴリー名</td>
　　<td>ID</td>
　　<td>ページ名</td>
　　<td>簡易説明</td>
　　<td>処理</td>
　</tr>
</table>
HERE;
のようなヒア構文を使うことでソースが(生成されるhtml文も)見やすくなるかと思われます。

補足

有難うございます。なるほど～と思いながら
直しました。有難うございます。
$i=0の部分も直しました。
それで、allcategoryのファイルの中に保存してるものは、
cidとcategoryなのですが、
categoryの文字列をこのセレクトメニューに入れたいんですが。
allcategoryのファイルにcid<>category<>と書き込まれてます。
よろしくお願いします。