Discussion :

[JezUK]

Coucou,

J'essaie en vain de pouvoir updater des données dans ma BDD via un formulaire.

voilà le code que j'ai:

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
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
<?	  
include('conf.php');  
 
$firstname= $_POST['firstname'];  
$lastname= $_POST['lastname'];  
$country= $_POST['country'];  
 
 
if($firstname==" " || $lastname==" " || $country=="")  
    {  
    echo '<script language=javascript> alert ("Vous avez oublié de remplir un champs");</script>';  
    echo '<SCRIPT LANGUAGE="JavaScript"> document.location.href="profil.php" </SCRIPT>';  
exit;  
    }  
 
$sql = "SELECT firstname FROM Users WHERE firstname='$_POST[firstname]'";  
$req = mysql_query($sql) or die('Erreur SQL !<br>'.$sql.'<br>'.mysql_error());
 
$data = mysql_fetch_array($req);
 
 
if($data['firstname']==$firstname)       
    {  	
    echo '<script language=javascript> alert ("Are you sure ? ");</script>';  
    echo '<SCRIPT LANGUAGE="JavaScript"> document.location.href="profil.php" </SCRIPT>';  
exit;       
    }
 
else
{
 
$sql = "SELECT lastname FROM Users WHERE lastname='$_POST[lastname]'";  
$req = mysql_query($sql);
 
$data = mysql_fetch_array($req);
 
if($data['lastname']==$lastname)
    {
    echo '<script language=javascript> alert ("This last name already exists !");</script>';  
	echo '<SCRIPT LANGUAGE="JavaScript"> document.location.href="profil.php" </SCRIPT>';  
exit;
    }  
 
else
{
 
$sql = "SELECT country FROM Users WHERE country='$_POST[country]'";  
$req = mysql_query($sql);  
 
$data = mysql_fetch_array($req); 
 
if($data['country']==$country)  
    {	
    echo '<script language=javascript> alert ("hummm !");</script>';  
	echo '<SCRIPT LANGUAGE="JavaScript"> ocument.location.href="profil.php" </SCRIPT>';  
exit;  
 
    }
else
 
$mysql="UPDATE Users SET firstname='$firstname', lastname='$lastname', country='$country' WHERE id='$login";
 
$rs = mysql_query($mysql) or die('Erreur SQL !<br>'.$sql.'<br>'.mysql_error());
$nid = mysql_insert_id();
 
echo '  <p>your data has been added to the database, <a href="profil.php">click here</a> to check you profil.</p> ';  
 
} 
 
}
 
?>

Le formulaire est dans un autre fichier et j'appelle celui que je viens de mettre juste au dessus.
J'ai des message d'erreurs et je suis perdu ;(

Code :

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

$mysql="UPDATE Users SET firstname='$firstname', lastname='$lastname', country='$country' WHERE id='$login";

Autre petit problème j'ai mis une ligne de code pour ne pas avoir le mm nom ou prenom qui existent 2 fois mais en fait je me suis planté et impossible de retirer la bonne ligne de code maintenant, je comprends pas...

[lenono]

Il est bizarre ton code...
tu fais ça :

Code :

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

1
2
3
$firstname= $_POST['firstname'];  
$lastname= $_POST['lastname'];  
$country= $_POST['country'];

Et après tu réutilises tes variables $_POST dans les requêtes
===>>>> GROSSE ERREUR (pb d'injections SQL, tu connais ?)

Fais un echo $mysql de ta requête, à mon avis, ton erreur apparaitra toute seule....

[FirePrawn]

Bonjour,

Vérifie le contenu de $_POST également, peut-être que tes valeurs sont mal passées.

[Invité]

Bonjour,
mélanger PHP et javascript n'est pas très heureux...

Vérification d'un formulaire :
1/ on fait une PREMIERE vérification (facultative) en javascript AVANT d'envoyer le formulaire (onsubmit ...)
2/ on fait une DEUXIEME vérification (obligatoire) en php (tout php !). Au cas où javascript est désactivé, c'est la dernière "barrière" !

tu peux t'inspirer de cette méthode -> FORMULAIRE ET TRAITEMENT sur la même page

[feldi]

Le formulaire est dans un autre fichier et j'appelle celui que je viens de mettre juste au dessus.
J'ai des message d'erreurs et je suis perdu ;(

Code :

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

$mysql="UPDATE Users SET firstname='$firstname', lastname='$lastname', country='$country' WHERE id='$login'";

Je ne sais pas si ça va changer grand chose mais il te manquait une ' avant le " de fin d'update.

[cragoroth]

salut,

ton code est une usine à gaz quand même.

Plusieurs solutions :

- très simple tu peux trouver ça sur le net, un validator en Jquery facile d'utilisation, dès qu'un champ est oublié directement message d'erreur à côté de ton input ou autre champ de ton formulaire.
- fait un fichier javascript, moins le bazar et surtout tes boucles tu peux faire mieux au niveau de ton code
- je ferrais ceci :

Code :

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

1
2
 
$mysql="UPDATE Users SET firstname='".$firstname."', lastname='".$lastname."', country='".$country."' WHERE id='".$login."'";

tiens au courant si ça marche