[Php/Sql] Problème Order By

Marsh Posté le 02-05-2011 à 11:39:52

Bonjour a vous tous

Je code actuellement un site ou un professionnel peut venir ajouter une pub,jusque la pas de soucis sauf que dans mon administration je voudrais afficher un tableau avec toutes les pubs enregistrés dans la BD et les pubs serait triés selon des critères choisis par l'admin le problème c'est que le trie (ORDER BY) ne fonctionne pas

Voila les parties du code concernés :

Class Pub:

Code :

class pub{
public $id,$id_pro,$nom,$prixclick,$dimension,$lienimg,$nb_affichage,$nb_click,$lienurl,$id_domaine,$date_creation,$date_activation,$date_fin,$active;
public function __construct($n,$idp,$d,$p,$l,$nba,$nbc,$url,$iddom,$dc,$da,$df,$a=false,$i=false){
$this->id=$i;
$this->id_pro=$idp;
$this->nom=$n;
$this->prixclick=$p;
$this->dimension=$d;
$this->active=$a;
$this->lienimg=$l;
$this->nb_affichage=$nba;
$this->nb_click=$nbc;
$this->lienurl=$url;
$this->id_domaine=$iddom;
$this->date_creation=$dc;
$this->date_activation=$da;
$this->date_fin=$df;
}
public function save(){
if($this->id==false){
$sql = "INSERT INTO pub VALUES (NULL,'$this->nom','$this->id_pro','$this->dimension','$this->prixclick','$this->lienimg','$this->nb_affichage','$this->nb_click','$this->lienurl','$this->id_domaine','$this->date_creation','$this->date_activation','$this->date_fin','$this->active')";
mysql_query($sql);
$this->id = mysql_insert_id();
}
else{
$sql= "UPDATE pub SET id_pro='$this->id_pro',nom='$this->nom',prix='$this->prixclick',dimension='$this->dimension',lienimg='$this->lienimg',nb_affichage='$this->nb_affichage',nb_click='$this->nb_click',lienurl='$this->lienurl',id_domaine='$this->id_domaine',date_creation='$this->date_creation',date_validation='$this->date_activation',date_fin='$this->date_fin',active='$this->active' WHERE id='$this->id'";
mysql_query($sql);
}
}
...
}

class PubTable:

Code :

<?php
class PubTable{
public static function getAlltrie($order) {
$sql="Select * from pub order by '$order'";
echo $sql;
$res = mysql_query($sql);
$toto = array();
while($line=mysql_fetch_array($res)){
$toto[] = new pub($line['nom'],$line['id_pro'],$line['dimension'],$line['prix'],$line['lienimg'],$line['nb_affichage'],$line['nb_click'],$line['lienurl'],$line['id_domaine'],$line['date_creation'],$line['date_validation'],$line['date_fin'],$line['active'],$line['id']);
}
return $toto;
}
}
?>

Appel de la fonction

Code :

$res=PubTable::getAlltrie($order);

J'ai fait tout les echo possible et le $order est bien valide que ce soit ici (admin.php) ou dans la class PubTable.class.php

Voila je ne sais pas du tout pour quoi ca ne marche pas

Reply

Marsh Posté le 02-05-2011 à 11:39:52

Reply

Marsh Posté le 02-05-2011 à 11:52:18

Habituellement, il est recommandé d'éviter les select * et les remplacer par des select en précisant le nom de chaque colonne dans le select, surtout si après on utilise un nom de colonne dans un order ou un group by.

Reply

Marsh Posté le 02-05-2011 à 13:03:12

- Quel est le résultat de ton echo $sql?
- On évite en général de mélanger français et anglais ( getAlltrie )
- Essaie de respecter les standards de mise en majuscule (getAllTrie)
- Arrête d'utiliser mysql_query + requête écrite en dûr et passe plutôt par DBO et les prepared statements.

---------------
Si la vérité est découverte par quelqu'un d'autre,elle perd toujours un peu d'attrait

Reply

Marsh Posté le 02-05-2011 à 13:48:54

Mon echo de $sql avec par exemple $order=nom: Select * from pub order by 'nom'

Edit:problème résolue !

En fait il fallait juste enlevé les simple quote autour du $order dans la variable $sql de la fonction getAlltrie()

Message édité par MonTheDim le 02-05-2011 à 14:06:36

Reply

[Php/Sql] Problème Order By

Sujets relatifs:

Leave a Replay