Discussion :

[Tif26fany]

Bonjour,
J'ai un problème depuis plusieurs jour avec un select, je veux récupérer sa valeur et l'envoyer dans ma BDD sauf que ça me renvoie dans la bdd un champs vide..

J'écris d'abord mon select en recherchant les valeurs de celui-ci dans ma bdd, ce qui fonctionne très bien.

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
 
<?php
$pdo = new PDO("mysql:dbname=base_omaj;host=localhost", "root", "", array(PDO::MYSQL_ATTR_FOUND_ROWS => true ));
$st1 = $pdo -> prepare ("select Nom_poste_source from poste_source");
$st1->execute();
$res = null;
$input_name1 = "source";
 
 
if ((isset($_GET[$input_name1]) === true) AND (strlen($_GET[$input_name1]) >0 ))
{
    echo "Vous avez choisie " .$_GET[$input_name1];
}
 
?>
 
 <form action="Remplissage_omaj.php" methode="post">
 
                <!-- REMPLISSAGE POSTE SOURCE -->   
                <!-- Poste source -->    
                Poste source :
                <select name="source" >
                <?php
                    echo "<option></option>\n>";
                    while ($res =$st1->fetch(PDO::FETCH_NUM))
                    {
                        echo "<option>" .$res[0]. "</option>";
                    }
                ?>
                </select>
                <input name="Valider" type="submit" value="Valider" />
                <input name="Annuler" type="submit" value="Annuler" />
 
</form>

Puis je les récupère la valeur selectionnée et la renvoie sur ma bdd (j'ai 3 select comme celui)

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
<?php
$host = "localhost";
$dbusername = "root";
$dbpassword = "";
$dbname = "base_omaj";
 
$conn = new mysqli($host, $dbusername, $dbpassword, $dbname);
if ($conn->connect_error) {
    die("Connection failed: " . $conn->connect_error);
}
 
 
 
$Nom_source = !empty($_POST['source']) ? $_POST['source'] : NULL;
$Nom_depart = !empty($_POST['depart']) ? $_POST['depart'] : NULL;
$Nom_type = !empty($_POST['type']) ? $_POST['type'] : NULL;
$Description = !empty($_POST['description']) ? $_POST['description'] : NULL;
 
 
 
$sql = "INSERT INTO omaj (Numero_omaj, Nom_source, Nom_depart, Nom_type, Date, Description, Id_etat,Nom_etat) VALUES ('46','$Nom_source','$Nom_depart','$Nom_type','2018-12-10','$Description','1','Prepare')"; 
 
if ($conn->query($sql) === TRUE) {
    echo "Omaj ajoutéé";
        header( "refresh:5;url=Pr%C3%A9par%C3%A9.php");
    } else {
        echo "Error: " . $sql . "<br>" . $conn->error;
        header( "refresh:5;url=Pr%C3%A9par%C3%A9.php" );
    }
 
?>

J'espère avoir été clair, merci pour l'aide

[Invité]

Bonjour,

1- pourquoi utilises-tu PDO dans un script, et mysqli_ dans l'autre ??

Contente-toi de PDO.

2- Il est préférable de mettre des value aux <option>.
Ce qui permet notamment de récupérer les ids au lieu des noms.

ex. :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

      echo "<option value=\"" .$res['id']. "\""">" .$res['nom']. "</option>";

3- La bonne pratique veut qu'on utilise une requête préparée.

• A LIRE : PDO une soupe et au lit !

4-

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

         header( "refresh:5;url=Pr%C3%A9par%C3%A9.php" );

EVITE de mettre des accents !! (noms de fichiers, de variables, de colonnes en BDD....)

[Tif26fany]

Bonjour, merci pour les conseils
Je vais essayer d'appliquer tout ça!

[Invité]

Bonjour,
j'ajoute ceci:

• PRINCIPE de FORMULAIRE et TRAITEMENT sur la même page

• ça permet de faire une "gestion d'erreur" (et de réafficher le formulaire en cas d'erreur)
• et ça évite de faire un header inutile à la fin

[Tif26fany]

J'ai essayer de corriger un maximum en suivant les conseils mais ça ne fonctionne toujours pas, cela m'ajoute un champs vide dans la BDD.

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
<?php
$pdo = new PDO("mysql:dbname=base_omaj;host=localhost", "root", "", array(PDO::MYSQL_ATTR_FOUND_ROWS => true ));
$st1 = $pdo -> prepare ("select Nom_poste_source Id_poste_source from poste_source");
$st1->execute();
$res = null;
$input_name1 = "source";
 
if ((isset($_GET[$input_name1]) === true) AND (strlen($_GET[$input_name1]) >0 ))
{
    echo "Vous avez choisie " .$_GET[$input_name1];
}
 
?>
 
                <form action="Remplissage_omaj.php" methode="post">
 
                <!-- REMPLISSAGE POSTE SOURCE -->   
                <!-- Poste source -->    
                Poste source :
                <select name="source">
                <?php
                    echo "<option></option>\n>";
                    while ($res =$st1->fetch(PDO::FETCH_NUM))
                    {
                        echo "<option value=" .$res['Id_poste_source']. ">" .$res[0]. "</option>";
                    }
                ?>
                </select>
 
                <input name="Valider" type="submit" value="Valider" />
                <input name="Annuler" type="submit" value="Annuler" />
 
                </form>

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
$sql = "INSERT INTO omaj (Id,Numero_omaj, Nom_source, Nom_depart, Nom_type, Date, Description, Id_etat,Nom_etat) VALUES (NULL,?,?,?,?,?,?,?,?)"; 
$db = include 'Remplissage_tableau.php';
 
try { 
 
    $stmt = $pdo->prepare($sql);
 
    $Numero_omaj = '50';
    $Nom_source = !empty($_POST['source']) ? $_POST['source'] : NULL;
    $Nom_depart = !empty($_POST['depart']) ? $_POST['depart'] : NULL;
    $Nom_type = !empty($_POST['type']) ? $_POST['type'] : NULL;
    $Date = '2018-12-10';
    $Description = !empty($_POST['description']) ? $_POST['description'] : NULL;
    $Id_etat = '1';
    $Nom_etat = 'Prepare';
 
    $stmt->execute(array($Numero_omaj, $Nom_source, $Nom_depart, $Nom_type, $Date, $Description, $Id_etat, $Nom_etat)) ;
 
   $nb_insert = $stmt->rowCount();
   echo $nb_insert.' insertion effectuée<br/>';
   unset($db);
} 
catch (Exception $e) {
   //s'il y a un problème PHP ou SQL, tout s'affichera ici
   print "Erreur ! " . $e->getMessage() . "<br/>";
}
 
?>

De plus j'ai cette erreur là :

Erreur ! SQLSTATE[23000]: Integrity constraint violation: 1048 Le champ 'Nom_source' ne peut être vide (null)

Ce qui est normal étant donné que rien ne s'affiche dans la BDD.

Merci

[Tif26fany]

Cela fonctionne en changeant le POST par des GET

[Invité]

Bonjour,
Vu !!

Code html :

Sélectionner tout - Visualiser dans une fenêtre à part

                <form action="Remplissage_omaj.php" methode="post">

C'est method !! Pas methode ! (<form> -> method)

Un formulaire est par défaut en method="get". Ce qui explique pourquoi ça fonctionne en $_GET.

Pour l'avoir en $_POST, corrige :

Code html :

Sélectionner tout - Visualiser dans une fenêtre à part

                <form action="Remplissage_omaj.php" method="post">

[Tif26fany]

Bonjour,
oui j'ai vu
merci