sql文へのPHP変数の代入（2回目）

Question

sql文にPHP変数を代入しカラム名を配列に応じて変更したいのですが、文法エラーが発生してしまいます。
クォーテーションの違いを理解したはずでしたが...
ご教示いただけますよう、よろしくお願いいたします。

function update_active_days($dbh, $active_day_str){
  $sql = 'SELECT column_name from information_schema.columns where table_name = "schedule"';
  $stmt = $dbh->prepare($sql);
  $stmt->execute();
  $column_name = array_map('current', $stmt->fetchAll());
  //カラム名”id”を配列から削除
  while(($index = array_search("id", $column_name, true )) !== false ) {
		unset($column_name[$index]) ;
	}
  //カラム名の差分を取ってキーを0から始まる連番に振り直す
  $column_name_diff_old = array_values(array_diff($column_name, $active_day_str));
  $column_name_diff_new = array_values(array_diff($active_day_str, $column_name));
  $cnt = count($column_name_diff_old);
  if ($cnt == 0){
    exit;
  }elseif($cnt > 0){
    for ($i=0; $i < $cnt; $i++){
      $sql = "SELECT '$column_name_diff_old[$i]' from schedule AS '$column_name_diff_new[$i]'";
      $stmt = $dbh->prepare($sql);
      $stmt->execute();
    }
  }
}

エラー
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-09-01'' at line 1 in

Comments

[m.ts10806]

エラーが出たのであればそのエラー文をそのまま提示してください

[m.ts10806]

コードも切り出しすぎてちゃんと動くものではないですしそれぞれの配列がどのような値が入っているかわかりません。簡単に見える問題でも質問内容が不十分だと的確な回答は得られませんよ

[DaisukeKusakari]

毎度ありがとうございます。失礼いたしました。

Answer 1

変数にSQLを入れるのは良いのですが、その文法を確認せずに毎回PHPを実行して確認するのは効率が悪いです。
まずはechoで文字列出力し、想定通りのSQLが組上がっているか、実際にDBに投げてみて問題ないかを確認してください。
SQLもエラー番号が出ているのでしたらその番号で調べればエラーの原因が分かります。

ちなみに
シングルクォーテーション'はDBにおさめる値、配列で言えばkey,valueだとvalueに使うもので
keyにあたるカラム名やテーブル名にはバッククォート`を使います。
※全角文字をカラムやテーブルに使えるDBではまた別ですが今回は当てはまらないなで割愛

ついでにアドバイスをしておくとダブルクォート内ではで配列を展開する場合に限らず変数は{}で囲っておいた方が良いです。

Answer 2

    for ($i=0; $i < $cnt; $i++){
      $sql = "SELECT '$column_name_diff_old[$i]' from schedule AS '$column_name_diff_new[$i]'";

      var_dump($sql); // こうして意図した文字列になっているかを確認してください。

      $stmt = $dbh->prepare($sql);
      $stmt->execute();
    }

Comments

[DaisukeKusakari]

デバック方法すらなるほど！といったお粗末です。
ありがとうございます。