Bonjour,

Je suis en train de faire une page de recherche de membres dans mon espace membre et j'ai besoin de vos avis/suggestions pour me dire si mon code est bien ou si il faut l'améliorer.. (pas si mon code fonctionne, car je sais déjà qu'il fonctionne.)

voici la page de recherche:
Code : Sélectionner tout - Visualiser dans une fenêtre à part
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
<?php
logged_only(); // Vérifie que le membre est bien connecté.
 
$user_id = $_SESSION['auth']->id; // Récupère l'id du membre connecté.
 
if(!empty($_POST)){
  if(empty($_POST['genre'])){
    $_SESSION['flash']['danger'] = "Veuillez renseigner le genre recherché.";
  } else {
    $reponse = $pdo->prepare("SELECT id, civilite, prenom, date_naissance, confirmed_at
    FROM users
    WHERE id != $user_id AND civilite = ? AND confirmed_at IS NOT NULL
    ORDER BY confirmed_at DESC LIMIT 0, 20");
    $reponse->execute(array($_POST['genre']));
  }
} else {
  // Recherche par défaut si le formulaire de recherche n'a pas été envoyé donc pas besoin de requête préparé.
  $reponse = $pdo->query("SELECT id, civilite, prenom, date_naissance, confirmed_at
  FROM users
  WHERE id != $user_id AND confirmed_at IS NOT NULL
  ORDER BY confirmed_at DESC LIMIT 0, 20");
}
 
?>
 
<h1>Effectuer une recherche</h1>
 
<form action="" method="POST" class="form-horizontal">
  <div class="form-group">
    <div class="col-md-4 col-md-offset-4 col-sm-4 col-sm-offset-4">
      <label class="radio-inline">
        <input type="radio" name="genre" id="inlineRadio1" value="1"> Homme
      </label>
      <label class="radio-inline">
        <input type="radio" name="genre" id="inlineRadio2" value="2"> Femme
      </label>
    </div>
  </div>
  <div class="form-group">
    <div class="col-md-4 col-md-offset-4 col-sm-4 col-sm-offset-4">
      <button type="submit" class="btn btn-primary">Rechercher</button>
    </div>
  </div>
</form>
 
<h2>Résultat</h2>
 
<?php
while ($donnees = $reponse->fetch())
{
// exemple d'affichage..
?>
 
        <h4><?php echo htmlspecialchars($donnees->prenom); ?>, <?php echo age($donnees->date_naissance); ?> ans</h4>
 
<?php } ?>
Petite précision : Le formulaire de recherche comportera bien plus qu'un simple radio.. mais la c'est juste pour l'exemple schématique.

merci