where inが上手く書けない

削除、更新処理ともに1回にまとめたくて下記コードを書いてみたんですが、delete文に「Notice: Array to string conversion 」と表示され、恐らく($id)の箇所を指してるんだと思いますが、シングルクォーテーションを付けても全く改善しませんでした
私が今やりたいことは、各投稿のIDをvalueに設定し、そのIDをupdate・delete文で指定してnameとcommentを更新・削除するということです

チェックボックスにはname属性にchkid[]を指定しているのでchkid自体に何かしらの処理を行うべきではと思い、($id)の箇所を$_POST['chkid']としてみたんですがエラーが出てそこから先が全く分からなくなってしまいました

PHP
1	$link = mysqli_connect($host, $user, $pass, $dbname) or die("NG");
2
3	mysqli_select_db($link, "XXX");
4
5	if (isset($_POST['name'], $_POST['comment'], $_POST['bbs']) && $_POST['bbs'] == "post") {
6		$name = $_POST['name'];
7		$comment = $_POST['comment'];
8		mysqli_query($link, "INSERT INTO XXX (name, comment) VALUES ('$name', '$comment')");
9	} elseif (isset($_POST['bbs']) && $_POST['bbs'] == "update") {
10		$name = $_POST['name'];
11		$comment = $_POST['comment'];
12		$id = $_POST['chkid'];
13		mysqli_query($link, "UPDATE XXX SET 'name = '$name', comment = '$comment' WHERE id in($id);");
14	}  elseif (isset($_POST['bbs']) && $_POST['bbs'] == "delete") {
15		$id = $_POST['chkid'];
16		mysqli_query($link, "DELETE FROM XXX WHERE id in($id);");
17	}
18
19	$result = mysqli_query($link, "SELECT * FROM XXX ORDER BY time DESC");
20?>
21
22<html>
23	<head>
24		<meta http-equiv="content-type" content="text/html; charset=utf-8">
25	</head>
26	<body>
27		<form method="post" action="">
28			<table>
29				<tr>
30					<td>
31						名前：<input type="text" name="name">
32						内容：<textarea name="comment" cols="30" rows="3"></textarea>
33						<input type="radio" name="bbs" value="post" checked>投稿
34						<input type="radio" name="bbs" value="update">更新
35						<input type="radio" name="bbs" value="delete">削除
36						<input type="submit" value="投稿">
37					</td>
38				</tr>
39			</table>
40
41<?php
42			while($row = mysqli_fetch_array($result, MYSQLI_ASSOC)) {
43?>
44
45			<table>
46				<input type="checkbox" name="chkid[]" value="<?=$row['id'] ?>">
47				<tr>
48					<td>名前：
49						<?php echo $row['name'] ?>
50					</td>
51				</tr>
52				<tr>
53					<td>内容：
54						<?php echo $row['comment'] ?>
55					</td>
56				</tr>
57			</table>
58
59<?php
60			}
61			mysqli_free_result($result);
62
63			mysqli_close($link);
64?>
65
66		</form>
67	</body>
68</html>

kei344

2016/08/16 07:03

「Notice: Array to string conversion」で検索されましたか？

a-_.

2016/08/16 07:35

ご回答ありがとうございます調べたら英文のみのサイトばっかりで、唯一日本語で書かれたサイトがここしか見つかりませんでした http://nobuneko.com/blog/archives/2010/09/phpnotice_array_to_string_conv.html

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

回答3件

エラーが出ているのは２種類ですね。

まず、noticeというエラーですが、別に表示されていてもプログラムは普通に動きます。英文の内容は「文字列が入るべき所に配列が入ってますよ」というものです。具体的に言うと、

html
1 <input type="checkbox" name="chkid[]" value="<?=$row['id'] ?>">

このように記述されているので、$idには$_POST['chkid']から配列として送られます。この場合、たとえ、選択が１個であったとしても、$id[0]となります。ですが、$idに配列の宣言がないので、$idはデフォルトで文字列を格納する変数として認識するため、noticeが出ているのです。

したがって、

PHP
1$comment = $_POST['comment'];
2$id = array(); //配列の宣言
3$id = $_POST['chkid'];

と宣言すれば、noticeは出なくなります。

次に、SQLのIN構文内に入れるとすると

IN (1,2,3)

のようにしないといけません。ところが、現状$idに送られているのは配列なので、SQLが読み取れずエラーが起こります。なので、この配列を文字列に直してあげる必要があるのでimplode関数を使います(implode関数は頻繁に使うので、仕組みを覚えておいてください)。

PHP
1$id_str = implode(",",$id); //$id_strは文字列を格納する変数

補足
もしDBテーブル内のidをint型で宣言しているなら括弧で囲む必要ないですが、char型などの文字で宣言している場合は

IN ('1','2','3')

のように、値を逐一シングルクォートで囲む必要があります。その場合は

PHP
1$id_str = "'".implode(",",$id)."'";

こうすれば大丈夫のようです。

投稿2016/08/16 08:56

編集2016/08/16 09:33

FKM

総合スコア3619

a-_.

2016/08/17 01:40

ご回答ありがとうございます $id_str = implode(",",$id);を入れるとすると更新部分でいうとupdate文中という事でしょうか

FKM

2016/08/17 02:24

SQLはプログラムとして考えるより、文字列として考えた方がいいです。一応、そのまま文中に入れても動かないことはないですが分けた方が確実です。 $id_str = implode(",",$id); ここで取得した$id_strはSQL文には次のように挿入します。 "UPDATE XXX SET 'name = '$name', comment = '$comment' WHERE id in($id_str);" そのへんは、KotoriMaturiさんが詳しく説明されていますので合わせて参照してみてください。

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

ベストアンサー

他の回答者さんのおっしゃる通り、
先ずはimplode関数を使って配列を展開すると良いかなと思います。

さて今回のケースのような場合は、
最終的に組み立てるべきSQLの完成像を意識しつつ取り組んでいただきたいのですが、
先ず実行する必要のあるSQLの完成像は以下のようなイメージとなることを掴んでおいて下さい。

UPDATE

SQL
1UPDATE XXX 
2SET 
3    name = '太郎さん'  -- 「'name = '$name'」の先頭の「'」は打ち間違い？
4,   comment = 'IN句を使った更新'
5WHERE 
6    id IN(1, 2, 3)
7--    id IN('1', '2', '3') ←id列の定義が文字列ならシングルクォートで囲む方が良い

DELETE

SQL
1DELETE XXX 
2FROM
3WHERE 
4    id IN(1, 2, 3)
5--    id IN('1', '2', '3') ←id列の定義が文字列ならシングルクォートで囲む方が良い

上記の2つのSQLのIN句の括弧内の値の入り方に注目して下さい。
質問者さんの掲示コードの該当箇所から引っ張ってくると、

PHP
1WHERE id IN($id) // この時の$idの値が「1, 2, 3」のような形式である必要がある！

ソースコメントにも書いてますが、
今ケースのように値をリスト化してSQLを組み立てる際、IN句の括弧内にはカンマ区切りの値リストを引き渡す必要があります。

最初にも書きましたが、
カンマ区切りリストを得る場合は、
implode関数を利用しましょう。

###IN句について
最後にもしかしたら誤解しているかもしれないので念のため補足しますが、
SQLで指定している「id in(〜)」の記述内に出てくるIN句ですが、
これはPHP側の構文ではなくSQL側(MySQL)の構文となります。
詳しくは以下リンク先をどうぞ。
MySQL IN句(リストで比較)

SQL構文の知識に不安がある場合は、
確認の意味でも上記サイトを簡単に巡回すると良いかもしれませんね。

###コメントを受けまして
先ずDBアクセスが発生する処理を整理しましょう。
今回提示していただいたソースコードでDBとの通信が生じるのは、

mysqli_connect
mysqli_select_db
mysqli_query

の3点です。

**「上記処理の実行回数 = データベースとの通信回数」**と考えてください。

foreachではいけないという指摘は、
３番目のmysqli_query関数がループ処理１回ごとに呼び出されるという点から受けたものと考えられます。

またimplode関数というのはデータベース(MySQL側)の処理ではなく、
PHP側の処理となります。

MySQL側の処理とPHP側の処理というのは異なるものなので、
上記２つは基本的には切り離して考えてください。

更に付け加えると、
PHP側の処理のうちmysqli_*(mysqli始まりの関数群)というのが、
PHPとMySQLを橋渡ししている処理となります。

つまり今回のケースでは、
「mysqli_*」の関数以外ではDBとの通信処理が発生するはずがないということを抑えてもらえればと思います。

投稿2016/08/16 10:23

編集2016/08/17 03:19

Panzer_vor

総合スコア1636

a-_.

2016/08/17 07:23 編集

ご回答ありがとうございます以前更新と削除の処理にforeach文を用いてたんですが、foreachは例えばIDを複数処理したいときに処理する毎に毎回データベースへアクセスする文なので可能な限り少ない回数でアクセスするように変更した方がいいと指摘され、foreach文を用いらないコードを書いてみたいんですが、implodeは処理する毎に毎回データベースするという関数ではないのでしょうか

Panzer_vor

2016/08/17 03:17

＞ a-_.さん回答追記しました。ご確認よろしくお願いします。

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

$idが配列なので"...($id).."としたときに文字列に変換できません。
配列をカンマ区切りで文字列化するためにimplode(' ,', $id)を使いましょう。

投稿2016/08/16 07:26