PHP（HTML）でのエラーについて

前提・実現したいこと

一気に書き込める機能を実装中に以下のエラーメッセージが発生しました。

発生している問題・エラーメッセージ

Parse error: syntax error, unexpected end of file in /home/cfabond/cfabond.starfree.jp/public_html/rentaru/class/j21/0710/number.php on line 61

該当のソースコード

html
1<?php 
2$list = array( 
3array('to'=>'46.txt', 'from'=>'23.txt', 'name'=>'inp_01', 'disp'=>'クーピー', 'qty'=>0), 
4); 
5// 取得した値を保管する変数を初期化 
6$values = array(); // これ追加します 
7foreach($list as $key=>$item) { 
8// オプションを指定すれば入力範囲の制限も可能 
9$options=array('options'=>array('min_range'=>0, 'max_range'=>9)); 
10// 制限範囲内の整数を取得する 
11$value = filter_input(INPUT_POST, $item['name'], FILTER_VALIDATE_INT, $options); 
12if (($value === NULL) or ($value === FALSE)) { 
13// 入力欄が無かった場合、 
14// 数値でなかった場合、制限範囲外の場合は以降をスキップ 
15continue; 
16} 
17// スキップされていない＝数値が入力された 
18// $item や $value を使って $msg を生成する  
19// $item や $value を使ってファイルに書き出す 
20} 
21// スキップされていない＝数値が入力された 
22$values[$key] = $value; // これを追加します 
23// $field を生成する 
24$field = ''; 
25foreach ($list as $key=>$item) { 
26if ($item['qty'] == 0) { 
27// 不要な入力欄を作らない為にスキップ 
28continue; 
29} 
30$field .= <<<HTML
31<div class="field"> 
32<div class="item"> 
33<span class="name">{$item['disp']}</span>：<span class="number">{$item['qty']}</span> 
34</div> 
35<div class="inputField"> 
36<input type="text" name="{$item['name']}" value="{$values[$key]}"> 
37</div> 
38HTML; 
39// ここで htmlspecialchars() を使っていないのは 
40// 自分で設定した値、及びフィルターを通して 
41// 整数である事が確定済みで、無害であると判断したため 
42// 最初に「数値である事を」と書いたのはその為  
43?> 
44<!DOCTYPE html> 
45<html lang="ja"> 
46<head> 
47<meta charset="UTF-8"> 
48<title>必要な物を</title> 
49</head> 
50<body> 
51<?= count($msg) > 0 ? implode('', $msg) . "備品のナンバーを書き込みました。 <hr>" 
52'' ?>  
53<?=$msg;?> 
54<div class="form"> 
55<form action="number.php" method="post"> 
56<?=$filed:?> 
57<input type="submit" name="btnSubmit" value="送信"> 
58</form> 
59</div> 
60</body> 
61</html>

試したこと

{ } ( ) " " ' ' などは一応チェックしました。

kei344

2019/09/09 12:38

PHPのエラーです。質問タグとタイトルを修正してください。

daichi-onoue

2019/09/09 12:39

修正しました。

m.ts10806

2019/09/09 19:01

コードインデントをしっかりつけてください。

行動規範の内容に同意します

回答2件

foreach閉じてない

【追記】
私のやった確認方法を添えておきます。
まず、3v4lを開きます。
https://3v4l.org/
オンラインのphp実行環境です。
ここではブロック要素の{～}の対が分かる表示機能があります。
それを確認しながら、提示のソースをインデントしていくと、
後半のforeachループの終わりがわからないようになっています。

php
1<?php
2
3$list = array( 
4array('to'=>'46.txt', 'from'=>'23.txt', 'name'=>'inp_01', 'disp'=>'クーピー', 'qty'=>0), 
5); 
6// 取得した値を保管する変数を初期化 
7$values = array(); // これ追加します 
8foreach($list as $key=>$item) { 
9    // オプションを指定すれば入力範囲の制限も可能 
10    $options=array('options'=>array('min_range'=>0, 'max_range'=>9)); 
11    // 制限範囲内の整数を取得する 
12    $value = filter_input(INPUT_POST, $item['name'], FILTER_VALIDATE_INT, $options); 
13    if (($value === NULL) or ($value === FALSE)) { 
14        // 入力欄が無かった場合、 
15        // 数値でなかった場合、制限範囲外の場合は以降をスキップ 
16        continue; 
17    } 
18    // スキップされていない＝数値が入力された 
19    // $item や $value を使って $msg を生成する  
20    // $item や $value を使ってファイルに書き出す 
21} 
22// スキップされていない＝数値が入力された 
23$values[$key] = $value; // これを追加します 
24// $field を生成する 
25$field = ''; 
26foreach ($list as $key=>$item) { 
27    if ($item['qty'] == 0) { 
28        // 不要な入力欄を作らない為にスキップ 
29        continue; 
30    } 
31    $field .= <<<HTML
32<div class="field"> 
33<div class="item"> 
34<span class="name">{$item['disp']}</span>：<span class="number">{$item['qty']}</span> 
35</div> 
36<div class="inputField"> 
37<input type="text" name="{$item['name']}" value="{$values[$key]}"> 
38</div> 
39HTML; 
40    // ここで htmlspecialchars() を使っていないのは 
41    // 自分で設定した値、及びフィルターを通して 
42    // 整数である事が確定済みで、無害であると判断したため 
43    // 最初に「数値である事を」と書いたのはその為  
44    ?> 
45    <!DOCTYPE html> 
46    <html lang="ja"> 
47    <head> 
48    <meta charset="UTF-8"> 
49    <title>必要な物を</title> 
50    </head> 
51    <body> 
52    <?= count($msg) > 0 ? implode('', $msg) . "備品のナンバーを書き込みました。 <hr>" 
53    '' ?>  
54    <?=$msg;?> 
55    <div class="form"> 
56    <form action="number.php" method="post"> 
57    <?=$filed:?> 
58    <input type="submit" name="btnSubmit" value="送信"> 
59    </form> 
60    </div> 
61    </body> 
62    </html>

それだけでなく、<?=$filed:?>のコロンも文法エラーを引き起こすでしょうし、

    <?= count($msg) > 0 ? implode('', $msg) . "備品のナンバーを書き込みました。 <hr>" 
    '' ?>

で改行を挟んでいる箇所が文字列の連結演算子も置かないために文法エラーを引き起こすだろうなと。

投稿2019/09/09 13:39

編集2019/09/10 00:52

退会済みユーザー

総合スコア0

daichi-onoue

2019/09/10 00:38

どこで閉じればいいのでしょうか？

行動規範の内容に同意します

ベストアンサー

グチャグチャなコードは問題をどんどん潜在化します。
構文チェック機能のあるエディタ、コードフォーマット機能のあるエディタは必須アイテムです。
できればデバッグまでできたほうがいいのでIDEに乗り換えるべき。
もしプラグインなどを入れるものであれば一通り取り入れるべき。

下記はIDEであるEclipseの例。

構文チェックだけでなく、未定義の変数・定義されながら未使用の変数も警告で教えてくれます。
関数の補完とか、予測変換、{の対応などもしてくれます。

コードフォーマットも使えば多少は読みやすくなります。

php
1<?php
2$list = array(
3    array('to'=>'46.txt', 'from'=>'23.txt', 'name'=>'inp_01', 'disp'=>'クーピー', 'qty'=>0),
4);
5// 取得した値を保管する変数を初期化
6$values = array(); // これ追加します
7foreach($list as $key=>$item) {
8    // オプションを指定すれば入力範囲の制限も可能
9    $options=array('options'=>array('min_range'=>0, 'max_range'=>9));
10    // 制限範囲内の整数を取得する
11    $value = filter_input(INPUT_POST, $item['name'], FILTER_VALIDATE_INT, $options);
12    if (($value === NULL) or ($value === FALSE)) {
13        // 入力欄が無かった場合、
14        // 数値でなかった場合、制限範囲外の場合は以降をスキップ
15        continue;
16    }
17    // スキップされていない＝数値が入力された
18    // $item や $value を使って $msg を生成する
19    // $item や $value を使ってファイルに書き出す
20}
21// スキップされていない＝数値が入力された
22$values[$key] = $value; // これを追加します
23// $field を生成する
24$field = '';
25foreach ($list as $key=>$item) {
26    if ($item['qty'] == 0) {
27        // 不要な入力欄を作らない為にスキップ
28        continue;
29    }
30    $field .= <<<HTML
31<div class="field">
32<div class="item">
33<span class="name">{$item['disp']}</span>：<span class="number">{$item['qty']}</span>
34</div>
35<div class="inputField">
36<input type="text" name="{$item['name']}" value="{$values[$key]}">
37</div>
38HTML;
39    // ここで htmlspecialchars() を使っていないのは
40    // 自分で設定した値、及びフィルターを通して
41    // 整数である事が確定済みで、無害であると判断したため
42    // 最初に「数値である事を」と書いたのはその為
43    ?>
44<!DOCTYPE html>
45<html lang="ja">
46<head>
47<meta charset="UTF-8">
48<title>必要な物を</title>
49</head>
50<body> 
51<?= count($msg) > 0 ? implode('', $msg) . "備品のナンバーを書き込みました。 <hr>" 
52'' ?>  
53<?=$msg;?> 
54<div class="form">
55		<form action="number.php" method="post"> 
56<?=$filed:?> 
57<input type="submit" name="btnSubmit" value="送信">
58		</form>
59	</div>
60</body>
61</html>
62

あと、各所で書かれているコメントがほぼ意味を成していません。
「これ」とか「あれ」とかでは後から読んだ際に理解できません。
無駄なコメント・冗長なコメント・コードと合っていないものも見受けられます。
コードが長くなればそれだけ読むのがしんどくなります。
コメントは的確に、端的に、ダラダラと書かないようにしましょう。

下記のような記事は読んでおくべきです。

軽くヒント:コメントの書き方

要件は分からないですが、例として。
※コードフォーマットをかけてコメントを書き直しただけです。

php
1<?php
2//入力内容取得・入力コントロールHTML生成のための設定配列
3$list = array(
4    array(
5        'to'=>'46.txt', 
6        'from'=>'23.txt',
7        'name'=>'inp_01',
8        'disp'=>'クーピー', 
9        'qty'=>0
10    ),
11);
12
13//フィルターOKのもののみ保管する配列
14$values = array();
15foreach($list as $key=>$item) {
16    $options=array('options'=>array('min_range'=>0, 'max_range'=>9));
17
18    // POST値を整数型 最小0,最大9でチェック
19    $value = filter_input(INPUT_POST, $item['name'], FILTER_VALIDATE_INT, $options);
20
21    //フィルターNGの場合は次へ
22    if (($value === NULL) or ($value === FALSE)) {
23        continue;
24    }
25}
26//フィルターOKのもののみ保管
27$values[$key] = $value;
28
29$field = '';
30foreach ($list as $key=>$item) {
31    //空行のためスキップ
32    if ($item['qty'] == 0) {
33        continue;
34    }
35
36    //各入力コントロールのHTML
37    $field .= <<<HTML
38<div class="field">
39<div class="item">
40<span class="name">{$item['disp']}</span>：<span class="number">{$item['qty']}</span>
41</div>
42<div class="inputField">
43<input type="text" name="{$item['name']}" value="{$values[$key]}">
44</div>
45HTML;
46
47//メインHTML
48?>
49<!DOCTYPE html>
50<html lang="ja">
51<head>
52<meta charset="UTF-8">
53<title>必要な物を</title>
54</head>
55<body> 
56<?= count($msg) > 0 ? implode('', $msg) . "備品のナンバーを書き込みました。 <hr>" 
57'' ?>  
58<?=$msg;?> 
59<div class="form">
60    <form action="number.php" method="post"> 
61        <?=$filed:?> 
62        <input type="submit" name="btnSubmit" value="送信">
63    </form>
64</div>
65</body>
66</html>

明らかに次のSyntaxが出そうなコードですけどそこまでやってたらキリがないので、
あとはがんばってください。
エラー読めるようになったらとりあえず初心者から少しだけ進みます。

投稿2019/09/10 00:28

編集2019/09/10 01:33

m.ts10806

総合スコア80875

maisumakun

2019/09/10 00:32

> 下記は冗長な記述と思います。 FILTER_VALIDATE_INTである以上、正当な入力として「0」を想定する必要があります（文字列の'0'すらPHPではfalsyですが）。if(!$v)では、0も除外されてしまいます。

m.ts10806

2019/09/10 00:35 編集

すみません。そこだけ見てました。 integer の 0 (ゼロ)が引っかかりますね。

daichi-onoue

2019/09/10 00:37

御提示いただいたソースでは、 52行目でエラーが発生してしまいます。

m.ts10806

2019/09/10 00:54 編集

よく読んでください。直したソースを提示したと書いていません。コードフォーマットをかけただけの同じソースです。フォーマット掛けなくても私のほうでは最初に提示した画像のように同じ52行目ででてますよ。

m.ts10806

2019/09/10 00:48

maisumakunさん不適当なアドバイスでしたので「コメントの書き方」の方にシフトしました。

m.ts10806

2019/09/10 00:53

daichi-onoueさんところで、コードフォーマット掛けたら色々見えてきませんか？ここにこの処理があるのはマズいとか、ちゃんと入るべきところに入ってないとか構文エラー直したところで想定通り動かないのが丸見えです。私が、特にPHPではそのまんま現在の解決策を提示することはあまり多くないです。なぜなら起きている問題を解決したところで今後必ず起きる問題まで解決にはならないからです。自己解決出来たら一番早いのは明白です。環境を整えること、デバッグをすること、エラーを読むことこの３つをするだけで初心者のうちに起きている問題の９割以上は解決できます。質問して回答待って、もらって、試して　　時間が勿体ないんですよ。

行動規範の内容に同意します

あなたの回答