Quel est le problème avec mon code?

Question

J'essaie de permettre aux utilisateurs de trier les requêtes mysql en fonction du nom, du prix, etc. Ce menu déroulant leur donne le pouvoir de le faire, il modifie la clause "ORDER BY" en fonction de ce que l'utilisateur choisit. Il est trois heures ici et je ne peux pas repérer l'erreur:

<?php 
     $sortBy = $_POST['sortBy']; 
     if ($sortBy) { 

      $priceLowToHigh = $_POST['price-low-to-high']; 
      $priceHighToLow = $_POST['price-high-to-low']; 
      $dateMostRecent = $_POST['date-most-recent']; 
      $dateOldest = $_POST['date-oldest']; 
      $alphabeticalOrder = $_POST['alphabetical-order']; 

      if ($priceLowToHigh) { 
       $sortOrder = "price ASC"; 
      } elseif ($priceHighToLow) { 
       $sortOrder = "price DESC"; 
      } elseif ($dateMostRecent) { 
       $sortOrder = "date DESC"; 
      } elseif ($dateOldest) { 
       $sortOrder = "date ASC"; 
      } elseif ($alphabeticalOrder) { 
       $sortOrder = "name ASC"; 
      } 

     } else { 
      $sortOrder = "date DESC"; 
     } 
    ?> 
    <form action="" method="post"> 
     <select name="sortBy" onchange="this.form.submit()"> 
      <option>Sort By</option> 
      <option value="price-low-to-high">Price (low to high)</option> 
      <option value="price-high-to-low">Price (high to low)</option> 
      <option value="date-most-recent">Date (most recent)</option> 
      <option value="date-oldest">Date (oldest)</option> 
      <option value="alphabetical-order">Alphabetical Order</option> 
     </select> 
    </form> 

plus tard la clause ORDER BY est utilisée comme ceci:

$query = mysql_query("SELECT * FROM products WHERE category = $categoryId ORDER BY $sortOrder"); 

Answer 1

Vous ne traitez pas correctement votre menu. Votre variable $ sortBy sera déjà ce que l'utilisateur sélectionne, pas les champs $ _POST séparés.

C'est ce que vous devez faire à la place -

<?php 
    $sortBy = $_POST['sortBy']; 
    if($sortBy == 'price-low-to-high') { 
     $sortOrder = "price ASC"; 
    } else if($sortBy == 'price-high-to-low') { 
     $sortOrder = "price DESC"; 
    } else if($sortBy == 'date-most-recent') { 
     $sortOrder = "date DESC"; 
    } else if($sortBy == 'date-oldest') { 
     $sortOrder = "date ASC"; 
    } else if($sortBy == 'alphabetical-order') { 
     $sortOrder = "name ASC"; 
    } else { 
     $sortOrder = "date DESC"; 
    } 
?> 

Answer 2

Il semble que votre code est entre guillemets quand il devrait être en single pour avoir php faire le remplacement de votre code. Vous devriez imprimer ce sql à l'écran et vous verrez qu'il ne remplacera probablement pas la valeur. Je veux parler de la ligne où vous faites l'appel mysql_query.

Answer 3

Votre code de base est un peu mal. Ce devrait être le suivant: -

<?php 
$sortBy = $_POST['sortBy']; 

if($sortBy == 'price-low-to-high') 
    $sortOrder = "price ASC"; 
else if($sortBy == 'price-high-to-low') 
    $sortOrder = "price DESC"; 
else if($sortBy == 'date-most-recent') 
    $sortOrder = "date DESC"; 
else if($sortBy == 'date-oldest') 
    $sortOrder = "date ASC"; 
else if($sortBy == 'alphabetical-order') 
    $sortOrder = "name ASC"; 
else 
    $sortOrder = "date DESC"; 
?> 

<form action="" method="post"> 
     <select name="sortBy" onchange="this.form.submit()"> 
      <option>Sort By</option> 
      <option value="price-low-to-high">Price (low to high)</option> 
      <option value="price-high-to-low">Price (high to low)</option> 
      <option value="date-most-recent">Date (most recent)</option> 
      <option value="date-oldest">Date (oldest)</option> 
      <option value="alphabetical-order">Alphabetical Order</option> 
     </select> 
</form> 

Je pense que ceci fonctionnera maintenant, si vous changez votre code au code ci-dessus.

Espérons que ça aide.

Answer 4

Je ne peux pas commenter sur votre réponse Raphael parce que je ne suis pas assez élevé, mais je veux juste mentionner éventuellement en utilisant une déclaration de commutateur pour la lisibilité. Mais vous êtes sur place là-bas. Upvote