素材を比較したidと共通項目数をデータベースに格納したい

「まだ回答を求めています」

質問に「まだ回答を求めています」と表示し、閲覧者に状況を知らせることで、新たな回答を得られやすくなります。以下のような場合にご利用ください。
もらった回答を調べてみたが、問題を解決できなかった
質問投稿から時間がたってしまった
ボタンを押してから1週間が経過するか、新たな回答を得られると自動で非表示になります。
」

始めて質問いたしますphp初心者です。
どぞよろしくお願いいたします。

###前提・実現したいこと
PHPで素材の特徴の共通項目数を求めるシステムを作っています。

データベース'compare'のテーブル'answers'に既にいくつかの回答(データ）が入っている状態で質問formから受け取った質問の回答をテーブル'answers'のデータと比較し幾つ共通項目があるかをデータベース'compare'の'result'テーブルのidナンバー（idフィールド）と共通項目数（commonsフィールド）を格納することを目指いしています。

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

idと共通項目数の値を求めるところまではできているよう思いますがそれらの値をresultテーブルに格納することができません。

printと<br>等を使って表示させるとそれぞれの数字は問題なく出ています。
いろいろ試しエラーメッセージが出ることはないのですが値がテーブルに格納されるという結果が得られません。

御助力いただけますようお願いいたします。

###該当のソースコード
質問form

html
1<html>
2<head>
3<meta charset="UTF-8">
4<title>特徴マッチ度</title>
5</head>
6<body>
7<form method="post" action="question_check.php">
8熱に強い<br/>
9<input type="radio" name="q1" value="1">Yes <input type="radio" name="q1" value="2">No<br/>
10
11柔軟性がある<br/>
12<input type="radio" name="q2" value="1">Yes <input type="radio" name="q2" value="2">No<br/>
13
14１、手で破く事ができる２、手で破く事はできない３鋏を使えば切れる４、鋏でも切れない<br/>
15<input type="radio" name="q3" value="1">1 <input type="radio" name="q3" value="2">2<input type="radio" name="q3" value="3">3 <input type="radio" name="q3" value="4">4<br/>
16<br/>
17<input type="button" onclick="history.back()" value="戻る">
18<input type="submit" value="OK">
19</form>
20</body>
21</html

これの回答チェックをし

PHP
1<html>
2<head>
3<meta charaset="UTF8">
4<title>回答チェック</title>
5</head>
6<body>
7
8<?php
9$question1=$_POST['q1'];
10$question2=$_POST['q2'];
11$question3=$_POST['q3'];
12
13if($question1=='')
14{
15    	print'1番の回答がありません。<br/>';
16}
17else
18{
19    	print'1番の回答';
20    	print$question1;
21    	print'<br/>';
22}
23if($question2=='')
24{
25    	print'2番の回答がありません。<br/>';
26}
27if (isset($_POST["q2"])) 
28{
29    	print'2番の回答';
30    	print$question2;
31    	print'<br/>';
32}
33
34if($question3=='')
35{
36    	print'3番の回答がありません。<br/>';
37}
38else
39{
40    	print'3番の回答';
41    	print$question3;
42    	print'<br/>';
43}
44
45if($question1==''||$question2==''||$question3=='')
46{
47	print'<form>';
48	print'<input type="button" onclick="history.back()"value="戻る"	>';
49	print'</form>';
50}
51else
52{
53	print'<form method="post" action="compair.php">';
54    	print'<input type="hidden" name="q1" value="'.$question1.'">';
55    	print'<input type="hidden" name="q2" value="'.$question2.'">';
56    	print'<input type="hidden" name="q3" value="'.$question3.'">';
57    	print'<br/>';
58    	print'<input type="button" onclick="history.back()" value="戻る">';
59    	print'<input type="submit" value="OK">';
60    	print'</form>';
61}
62?>
63</body>
64</html>

これを
###試したこと

PHP
1<html>
2<head>
3<meta charaset="UTF8">
4<title>素材比較</title>
5</head>
6<body>
7
8<?php
9
10$question1=$_POST['q1'];
11$question2=$_POST['q2'];
12$question3=$_POST['q3'];
13
14$dsn='mysql:dbname=compare;host=localhost';
15$user='root';
16$password='';
17$dbh=new PDO($dsn, $user, $password);
18$dbh->query('SET NAMES utf8');
19
20$sql='SELECT code,r1,r2,r3 FROM answers WHERE 1';
21$stmt=$dbh->prepare($sql);
22$stmt->execute();
23
24$dbh=null;
25
26//回答が共通する計を求める。
27
28while(true)
29{
30    $rec=$stmt->fetch(PDO::FETCH_ASSOC);
31    if($rec==false)
32    {
33             break;
34    }
35
36	if($rec['r1']==$question1)
37		{                       
38			$s=$s+1;
39		}
40        else
41		{
42			$s=0;
43		}
44	if($rec['r2']==$question2)
45		{
46			$s=$s+1;
47		}
48	if($rec['r3']==$question3)
49		{
50			$s=$s+1;
51		}
52//試した一例
53
54$dsn='mysql:dbname=compare;host=localhost';
55$user='root';
56$password='';
57$dbh=new PDO($dsn, $user, $password);
58$dbh->query('SET NAMES utf8');
59
60$sql = "INSERT INTO results (id, commons) VALUES (:id, :commons)";
61 
62$stmt = $dbh->prepare($sql);
63 
64$id= array($rec['code']=>$s);
65 
66// foreachで挿入する値を1つずつループ処理
67foreach ($id as $key => $s) {
68 
69  // 連想配列のキーを :name に、値を :population にセットし、executeでSQLを実行
70  $stmt->execute(array(':id' => $key, ':commons' => $s));
71 
72}
73//ここまで一例
74
75//上記の一例の部分を書かずにこの下のprintによって表示される値の様にをデータベースのテーブルresultsのフィールドidとフィールドcommonsに格納したい。
76
77print" |";
78print $rec['code'];
79print" | ";
80print $s;
81print" |";
82print '<br/>';
83
84//これより上。
85
86
87
88$s=0;
89}
90
91//新規の素材の回答をテーブルanswersの最後に格納。
92
93$dsn='mysql:dbname=compare;host=localhost';
94$user='root';
95$password='';
96$dbh=new PDO($dsn, $user, $password);
97$dbh->query('SET NAMES utf8');
98
99$sql='INSERT INTO answers(r1,r2,r3) VALUES (?,?,?)';
100$stmt=$dbh->prepare($sql);
101$data[]=$question1;
102$data[]=$question2;
103$data[]=$question3;
104$stmt->execute($data);
105$dch=null;
106?>
107
108</body>
109</html>

###補足情報
サーバの種類： MySQL
サーバのバージョン： 5.6.20
Apache/2.4.10 (Win32) OpenSSL/1.0.1i PHP/5.5.15

どうぞ宜しくお願いいたします。
始めて質問いたしますphp初心者です。
どぞよろしくお願いいたします。

###前提・実現したいこと
PHPで素材の特徴の共通項目数を求めるシステムを作っています。

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

idと共通項目数の値を求めるところまではできているよう思いますがそれらの値をresultテーブルに格納することができません。

printと<br>等を使って表示させるとそれぞれの数字は問題なく出ています。
いろいろ試しエラーメッセージが出ることはないのですが数字がテーブルに格納されるという結果が得られません。

御助力いただけますようお願いいたします。

###該当のソースコード
質問form

html
1<html>
2<head>
3<meta charset="UTF-8">
4<title>特徴マッチ度</title>
5</head>
6<body>
7<form method="post" action="question_check.php">
8熱に強い<br/>
9<input type="radio" name="q1" value="1">Yes <input type="radio" name="q1" value="2">No<br/>
10
11柔軟性がある<br/>
12<input type="radio" name="q2" value="1">Yes <input type="radio" name="q2" value="2">No<br/>
13
14１、手で破く事ができる２、手で破く事はできない３鋏を使えば切れる４、鋏でも切れない<br/>
15<input type="radio" name="q3" value="1">1 <input type="radio" name="q3" value="2">2<input type="radio" name="q3" value="3">3 <input type="radio" name="q3" value="4">4<br/>
16<br/>
17<input type="button" onclick="history.back()" value="戻る">
18<input type="submit" value="OK">
19</form>
20</body>
21</html

これの回答チェックをし

PHP
1<html>
2<head>
3<meta charaset="UTF8">
4<title>回答チェック</title>
5</head>
6<body>
7
8<?php
9$question1=$_POST['q1'];
10$question2=$_POST['q2'];
11$question3=$_POST['q3'];
12
13if($question1=='')
14{
15    	print'1番の回答がありません。<br/>';
16}
17else
18{
19    	print'1番の回答';
20    	print$question1;
21    	print'<br/>';
22}
23if($question2=='')
24{
25    	print'2番の回答がありません。<br/>';
26}
27if (isset($_POST["q2"])) 
28{
29    	print'2番の回答';
30    	print$question2;
31    	print'<br/>';
32}
33
34if($question3=='')
35{
36    	print'3番の回答がありません。<br/>';
37}
38else
39{
40    	print'3番の回答';
41    	print$question3;
42    	print'<br/>';
43}
44
45if($question1==''||$question2==''||$question3=='')
46{
47	print'<form>';
48	print'<input type="button" onclick="history.back()"value="戻る"	>';
49	print'</form>';
50}
51else
52{
53	print'<form method="post" action="compare.php">';
54    	print'<input type="hidden" name="q1" value="'.$question1.'">';
55    	print'<input type="hidden" name="q2" value="'.$question2.'">';
56    	print'<input type="hidden" name="q3" value="'.$question3.'">';
57    	print'<br/>';
58    	print'<input type="button" onclick="history.back()" value="戻る">';
59    	print'<input type="submit" value="OK">';
60    	print'</form>';
61}
62?>
63</body>
64</html>

これを
###試したこと(compare.php)

PHP
1<html>
2<head>
3<meta charaset="UTF8">
4<title>素材比較</title>
5</head>
6<body>
7
8<?php
9
10$question1=$_POST['q1'];
11$question2=$_POST['q2'];
12$question3=$_POST['q3'];
13
14$dsn='mysql:dbname=compare;host=localhost';
15$user='root';
16$password='';
17$dbh=new PDO($dsn, $user, $password);
18$dbh->query('SET NAMES utf8');
19
20$sql='SELECT code,r1,r2,r3 FROM answers WHERE 1';
21$stmt=$dbh->prepare($sql);
22$stmt->execute();
23
24$dbh=null;
25
26//回答が共通する計を求める。
27
28while(true)
29{
30    $rec=$stmt->fetch(PDO::FETCH_ASSOC);
31    if($rec==false)
32    {
33             break;
34    }
35
36	if($rec['r1']==$question1)
37		{                       
38			$s=$s+1;
39		}
40        else
41		{
42			$s=0;
43		}
44	if($rec['r2']==$question2)
45		{
46			$s=$s+1;
47		}
48	if($rec['r3']==$question3)
49		{
50			$s=$s+1;
51		}
52//試した一例
53
54$dsn='mysql:dbname=compare;host=localhost';
55$user='root';
56$password='';
57$dbh=new PDO($dsn, $user, $password);
58$dbh->query('SET NAMES utf8');
59
60$sql = "INSERT INTO results (id, commons) VALUES (:id, :commons)";
61 
62$stmt = $dbh->prepare($sql);
63 
64$id= array($rec['code']=>$s);
65 
66// foreachで挿入する値を1つずつループ処理
67foreach ($id as $key => $s) {
68 
69  // 連想配列のキーを :name に、値を :population にセットし、executeでSQLを実行
70  $stmt->execute(array(':id' => $key, ':commons' => $s));
71 
72}
73//ここまで一例
74
75//上記の一例の部分を書かずにこの下のprintによって表示される値の様にをデータベースのテーブルresultsのフィールドidとフィールドcommonsに格納したい。
76
77print" |";
78print $rec['code'];
79print" | ";
80print $s;
81print" |";
82print '<br/>';
83
84//これより上。
85
86
87
88$s=0;
89}
90
91//新規の素材の回答をテーブルanswersの最後に格納。
92
93$dsn='mysql:dbname=kachikan;host=localhost';
94$user='root';
95$password='';
96$dbh=new PDO($dsn, $user, $password);
97$dbh->query('SET NAMES utf8');
98
99$sql='INSERT INTO answers(r1,r2,r3) VALUES (?,?,?)';
100$stmt=$dbh->prepare($sql);
101$data[]=$question1;
102$data[]=$question2;
103$data[]=$question3;
104$stmt->execute($data);
105$dch=null;
106?>
107
108</body>
109</html>

###補足情報
サーバの種類： MySQL
サーバのバージョン： 5.6.20
Apache/2.4.10 (Win32) OpenSSL/1.0.1i PHP/5.5.15

どうぞ宜しくお願いいたします。

エラーはでていません。

テーブル'answers'はフィールドがcode(int11)AI主,r1(int11),r2(int11),r3(int11)です。
テーブル'results'はフィールドid(int11)主,commons(int11)です。

m.ts10806

2017/07/21 05:24

エラーがあれば追記願います（実際に実行されたSQLに何か問題があるかも）。また、それぞれのテーブル定義はありますか？

pintxos

2017/07/27 15:32

ご質問頂いていたのに本文の編集の方でお答えしていました。大変失礼いたしました。エラーはでていません。テーブル'answers'はフィールドがcode(int11)AI主,r1(int11),r2(int11),r3(int11)です。テーブル'results'はフィールドid(int11)主,commons(int11)です。

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

回答1件

SQL実行時のエラーを確認しないと詳細はわかりませんが、おかしいと思ったところを指摘します。

php
1$dbh=new PDO($dsn, $user, $password);
2$dbh->query('SET NAMES utf8');
3
4$sql = "INSERT INTO results (id, commons) VALUES (:id, :commons)";
5
6$stmt = $dbh->prepare($sql);
7
8$id= array($rec['code']=>$s);
9
10// foreachで挿入する値を1つずつループ処理
11foreach ($id as $key => $s) {
12
13  // 連想配列のキーを :name に、値を :population にセットし、executeでSQLを実行
14  $stmt->execute(array(':id' => $key, ':commons' => $s));
15
16}

この部分、executeをループさせていますが、これだと動かないと思います。
生成されたステートメントに対して、値のバインドは１回のみじゃないですか？

なので、こうじゃないかと私は思います。

php
1$dbh=new PDO($dsn, $user, $password);
2$dbh->query('SET NAMES utf8');
3
4$sql = "INSERT INTO results (id, commons) VALUES (:id, :commons)";
5
6$id= array($rec['code']=>$s);
7
8// foreachで挿入する値を1つずつループ処理
9foreach ($id as $key => $s) {
10  // ステートメントの生成は毎回行わないと・・・。
11  $stmt = $dbh->prepare($sql);
12  // 連想配列のキーを :name に、値を :population にセットし、executeでSQLを実行
13  $stmt->execute(array(':id' => $key, ':commons' => $s));
14
15}

コメント頂いた部分について

なるほど、仕様はわかりました。
そうなると、resultsテーブルへ格納するレコードは、while(true)の各ループで1件だけですよね。
そうであれば、foreachはいらないです。

根本的にこれという原因はソースコードから見つけられませんでしたが、resultsにデータが入っていないということと、resultsテーブルへの格納SQLが実行されているのであれば、必ずエラーが出ています。
apacheのerror_logに吐き出されるはずなので、そちらを確認してください。

とりあえず、resultへのinsert処理で値のバインド処理が省略されていた形でしたので、明示的にバインドするソースに書き換えてみました。
こちらでどうなるか、試してみていただけないでしょうか？

php
1<html>
2<head>
3<meta charaset="UTF8">
4<title>素材比較</title>
5</head>
6<body>
7
8<?php
9
10$question1=$_POST['q1'];
11$question2=$_POST['q2'];
12$question3=$_POST['q3'];
13
14$dsn='mysql:dbname=compare;host=localhost';
15$user='root';
16$password='';
17$dbh=new PDO($dsn, $user, $password);
18$dbh->query('SET NAMES utf8');
19
20$sql='SELECT code,r1,r2,r3 FROM answers WHERE 1';
21$stmt=$dbh->prepare($sql);
22$stmt->execute();
23
24$dbh=null;
25
26//回答が共通する計を求める。
27
28while(true)
29{
30    $rec=$stmt->fetch(PDO::FETCH_ASSOC);
31    if($rec==false)
32    {
33             break;
34    }
35
36    if($rec['r1']==$question1)
37        {
38            $s=$s+1;
39        }
40        else
41        {
42            $s=0;
43        }
44    if($rec['r2']==$question2)
45        {
46            $s=$s+1;
47        }
48    if($rec['r3']==$question3)
49        {
50            $s=$s+1;
51        }
52//試した一例
53$sql = "INSERT INTO results (id, commons) VALUES (:id, :commons)";
54$stmt = $dbh->prepare($sql);
55$stmt->bindParam(':id', $rec['code'], PDO::PARAM_INT);
56$stmt->bindParam(':commons', $s, PDO::PARAM_INT);
57$stmt->execute();
58//ここまで一例
59
60//上記の一例の部分を書かずにこの下のprintによって表示される値の様にをデータベースのテーブルresultsのフィールドidとフィールドcommonsに格納したい。
61
62print" |";
63print $rec['code'];
64print" | ";
65print $s;
66print" |";
67print '<br/>';
68
69//これより上。
70
71
72
73$s=0;
74}
75
76//新規の素材の回答をテーブルanswersの最後に格納。
77$sql='INSERT INTO answers(r1,r2,r3) VALUES (?,?,?)';
78$stmt=$dbh->prepare($sql);
79$data[]=$question1;
80$data[]=$question2;
81$data[]=$question3;
82$stmt->execute($data);
83$dch=null;
84?>
85
86</body>
87</html>
88

投稿2017/07/21 08:36

編集2017/07/23 07:33

Iya712

総合スコア158

pintxos

2017/07/21 13:47

ご回答ありがとうございます。ご指摘の内容を反映してみたところresultsテーブルのidフィールドとcommonsフィールドに各1つ値が入りました。ただ私が求めているものとは少し違っていました。私の書き方が要領を得てないのでわかりにくいかと思います(スミマセン）。求めている結果と言うのは、テーブル'answers'にすでに3つの素材の特徴の回答が下記の様に入っていたとします。 code | r1 | r2 | r3 | 1 | 2 | 1 | 4 | 2 | 1 | 2 | 3 | 3 | 2 | 3 | 1 | 新規に1つの素材(code 4)の回答(2,2,4)をformから送った場合 code 4とcode 1の共通項目はr1とr3で共通項目数は2。 code 4とcode 2の共通項目はr2のみで共通項目数は1。 code 4とcode 3の共通項目はr1のみで共通項目数は1。テーブル'results'には id | commons| 1 | 2 | 2 | 1 | 3 | 1 | という結果が格納されてほしく（codeとidの内容は同じで同一の素材です）テーブル'answers'にはcode 4の回答を最後に格納して code | r1 | r2 | r3 | 1 | 2 | 1 | 4 | 2 | 1 | 2 | 3 | 3 | 2 | 3 | 1 | 4 | 2 | 2 | 4 | と言う結果になることを求めています。ご指摘を頂いた部分の理解を深めて自分自身でも解決を模索いたしますが、さらにご助言頂けるようでしたら幸いです。どうぞよろしくお願いいたします。

pintxos

2017/07/24 17:01

ご回答有難うございます。また返信が遅くなりまして大変失礼しました。＞while(true)の各ループで1件だけですよね。私がご質問の意味を理解しているかどうかが不安ですが、各ループ id1件、共通項目数1件になります。今回ご提案頂いたソースをソースを試させて頂いた結果ですが、テーブル 'results'には何も格納されませんでした。 apacheのerror_logですが知識がなく今の所読み解けておりません。この辺りかなと言うところを下記に記します。お助けいただければ幸いです。 [Tue Jul 25 01:25:24.365393 2017] [ssl:warn] [pid 2864:tid 276] AH01909: www.example.com:443:0 server certificate does NOT include an ID which matches the server name [Tue Jul 25 01:25:24.412193 2017] [core:warn] [pid 2864:tid 276] AH00098: pid file C:/xampp/apache/logs/httpd.pid overwritten -- Unclean shutdown of previous Apache run? [Tue Jul 25 01:25:24.724194 2017] [ssl:warn] [pid 2864:tid 276] AH01909: www.example.com:443:0 server certificate does NOT include an ID which matches the server name [Tue Jul 25 01:25:24.848994 2017] [mpm_winnt:notice] [pid 2864:tid 276] AH00455: Apache/2.4.10 (Win32) OpenSSL/1.0.1i PHP/5.5.15 configured -- resuming normal operations [Tue Jul 25 01:25:24.848994 2017] [mpm_winnt:notice] [pid 2864:tid 276] AH00456: Apache Lounge VC11 Server built: Jul 17 2014 11:50:08 [Tue Jul 25 01:25:24.848994 2017] [core:notice] [pid 2864:tid 276] AH00094: Command line: 'c:\\xampp\\apache\\bin\\httpd.exe -d C:/xampp/apache' [Tue Jul 25 01:25:24.848994 2017] [mpm_winnt:notice] [pid 2864:tid 276] AH00418: Parent: Created child process 9888 [Tue Jul 25 01:25:25.535395 2017] [ssl:warn] [pid 9888:tid 288] AH01909: www.example.com:443:0 server certificate does NOT include an ID which matches the server name [Tue Jul 25 01:25:25.956596 2017] [ssl:warn] [pid 9888:tid 288] AH01909: www.example.com:443:0 server certificate does NOT include an ID which matches the server name [Tue Jul 25 01:25:25.987796 2017] [mpm_winnt:notice] [pid 9888:tid 288] AH00354: Child: Starting 150 worker threads. どうぞよよろしくお願いいたします。

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