duda con sort de bd

[Triby]

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:

// 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:

$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.