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

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

ただいまの
回答率

88.10%

エラーの解決法がわかりません

受付中

回答 3

投稿

  • 評価
  • クリップ 0
  • VIEW 3,027

score 6

 前提・実現したいこと

エラー
を解決したい
ここに質問の内容を詳しく書いてください。
(例)PHP(CakePHP)で●●なシステムを作っています。
■■な機能を実装中に以下のエラーメッセージが発生しました。

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

```
エラーメッセージ
Parse error: syntax error, unexpected ']', expecting '-' or identifier (T_STRING) or variable (T_VARIABLE) or number (T_NUM_STRING) in C:\xampp\htdocs\staff\staff_add_done.php on line 28

 該当のソースコード

ここに言語名を入力PHP
ソースコード
<!DOCTYPE html>
<html>
<head>
<meta charset="UTF-8">
<title>ろくまる農園</title>
</head>
<body>

<?php

try
{

$staff_name = $_POST['name'];
$staff_pass = $_POST['pass'];

$staff_name = htmlspecialchars($staff_name,ENT_QUOTES,'UTF-8');
$staff_pass = htmlspecialchars($staff_pass,ENT_QUOTES,'UTF-8');

$dsn='mysql:dbname=shop;host=localhost;charset=utf8';
$user='root';
$password=";
$dbh=new PDO($dsn,$user,$password);
$dbh->setAttribute(PDO::ATTR_ERRMODE,PDO::ERRMODE_EXCEPTION);

$sql='INSERT INTO mst_staff(name,password) VALUES (?,?)';
$stmt=$dbh->prepare($sql);
$data[]= $staff_name;
$data[]= $staff_pass;
$stmt->execute($data);

$dbh=null;

print $staff_name;
print'さんを追加しました。<br />';


catch (Exception $e)
{
print'ただいま障害により大変ご迷惑をおかけしております。';
exit();
}

?>

<a href="staff_list.php">戻る</a>

</body>
</html>

 試したこと

ここに問題に対して試したことを記載してください。

 補足情報(FW/ツールのバージョンなど)

ここにより詳細な情報を記載してください。

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

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

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

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • m6u

    2018/08/07 16:16

    質問テンプレートに従って、それぞれの箇所を適切に書き換えてくださいね。それと、ソースコードを貼り付けた箇所が読みづらいので、該当箇所を編集し直して「<code>」ってボタンで囲うと少しは改善されますよ。

    キャンセル

  • m.ts10806

    2018/08/07 16:29 編集

    既に同じ指摘が入っていたため編集削除。

    キャンセル

回答 3

+2

<!DOCTYPE html>
<html>
<head>
<meta charset="UTF-8">
<title>ろくまる農園</title>
</head>
<body>

<?php

try
{

    $staff_name = $_POST['name'];
    $staff_pass = $_POST['pass'];

    $staff_name = htmlspecialchars($staff_name,ENT_QUOTES,'UTF-8');
    $staff_pass = htmlspecialchars($staff_pass,ENT_QUOTES,'UTF-8');

    $dsn='mysql:dbname=shop;host=localhost;charset=utf8';
    $user='root';
    $password='';
    $dbh=new PDO($dsn,$user,$password);
    $dbh->setAttribute(PDO::ATTR_ERRMODE,PDO::ERRMODE_EXCEPTION);

    $sql='INSERT INTO mst_staff(name,password) VALUES (?,?)';
    $stmt=$dbh->prepare($sql);
    $data[]= $staff_name;
    $data[]= $staff_pass;
    $stmt->execute($data);

    $dbh=null;

    print $staff_name;
    print 'さんを追加しました。<br />';

}
catch (Exception $e)
{
    print 'ただいま障害により大変ご迷惑をおかけしております。';
    exit();
}

?>

<a href="staff_list.php">戻る</a>

</body>
</html> 

何箇所か直しました。
catch直前の}が全角だったり、
$password=";←ダブルクォート一発のみだったり。

気になるのが、DB保存用データに対して htmlspecialchars() を適用している所。
変数をバインドして与えるときは自動的にエスケープ処理されるので
本来不要なのですが。
DB保存時にはエスケープ処理を加えず実データのまま保持し、
あくまでhtml上に出力する直前に加工するべきものです。

直接この処理のurlを叩くことは想定していないようですが、
直に$_POST['name']とか参照すると
Notice: undefined index
とか言われますので、ちゃんと値を受信しているかどうかの点検をしましょう。
他の方の回答にあるような、filter_input()を使うと良いです。

肝心の「エラーの解決法」についてですが、
「php デバッグ方法」でネット検索すればいろんな記事が見つかります。
まず、php.iniを変更してデバッグしやすくします、display_errorやerror_reportingを見直します。
次に、文法エラーを見抜きやすくするために、
phpの文法に沿ってカラー表示してくれるエディタや統合開発環境を導入します。
「php エディタ おすすめ」などでネット検索してみてください。
あと、なるべくXDebugモジュールを活用したデバッグ環境を構築すると良いです。
「php デバッグ方法 XDebug」でネット検索(以下略

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

+2

とりあえず例外処理のウチ、pdoでのcatchは

try{
・・・
}catch(PDOException $e){
  die($e->getMessage());
}

あとは、どこが28行目なのかよくわからないですが
その前後の文法まずそうですね

POSTの受け渡しも

$staff_name = $_POST['name'];
$staff_pass = $_POST['pass']; 

ではなく

$staff_name = filter_input(INPUT_POST,'name');
$staff_pass = filter_input(INPUT_POST,'pass'); 


としたほうが良いでしょう

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/08/07 16:40

    filter_input() は忘れがち。これでNoticeを回避できるし。

    キャンセル

+1

37行目のかっこが全角のような気がする…

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/08/07 16:26

    あと、22行目で
    $password=";
    のダブルクォートが1つ足りないです。

    キャンセル

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

  • ただいまの回答率 88.10%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る