[SOLUCIONADO] duda con sort de bd

xSkArx · #1 (**permalink**) 01/08/2013, 20:00

Hola, quiero mostrar los datos de x tabla de mi db en mysql en tablas html (tipo datagrid en vb.net) y ordenarlos segun peticion de la url a traves de $_GET['sort'] de manera ascendiente

si la url fueses asi: asdf.php?sort=id segun yo deberia ir asi la consulta,

Código PHP:

Ver original$sort = $_GET['sort']
$query="SELECT * FROM tabla ORDER BY ".$sort. "ASC";
$res= mysql_query($query, $conn);
while($row= mysql_fetch_assoc($res){
//todo lo que necesite para la tabla
}

muestra los datos ordenados segun el id de la fila, o segun el nombre del campo, estara bien asi?

Triby · #2 (**permalink**) 02/08/2013, 00:08

Qué tardará más?
a) Que tú mismo realices las pruebas y veas si funciona o no
b) Esperar que alguien responda tu duda en el foro

No lo tomes a mal, si realmente quieres programar, debes tener iniciativa, de lo contrario, estarás preguntando en el foro por cualquier duda y perderás tiempo valioso que podrías dedicar a avanzar con tu proyecto.

xSkArx · #3 (**permalink**) 02/08/2013, 09:53

En el momento que formule la pregunta no tenia como probarlo, llegue a casa lo probe y no funciono, ponga lo qye ponga comi parametro en la url lo ordena por ID ASC, se que algo esta fallando en la parte del $_GET['sort'], pero ahora no tengo como probarlo, solo quiero saber donde esta el error, gracias.

jonni09lo · #4 (**permalink**) 02/08/2013, 10:24

Hola si has notado de que tienes muchos errores de sintaxis, no has depurado tus consultas y que no estan bien formulados tu sql (teniendo en cuenta de que tome el mismo codigo que diste)

Si no tienes donde probar mal por ti ya que con esa actitud de solo quiero saber que esta mal no obtendras nada, cuando hayas depurado tu codigo y verificar que el sql esta bien formado y lo PRUEBAS nos muestras y te podemos seguir orientando

Saludos

xSkArx · #5 (**permalink**) 02/08/2013, 15:22

Hola, les comento que he realizado varias pruebas y he encontrado la solucion, aunque es momentanea porque me falta agregarle seguridad al script, aqui va:

Código PHP:

Ver original<?php
if ($_GET['sort']== NULL){
$order= "ID";
}else{
$order=$_GET['sort'];
$order=strtoupper($order);
}
include('config.php');
$q="SELECT * FROM noticias ORDER BY ".$order." DESC";
$r=mysql_query($q,$conexion);
$i=0;
while($row=mysql_fetch_array($r)){
$id=$row['ID'];
$title=$row['TITULO'];
echo $id." el ".$title;
echo "<br/>";
$i++;
}
?>

Estoy usando strtoupper porque el nombre de los campos estan en mayusculaz y por el momento solo lo he probado con 2 campos, tambien tengo hacer pruebas para despues hacer el sort asc o desc, gracias a ambos por los consejos

Triby · #6 (**permalink**) 02/08/2013, 19:08

Excelente, la iniciativa deja buenos resultados!

Sólo un comentario, si 'sort' no fue proporcionada en GET seguramente obtendrás un mensaje de error con ese if($_GET['sort'] == NULL) porque no puedes comparar una variable (o índice de array) no definida.

Te sugiero usar asignación ternaria, que es como un if, pero en una sola línea, y realizando la comparación de forma adecuada:

Código PHP:

Ver original// Con if normal
(if(empty($_GET['sort'])) {
     // Empty devuelve verdadero si la variable (o índice):
     //  No está definida, su valor es null, cero, false o cadena vacía
     $order = 'ID';
} else {
     $order = strtoupper($_GET['sort']);
}
 
// Con asignación ternaria:
$order = (empty($_GET['sort'])) ? 'ID' : strtoupper($_GET['sort']);

Ahora, la cuestión de seguridad es muy fácil de resolver creando un array con los campos disponibles para ordenar, ejemplo:

Código PHP:

Ver original$campos = array('ID', 'NOMBRE', 'CATEGORIA', 'ETC');
 
// Defines el orden:
$order = (empty($_GET['sort'])) ? 'ID' : strtoupper($_GET['sort']);
 
// Haces una validación
if(!in_array($order, $campos)) {
       // El campo para ordenar no es válido
       // Aquí puedes generar un mensaje de error
       // O simplemente asignar el campo por default
       $order = 'ID';
}

Lo del ASC o DESC es mucho más sencillo si adaptas lo mismo que hagas para el orden.

xSkArx · #7 (**permalink**) 03/08/2013, 14:29

Hola, he probado y funciona la primera parte, intente agregar al script lo de ASC y DESC pero no funciona,

Código PHP:

Ver original<html>
<head>
<?php
$ord = array('ASC', 'DESC');
$campos = array('ID', 'TITULO', 'DESCRIPCION', 'FOTO', 'VIDEO', 'HORA', 'FECHA');
// Defines el orden:
$order = (empty($_GET['sort'])) ? 'ID' : strtoupper($_GET['sort']);
$sorter= (empty($_GET['sorter'])) ? 'ASC' : strtoupper($_GET['sorter']);
// Haces una validación
if(!in_array($order, $campos)) {
       $order = 'ID';
} 
if(!in_array($sorter, $ord)) {
$sorter= 'ASC';
}
include('config.php');
?>
</head>
<body>
<table border="2">
<?php
$q="SELECT * FROM noticias ORDER BY '$sort' '$sorter'";
$r=mysql_query($q,$conexion);
$i=0;
while($row=mysql_fetch_assoc($r)){
$id=$row['ID'];
$title=$row['TITULO'];
$des= $row['DESCRIPCION'];
echo "<tr>";
echo "<td>".$id."</td><td>".$title."</td><td>".substr($des,0, 10)."</td>";
echo "</tr>";
$i++;
}
?>
</table>
</body>
</html>

Si no pongo ningun parametro en la url lo ordena por ID y ASC. Y si pongo parametros sigue igual
Pienso que el error esta en la consulta

Código PHP:

Ver original$q="SELECT * FROM noticias ORDER BY '$order' '$sorter'";

he puesto las. Variables con comilla simple y con doble comilla perk no funciona

Triby · #8 (**permalink**) 03/08/2013, 21:12

Esas dos variables se supone que corresponden al nombre de un campo y tipo de orden, no son un valor que se vaya a guardar, entonces... no necesitan comillas, ni dobles ni simples.

xSkArx · #9 (**permalink**) 04/08/2013, 20:29

Bueno, despues de muchas pruebas pude conseguir lo que queria, era problema de comillas el asunto :D

Código PHP:

Ver original<?php
$campos = array('ID', 'TITULO', 'DESCRIPCION', 'FOTO', 'VIDEO', 'HORA', 'FECHA');
$order = (empty($_GET['sort'])) ? 'ID' : strtoupper($_GET['sort']);
if(!in_array($order, $campos)) {
      $order = 'ID';
}
$orden= array('ASC', 'DESC');
$sorter= (empty($_GET['sorter'])) ? 'ASC' : strtoupper($_GET['sorter']);
if(!in_array($sorter, $orden)){
    $sorter= 'ASC';
}
include('config.php');
?>
<table border="2">
<?php
$q="SELECT * FROM noticias ORDER BY ".$order." ".$sorter;

Gracias por la ayuda y la paciencia :D