Question

私はこのコードのトラブルサイコーいます：データベースで

if ($_GET['offline']) {$extranet = $_GET['offline'];} else {$extranet = $online;} 
$sql = mysqli_query($db,"UPDATE tbl_system SET value = '$extranet' WHERE property = 'extranet'"); 
echo $_GET['offline']; 
echo $extranet; 
echo $online; 

、プロパティ=エクストラネットは、エコーの1 結果であるフィールドの値：

$_GET['offline'] = 0 
$extranet  = 1 
$online   = 1 

を

今、明らかに、何かがここで間違っている

このGETを与えるためのURLは次のとおりです。。?app=admincp&offline=0、そう$_GET['offline']は問題ではありません

問題はifステートメントにある必要がありますが、わかりません。

アイデアはありますか？

Answer 1

$ _GET ['offline']は0で、0はfalseと評価されるため、ステートメントは正しく処理されます。オフラインが引数として渡されているかどうかを確認する場合は、使用する必要があります。

if (!isset($_GET['offline'])) {$extranet = $_GET['offline'];} else {$extranet = $online;} 

Answer 2

if ($_GET['offline']) 

PHPでは "0"という文字列がfalseと評価されるため、falseが返されるため、if文の条件はtrueに評価されません。代わりにissetまたはarray_key_existsを使用してください。例えば：

if (isset($_GET['offline'])) { ... } 

か：あなたのケースでは

if (array_key_exists('offline',$_GET)) { ... } 

、あなたは値が正確に文字列をゼロ「0」であるかどうかを知りたいでしょう。私はstrcmpを使用することをお勧めします：

if (strcmp($_GET['offline'],"0") === 0) { 
    $extranet = $_GET['offline']; 
} else { 
    $extranet = $online; 
} 
$sql = mysqli_query($db,"UPDATE tbl_system SET value = '$extranet' WHERE property = 'extranet'"); 
echo $_GET['offline']; 
echo $extranet; 
echo $online; 

Answer 3

$_GET['offline']は、あなたが提供するURLにはゼロであるので、そのブロック内if($_GET['offline'])とすべてがヒットすることはありません。代わりにisset()を使用してください。

Answer 4

Yikes！ karimのコメントに加えて、あなたの入力を害さない！あなたはURLパラメータを取って、それをSQLクエリに直接入れています。 AのURL

?app=admincp&offline=';DROP table tbl_system;-- 

あなたを台無しにするでしょうか？

Answer 5

if (strcmp($_GET['offline'],"0") === 0) { 
    $extranet = $_GET['offline']; 
} else { 
    $extranet = $online; 
} 

これは$ _GET [「オフラインが」] !isset($_GET['offline'])