2017-09-07 7 views
-7
<?php 
    include("conn.php"); 
    if(isset($_POST['save'])){ 

     $title=$_POST['title']; 
     $genre=$_POST['genre']; 
     $year=$_POST['year']; 

     $sql = "INSERT INTO `add`(`title`, `genre`, `year`) VALUES ('$title','$genre','$year')" ; 
      $query = mysql_query($sql); 
     if (empty ($_POST["title"] || ["genre"] || ["year"])){ 
      echo "You did not fill out the required fields."; 
     } 
     else ($query){ 
      echo "Successfully Inserted . <a href = 'index.php'> Ok </a>"; 
     } 

    } 


?> 

私は、コードを実行するたびに、このエラーが述べている:このPHPコードのエラーは何ですか?

"Parse error: syntax error, unexpected '{' in C:\xampp\htdocs\Practise PHOBIAmming Skills\sample1\add.php on line 24"

+1

**警告**:PHPを学んでいるだけの方は、[mysql_query'](http://php.net/manual/en/function.mysql-query.php)インタフェースを使用しないでください。それはPHP 7で削除されたのでとてもひどいと危険です。[PDOのようなものは学ぶのが難しくない](http://net.tutsplus.com/tutorials/php/why-you-should-be-using-phps -pdo-for-database-access /)と[PHP The Right Way](http://www.phptherightway.com/)のようなガイドがベストプラクティスを説明しています。あなたのユーザーデータは**適切にエスケープされていない**(http://bobby-tables.com/php.html)であり、[SQLインジェクションのバグ](http://bobby-tables.com/)があります。悪用される。 – tadman

+0

あなたのコードには24行があるとは思わない。コードの一部を隠しましたか? –

+0

これはわずか21行です。留置は重要であることを覚えておいてください。このような間違いを見つけるのに役立ちます。 – tadman

答えて

1

あなたが追加した条件をelse

else ($query){ 

それを削除するか、それをelseif作るためにあなたのロジックに従って

+0

はい、私はそれを得た。ありがとう:) –

+0

基本的なPHP構文エラーに関する質問には答えてはいけません。それらは、[PHP Parse/Syntax Errors]の複製として閉じなければなりません。どのようにそれらを解決するのですか?](// stackoverflow.com/questions/18050071)のみ。参照:[トピックから離れた質問について助言する必要がありますか?](// meta.stackoverflow.com/q/276572/1768232)トピック以外の質問は閉じて削除することができますが、とにかく投稿を無効にする可能性があります。 –

関連する問題