Json ve MySQL sorun

1 Cevap php

Burada json php için benim kod

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;

ben bu "req.php? kapak = 3434" gibi bu php veri ilanıyla

ve gördüğünüz gibi $id = $_GET['lid']; gibi "kapak" oluyor

ama benim mysql i yazarken WHERE label_id = '$id' çalışmıyor

herhangi bir öneriniz?

Teşekkürler

1 Cevap

Sen bir işlev içinde küresel $id başvuran. Eğer küresel olarak işaretlemek gerekir:

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

Ya da muhtemelen daha iyi bir çözümdür üçüncü parametre olarak işleve geçmek olabilir.

Bu kod, SQL enjeksiyon saldırılarına karşı savunmasız olduğunu unutmayın. Ya alıntı $id (örn. kullanarak gerektiği mysql_real_escape_string() ), or if it always an integer you could cast it, e.g. $id = (int) $id. Daha da iyisi, PDO kullanabilir ve [(4)] {kullanın }, hangi bu sorunu ortadan kaldırır.