2017-01-07 7 views
0

私は関数をビルドしようとしています(別のものを変更しています)。私の関数を使ってINSERTコマンドを実行すると、フォーミュレーションがオフになっているようです。私は過去2時間を探していて、どこに行ったかも知れません。PHP関数経由で挿入するエラー

以下のエラーコードがあります。ここ

public static function addCompany($toInsert = array()){ 
    self::construct(); 
    if(count($toInsert) == 0){ 
    echo "Nothing posted!"; 
    }else { 
    $keys = array_keys($toInsert); 
    $columns = implode(",", $keys); 
    $colVals = implode(",:", $keys); 

    $sql = self::$dbh->prepare("INSERT INTO companys {$columns} VALUES(:$colVals)"); 
    //$sql->bindValue(":id", $company); 
    foreach($toInsert as $key => $value){ 
    $value = htmlspecialchars($value); 
    $sql->bindValue(":$key", $value); 
    } 
    $sql->execute(); 
    return true; 
} 
} 

が配列に来ている。

$vname = $_POST["name"]; 
$vlogo = $_POST["logo"]; 
$vinfo = $_POST["info"]; 
$vsite = $_POST["site"]; 
$vest = $_POST["est"]; 
$data = array('name' => $vname, 'logo' => $vlogo, 'info' => $vinfo, 'site' => $vsite, 'est' => $vest); 

ここでエラーがあります。

2017/01/07 18:30:22 [error] 9682#9682: *3352 FastCGI sent in stderr: "PHP message: PHP Fatal error: Uncaught exception 'PDOException' with message 'SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'name,logo,info,site,est VALUES('test','test','test','test','test')' at line 1' in /var/www/xxxx/inc/xxxxxxx.php:938

どのようなヘルプも素晴らしいでしょう。

+0

なぜあなたは、このような列/値を構築しようとしていますか?これは第4番目の質問です。 – chris85

+0

あなたが構築していると思われるクエリ、つまり '$ sql =" INSERT INTO companys {$ columns} VALUES(:$ colVals) "をエコーすることから始めます。 echo $ sql; 'これは何をしているのでしょうか? – RiggsFolly

+0

** http://dev.mysql.com/doc/refman/5.7/en/insert.html** –

答えて

3

あなたのINSERTクエリが間違っています。列名をカプセル化したカッコが見つかりませんでした。

$sql = self::$dbh->prepare("INSERT INTO companys {$columns} VALUES(:$colVals)"); 
               ^^^^^^^^^^ 

あなたのクエリ文はこのようにする必要があり、ここを参照してください:

$sql = self::$dbh->prepare("INSERT INTO companys ({$columns}) VALUES(:$colVals)"); 
+0

あなたは今回の* Eagle Eye *でした。私は文字通り彼らが何を持っていたかを見るために(目を)すっごく見せなければならなかった。 –

+0

@ Fred-ii- * haha​​ha *、ええ、場合によっては、考えられる構文エラーを探すためにズームインすることもあります。 :-) –

+0

私はかなり高解像度の画面/解像度を使い果たしているからです。それは誤解を招くほどのものでした。良いキャッチ;-) –

関連する問題