Empêcher l'exécution d'un code php avant submit d'un formulaire

Version imprimable

Bonjour à tous,

Je suis grand débutant en html et php. Merci pour votre site, une vraie mine d'or notamment vos cours !!
Mon but : générer une page web pour interroger une BDD sur php myadmin.

Voici un bout de code php, une liste déroulante interroge ma BDD.
Je souhaiterais qu'une fois seulement le submit coché (dans la 1ère partie du code), la suite du code php s'exécute car sinon j'ai un message d'erreur (la requête SQL renvoie une erreur puisque qu'aucune variable n'est transmise).
Le problème vient de la ligne if (isset($_POST['inscription']) de la 2e partie de mon code.
J'ai sans doute oublié quelque chose.

Merci d'avance pour votre aide !
Gilles
Code:

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  <form name="inscription" method="post"> <div class="position_liste_deroulante"> <select name="num_magazine"> <?php $link=mysqli_connect("localhost", "root", "","guitarist_acoustic"); mysqli_set_charset($link, "utf8mb4"); $resultat=mysqli_query($link, "SELECT num_mag FROM magazine_ga"); while($ligne=mysqli_fetch_array($resultat)) {echo '<option>'.$ligne["num_mag"].'</option><br/>';} mysqli_close(); ?> </select> <input type="submit" name="valider" value="Rechercher"/> </div> </form>  <div class="position_magazine"> <?php if (isset($_POST['inscription']) { echo "Saisir un numéro de magazine dans la liste déroulante puis cliquez sur RECHERCHER"; } else { $variable1=$_POST["num_magazine"]; $n_mag='<img src="Couvertures\GA'.$variable1.'.JPG" height="250">'; echo "<p>".$n_mag."</p>";// Affiche le numéro de référence de l'image echo "<p>Mag. Guitarist Acoustic n°" .$variable1."</p>";// Affiche le numéro de référence de l'image ) ?> </div>

Puisque tu débutes, autant prendre tout de suite les bonnes habitudes ;)

Une BdD n'est pas sur phpMyAdmin, elle est sur MySQL, PostgreSQL, SQLite, etc. phpMyAdmin n'est qu'une interface d'administration
Il faut absolument séparer, autant que possible, récupération des données, traitement des données et affichage
PHP propose PDO ( https://php.net/pdo ), il vaut mieux l'utiliser plutôt que mysqli
Quand on fait des conditions ou boucles dans du HTML, mieux vaut utiliser la syntaxe alternative ( https://www.php.net/manual/fr/contro...ive-syntax.php ), le script est ainsi beaucoup plus lisible
Dans le HTML, il faut afficher le moins possible de chaînes littérales, et on peut utiliser "<?=" au lieu de "<?php echo" (toujours la lisibilité)
On évite de nommer les variables $variable1, il faut donner un nom explicite aux variables, comme $num_magazine
Dans un chemin d'accès (IMG) il vaut mieux utiliser le séparateur de dossiers / qui passe sur tous les systèmes, contrairement à \
Lors de l'affichage il faut échapper avec htmlspecialchars() les valeurs utilisateurs/externes, cela protège des injections malveillantes

Pour le fond du problème, tu détermines si le form a été soumis avec if (isset($_POST['inscription']), or, de ce que tu montres, $_POST['inscription'] ne sera jamais valorisé (le name d'un form n'est pas transmis)
=> Fie-toi plutôt aux valeurs attendues avec ton form
=> Ici comme ton form n'a qu'une valeur, num_magazine, c'est ce qui sera utilisé

Proposition prenant tout cela en compte :
Code:

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 <?php // 1re étape : Récupération des données $link = mysqli_connect("localhost", "root", "", "guitarist_acoustic"); mysqli_set_charset($link, "utf8mb4"); $resultat = mysqli_query($link, "SELECT num_mag FROM magazine_ga"); $magazines = []; // À tester si à la place du while() tu peux utiliser mysqli_fetch_all() => https://www.php.net/manual/fr/mysqli-result.fetch-all.php while ($ligne = mysqli_fetch_array($resultat)) { $magazines[] = $ligne; } mysqli_close($link); ?>  <pre>DEBUG FORM SOUMIS : <?php print_r($_POST) ?></pre> <form method="post"> <div class="position_liste_deroulante"> <select name="num_magazine"> <?php foreach ($magazines as $ligne): ?> <option><?= $ligne["num_mag"] ?></option> <?php endforeach ?> </select> <input type="submit" value="Rechercher"/> </div> </form> <div class="position_magazine"> <?php if (!isset($_POST['num_magazine'])): // Pas de formulaire soumis ? ?> <p>Saisir un numéro de magazine dans la liste déroulante puis cliquez sur RECHERCHER</p> <?php else: // Formulaire soumis, num_magazine défini ?> <p><img src="Couvertures/GA<?= htmlspecialchars($_POST['num_magazine']) ?>.JPG" height="250"> <p>Mag. Guitarist Acoustic n° <?= htmlspecialchars($_POST['num_magazine']) ?></p> <?php endif ?> </div>

Merci Séb.

Quelle pédagogie et quelle qualité de réponse. Merci beaucoup pour vos conseils !

J'ai testé et effectivement le <num_magazine> était clairement le problème, j'aurais du être plus vigilant.

Pour la 2e partie du code j'ai suivi vos conseils pour la syntaxe de la structure conditionnelle, c'est vrai que c'est beaucoup plus clair.
Je n'ai en revanche pas cerné l'idée d'ajouter une surcouche avec une nouvelle zone de liste déroulante et de bouton submit. Cela empêche le bon fonctionnement.

Pour le PDO j'avais initialement commencer par l'utiliser, le problème c'est que je ne suis pas parvenu au remplissage de la zone déroulante.

Voici ci-dessous le code qui fonctionne en attendant de l'améliorer avec vos conseils (PDO plutôt que mysqli notamment).
Code:

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  <form name="inscription" method="post"> <div class="position_liste_deroulante"> <select name="num_magazine"> <?php // 1re étape : Récupération des données $link = mysqli_connect("localhost", "root", "", "guitarist_acoustic"); mysqli_set_charset($link, "utf8mb4"); $resultat = mysqli_query($link, "SELECT num_mag FROM magazine_ga"); $magazines = []; // À tester si à la place du while() tu peux utiliser mysqli_fetch_all() => https://www.php.net/manual/fr/mysqli-result.fetch-all.php while ($ligne = mysqli_fetch_array($resultat)) {echo '<option>'.$ligne["num_mag"].'</option><br/>';} //{$magazines[] = $ligne;} => ce code conseillé ne marche pas, à revoir mysqli_close($link); ?> </select> <input type="submit" name="valider" value="Rechercher"/> </div> </form>  <?php if (!isset($_POST['num_magazine'])): // Pas de formulaire soumis ? ?>  <p style="color:red;position: absolute;top: 220px;left: 55px;width: 200px;height: 10px;">Saisir un numéro de magazine dans la liste déroulante puis cliquez sur RECHERCHER</p> <?php else: // Formulaire soumis, num_magazine défini ?> <div class="position_magazine"> <p><img src="Couvertures/GA<?= htmlspecialchars($_POST['num_magazine']) ?>.JPG" height="250"> <p>Mag. Guitarist Acoustic n° <?= htmlspecialchars($_POST['num_magazine']) ?></p> </div> <?php endif ?>