Json и Mysql проблема

Question

вот мой код для json php

include("connect.php"); 
$id = $_GET['lid'];
function countRec($fname,$tname) {
$sql = "SELECT * FROM `mail` WHERE confirmed = 'no' AND label_id = '". $id ."'";
$result = mysql_query($sql) or die ('test'); 
$num = mysql_num_rows($result);
return $num;
}

$page = $_POST['page'];
$rp = $_POST['rp'];
$sortname = $_POST['sortname'];
$sortorder = $_POST['sortorder'];

if (!$sortname) $sortname = 'ID';
if (!$sortorder) $sortorder = 'desc';

    $sort = "ORDER BY $sortname $sortorder";

if (!$page) $page = 1;
if (!$rp) $rp = 10;

$start = (($page-1) * $rp);

$limit = "LIMIT $start, $rp";

$sql = "SELECT * FROM `mail` WHERE confirmed = 'no' AND label_id = '". $id ."' $sort $limit";
$result = mysql_query($sql) or die ('test'); 

$total = countRec();

header("Expires: Mon, 26 Jul 1997 05:00:00 GMT" );
header("Last-Modified: " . gmdate( "D, d M Y H:i:s" ) . "GMT" );
header("Cache-Control: no-cache, must-revalidate" );
header("Pragma: no-cache" );
header("Content-type: text/x-json");
$json = "";
$json .= "{\n";
$json .= "page: $page,\n";
$json .= "total: $total,\n";
$json .= "rows: [";
$rc = false;
while ($row = mysql_fetch_array($result)) {
if ($rc) $json .= ",";
$json .= "\n{";
$json .= "id:'".$row['ID']."',";
$json .= "cell:['".$row['email']."'";
$json .= ",'".addslashes($row['name'])."'";
$json .= ",'".addslashes($row['country'])."'";
$json .= ",'".addslashes($row['bus'])."'";
$json .= ",'".addslashes($row['website'])."'";
$json .= ",'".addslashes($row['music'])."'";
$json .= ",'".addslashes($row['radio'])."']";
$json .= "}";
$rc = true;
}
$json .= "]\n";
$json .= "}";
echo $json;

Я отправляю данные на этот php как "req.php? Lid = 3434"

и получение "крышки", как $id = $_GET['lid'];, как вы можете видеть

но в моем mysql, когда я пишу WHERE label_id = '$id', это не работает

есть предложения?

Спасибо

Answer 1

Вы ссылаетесь на глобальный $id внутри функции. Вы должны пометить его как глобальный:

function countRec($fname,$tname) {
    global $id;
    //etc
}

Или вы можете передать его функции в качестве третьего параметра, что, вероятно, является лучшим решением.

Обратите внимание, что этот код уязвим для атак SQL-инъекций. Вы должны либо цитировать $id (например, используя mysql_real_escape_string()), либо, если это всегда целое число, вы могли бы его разыграть, например. $id = (int) $id. Более того, вы можете использовать PDO и использовать подготовленные отчеты и связанные параметры , что устраняет эту проблему.