comment enlever les notices [Résolu]

[arckaniann]

Bonjour,

voila j'ai des erreurs de notices e rapport a des variables non défini,

je sais pourquoi elle ne sont pas défini car je ne les utilise pas,

j'explique:j'utilise la mêmes pages de traitement pour les modification et l'insertion,donc quand j'insert une données j'ai un indéfini index pour la variable id

j'ai bien utilisé le isset ou le empty mais j'ai toujours le mêmes problèmes,je peux aussi utilisé la commande pour cacher les notices mais sa je ne veux pas


merci pour votre aide

[Séb.]

je sais pourquoi elle ne sont pas défini car je ne les utilise pas,

Le problème est que tes variables ne sont pas définies et que malgré tout tu les utilises.

j'explique:j'utilise la mêmes pages de traitement pour les modification et l'insertion,donc quand j'insert une données j'ai un indéfini index pour la variable id

Bizarre, à l'insertion tu fais un INSERT, donc pas besoin de donner le ID qui est généralement auto-généré, l'embranchement de ton script devant éviter naturellement de la mentionner.

[arckaniann]

je me suis mal expliquer,disons que j'insert jeu,stock,image,lien et prix mais dans la modif je ne change que le prix,le stock et le nom donc l'indefini index est le lien,pour l'id c'est pareil,je ne l'utilise pas pour l'insert mais pour la modif oui

[Séb.]

Je ne vois pas le problème. Faut juste organiser correctement ton script.
Eventuellement montre-le pour qu'on puisse te conseiller.

[arckaniann]

je vous met une partie du code mais je sens les critiques sur l'indentation,je les faites avec netbeans

Code php :

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
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
 
if(empty($_POST['jeu_occasion'])&&empty($_POST['lien_jeu_occasion'])&&empty($_POST['prix_jeu_occasion'])
&&empty($_POST['stock_jeu_occasion'])&&empty($_GET['id']))
 
 
 
$jeu_occasion = htmlspecialchars(addslashes($_POST['jeu_occasion']));
$lien_jeu_occasion = htmlspecialchars(addslashes($_POST['lien_jeu_occasion']));
$prix_jeu_occasion = htmlspecialchars(addslashes($_POST['prix_jeu_occasion']));
$stock_jeu_occasion = htmlspecialchars(addslashes($_POST['stock_jeu_occasion']));
 
 
if (!empty($_FILES['jeu_occasion']['size'])) {
	//On définit les variables :
	$maxsize = 35000; //Poid de l'image
	$maxwidth = 300; //Largeur de l'image
	$maxheight = 300; //Longueur de l'image
	$extensions_valides = array('jpg', 'jpeg', 'gif', 'png', 'bmp'); //Liste des extensions valides
 
	if ($_FILES['jeu_occasion']['error'] > 0) {
		$avatar_erreur = "Erreur lors du tranfsert de l'avatar : ";
	}
	if ($_FILES['jeu_occasion']['size'] > $maxsize) {
		$i++;
		$image_erreur1 = "Le fichier est trop gros : (<strong>" . $_FILES['jeu_occasion']['size'] . " Octets</strong>    contre <strong>" . $maxsize . " Octets</strong>)";
	}
 
	$image_sizes = getimagesize($_FILES['jeu_occasion']['tmp_name']);
	if ($image_sizes[0] > $maxwidth OR $image_sizes[1] > $maxheight) {
		$i++;
		$image_erreur2 = "Image trop large ou trop longue :
						(<strong>" . $image_sizes[0] . "x" . $image_sizes[1] . "</strong> contre <strong>" . $maxwidth . "x" . $maxheight . "</strong>)";
	}
 
	$extension_upload = strtolower(substr(strrchr($_FILES['jeu_occasion']['name'], '.'), 1));
	if (!in_array($extension_upload, $extensions_valides)) {
		$i++;
		$image_erreur3 = "Extension de l'avatar incorrecte";
	}
}
 
 
 
$nomimage = (!empty($_FILES['jeu_occasion']['size'])) ? move_image($_FILES['jeu_occasion']) : '';
 
 print_r($_FILES);
 
 
$query = $db->prepare("SELECT COUNT(*) AS nombre_entree FROM jeu WHERE id = :id");
$query->bindValue(':id', $_POST['id'], PDO::PARAM_INT);
$query->execute();
$donnees = $query->fetch();
 
 
 
 
 
 
if ( $donnees['nombre_entree'] == 0) 
    {
 
 $query = $db->prepare('INSERT INTO jeu_occasion   (jeu_occasion,image,
 
									lien_jeu_occasion,
								        prix_jeu_occasion,
									stock_jeu_occasion)
 
							VALUES     (:jeu_occasion,:nomimage,
								    :lien_jeu_occasion,
								    :prix_jeu_occasion,
								     :stock_jeu_occasion
								)');
 
 
 
 
	$query->bindvalue(':jeu_occasion', $jeu_occasion, pdo::PARAM_STR);
	$query->bindvalue(':lien_jeu_occasion', $lien_jeu_occasion, pdo::PARAM_STR);
	$query->bindvalue(':prix_jeu_occasion', $prix_jeu_occasion, pdo::PARAM_INT);
	$query->bindvalue(':stock_jeu_occasion', $stock_jeu_occasion, pdo::PARAM_INT);
	$query->bindvalue(':nomimage', $nomimage, pdo::PARAM_STR);
	$query->execute() or die(print_r($db->errorinfo()));
 
 
	}else{ 
 
 
	$query = $db->prepare("UPDATE jeu_occasion
 
		                      SET  jeu_occasion = :jeu_occasion,
			                   prix_jeu_occasion = :prix_jeu_occasion,
					   stock_jeu_occasion = :stock_jeu_occasion
				    WHERE  id= :id
					");
        $query->bindvalue(':id', $_GET['id'], pdo::PARAM_INT);
	$query->bindvalue(':jeu_occasion', $jeu_occasion, pdo::PARAM_STR);
	$query->bindvalue(':lien_jeu_occasion', $lien_jeu_occasion, pdo::PARAM_STR);
	$query->bindvalue(':prix_jeu_occasion', $prix_jeu_occasion, pdo::PARAM_INT);
	$query->bindvalue(':stock_jeu_occasion', $stock_jeu_occasion, pdo::PARAM_INT);
	$query->execute() or die(print_r($db->errorinfo()));
 
	}
if ($i==0){		
	echo'<p>les données du jeu d\'occasion a bien été rentrée !<br /> Cliquez <a href="../tests/magasin.php">ici</a> 
            pour revenir Ã* l\'accueil</p>';
 
}else{ //on affiche les erreurs//
        echo'<h1>Inscription interrompue</h1>';
        echo'<p>Une ou plusieurs erreurs se sont produites pendant la rentré des données</p>';
        echo'<p>'.$i.' erreur(s)</p>';
        echo'<p>'.$image_erreur.'</p>';
        echo'<p>'.$image_erreur1.'</p>';
		echo'<p>'.$image_erreur2.'</p>';
        echo'<p>'.$image_erreur3.'</p>';
 
 
		echo'<p>cliquez <a href = "../tests/site.php">ici</a> </p>';
	$query->CloseCursor();
 
}

[Séb.]

Code :

1
2
3
4
5
6
7
if(empty($_POST['jeu_occasion'])&&empty($_POST['lien_jeu_occasion'])&&empty($_POST['prix_jeu_occasion'])
&&empty($_POST['stock_jeu_occasion'])&&empty($_GET['id']))
 
$jeu_occasion = htmlspecialchars(addslashes($_POST['jeu_occasion']));
$lien_jeu_occasion = htmlspecialchars(addslashes($_POST['lien_jeu_occasion']));
$prix_jeu_occasion = htmlspecialchars(addslashes($_POST['prix_jeu_occasion']));
$stock_jeu_occasion = htmlspecialchars(addslashes($_POST['stock_jeu_occasion']));

1. Pour déterminer si une variable existe il faut utiliser isset( ) et pas empty( )
2. Manque une { après le if( ), sinon seule la 1re instruction suivante est considérée
3. Pour échapper les données SQL il ne faut pas utiliser addslashes( ), et puis je crois que PDO peut s'en charger, à voir dans la doc
4. Il ne faut pas enregistrer du HTML dans ta BdD, il faut laisser les données le plus neutre possible, donc exit le htmlspecialchars( )

Code :

1
2
$query = $db->prepare("SELECT COUNT(*) AS nombre_entree FROM jeu WHERE id = :id");
$query->bindValue(':id', $_POST['id'], PDO::PARAM_INT);

Donc là je suppose que tu as une notification lors d'un INSERT.

Il suffirait de faire :

Code :

1
2
3
4
5
if ( isset($_POST['id']) ) {
    // UPDATE
} else {
    // INSERT
}

[arckaniann]

Code php :

1
2
3
 
$query = $db->prepare("SELECT COUNT(*) AS nombre_entree FROM jeu WHERE id = :id");
$query->bindValue(':id', $_POST['id'], PDO::PARAM_INT);

cette ligne me permet de savoir s'il y a déjà un identifiant dans la bdd si oui il update sinon il insert donc je vois pas ou je pourrais mettre le code ci dessous dans mon traitement

Code php :

1
2
3
4
5
6
 
if ( isset($_POST['id']) ) {
    // UPDATE
} else {
    // INSERT
}

ou bien est ce que je peut faire

Code php :

1
2
3
4
5
6
7
8
9
 
 
if ( $donnees['nombre_entree'] == 0) 
    {insert
 
}elseif ( isset($_POST['id']) ) {
    // UPDATE
 
}

[Séb.]

Le SELECT ne sert à rien car :
Si $_POST['id'] n'existe pas tu pars du principe que tu dois faire un INSERT ;
Sinon tu fais un UPDATE, au pire si $_POST['id'] est mal renseigné le UPDATE ne changera rien à la BdD.

[arckaniann]

ok merci pour l'aide,petit question qui a rien a voir mais je voulais savoir se que voulais dire cette phrase


#1064 - You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near ''console' WHERE 'console' LIKE '%:recherche%' UNION SELECT 'console_occasion'' at line 13

merci

[Séb.]

Retire les ' autour du nom de champ console ;-)
Si le pb persiste colle ici la requête soumise au serveur SQL.

[arckaniann]

ok je vais voir sa

merci

[arckaniann]

bon alors voila la requête mais elle est très grosse car c'est pour une recherche,je pense qu'il y a des erreurs dans la requête et voici l’erreur:


Warning: PDOStatement::execute() [pdostatement.execute]: SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'SELECT console AS console, id, lien_console_neuf AS lien_console_neuf, console A' at line 12 in C:\wamp\www\tests\recherche_1.php on line 100
Array ( [0] => 42000 [1] => 1064 [2] => You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'SELECT console AS console, id, lien_console_neuf AS lien_console_neuf, console A' at line 12 ) 1

Code php :

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
 
 
$query = $db->prepare("
 
SELECT jeu AS `jeu`, id, lien_jeu_neuf AS lien_jeu_neuf, jeu AS jeu, prix_jeu_neuf AS prix_jeu_neuf, stock_jeu_neuf AS stock_jeu_neuf, image
FROM jeu
WHERE jeu LIKE '%:recherche%'
UNION
SELECT jeu_occasion AS jeu_occasion, id, lien_jeu_occasion AS lien_jeu_occasion, jeu_occasion AS jeu_occasion, prix_jeu_occasion AS prix_jeu_occasion, stock_jeu_occasion AS stock_jeu_occasion, image
FROM jeu_occasion
WHERE jeu_occasion LIKE '%:recherche%'
 
SELECT console AS console, id, lien_console_neuf AS lien_console_neuf, console AS console, prix_console_neuf AS prix_console_neuf, stock_console_neuf AS stock_console_neuf, image
FROM console
WHERE console LIKE %:recherche%
UNION
SELECT console_occasion AS console_occasion, id, lien_console_occasion AS lien_console_occasion, lien_console_occasion AS lien_console_occasion, prix_console_occasion AS prix_console_occasion, stock_console_occasion AS stock_console_occasion, image
FROM console_occasion
WHERE console_occasion LIKE %:recherche% 
 
 
SELECT manga AS manga, id, lien_manga_neuf AS lien_manga_neuf, manga AS manga, prix_manga_neuf AS prix_manga_neuf, stock_manga_neuf AS stock_manga_neuf, image
FROM manga
WHERE manga LIKE '%:recherche%'
 UNION
 SELECT manga_occasion AS manga_occasion, id, lien_manga_occasion AS lien_manga_occasion, manga_occasion AS manga_occasion, prix_manga_occasion AS prix_manga_occasion, stock_manga_occasion AS stock_manga_occasion, image
FROM manga_occasion
WHERE `manga_occasion` LIKE '%:recherche%'
 
 
SELECT 'blu_ray' AS `bluray`, id, `lien_blu_ray_neuf` AS `lien_blu_ray_neuf`, `blu_ray` AS `blu_ray`, `prix_blu_ray_neuf` AS `prix_blu_ray_neuf`, `stock_blu_ray_neuf` AS `stock_blu_ray_neuf`, `image`
FROM bluray
WHERE `blu_ray` LIKE '%:recherche%'
UNION 
SELECT 'blu_ray_occasion' AS `bluray_occasion`, id, `lien_blu_ray_occasion` AS `lien_blu_ray_occasion`, `lien_blu_ray_occasion` AS `lien_blu_ray_occasion`, `prix_blu_ray_occasion` AS `prix_blu_ray_occasion`, `stock_blu_ray_occasion` AS `stock_blu_ray_occasion`, `image`
FROM bluray_occasion`
WHERE `blu_ray_occasion` LIKE '%:recherche%'
 
 
SELECT 'dvd' AS `console`, id, `lien_dvd_neuf` AS `lien_dvd_neuf`, `dvd` AS `dvd`, `prix_dvd_neuf` AS `prix_dvd_neuf`, `stock_dvd_neuf` AS `stock_dvd_neuf`, `image`
FROM dvd
WHERE `dvd` LIKE '%:recherche%' 
UNION
SELECT 'dvd_occasion' AS `dvd_occasion`, id, `lien_dvd_occasion` AS `lien_dvd_occasion`, `lien_dvd_occasion` AS `lien_dvd_occasion`, `prix_dvd_occasion` AS `prix_dvd_occasion`, `stock_dvd_occasion` AS `stock_dvd_occasion`, `image`
FROM dvd_occasion
WHERE `dvd_occasion` LIKE '%:recherche%'
 
 
SELECT 'figurine' AS `figurine`, id, `lien_figurine_neuf` AS `lien_figurine_neuf`, `figurine` AS `figurine`, `prix_figurine_neuf` AS `prix_figurine_neuf`, `stock_figurine_neuf` AS `stock_figurine_neuf`, `image`
FROM figurine
 
WHERE `figurine` LIKE '%:recherche%'
UNION
SELECT 'figurine_occasion' AS `figurine_occasion`, id, `lien_figurine_occasion` AS `lien_figurine_occasion`, `figurine_occasion` AS `figurine_occasion`, `prix_figurine_occasion` AS `prix_figurine_occasion`, `stock_figurine_occasion` AS `stock_figurine_occasion`, `image`
FROM figurine_occasion
WHERE `figurine_occasion` LIKE '%:recherche%'
 
SELECT 'carte' AS `carte`, id, `lien_carte` AS `lien_carte`, `carte` AS `carte`, `prix_carte` AS `prix_carte`, `stock_carte` AS `stock_carte`, `image`
FROM carte
WHERE `carte` LIKE '%:recherche%'
UNION
SELECT 'carte_occasion' AS `carte_occasion`, id, `lien_carte_occasion` AS `lien_carte_occasion`, `carte_occasion` AS `carte_occasion`, `prix_carte_occasion` AS `prix_carte_occasion`, `stock_carte_occasion` AS `stock_carte_occasion`, `image`
FROM carte_occasion
WHERE `carte_occasion` LIKE '%:recherche%'");
$query->bindvalue(':recherche',$recherche, pdo::PARAM_STR);
$query->execute() or die(print_r($query->errorInfo()));

[Séb.]

Manque des UNION comme ici :

Code :

1
2
3
WHERE jeu_occasion LIKE '%:recherche%'
 
SELECT console AS console, id,

Il reste encore des noms de champs délimités avec des ' ou mal délimités comme FROM bluray_occasion`
La plupart de tes AS sont inutiles, ex. lien_console_neuf AS lien_console_neuf


En outre il est malvenu de créer une table par type de produits, tel que je le vois là c'est totalement inmaintenable et conduit à des situations inextricables comme celle vers laquelle tu sembles te diriger.

Mieux vaut avoir une table produits et une tables types :

Code :

1
2
3
4
5
6
7
produits # Description générale des produits
produit_id
produit_nom
produit_description
produit_prix
produit_stock
produit_type # Clef étrangère vers types.type_id

Code :

1
2
3
types
type_id
type_nom # DVD, Blu-Ray, etc.

Extraire tous tes produits et leur type :

Code :

1
2
SELECT produit_nom, type_nom
FROM produits INNER JOIN types ON produit_type = type_id

Cela vaut mieux que tes X UNION non ?

Pour chercher un produit selon son nom et parmi tous les types de produits :

Code :

1
2
3
SELECT produit_id, produit_nom
FROM produits
WHERE produit_nom LIKE '%$nom%'

Pour restreindre la recherche d'un produit à son nom et son type :

Code :

1
2
3
SELECT produit_id, produit_nom
FROM produits
WHERE produit_nom LIKE '%$nom%' AND produit_type = $type

Plus tard quand tu rajouteras un nouveau type de produits il suffira d'ajouter une entrée à la table types et le reste du script ne bougera pas.

[arckaniann]

au début que je fasse cela j'avais une table ou tout était regroupé mais j'avais des problèmes pour faire les mises a jours,c'est pour sa que j'avais fait sa,c'est vrai que c'est plus facile,je vais devoir tout refaire,quel boulot

[Séb.]

au début que je fasse cela j'avais une table ou tout était regroupé mais j'avais des problèmes pour faire les mises a jours,c'est pour sa que j'avais fait sa

Quels problèmes ?

[arckaniann]

bin comme je n'arrivais pas a mettre l'id dans l'update sois sa me fessait des mises a jours de tout mes champs sois sa inséré pas,j'ai failli m'arracher les cheveux ,donc j'avais fait plusieurs table mais sa en revenais au mêmes

alors j'ai demander sur un autre forum et on ma donné la réponse de faire insert et une modification sur des formulaires différents mais rien que le code fait plus de 800 lignes

par contre on peut pas faire d’insére sur deux tables

[Séb.]

je n'arrivais pas a mettre l'id dans l'update sois sa me fessait des mises a jours de tout mes champs

Parce que tu oubliais de restreindre le UPDATE à un enregistrement :

Code :

1
2
3
UPDATE une_table
SET un_champ = 'Une valeur', un_autre_champ = 'Une autre valeur'
WHERE id = $id

sois sa inséré pas

C'est juste un problème d'organisation du script.

[arckaniann]

la je suis en train de réfléchir a comment exposer ton idée qui est bonne a tout se que j'ai,sa me donne mal a la tète

[arckaniann]

j'ai fais comme tu m'avais dit mais comment je fais pour inséré un id comme clé etrangeres

[Séb.]

Pas compris ta question.