Erreur $sql undefined Variable

**superstarz** · 26/04/2011, 15h39

Bonjour,

J'ai creer un script qui insère des valeurs dans une bdd, lorsque je lance le script voila l'erreur qui apparait
Notice: Undefined variable: sql in C:\Program Files\EasyPHP-5.3.6.0\www\webdevproject\newtopic.php on line 23
Erreur SQL !
Query est vide

En gros l'erreur typique d'un sql query sans valeur, cependant je déclare $sql juste au dessus !!

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
73
74
75
76
77
78
79
80
81
<?php
session_start();
include ("bdd.php");
$base = mysql_connect ('localhost', 'root', '');
mysql_select_db ('supforum', $base);
if (isset ($_GET['cat'])) {
	$_GET['cat'] = $_POST['cat'];
}
 
if (isset ($_POST['go'])) { 
   // on teste la déclaration de nos variables
   if (!isset($_POST['description']) || !isset($_POST['titre']) || !isset($_POST['content'])) { 
      $erreur = 'Les variables nécessaires au script ne sont pas définies.'; 
   } 
   else
   { 
      if ((empty($_POST['titre'])) || (empty($_POST['content'])) || (empty($_POST['description']))) { 
         $erreur = 'Au moins un des champs est vide.'; 
      } 
      else {
	$sql = 'INSERT INTO topics VALUES("", "'.mysql_escape_string($_POST['titre']).'" , "'.mysql_escape_string($_POST['description']).'"  , "'.$_SESSION['id'].'" , "'.$_POST['cat'].'" ,  "")'; 
      }
      mysql_query($sql) or die('Erreur SQL !'.$sql.'<br />'.mysql_error());
      $id_sujet = mysql_insert_id();
	$sql = 'INSERT INTO messages VALUES("", "'.mysql_escape_string($_POST['content']).'", "'.$_SESSION['id'].'" ,"'.$id_sujet.'")';
	mysql_query($sql) or die('Erreur SQL !'.$sql.'<br />'.mysql_error());
	mysql_close();
	header('Location: membre.php');
	exit();
	}
}
?>
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
 
<meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
<title>SUPFORUM</title>
<style type="text/css">
.Titre {
	text-align: center;
	font-size: 24px;
	font-weight: bold;
	color: #000;
}
.Text1 {
	text-align: center;
}
 
<?php include ("wyhead.php"); ?>
 
 
</style>
</head>
 
<body>
<p>&nbsp;</p>
<?php include ("entete2.php"); ?>
<form action="newtopic.php" method="post">
<tr >
    <th>
    <input type="text" name="titre" value="<?php if (isset($_POST['text'])) echo htmlentities(trim($_POST['text'])); ?>">
	<input type="text" name="description" value="<?php if (isset($_POST['text'])) echo htmlentities(trim($_POST['text'])); ?>">
	$
    </th>
 
</tr>
<tr>
	<th>
		<?php include ("wybody.php") ?>
	</th>
</tr>
</form>	
 
<?php include ("baspage2.php"); ?>
</body>
</html>
 
<?php
if (isset($erreur)) echo ' <br /> ', $erreur;
?>

Je ne vois pas du tout d'où peut venir l'erreur .. Pouvez-vous m'aider ?

**Nesmontou** · 26/04/2011, 15h42

Bonjour,

En effet, tu déclares $sql juste au dessus, mais dans la clause else.

Si tu ne passes pas dedans, $sql n'est pas défini.

**TomTom_** · 26/04/2011, 15h45

Salut,

En lisant ton code, je me rends comptes qu'il ya un plan d'exécution où ta variable $sql ne sera pas initialisée :

Code :

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

1
2
3
4
5
6
if ((empty($_POST['titre'])) || (empty($_POST['content'])) || (empty($_POST['description']))) { 
         $erreur = 'Au moins un des champs est vide.'; 
      } 
      else {
	$sql = 'INSERT INTO topics VALUES("", "'.mysql_escape_string($_POST['titre']).'" , "'.mysql_escape_string($_POST['description']).'"  , "'.$_SESSION['id'].'" , "'.$_POST['cat'].'" ,  "")'; 
      }

Dans ce cas tu ne passeras pas dans le 'else' et ta variable ne sera pas initialisée, non?

TomTom_

**superstarz** · 26/04/2011, 16h09

Enfin dans mon idée je passais forcément dans le else ^^.
Donc bon j'ai levé cette condition et la maintenant cela marche.

Par contre, je sais pas si vous le voyez, mais je met sert d'une methode GET pour recuperer l'id de la catégorie, cependant dans ma table SQL celui-ci reste a 0.
Je sais pourquoi, car en faite lorsque j'ouvre la page j'ai bien newtopics.php?cat=5, mais lorsque je termine la rédaction de mon formulaire et que j'envoie les paramètre j'ouvre : newtopics.php (dans le form)
Quelles solutions s'offres à moi ? Dans le form rajouter "?rcat=' , $_GET['cat'] ,""
ça se fait ?

Dernière petite interogation, dans la bdd, j'ai initialiser mes date en timestamp avec en valeur par default : current timestamp.
Or tout reste a 0 alors qu'il aurai du me mettre la date non ?
Sinon un "time()" dans la requete SQL suffit ?

Merci pour vos réponses déjà vous avez été super rapide :p

**Shikiryu** · 26/04/2011, 16h42

Code :

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

1
2
3
if (isset ($_GET['cat'])) { // si on reçoit 'cat' en GET
	$_GET['cat'] = $_POST['cat']; // on lui donne quand même la valeur en POST (aka $_GET['cat'] est remplacé)
}

Je ne sais pas si c'est voulu comme ça vu que vous utilisez $_POST['cat'] dans la requête.

Cela devrait être

Code :

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

1
2
3
if (isset ($_GET['cat'])) { 
	$_POST['cat'] = $_GET['cat'];
}

(mais c'est quand même bien laid.)

**superstarz** · 26/04/2011, 17h09

Ah oui rolalala je suis fatigué on dirait ...
Enfin c'est le seul moyen que j'ai trouvé pour passer ma variable pour m'en resservir.
Et même comme ça, cela ne fonctionne pas

Erreur $sql undefined Variable

Langage PHP

Discussions similaires

Partager

Partager