mysql_num_rows(): supplied argument is not a valid MySQL result resource in

[roman]

En PHP5 existe ya manejo de excepciones al estilo clásico de try-catch. Pero más allá de eso, seguirías ocultando el error y ésa no es la solución. Ese mensaje de error se debe, las más de la veces, a que la consulta previa con mysql_query arrojó algún error y por tanto su resultado no es válido.

// Saludos

[richy08]

ok gracias roman por el dato sigo en php 4 un dia de estos que sea mas estable el php5 tendre que emprender el vuelo a la version 5 en mi caso este error me da por que el query no obtiene ningun resultado, por que cuando el query encuentra coincidencias no me tira ningun ningun error pero no se como controlarlo el codigo que uso es este

Código PHP:

[left]$ssql = "SELECT ahelp, concat( e.sLastName, '_', e.sFirstName ) AS empleado, scompany, Date, sTitle
FROM sitiempleados e, siticontactos c, helpdesk h
WHERE e.aempleados = h.scliente
AND c.acode = sAssigned
AND sOrdClo <> 'Y'";[/left]

$rs = mysql_query($ssql,$link);
$num_total_registros = mysql_num_rows($rs); 


sera que tendre que usar la @ gracias.

[roman]

PHP5 es estable desde hace ya varios años. Y no creo que el error se deba a que la consulta no encuentre coincidencias, porque en ese caso, el resultado de mysql_query sería válido y mysql_num_rows devolvería 0.

Prueba con:

Código PHP:

$rs = mysql_query($ssql,$link) or die(mysql_error()); 


para que puedas ver cuál es el error que está generando la consulta.

// Saludos

[richy08]

si efectivamente tengo un error de sintaxis que extraño que cuando encuentre resultados no lo tome en cuenta buscare donde le concatena ese apostrofe al 8 ya que es una variable que paso por el metodo GET de un form mil gracias.

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 'where DateC>='2008-01-01' and DateC<='2008-01-31' and sClient=8'

[roman]

Creo que ese apóstrofe sólo es parte del mensaje de error para enmarcar la parte problemática:

'where .... 8[size="4"]'

¿Cuál es el tipo de datos del campo sClient? ¿No tendrás que entrecomillarlo también?

// Saludos

[richy08]

uffs ya revise y el error es que voy concatenando cadenas y encontre que tenia dos where en la sentencia y ese es el problema muchisimas gracias por tu ayuda roman.

[D-MO]

Bueno, según se vé, el problema ya está solucionado, sin embargo aún quedan las dudas:

• ¿habrá que usar las '@' en estos casos?,
• ¿Otra forma de evitar esto?...

Bueno, puesto que el error se debe a que el SQL no era válido, pueda que exista alguna aplicación donde exista la necesidad de que el usuario sea quién escriba la consulta. Para evitar este mensaje en un lugar inapropiado podemos hacer lo siguiente:

Código PHP:

$r = mysql_query('SELECT ...');
if($r){
  $total = mysql_num_rows($r);
} else {
// Informar al usuario del error
} 


A lo mejor no sea algo realmente común (a excepción de aplicaciones como phpMyAdmin claro ), pero pueda que en algún momento se necesite.

Saludos.

PS// Bendita maña de escribir unas cuantas líneas de código y guardar el documento [ CTRL ] + [ S ], lo acabo de hacer y ya saben lo que pasó

[dec]

Hola,

Yo he visto utilizar para esto la función "mysql_error()", puesto que retorna el mensaje de error de MySQL, o una cadena vacía, si no hay error alguno. No sé si servirá o no decirlo aquí ahora.

[richy08]

D-Mo la idea de que le usuario pueda meter sentencias sql es un poco extraña no te parece , si de por si ya es bastante teidoso estar cuidandote de la inyeccion de sql para que le darias a un usuario oportunidad de que te ejecute en una aplicacion sentencias sql , y la funcion de mysql_error no la habia visto le echare unvistazo mil gracias.

[D-MO]

Empezado por dec

Hola,

Yo he visto utilizar para esto la función "mysql_error()", puesto que retorna el mensaje de error de MySQL, o una cadena vacía, si no hay error alguno. No sé si servirá o no decirlo aquí ahora.

Que tal David, ¿Como te vá?... La función la utilizas para lo que yó he dicho, o haces referencia a lo que dijo román?

Empezado por richy08

D-Mo la idea de que le usuario pueda meter sentencias sql es un poco extraña no te parece , si de por si ya es bastante teidoso estar cuidandote de la inyeccion de sql para que le darias a un usuario oportunidad de que te ejecute en una aplicacion sentencias sql , y la funcion de mysql_error no la habia visto le echare unvistazo mil gracias.

Bueno, como expresé arriba, es poco común pero puede darse el caso, phpMyAdmin y/o TurboDBAdmin por ejemplo.

Saludos

Saludos.

[richy08]

Empezado por D-MO

Que tal David, ¿Como te vá?... La función la utilizas para lo que yó he dicho, o haces referencia a lo que dijo román?

Bueno, como expresé arriba, es poco común pero puede darse el caso, phpMyAdmin y/o TurboDBAdmin por ejemplo.

Saludos

Saludos.

Si en esos casos tienes mucha razon

[roman]

Empezado por dec

Yo he visto utilizar para esto la función "mysql_error()", puesto que retorna el mensaje de error de MySQL, o una cadena vacía, si no hay error alguno.

Empezado por D-MO

Que tal David, ¿Como te vá?... La función la utilizas para lo que yó he dicho, o haces referencia a lo que dijo román?

Soy una persona autocontradictoria Vi el comentario de dec y pensé mencionar un punto en contra, pero luego vi el de D-MO y me di cuenta que yo mismo ya lo había mencionado.

En fin, lo que quiero comentar, y de esto me parece que dec está muy al tanto, es que mysql_error hay que usarla con cuidado. Es decir, vale dejar el mensaje durante las pruebas pero no ya "en producción" puesto que la información puede proporcionar detalles acerca de la estructura de nuestras tablas.

// Saludos