Buscar registros por criterio no muestra nada

Question

Hago esta consulta en PHP/MySQL y quiero que los registros estén ordenados por el campo nom Pero no muestra nada.

$stm = $BD->prepare("SELECT a.cod, a.nom, b.id_pago, c.planilla
FROM sc_personal a,
sc_condicion b,
sc_planilla c
WHERE  a.id_condicion = b.id_condicion
AND  a.id_trab = c.id_trab
AND  a.cod ='.$valor'. ORDER BY a.nom ASC");

Antes terminaba en:

AND  a.cod =".$valor);

Y no había problema, ahora le agregué el ORDER BY y no muestra nada. Cuál es el problema?

Estas concatenando mal, pero la pregunta es ¿Por que concatenas valores si estas utilizando sentencias preparadas? — Xerif, Sep 04 '17 at 22:17
La respuesta de @Xerif enfoca muy bien el problema , aunque también yo haría mi consulta usando `JOIN`... así el código es más fácil de entender y mantener. — A. Cedano, Sep 05 '17 at 02:23
Cierto @A.Cedano no me fije en la consulta sinceramente, pero bien visto. — Xerif, Sep 05 '17 at 09:22
@Piropeator **[yo plantee esa duda hace un tiempo](https://es.stackoverflow.com/q/74498/29967)** y al parecer no hay diferencia en el rendimiento. De todos modos, creo que con `JOIN` el código queda más fácil de entender. — A. Cedano, Sep 05 '17 at 20:51

score 4 · Accepted Answer · answered Sep 04 '17 at 22:23

No deberías concatenar valores en las sentencias preparadas. Los valores deberían ser ligados. Ademas tienes un error concatenando.

Ejemplo de como podría quedar:

$stm = $BD->prepare("SELECT a.cod, a.nom, b.id_pago, c.planilla
FROM sc_personal a,
sc_condicion b,
sc_planilla c
WHERE  a.id_condicion = b.id_condicion
AND  a.id_trab = c.id_trab
AND  a.cod = ? ORDER BY a.nom ASC");

// Ligamos el valor
$stmt->bind_param('s', $valor);
// Ejecutamos
$stmt->execute();

score 0 · Answer 2 · answered Sep 04 '17 at 22:12

Estás concatenando mal, intenta de esta forma:

$stm = $BD->prepare("SELECT a.cod, a.nom, b.id_pago, c.planilla
                     FROM sc_personal a,
                          sc_condicion b,
                          sc_planilla c
                     WHERE  a.id_condicion = b.id_condicion
                     AND  a.id_trab = c.id_trab
                     AND  a.cod ='" . $valor . "' ORDER BY a.nom ASC");

Buscar registros por criterio no muestra nada

2 Answers2