[SOLUCIONADO] Error en consultas Sql.

satjaen · #1 (**permalink**) 24/02/2013, 15:39

Hola tengo unas consultas Sqls y me dan un error porque estan mal construidas o son varias sqls. Se podrían juntar en una única Sql. Por favor me podeis ayudar a recomponerlas?

Código:

if ( ! empty($vte)) { 
$sql ="(SELECT 0 as qnum,A.id_aviso,A.telefonos,A.telefonos3,A.movil,A.dni,A.name,A.apellidos,B.localidad,B.calle
 
FROM clientes A,dir_clientes B WHERE A.id_aviso=B.id_aviso AND B.calle != '' AND  A.telefonos='$vte')
 
UNION ALL
 
(SELECT 1 as qnum,id_aviso,telefonos,telefonos3,movil,dni,name,apellidos,localidad,calle
 
FROM avisos WHERE telefonos='$vte' )
 
UNION ALL
 
(SELECT 2 as qnum,U.id_aviso,U.telefonos,U.telefonos3,U.movil,U.dni,U.name,U.apellidos,D.localidad,D.calle
 
FROM usuarios U,dir_usuarios D WHERE  U.id_aviso=D.id_aviso AND D.calle != ''AND  U.telefonos='$vte')";
}
if ( ! empty($movil)) { 
$sql ="(SELECT 0 as qnum,A.id_aviso,A.telefonos,A.telefonos3,A.movil,A.dni,A.name,A.apellidos,B.localidad,B.calle
 
FROM clientes A,dir_clientes B WHERE A.id_aviso=B.id_aviso AND B.calle != '' AND  A.movil='$movil')
 
UNION ALL
 
(SELECT 1 as qnum,id_aviso,telefonos,telefonos3,movil,dni,name,apellidos,localidad,calle
 
FROM avisos WHERE movil='$movil' )
 
UNION ALL
 
(SELECT 2 as qnum,U.id_aviso,U.telefonos,U.telefonos3,U.movil,U.dni,U.name,U.apellidos,D.localidad,D.calle
 
FROM usuarios U,dir_usuarios D WHERE  U.id_aviso=D.id_aviso AND D.calle != ''AND  U.movil='$movil')";
}
if ( ! empty($dni)) { 
$sql ="(SELECT 0 as qnum,A.id_aviso,A.telefonos,A.telefonos3,A.movil,A.dni,A.name,A.apellidos,B.localidad,B.calle
 
FROM clientes A,dir_clientes B WHERE A.id_aviso=B.id_aviso AND B.calle != '' AND  A.dni='$dni')
 
UNION ALL
 
(SELECT 1 as qnum,id_aviso,telefonos,telefonos3,movil,dni,name,apellidos,localidad,calle
 
FROM avisos WHERE dni='$dni' )
 
UNION ALL
 
(SELECT 2 as qnum,U.id_aviso,U.telefonos,U.telefonos3,U.movil,U.dni,U.name,U.apellidos,D.localidad,D.calle
 
FROM usuarios U,dir_usuarios D WHERE  U.id_aviso=D.id_aviso AND D.calle != ''AND  U.dni='$dni')";
}
if ( ! empty($calle)) { 
$sql ="(SELECT 0 as qnum,A.id_aviso,A.telefonos,A.telefonos3,A.movil,A.dni,A.name,A.apellidos,B.localidad,B.calle
 
FROM clientes A,dir_clientes B WHERE A.id_aviso=B.id_aviso AND B.calle != '' AND  B.calle like '%$calle%')

UNION ALL
 
(SELECT 1 as qnum,id_aviso,telefonos,telefonos3,movil,dni,name,apellidos,localidad,calle
 
FROM avisos WHERE calle like '%$calle%')
 
UNION ALL

(SELECT 2 as qnum, U.id_aviso,U.telefonos,U.telefonos3,U.movil,U.dni,U.name,U.apellidos,D.localidad,D.calle
 
FROM usuarios U,dir_usuarios D WHERE  U.id_aviso=D.id_aviso AND D.calle != ''AND  D.calle like '%$calle%')";
}

if ( ! empty($ape)) { 
$sql ="(SELECT 0 as qnum,A.id_aviso,A.telefonos,A.telefonos3,A.movil,A.dni,A.name,A.apellidos,B.localidad,B.calle
 
FROM clientes A,dir_clientes B WHERE A.id_aviso=B.id_aviso AND B.calle != '' AND  A.apellidos like '%$ape%')

UNION ALL
 
(SELECT 1 as qnum,id_aviso,telefonos,telefonos3,movil,dni,name,apellidos,localidad,calle
 
FROM avisos WHERE apellidos like '%$ape%')
 
UNION ALL

(SELECT 2 as qnum, U.id_aviso,U.telefonos,U.telefonos3,U.movil,U.dni,U.name,U.apellidos,D.localidad,D.calle
 
FROM usuarios U,dir_usuarios D WHERE  U.id_aviso=D.id_aviso AND D.calle != ''AND  U.apellidos like '%$ape%')";
}
if ( ! empty($name)) { 
$sql ="(SELECT 0 as qnum,A.id_aviso,A.telefonos,A.telefonos3,A.movil,A.dni,A.name,A.apellidos,B.localidad,B.calle
 
FROM clientes A,dir_clientes B WHERE A.id_aviso=B.id_aviso AND B.calle != '' AND  A.name like '%$name%')

UNION ALL
 
(SELECT 1 as qnum,id_aviso,telefonos,telefonos3,movil,dni,name,apellidos,localidad,calle
 
FROM avisos WHERE name like '%$name%')
 
UNION ALL

(SELECT 2 as qnum, U.id_aviso,U.telefonos,U.telefonos3,U.movil,U.dni,U.name,U.apellidos,D.localidad,D.calle
 
FROM usuarios U,dir_usuarios D WHERE  U.id_aviso=D.id_aviso AND D.calle != ''AND  U.name like '%$name%')";
}
if ( ! empty($loca)) { 
$sql ="(SELECT 0 as qnum,A.id_aviso,A.telefonos,A.telefonos3,A.movil,A.dni,A.name,A.apellidos,B.localidad,B.calle
 
FROM clientes A,dir_clientes B WHERE A.id_aviso=B.id_aviso AND B.calle != '' AND  B.localidad like '%$loca%')

UNION ALL
 
(SELECT 1 as qnum,id_aviso,telefonos,telefonos3,movil,dni,name,apellidos,localidad,calle
 
FROM avisos WHERE localidad like '%$loca%')
 
UNION ALL

(SELECT 2 as qnum, U.id_aviso,U.telefonos,U.telefonos3,U.movil,U.dni,U.name,U.apellidos,D.localidad,D.calle
 
FROM usuarios U,dir_usuarios D WHERE  U.id_aviso=D.id_aviso AND D.calle != ''AND  D.localidad like '%$loca%')";
}

Gracias

quimfv · #2 (**permalink**) 25/02/2013, 04:17

Intenta publicar las de manera legible (bien identadas), explica que hacen y que pretendes hacer, que errores dan ... etc tenemos intención de ayudar... pero ponlo facil.

satjaen · #3 (**permalink**) 25/02/2013, 04:56

Cita:

Iniciado por quimfv

Intenta publicar las de manera legible (bien identadas), explica que hacen y que pretendes hacer, que errores dan ... etc tenemos intención de ayudar... pero ponlo facil.

Gracias, ahora mismo funciona bien si me da algún error te lo digo.

gnzsoloyo · #4 (**permalink**) 25/02/2013, 07:12

Lo que te están diciendo es que el código es ilegible como lo pones.
En SQL uno tiende a escribir las consultas de una forma más estructurada, de modo que sea sencillo de recorrer de arriba a abajo.
Además estás poniendo código PHP, cosa no permitida en estos foros, y usando la etiqueta equivocada (no es CODE ni PHP, o es el Highlight SQL o es MySQL).

Minimamente habría que ponerlo así:

Código MySQL:

Ver original(SELECT 0 qnum, A.id_aviso, A.telefonos, A.telefonos3, A.movil, A.dni, A.name, A.apellidos, B.localidad, B.calle
FROM clientes A, dir_clientes B 
WHERE A.id_aviso=B.id_aviso AND B.calle != '' AND A.telefonos='$vte')
UNION ALL
(SELECT 1 qnum, id_aviso, telefonos, telefonos3, movil, dni, name, apellidos, localidad, calle
FROM avisos 
WHERE telefonos='$vte' )
UNION ALL
(SELECT 2 qnum, U.id_aviso, U.telefonos, U.telefonos3, U.movil, U.dni, U.name, U.apellidos, D.localidad, D.calle
FROM usuarios U, dir_usuarios D 
WHERE U.id_aviso=D.id_aviso AND D.calle != ''AND U.telefonos='$vte');
 
(SELECT 0 qnum, A.id_aviso, A.telefonos, A.telefonos3, A.movil, A.dni, A.name, A.apellidos, B.localidad, B.calle
FROM clientes A, dir_clientes B 
WHERE A.id_aviso=B.id_aviso AND B.calle != '' AND A.movil='$movil')
UNION ALL
(SELECT 1 qnum, id_aviso, telefonos, telefonos3, movil, dni, name, apellidos, localidad, calle
FROM avisos 
WHERE movil='$movil' )
UNION ALL
(SELECT 2 qnum, U.id_aviso, U.telefonos, U.telefonos3, U.movil, U.dni, U.name, U.apellidos, D.localidad, D.calle
FROM usuarios U, dir_usuarios D 
WHERE U.id_aviso=D.id_aviso AND D.calle != ''AND U.movil='$movil');
 
(SELECT 0 qnum, A.id_aviso, A.telefonos, A.telefonos3, A.movil, A.dni, A.name, A.apellidos, B.localidad, B.calle
FROM clientes A, dir_clientes B 
WHERE A.id_aviso=B.id_aviso AND B.calle != '' AND A.dni='$dni')
UNION ALL
(SELECT 1 qnum, id_aviso, telefonos, telefonos3, movil, dni, name, apellidos, localidad, calle
FROM avisos 
WHERE dni='$dni' )
UNION ALL
(SELECT 2 qnum, U.id_aviso, U.telefonos, U.telefonos3, U.movil, U.dni, U.name, U.apellidos, D.localidad, D.calle
FROM usuarios U, dir_usuarios D 
WHERE U.id_aviso=D.id_aviso AND D.calle != ''AND U.dni='$dni');
 
(SELECT 0 qnum, A.id_aviso, A.telefonos, A.telefonos3, A.movil, A.dni, A.name, A.apellidos, B.localidad, B.calle
FROM clientes A, dir_clientes B 
WHERE A.id_aviso=B.id_aviso AND B.calle != '' AND B.calle like '%$calle%')
UNION ALL
(SELECT 1 qnum, id_aviso, telefonos, telefonos3, movil, dni, name, apellidos, localidad, calle
FROM avisos 
WHERE calle like '%$calle%')
UNION ALL
(SELECT 2 qnum, U.id_aviso, U.telefonos, U.telefonos3, U.movil, U.dni, U.name, U.apellidos, D.localidad, D.calle
FROM usuarios U, dir_usuarios D 
WHERE U.id_aviso=D.id_aviso AND D.calle != ''AND D.calle like '%$calle%');
 
(SELECT 0 qnum, A.id_aviso, A.telefonos, A.telefonos3, A.movil, A.dni, A.name, A.apellidos, B.localidad, B.calle
FROM clientes A, dir_clientes B 
WHERE A.id_aviso=B.id_aviso AND B.calle != '' AND A.apellidos like '%$ape%')
UNION ALL
(SELECT 1 qnum, id_aviso, telefonos, telefonos3, movil, dni, name, apellidos, localidad, calle
FROM avisos 
WHERE apellidos like '%$ape%')
UNION ALL
(SELECT 2 qnum, U.id_aviso, U.telefonos, U.telefonos3, U.movil, U.dni, U.name, U.apellidos, D.localidad, D.calle
FROM usuarios U, dir_usuarios D 
WHERE U.id_aviso=D.id_aviso AND D.calle != ''AND U.apellidos like '%$ape%');
 
(SELECT 0 qnum, A.id_aviso, A.telefonos, A.telefonos3, A.movil, A.dni, A.name, A.apellidos, B.localidad, B.calle
FROM clientes A, dir_clientes B 
WHERE A.id_aviso=B.id_aviso AND B.calle != '' AND A.name like '%$name%')
UNION ALL
(SELECT 1 qnum, id_aviso, telefonos, telefonos3, movil, dni, name, apellidos, localidad, calle
FROM avisos 
WHERE name like '%$name%')
UNION ALL
(SELECT 2 qnum, U.id_aviso, U.telefonos, U.telefonos3, U.movil, U.dni, U.name, U.apellidos, D.localidad, D.calle
FROM usuarios U, dir_usuarios D 
WHERE U.id_aviso=D.id_aviso AND D.calle != ''AND U.name like '%$name%');
 
(SELECT 0 qnum, A.id_aviso, A.telefonos, A.telefonos3, A.movil, A.dni, A.name, A.apellidos, B.localidad, B.calle
FROM clientes A, dir_clientes B 
WHERE A.id_aviso=B.id_aviso AND B.calle != '' AND B.localidad like '%$loca%')
UNION ALL
(SELECT 1 qnum, id_aviso, telefonos, telefonos3, movil, dni, name, apellidos, localidad, calle
FROM avisos 
WHERE localidad like '%$loca%')
UNION ALL
(SELECT 2 qnum, U.id_aviso, U.telefonos, U.telefonos3, U.movil, U.dni, U.name, U.apellidos, D.localidad, D.calle
FROM usuarios U, dir_usuarios D 
WHERE U.id_aviso=D.id_aviso AND D.calle != ''AND D.localidad like '%$loca%');

Por otro lado, a nivel de implementación, te estás complicando la vida.
¿Para qué escribir tantas veces la misma consulta que sólo cambia un determinado parámetro en cada una?
Es más sencillo hacer una sola consulta y determinar qué contendrá el parámetro a agregar, y sólo ese segmento. De ese modo simplificarías el asunto.
Creo que luego te lo agregaré en el post de PHP...

Hay algunso detalles más, pero esos los veremos después.

satjaen · #5 (**permalink**) 25/02/2013, 07:34

Cita:

Iniciado por gnzsoloyo

Por otro lado, a nivel de implementación, te estás complicando la vida.
¿Para qué escribir tantas veces la misma consulta que sólo cambia un determinado parámetro en cada una?
Es más sencillo hacer una sola consulta y determinar qué contendrá el parámetro a agregar, y sólo ese segmento. De ese modo simplificarías el asunto.

Gracias, es precisamente lo que quisiera hacer si se puede una única consulta.
Un saludo