Discussion :

[Rizcolas62]

Bonjour,
j'ai une succesion de conditions dans mon code php mais les données ne s'insére pas comme je veux :/
pouvez vous me dire ce qui cloche? et comment l'optimiser?

merci d'avance
voici mon formulaire:

et le php

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
<?php
// Récupération des données
$nom_ajout1=$_GET['nom1'];
$prenom_ajout1=$_GET['prenom1'];
$sexe1=$_GET['sexe'];
$date_naissance_ajout1=$_GET['naissance1'];
$situation=$_GET['situation'];
$uid=uniqid();
$nom_ajout2=$_GET['nom2'];
$prenom_ajout2=$_GET['prenom2'];
$sexe2=$_GET['s2'];
$date_naissance_ajout2=$_GET['naissance2'];
// $nb_enfants=$_GET['enfants1'];
// 2:Requête en fonction de la situation
if ($situation == 'Célibataire')
{
	// 1:Requête en fonction du sexe
	if ($sexe == 'M1')
	{
	$req="INSERT INTO t_h (nom, prenom, date_naissance, id, situation) VALUES ('".$nom_ajout1."', '".$prenom_ajout1."', '".$date_naissance_ajout1."', '".$uid."', '".$situation."')"; 
	}
	else
	{
	$req="INSERT INTO t_f (nom, prenom, date_naissance, id, situation) VALUES ('".$nom_ajout1."', '".$prenom_ajout1."', '".$date_naissance_ajout1."', '".$uid."', '".$situation."')"; 
	}
}
else
{
	if ($sexe1 == 'M1')
	{
	$req="INSERT INTO t_h (nom, prenom, date_naissance, id, situation) VALUES ('".$nom_ajout1."', '".$prenom_ajout1."', '".$date_naissance_ajout1."', '".$uid."', '".$situation."')"; 
	}
	else
	{
	$req="INSERT INTO t_f (nom, prenom, date_naissance, id, situation) VALUES ('".$nom_ajout1."', '".$prenom_ajout1."', '".$date_naissance_ajout1."', '".$uid."', '".$situation."')"; 
	}
	if ($sexe2 == 'M2')
	{
	$req="INSERT INTO t_h (nom, prenom, date_naissance, id, situation) VALUES ('".$nom_ajout2."', '".$prenom_ajout2."', '".$date_naissance_ajout2."', '".$uid."', '".$situation."')"; 
	}
	else
	{
	$req="INSERT INTO t_f (nom, prenom, date_naissance, id, situation) VALUES ('".$nom_ajout2."', '".$prenom_ajout2."', '".$date_naissance_ajout2."', '".$uid."', '".$situation."')"; 
	}
}
// Execution de la requête SQL
$res_ajout = pg_query($req) or die('Échec de la requête : ' . pg_last_error());
// Ferme la connexion
pg_close($dbconn);
?>

l'html:

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
<h2> Formulaire </h2>
		<FORM NAME=Formul method=GET action=ajout.php>
		<!-- Formulaire1 -->
		Nom : 
			<INPUT type=text name=nom1 autocomplete=on required>  
		Prénom : 
			<INPUT type=text name=prenom1 autocomplete=on required><BR>
		Sexe : 
			<INPUT type="radio" name="sexe" value="M1" id="M1"/>
			<label for="M1">M</label>
			<INPUT type="radio" name="sexe" value="F1" id="F1"/>
			<label for="F1">F</label><BR>
		Date de naissance :
			<INPUT type="date" name="naissance1" autocomplete=on/><BR>
 
		Situation :
			<INPUT type="radio" name="situation" value="Célibataire" id="C"/>
			<label for="Célibataire">Célibataire</label>			
			<INPUT type="radio" name="situation" value="En Couple" id="EC"/>
			<label for="En Couple">En Couple</label><BR>
		Nombre d'enfants :
			<select name="enfants">
				<option value="0" selected="selected">0</option>
				<option value="1">1</option>
				<option value="2">2</option>
				<option value="3">3</option>
			</select><BR>
			<INPUT type="hidden" name="uid" value="<?=$uid;?>"/>
 
		<!-- Formulaire2 -->
		Nom : 
			<INPUT type=text name=nom2 autocomplete=on required> 
		Prénom : 
			<INPUT type=text name=prenom2 autocomplete=on required><BR>
		Sexe : 
			<INPUT type="radio" name="s2" value="M2" id="M2"/>
			<label for="M2">M</label>
			<INPUT type="radio" name="s2" value="F2" id="F2"/>
			<label for="F2">F</label><BR>
		Date de naissance :
			<INPUT type="date" name="naissance2" autocomplete=on required><BR>
 
		<INPUT type=submit value="Enregistrer","/>
		</FORM>	
	</p>

[Invité]

Bonjour,

1- Déjà, pose-toi la question de la pertinence d'avoir 2 tables t_h et t_f...
Une seule table avec une colonne "sexe" aurait suffit, et serait pour le coup plus pertinente et facile à gérer.

2- Idem pour la récupération en GET plutôt qu'en POST (où les variables ne passent pas par l'URL)

3- Ben.... après, c'est surtout une question de logique...