consulta MySql!

[aado29]

hola como estas? como saban he hecho un sistema de noticias con php y mysql.. cuento mi proble cuando trigo mi noticia hacien esto www.miweb.com/publicaciones.php?idNoticia=3&titulo=hola-mundo perteneciendo idNoticia = 3 y titulo = hola-mundo a la misma columna en mi base de datos pero cuando cambio ya se mi el titulo o mi idNoticia me lanza un error (mysql_fetch_assoc)...

Cual es la ayuda que nesecito? que me ensenen a crear un filtro que me diga que si el titulo y el idnoticia no equivalen me lanze un error que por su puesto yo pueda editar.

Muchas gracias Espero su respuesta.

[quimfv]

Intenta explicarte mejor.....


SELECT * FROM noticias WHERE idNoticia=3 and titulo="hola-mundo";

Si eso retorna false es que no ha encontrado una noticia que cumpla las dos cosas....si no retornara la noticia

[Triby]

Y se supone que tenemos que adivinar cómo lees la noticia? o quieres que alguien haga el trabajo por ti?

Muestra tu código, de otra forma no será posible ayudarte.

[aado29]

ok este es mi php:

Código PHP:

  <?

// iniciamos session
session_start ();

// archivos necesarios
require_once 'admin/config.php';
require_once 'admin/conexion.php';
require_once 'admin/esUsuario.php';

// obtengo puntero de conexion con la db
$dbConn = conectar();

// verificamos que este conectado el usuario
if ( !empty( $_SESSION['usuario'] ) && !empty($_SESSION['password']) ) {
    $arrUsuario = esUsuario( $_SESSION['usuario'], $_SESSION['password'], $dbConn );        
}

if ( !empty($_POST['submit']) ) {
    
    if ( !empty($_POST['comentario']) )     $comentario     = $_POST['comentario'];
    if ( !empty($_GET['idNoticia']) )    $idNoticia     = $_GET['idNoticia'];
    if ( !empty($arrUsuario['idUsuario']))    $idUsuario    = $arrUsuario['idUsuario'];
    
    // completamos la variable error si es necesario
    if ( empty($comentario) )     $error['comentario']         = true;
    if ( empty($idNoticia) )     $error['idNoticia']         = true;
    if ( empty($idUsuario) )     $error['idUsuario']         = true;
    
    // si no hay errores registramos al usuario
    if ( empty($error) ) {
        
        // inserto los datos de registro en la db
        $query  = "INSERT INTO `comentarios` (comentario, idUsuario, idNoticia) VALUES ('$comentario','$idUsuario','$idNoticia')";
        $result = mysql_query($query, $dbConn);
        
        header( 'Location: publicaciones.php?idNoticia='.$idNoticia );
        die;
        
    }
    
}

// traemos la noticia
$query = "SELECT noticias.idNoticia, noticias.titulo, noticias.copete, noticias.cuerpo, noticias.fCreacion, noticias.fModificacion, categorias.valor as categoria, usuarios.usuario FROM `noticias` 
INNER JOIN `categorias` ON categorias.idCategoria = noticias.idCategoria 
INNER JOIN `usuarios` ON usuarios.idUsuario = noticias.idUsuario 
WHERE idNoticia = '" . $_GET['idNoticia'] . "' AND titulo = '" . $_GET['titulo'] . "'" ;
$resultado = mysql_query ($query, $dbConn);
$noticia = mysql_fetch_assoc ($resultado);

// traemos los comentarios aprobados
$arrComentarios = array();
$query = "SELECT comentarios.idComentario, comentarios.comentario, usuarios.usuario  
FROM `comentarios` 
INNER JOIN `usuarios` ON comentarios.idUsuario = usuarios.idUsuario 
WHERE comentarios.estado = 'apto' AND comentarios.idNoticia = " . $_GET['idNoticia'] . " 
ORDER BY comentarios.idComentario DESC";
$resultado = mysql_query ($query, $dbConn);
while ( $row = mysql_fetch_assoc ($resultado)) {
    array_push( $arrComentarios,$row );
}

?>   

[eduardomxm]

Para empezar tienes mucha inseguridad realizando consultas de esa manera obteniendo valores get

[gnzsoloyo]

Código SQL:

SELECT 
    noticias.idNoticia, 
    noticias.titulo, 
    noticias.copete, 
    noticias.cuerpo, 
    noticias.fCreacion, 
    noticias.fModificacion, 
    categorias.valor AS categoria, 
    usuarios.usuario 
FROM `noticias` 
    INNER JOIN `categorias` ON categorias.idCategoria = noticias.idCategoria 
    INNER JOIN `usuarios` ON usuarios.idUsuario = noticias.idUsuario 
WHERE idNoticia = '" . $_GET['idNoticia'] . "' AND titulo = '" . $_GET['titulo'] . "'
 

Mira, si ya tienes el ID de noticia, se supone que el título es irrelevante. Y si tienes el título correctamente, el idNoticia no es necesario.
Pero si uno de los dos valores no llega o no coninciden al mismo tiempo con la oticia, esa consulta no devovlerá datos.
¿Tienes claro eso?
Si lo que quieres es que te evalúe uno u otro, pero al menos uno de ambos, entonces la sintaxis es otra:

Código SQL:

SELECT 
    noticias.idNoticia, 
    noticias.titulo, 
    noticias.copete, 
    noticias.cuerpo, 
    noticias.fCreacion, 
    noticias.fModificacion, 
    categorias.valor AS categoria, 
    usuarios.usuario 
FROM `noticias` 
    INNER JOIN `categorias` ON categorias.idCategoria = noticias.idCategoria 
    INNER JOIN `usuarios` ON usuarios.idUsuario = noticias.idUsuario 
WHERE idNoticia = " . $_GET['idNoticia'] . " OR titulo = '" . $_GET['titulo'] . "'
 

Por otro lado, usar el título como parte de la búsqueda puede ser un poco inseguro, si el valor ingresado en formulario no es exactamente el mismo. Con una sola letra de diferencia, y ese registro no lo encontrarás.

Además, no veo que estés validando en su script PHP que las consultas realmente estén devolviendo datos. Cometes el clásico error de asumir que la consulta funciona y devuelve datos, no lo que no es necesariamente cierto. Nunca.
Valida la respuesta (el result), antes de intentar hacer un fetch.

[aado29]

gnzsoloyo de verdad muchas gracias.. pero ya encontre lo que necesitaba no era algo que con leer un poco la documentacion php no resolviera.. me que algo asi :

Código PHP:

  $query = "SELECT noticias.idNoticia, noticias.titulo, noticias.copete, noticias.cuerpo, noticias.fCreacion, noticias.fModificacion, categorias.valor as categoria, usuarios.usuario FROM `noticias` 
INNER JOIN `categorias` ON categorias.idCategoria = noticias.idCategoria 
INNER JOIN `usuarios` ON usuarios.idUsuario = noticias.idUsuario 
WHERE idNoticia = '" . $_GET['idNoticia'] . "' AND titulo = '" . $_GET['titulo'] . "'" ;
$resultado = mysql_query ($query, $dbConn);
if (mysql_num_rows($resultado) == 0) {
    header('Location: ../'.$_GET['titulo'].'-'. $_GET['idNoticia'] ); 
}
else {
    $noticia = mysql_fetch_assoc ($resultado);
} 
   

esto lo que me dice es que si la consulta hecha por $_GET es igual a 0 que redireccione y si encuenta algo en base de datos me los muetre con, ejemplo

Código HTML:

<p><?php echo $noticia['idNoticia']; ?></p><br>
<p><?php echo $noticia['titulo']; ?></p>