Discussion :

[megaoctet]

Bonsoir à tous.
J'ai changé d'hébergeur en passant de Nétissime à 1&1 et depuis j'ai quelques soucis avec mon site internet.
En changeant d'hébergeur, j'ai changé de version de PHP et depuis il m'est impossible d'utiliser la fonction INSERT INTO, même en local.
J'ai tout mis à jour, mais là je sèche.

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
 
if(isset($_POST['civilite']))      $civilite=$_POST['civilite'];
else      $civilite="";
 
if(isset($_POST['nom']))      $nom=$_POST['nom'];
else      $nom="";
 
if(isset($_POST['prenom']))      $prenom=$_POST['prenom'];
else      $prenom="";
 
if(isset($_POST['email']))      $email=$_POST['email'];
else      $email="";
 
if(empty($nom) OR empty($prenom) OR empty($email)) 
    { 
    echo 'Attention, vous n\'avez pas rempli un champs obligatoire !';
    echo '<a href="javascript:history.go(-1);">Retour</a>';
    } 
 
else      
    { 
 
$sql = "SELECT id FROM newsletter WHERE email='$email'"; 
$req = mysqli_query($sql);  
 
$res = mysqli_num_rows($req); 
 
    if($res!=0)
        { 
	echo 'Merci de votre confiance,';
	echo '<br />';
    	echo 'mais votre adresse mail existe dans notre base !';
	echo '<br />';
	echo 'Pour consulter le site, <a href="index.php" target="_top">cliquez ici</a>';
        } 
    else 
        {    
$sql = "INSERT INTO newsletter(id,civilite,nom,prenom,email) VALUES('','$civilite','$nom','$prenom','$email')"; 
mysqli_query($sql) or die('Erreur SQL !'.$sql.'<br>'.mysqli_error());

Est-ce que vous avez une idée de ce qui se passe ?

Merci d'avance pour votre aide

[andry.aime]

Bonjour.

1- Ne concatène pas directement les valeurs à insérer dans la requête, utilise une requête préparée.
2- Pour l'id (clé primaire), en principe, on utilise une valeur auto-incrémentée si on a pas de générateur, ne pas mettre une valeur vide.
3- Même si l'utilisation d'un block {} n'est pas obligatoire quand il s'agit d'une seule instruction, utilise le pour une meilleur lisibilité de ton code.
4- Il faut faire attention avec l'utilisation de l'opérateur OR

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
<?php
$a = "";
$b = "test";
if(empty($a) or empty($b)){
  echo ' true <br />';
} else {
  echo ' false <br />';
}
if(empty($b) or empty($a)){
  echo ' true <br />';
} else {
  echo ' false <br />';
}
if(empty($a) || empty($b)){
  echo ' true <br />';
} else {
  echo ' false <br />';
}
if(empty($b) || empty($a)){
  echo ' true <br />';
} else {
  echo ' false <br />';
}
 
 
$c = empty($a) or empty($b);
echo '$c: ';
var_dump($c);
echo '<br />';
 
$d = empty($b) or empty($a);
echo '$d: ';
var_dump($d);
echo '<br />';
 
$e = empty($a) || empty($b);
echo '$e: ';
var_dump($e);
echo '<br />';
 
$f = empty($b) || empty($a);
echo '$f: ';
var_dump($f);
 
 ?>

$d = empty($b) or empty($a); vaut à écrire ($d = $empty($b)) or $empty($a)

A+.