sql文へのPHP変数の代入

Question

PHPで$active_day_str0として受け取った文字列をカラム名としてカラムを追加したいのですが、うまく代入できません。表記方法をご教示いただけないでしょうか。

function update_schedule($dbh, $id, $active_day_str0, $active_day_str1, $active_day_str2, $active_day_str3, $active_day_str4, $active_day_str5, $active_day_str6, $active_day_str7){
  $sql = 'ALTER TABLE schedule ADD active_day_str0 int(1)';
  // PDO::PARAM_STR-->プリペアドステートメント
  $stmt = $dbh->prepare($sql);
  // SQLに変数を代入
  $stmt->bindValue(':active_day_str0', $active_day_str0, PDO::PARAM_STR);
  if($stmt->execute()){
      return TRUE;
  }else{
      return FALSE;
  }
}

Answer 1

まず、プリペアドステートメントの基本として、

$sql = 'ALTER TABLE schedule ADD active_day_str0 int(1)';

↓

$sql = 'ALTER TABLE schedule ADD :active_day_str0 int(1)';

コロンが抜けています。

active_day_str0 はカラム名ですよね？

$stmt->bindValue(':active_day_str0', $active_day_str0, PDO::PARAM_STR);

string で値を与えようとした場合、実行されるSQLは

ALTER TABLE schedule ADD 'column_name' int(1)

となり、クォーテーションで括られることはお分かりですか？
この場合、SQLの文法エラーとなり、実行できません。

本来、ALTER 文は以下のように作られるべきです。シングルクォーテーションとバッククォートの違いはわかりますよね？

ALTER TABLE schedule ADD `column_name` int(1)

以上のことから、プリペアドステートメントにおいて、カラム名を外部から与える場合に用いることはできません。そういう目的で作られたものではないのです。

つまり、それはなぜかというと、運用の中でカラムを追加したり、削除したりする必要があるとしたら、それは設計上のミスだからです。
DBは正規化された設計であるべきだし、そうあるものとしてPHPも設計されているからです。

Comments

[m.ts10806]

作りに違和感あったのですがそこですね。カラム名に使うのが間違いですね。

[DaisukeKusakari]

>>シングルクォーテーションとバッククォートの違いはわかりますよね？
わかりませんでした。

>>運用の中でカラムを追加したり、削除したりする必要があるとしたら、それは設計上のミスだからです。
でしたら、そもそも設計方針が間違っていました。

ありがとうございました。

[退会済みユーザー]

>>>シングルクォーテーションとバッククォートの違いはわかりますよね？
>わかりませんでした。

ご自身でお調べになって理解したと受け取ってよいのでしょうか？

[DaisukeKusakari]

>>ご自身でお調べになって理解したと受け取ってよいのでしょうか？

はい、調べました。基本的なところを理解しておらず申し訳ございません。

Answer 2

バインド元が設置されてないからでは？

ADD active_day_str0 int(1)

↓

ADD :active_day_str0 int(1)

今回の問題とは関係ないですが引数が多すぎます。
情報が増えれば引数を追加しなければなりませんし、連番なのは中身が何かわかりづらくなるのではないでしょうか。
連想配列で受け取りそのkey,valueで対応された方が可読性も良く汎用性も高くなると思います。

Comments

[DaisukeKusakari]

ありがとうございます。
ご指摘の通りにしたところ別のエラーが発現しました。
SQL文の文法に誤りがあるそうですが、ご指導いただけないでしょうか。

SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near '= '2018-08-05' int(1)' at line 1

[DaisukeKusakari]

訂正：

Uncaught PDOException: SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near ''2018-08-05' int(1)' at line 1

[m.ts10806]

$active_day_str0 に日付が渡ってきてますかね。

[m.ts10806]

Kosuke_Shibuyaさんが回答してくださっていますね。
カラム名をプリペアドで設定するのは間違いです。値にすべきです。