私のmysqlクラスの関数の中に問題がありますか？

は、私は、テーブル内のレコーダーを更新するために書いたいくつかのMySQLのPHPのクラスは、更新のように提案し、すべてのものがのみ1機能ではありませんので、よくその機能をOKになるだろう、テーブルにデータを挿入書い私のmysqlクラスの関数の中に問題がありますか？

function update($table=null,$array_of_values=array(),$conditions='FALSE') { 
     if ($table===null || empty($array_of_values)) return false; 
     $what_to_set = array(); 
     foreach ($array_of_values as $field => $value) { 
      if (is_array($value) && !empty($value[0])) $what_to_set[]="'$field'='{$value[0]}'"; 
      else $what_to_set []= "'$field'='".mysql_real_escape_string($value,$this->con)."'"; 
     } 
     $what_to_set_string = implode(',',$what_to_set); 
     return $this->execute("UPDATE $table SET $what_to_set_string WHERE $conditions"); 
    }

いくつかのフィールドを更新するために使用しようとすると、エラーは返されず、すべてのエラーは返されません。

出典

2011-07-29 Marco

データベースコントローラが必要です。 – afuzzyllama

トラブルシューティングのテクニックですが、実際に実行しているステートメントを印刷してください。次に、mysqlコマンドラインでその正確な文を実行してみてください（基本的にはphpを画像から取り除きます）。また、phpはエラーを表示するためにphp.iniファイルに設定された適切なエラー報告レベルを持っている必要があります。 –

私は既にケビンとこれを行ったことがあります。私は問題を見つけました。偉大なコメントをありがとうございます。 – Marco

あなたはカラム名を一重引用符で囲みました。エイリアスを宣言するときを除いて、識別子は一重引用符で囲まれません。

UPDATE table SET color = 'red'

ませ

UPDATE table SET 'color' = 'red'

あなたのコードが生成するものです。予約語を識別子として許可したい場合は、それらを `一重引用符 'ではなく`バッククォート `で囲みます。

出典

2011-07-29 00:21:11

アドバイスいただきありがとうございますが、まだうまくいきません。 – Marco

1）わからないのでもう助けてもらえません。あなたが正しい変更をしたかどうか –

2）この関数は存在してはいけません。 PDOまたはmysqliプリペアドステートメントを使用します。これをあなた自身でロールする必要はなく、あなたはそれをうまくやっていません。すべての値は一重引用符で囲まれていません。 –

私のmysqlクラスの関数の中に問題がありますか？

答えて

関連する問題