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

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

新規登録して質問してみよう
ただいま回答率
85.48%
SQL Server

SQL Serverはマイクロソフトのリレーショナルデータベース管理システムです。データマイニングや多次元解析など、ビジネスインテリジェンスのための機能が備わっています。

PHP

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

Q&A

解決済

2回答

4675閲覧

php executeのエラーを解消したい

3time

総合スコア12

SQL Server

SQL Serverはマイクロソフトのリレーショナルデータベース管理システムです。データマイニングや多次元解析など、ビジネスインテリジェンスのための機能が備わっています。

PHP

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

0グッド

0クリップ

投稿2019/06/20 02:54

編集2019/06/20 05:52

いつも質問や回答を拝見しています。
最近会社で必要なためphpを勉強し始めた初心者です。
何か質問に失礼や不備があれば改善しますので、お伝えくださいますと幸いです。

前提・実現したいこと

本を参考にphpでミニ掲示板を作成しています。
その際DBへの登録処理でエラーが出てしまい困っております。
エラー箇所を見つけるのに、お力を貸していただきたいと思っています。

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

Warning: PDOStatement::execute() [function.PDOStatement-execute]: message: Incorrect syntax near '('. (severity 15) in /var/www/html/ono/hi/join/check.php on line 18 Warning: Cannot modify header information - headers already sent by (output started at /var/www/html/ono/hi/join/check.php:18) in /var/www/html/ono/hi/join/check.php on line 21

header部分のエラーについては、executeのエラーによって何らかの文字が
出力されたために起こっているのではないかと考えています。
ですので、とりあえずexecute部分について解消したいです。

該当のソースコード

php

1<?php 2session_start(); 3require('../dbconnect.php'); 4 5if (!isset($_SESSION['join'])) { //5行目 6 header('Location: index.php'); 7 exit(); 8} 9 10if (!empty($_POST)) { 11 // 登録処理 この下から 12 $statement = $db->prepare('INSERT INTO mini_bbs(  13 name, 14 email, 15 password, 16 picture, 17 created 18 ) 19 VALUES (?, ?, ?, ?, GETDATE())'); //ここまで本来12行目 20 echo $ret = $statement->execute(array( 21 $_SESSION['join']['name'], 22 $_SESSION['join']['email'], 23 sha1($_SESSION['join']['password']), 24 $_SESSION['join']['image'] 25 )); //※エラー箇所 18行目 26 unset($_SESSION['join']); 27 28 header('Location: thanks.php'); //21行目 29 exit(); 30} 31?>

こちらに掲載するにあたって、SQL文が長くなり見づらくなったため改行を入れております。
そのため行数が少し変わっておりますので、コメントにて大まかな行数を付け加えています。

試したこと

〇自分のコードの書き方が悪いのかと思い、付録としてDLしたソースコードを貼り付けてみましたが同じエラーが出ました。
INSERT文の中身のみ使用しているSQLserverに合わせて書き換えておりますが、
それ以外の部分はDLしたものをそのまま貼り付けています。

〇見たところ間違ってはいないと思ったのですが、気づいていないだけで
()が多い・足りない箇所があるのかと思い18行目の閉じ括弧を1,2,3個にしてみましたが、やはり改善しませんでした。

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

php(5.3以前)
※会社のサーバーを使っており、はっきりとしたバージョンが分かりません。
申し訳ありません……。
SQLserver2012

他にページ全体のコードや、index.phpのコードが必要でしたら掲載いたします。
どの情報が解決に必要か判断しきれず申し訳ない限りですが、
どうかお力添えいただけますと幸いです。
よろしくお願いいたします。

###追記(修正等)
・SELECT文で試す
→試してみたのですが表示されないため、もしかしたら接続で何か躓いているのかもしれません……

dbconnect.php

php

1<?php 2try { 3 $db = new PDO('dblib:dbname=mini_bbs;host=****;charaset=Shift-Jis','****','**********'); 4} catch(PDOException $e) { 5 echo 'DB接続エラー: '.$e ->getMessage(); 6} 7?>

表示するために元のファイルの中に書いたphp

php

1$members = $db -> query('SELECT * FROM members'); 2while ($member = $members -> fetch()) { 3print($record['email']. "\n");

・echoしたあとheader効かない
→headerは何かを読み込む前、との認識だったので不思議に思いながらも、
何度か改訂された本の情報なので大丈夫なのかな、と思い特に編集せず書いております。
使えるのであればなぜ使えるのか調べよう、と思っておりました。

・全角の何かが入っている
→teratailに記載するにあたり、長くて見づらくなったため、
こちらのエディタで改行したところ入ってしまったようです。紛らわしいことをして申し訳ありません。

・使う予定のSQL

SQL

1INSERT INTO mini_bbs (name, email, password, picture, created) VALUES('test','test','test',NULL,GETDATE())

こちらを利用してみたところ、SQLserverでは通るものの、php側ではエラーが出ました。
以下該当箇所の記述(それ以外は変更していません)とエラー内容です。

php

1$statement = $db->prepare('INSERT INTO members (name, email, password, picture, created) VALUES('wawaaa','wawawaaa','waaa',NULL,GETDATE())'); //12行目
syntax error, unexpected T_STRING in /var/www/html/ono/hi/join/check.php on line 12

文章のおかしな点など多々あると思います、申し訳ありません。
少し気持ちが焦っているので、見当違いなことをしていましたらご指摘くださいますと助かります。

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

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

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

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

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

yoorwm

2019/06/20 03:02

「INSERT INTO(」となっていますが、どこのテーブルにINSERTするかの情報が無いのではないですか?
3time

2019/06/20 03:08

yoorwmさん ありがとうございます。失礼いたしました、修正しておきます。 実際お恥ずかしながら入力を忘れていたのですがご報告までに、 入力してもエラー内容は変わらず、DBへの登録もされておりませんでした。
kariya

2019/06/20 03:17

セッション変数が正しく設定されているか、テーブル定義とSQLクエリが正しいか など、問題の可能性があるかと思います。 INSERT ではなく、シンプルなSELECTにして、プリペアドステートメントもやめて、 それでエラーや警告がでないかどうかを確認されてはいかがでしょうか?
m.ts10806

2019/06/20 03:32

使う予定のSQLを実際に実行して通るかどうか確かめてください。
3time

2019/06/20 04:32

kariyaさん mts10806さん ありがとうございます。回答も含めて色々試してみます! 経過など、また報告しますが取り急ぎお礼を伝えさせてください。
guest

回答2

0

MySQL構文チェック通すと
https://jp.piliapp.com/mysql-syntax-check/

mini_bbs( ←ここに全角の何かが入ってる

みたいですね。
他にもあるかもしれません。ご確認ください。

色々なところでアドバイスしていることでもありますが、
プログラムから利用する前に直接実行して通ったSQLを採用してください。
PHPからすればDBは外部の仕組みです。直接実行して通らないものはPHP通しても当然通りません。

投稿2019/06/20 03:35

編集2019/06/20 03:38
m.ts10806

総合スコア80850

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

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

退会済みユーザー

退会済みユーザー

2019/06/20 04:01

echoやheader()のところも全角空白がエラー原因になるので対策が必要。 ところで、一度でもechoしたら、header()関数効かないんだけど。
m.ts10806

2019/06/20 04:05

そうですね。その直前でエラーになってるからまだ出てきてないですけど下記、バグの元ですね echo $ret =
退会済みユーザー

退会済みユーザー

2019/06/20 04:14

> header()関数効かないんだけど。 これ違反だけど、効くと思う。 思うと言っているのは、実験の範囲において確認しているだけなので、効く条件/効かない条件がわからないからだけど。
m.ts10806

2019/06/26 08:43

Locationするなら出力する意味は全くないですけどね・・。
退会済みユーザー

退会済みユーザー

2019/06/26 08:45

バッファの話なので、そもそも記述があっても出力されず破棄されるってことみたい。
3time

2019/06/27 01:44

リンク先を拝見したところphpのバージョンによって「output_buffering」なるものの設定が異なるようなので、本の通りに書いても動かなかったのかもしれません。バージョンにあった本を購入するべきでした……。 大変勉強になりました、ありがとうございます。
m.ts10806

2019/06/27 01:46

本がでたときにはバージョンが違うというのはよくあることなので、実際の設定を見て、PHPマニュアルを見て、簡単なコードを組んで設定を確認するのが良いと思います。
3time

2019/06/27 02:15

設定を確認したところ、output_bufferingがno valueになっておりました。 確認できることすら知りませんでした……不勉強で申し訳ない限りです。ありがとうございます!
m.ts10806

2019/06/27 02:19

追記されたコード $statement = $db->prepare('INSERT INTO members (name, email, password, picture, created) VALUES('wawaaa','wawawaaa','waaa',NULL,GETDATE())'); //12行目 もそうですが、クォートの概念やクォートのエスケープなど、文字列の基本のところでもありますので、こちらもおさえてくださいね。 https://www.php.net/manual/ja/regexp.reference.escape.php
guest

0

自己解決

アドバイスをくださった方々、ありがとうございました。
自己解決しましたのでご報告します。

php

1<?php 2session_start(); 3require('../dbconnect/dbconnect.php'); 4 5if (!isset($_SESSION['join'])) { 6 header('Location: index.php'); 7 exit(); 8} 9 10if (!empty($_POST)) { 11 // 登録処理をする 12 $statement = $db->prepare(' 13 INSERT INTO mini_bbs (name, email, password, picture, created) 14 VALUES (?, ?, ?, ?, GETDATE()) 15 '); //本来は改行していません 16 $ret = $statement->execute(array( 17 $_SESSION['join']['name'], 18 $_SESSION['join']['email'], 19 sha1($_SESSION['join']['password']), 20 $_SESSION['join']['image'] 21 )); 22 unset($_SESSION['join']); 23 header('Location: thanks.php'); 24 exit(); 25 } 26?> 27

結果、やはりご指摘いただいたechoが問題だったようです。
ただ、本にはechoが記載されており、配布されているサンプルコードにも載っていたことから、何かしら別の解決策があったのかもしれません。

投稿2019/06/26 08:37

3time

総合スコア12

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

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

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.48%

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

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

質問する

関連した質問