2012-10-23 3 views
0

Почему значение не равно 0 и вставлено в мою таблицу? Это мой код:

<?php 
    if ($_GET["subscribe"] == 0) 
     $Competition = $_GET["subscribe"]; 
     { 
      if (isset($_GET["subscribecontestant"])) 
      { 
       $sql1 = "INSERT INTO wedstrijdresultaten (WedstrijdID,DuifID,Username) VALUES (".$Competition.",".$_GET["subscribecontestant"].", '".$_SESSION['user']."')"; 
       $result1 = mysql_query($sql1); 
      } 
     } 
+5

Ваш код - беспорядок. Есть '' все вокруг, кроме того, где они должны быть. Например, первое, если не имеет одного, так что только следующая строка будет выполняться условно. – GolezTrol

+0

Может быть $ _GET ["subscribe"] равно null. http://stackoverflow.com/questions/1782759/php-considers-null-is-equal-to-zero – henkieee

+0

B.t.w. '$ _GET [" subscribe "] == 0' неверное сравнение. Он * может * оценивать значение true, но на самом деле он никогда не будет '0' точно, потому что параметры $ _GET всегда являются строками (или массивами строк, если PHP делает свою магию). Поэтому сравнение предполагает неправильное предположение. – GolezTrol

ответ

1

неправильное размещение {. Во-первых { должен быть рядом с первого if заявления

if ($_GET["subscribe"] == 0) { 
$Competition = $_GET["subscribe"]; 
if (isset($_GET["subscribecontestant"])) 
{ 
$sql1 = "INSERT INTO wedstrijdresultaten (WedstrijdID,DuifID,Username) VALUES (".$Competition.",".$_GET["subscribecontestant"].", '".$_SESSION['user']."')"; 
$result1 = mysql_query($sql1); 
} 
} 
1

у вас есть небольшая ошибка в коде с кронштейном if заявления. Попробуйте

if ($_GET["subscribe"] == 0) 
{ 
    $Competition = $_GET["subscribe"]; 

    if (isset($_GET["subscribecontestant"])) 
    { 
     $sql1 = "INSERT INTO wedstrijdresultaten (WedstrijdID,DuifID,Username) " 
       . "VALUES (".$Competition."," 
       . $_GET["subscribecontestant"] . ", '" 
       . $_SESSION['user']."')"; 
     $result1 = mysql_query($sql1); 
    } 
} 
-1

Там может быть несколько причин, почему это не работает, есть скобка отсутствует, как показано в других ответах, а также мы не знаем, что вы передаете в качестве параметров GET.

Но вы, вероятно, должны игнорировать их и проконсультироваться с http://www.phptherightway.com/#databases, прежде чем вы сделаете больше, ваш пример широко открыт для уязвимостей безопасности.

+0

И, таким образом, этот ответ не является ответом и должен быть комментарием. – GolezTrol

+0

Я осознал это и внес поправки в этот вопрос. – Steve

Смежные вопросы