質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.50%
PHP

PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

HTML

HTMLとは、ウェブ上の文書を記述・作成するためのマークアップ言語のことです。文章の中に記述することで、文書の論理構造などを設定することができます。ハイパーリンクを設定できるハイパーテキストであり、画像・リスト・表などのデータファイルをリンクする情報に結びつけて情報を整理します。現在あるネットワーク上のほとんどのウェブページはHTMLで作成されています。

Q&A

解決済

2回答

830閲覧

PHP(HTML)でのエラーについて

daichi-onoue

総合スコア53

PHP

PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

HTML

HTMLとは、ウェブ上の文書を記述・作成するためのマークアップ言語のことです。文章の中に記述することで、文書の論理構造などを設定することができます。ハイパーリンクを設定できるハイパーテキストであり、画像・リスト・表などのデータファイルをリンクする情報に結びつけて情報を整理します。現在あるネットワーク上のほとんどのウェブページはHTMLで作成されています。

0グッド

1クリップ

投稿2019/09/09 12:37

編集2019/09/09 12:39

前提・実現したいこと

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

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

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>

試したこと

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

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

kei344

2019/09/09 12:38

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

2019/09/09 19:01

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

回答2

0

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

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

0

ベストアンサー

グチャグチャなコードは問題をどんどん潜在化します。
構文チェック機能のあるエディタ、コードフォーマット機能のあるエディタは必須アイテムです。
できればデバッグまでできたほうがいいので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

総合スコア80765

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

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ではそのまんま現在の解決策を提示することはあまり多くないです。 なぜなら起きている問題を解決したところで今後必ず起きる問題まで解決にはならないからです。 自己解決出来たら一番早いのは明白です。 環境を整えること、デバッグをすること、エラーを読むこと この3つをするだけで初心者のうちに起きている問題の9割以上は解決できます。 質問して回答待って、もらって、試して  時間が勿体ないんですよ。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問