Entrar

Fallo en código

Rabs · · Solicitar retirada
Bien, pues como haze ya timepo que no hago, os voy a preguntar sobre un código:
<!DOCTYPE html> 
<html xmlns="http://www.w3.org/1999/xhtml" dir="ltr" lang="es-ES"> 
<!-- Zona de carga -->
<?php
include ("../enlaces.php");

?>
<!-- Fin zona de carga -->
<head> 
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1" />
<title>Sistema de noticias en PHP</title>
<?php 
include ("../conexionsql.php");
//conectamos a la base 
$connect=mysql_connect("xxxxx","xxxxxxxx" ,"xxxxxxxxxx"); 
//Seleccionamos la base 
mysql_select_db("xxxxxxxxxxxxxxx",$connect);

//hacemos las consultas 
$result=mysql_query("SELECT * FROM `$database`.`novedades`", $connect); 
$totalregistros=mysql_num_rows($result); 
?>
</head>

<body>

<? 
// Recibimos la variable llamada id, que es la que se pone en la URL (novedad.php?id=ID DE LA NOTICIA)
$id = htmlentities($_GET[id]);
$id = strip_tags($id);

//conectamos a la base 
$connect=mysql_connect("xxxxxxxxxxxxx","xxxxxxxxxx" ,"xxxxxxxxxxxxx"); 
//Seleccionamos la base 
mysql_select_db("xxxxxxxxxxxxxx",$connect);

// Hacemos las consultas para seleccionar la noticia correspondiente
$result=mysql_query("select * from `$database`.`novedades` where id_noticia=$id", $connect) or die (mysql_error());

// Aquí lo que hacemos es que si alguien pone en la url algo así como novedad.php?id=-1 le rediriga a una página en concreto de error, ya que poner el número negativo lo aprovechan los crackers. Este código me lo pasó Rabs, gracias!
if ($id<0) {
header ("location: URL");
}

// Aquí es lo mismo que arriba pero para si ponen de ID el número 0
if ($id==0) {
header ("location: URL");
}

if (mysql_num_rows($result)) { 
   while ($qry = mysql_fetch_array($result)) {

echo "$qry[titulo]<br/>Publicado por $qry[autor] el día $qry[fecha]<br/><br/>$qry[novedad]<br/><br/>";
}
}
?>
</body>
</html>

Está sacado del sistema de noticias de Doodleo, pero modificado.
El error es:
You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '' at line 1

A ver si me podies ayudar ;)
skaparate · · Solicitar retirada
Hola Rabs:

Cuando tengas un problema con SQL en el que hayan variables involucradas, lo mejor es mostrar el contenido de todas las variables y ver que ocurre. El error que mencionas es porque alguna variable esta vacía, así que debes revisar que $database y $id no estén vacías:

<!DOCTYPE html> 
<html xmlns="http://www.w3.org/1999/xhtml" dir="ltr" lang="es-ES"> 
<!-- Zona de carga -->
<?php
include ("../enlaces.php");

?>
<!-- Fin zona de carga -->
<head> 
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1" />
<title>Sistema de noticias en PHP</title>
<?php 
include ("../conexionsql.php");
//conectamos a la base 
$connect=mysql_connect("xxxxx","xxxxxxxx" ,"xxxxxxxxxx"); 
//Seleccionamos la base 
mysql_select_db("xxxxxxxxxxxxxxx",$connect);

//hacemos las consultas 
[COLOR="#FFA07A"]$tmp = "SELECT * FROM `$database`.`novedades`";
echo "Primera consulta: $tmp<";
$result=mysql_query($tmp, $connect);[/COLOR] 
$totalregistros=mysql_num_rows($result); 
?>
</head>

<body>

<? 
// Recibimos la variable llamada id, que es la que se pone en la URL (novedad.php?id=ID DE LA NOTICIA)
$id = htmlentities($_GET[id]);
$id = strip_tags($id);

//conectamos a la base 
$connect=mysql_connect("xxxxxxxxxxxxx","xxxxxxxxxx" ,"xxxxxxxxxxxxx"); 
//Seleccionamos la base 
mysql_select_db("xxxxxxxxxxxxxx",$connect);

// Hacemos las consultas para seleccionar la noticia correspondiente
[COLOR="#FFA07A"]$tmp = "select * from `$database`.`novedades` where id_noticia=$id";
echo "Segunda consulta: $tmp<br />";
$result=mysql_query($tmp, $connect) or die (mysql_error());[/COLOR]

// Aquí lo que hacemos es que si alguien pone en la url algo así como novedad.php?id=-1 le rediriga a una página en concreto de error, ya que poner el número negativo lo aprovechan los crackers. Este código me lo pasó Rabs, gracias!
if ($id<0) {
header ("location: URL");
}

// Aquí es lo mismo que arriba pero para si ponen de ID el número 0
if ($id==0) {
header ("location: URL");
}

if (mysql_num_rows($result) > 0) { 
   while ($qry = mysql_fetch_array($result)) {

echo "$qry[titulo]<br/>Publicado por $qry[autor] el día $qry[fecha]<br/><br/>$qry[novedad]<br/><br/>";
}
}
?>
</body>
</html>

Ejecútalo con esas modificaciones y busca las salidas "Primera Consulta" y "Segunda Consulta".

Suerte :D.