Discussion :

[yann123456]

Bonjour

J'ai tenté de faire la requête ci-dessous qui ne fonctionne pas:

Code :

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

1
2
3
4
5
6
7
8
9
10
 
mysql_query("INSERT INTO categories_articles (
		id_article,
		id_categorie) VALUES(
		'" . mysql_real_escape_string($result['id']) . "',
		'" . mysql_real_escape_string($resultat['id_categorie']) . "')
		IF NOT EXISTS
		(SELECT * FROM categories_articles WHERE
                 id_article='" . mysql_real_escape_string($result['id']) . "'
                 AND id_categorie='" . mysql_real_escape_string($resultat['id_categorie']) . "' )") or die(mysql_error());

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 'IF NOT EXISTS (SELECT * FROM categories_articles WHERE id_article='1' at line 6

Voyez_vous ou est le probléme ?
Peut-on faire une requête avec IF NOT EXISTS avec mysql5 ?

Merci

[CinePhil]

Déjà, je suppose que id_article et id_categorie sont des colonnes de type entier donc inutile de s'embarrasser avec les ' et les mysql_escape_string.

Ensuite, INSERT n'accepte pas de condition. Il faut donc d'abord interroger la base pour voir si la ligne est déjà présente et si le résultat retourné est vide alors lancer l'insertion.

[yann123456]

Déjà, je suppose que id_article et id_categorie sont des colonnes de type entier donc inutile de s'embarrasser avec les ' et les mysql_escape_string.

mysql_escape_string ne sert a rien sur des entiers ?

Ensuite, INSERT n'accepte pas de condition. Il faut donc d'abord interroger la base pour voir si la ligne est déjà présente et si le résultat retourné est vide alors lancer l'insertion.

Ok, je voulais gagner une étape.

Merci

[Maljuna Kris]

Dans le cadre, très restreint, de cette requête, où les deux colonnes constituent une clef UNIQUE (me semble-t'il), et s'il n'y a pas d'autres colonnes dans la table, tu peux envisager d'utiliser REPLACE.
Mais il faut impérativement que les deux conditions que j'ai exposées soient vérifiées.

[yann123456]

Dans le cadre, très restreint, de cette requête, où les deux colonnes constituent une clef UNIQUE (me semble-t'il), et s'il n'y a pas d'autres colonnes dans la table, tu peux envisager d'utiliser REPLACE.
Mais il faut impérativement que les deux conditions que j'ai exposées soient vérifiées.

La table ne contient effectivement que ces deux champs, mais un article peux avoir plusieurs catégories et une catégorie peut avoir plusieurs articles.

id_article | id_categorie

124 | 25
126 | 32
141 | 25
172 | 32
126 | 56
124 | 32

[Maljuna Kris]

Oui ce sont des clés étrangères je présume, et chaque couple est UNIQUE, non ?

[CinePhil]

Si le couple est clé primaire, comme il se devrait, le fait de vouloir insérer un couple déjà présent dans la table engendrera une erreur MySQL qu'il est possible de capturer ou carrément d'ignorer car l'insertion ne se fera pas.

Quant à ça :mysql_escape_string ne sert a rien sur des entiers ?
Si je comprends bien la définition de mysql_real_escape_string, ça ne sert effectivement à rien pour des entiers !
Si la variable donne un entier, on peut la passer directement à la requête.

Si par contre la variable vaut par exemple 'L\'étoile', là ça semble utile.

[yann123456]

Oui ce sont des clés étrangères je présume, et chaque couple est UNIQUE, non ?

Tout a fait, ce sont des clés étrangére:

id_categorie est une clé primaire (auto-increment) de la table categorie.
id_article est une clé primaire (auto-increment) de la table article.

Donc effectivement chaque couple est unique ou du moins doit être unique.

[yann123456]

Si le couple est clé primaire, comme il se devrait, le fait de vouloir insérer un couple déjà présent dans la table engendrera une erreur MySQL qu'il est possible de capturer ou carrément d'ignorer car l'insertion ne se fera pas.

Pour l'instant cette table n'a pas de clé primaire.
Mais si j'ai bien compris la citation ci-dessus, si je sélectionne mes deux champs dans ma table et que je les mets comme clé primaire, je ne pourrais plus ajouter un couple identique ?

Ce qui correspond a ce que je recherche a faire.

[Maljuna Kris]

Si le couple est clé primaire, comme il se devrait, le fait de vouloir insérer un couple déjà présent dans la table engendrera une erreur MySQL qu'il est possible de capturer ou carrément d'ignorer car l'insertion ne se fera pas.

Effectivement, INSERT IGNORE sera bien plus efficace que REPLACE, l'insertion n'intervenant que si c'est utile.

Décidément Quimper me mets trop à l'ouest

[CinePhil]

Pour l'instant cette table n'a pas de clé primaire.

Erreur grave !

Mais si j'ai bien compris la citation ci-dessus, si je sélectionne mes deux champs dans ma table et que je les mets comme clé primaire, je ne pourrais plus ajouter un couple identique ?

Ce qui correspond a ce que je recherche a faire.

Ben oui ! C'est le principe des tables associatives ! Elles prennent comme clé primaire l'ensemble des clés primaires provenant des tables concernées par l'association.

[yann123456]

Merci de vos réponses.

Je fais quelques essais et je vous redis ce qu'il en est.

[Maljuna Kris]

Si je comprends bien la définition de mysql_real_escape_string donnée par manuel php, ça ne sert effectivement à rien pour des entiers !
Si la variable donne un entier, on peut la passer directement à la requête.

Dans le cas précis d'une composante de VALUES, mais comme argument d'une clause WHERE ou HAVING, sans autre information sur la provenance de $result['id'] ni de $resultat['id_categorie'] (bien que l'on se doute qu'ils proviennent de résultats de requêtes préalables, ce qui devrait nous porter à nous interroger sur la pertinence d'une éventuelle requête INSERT IGNORE INTO ... SELECT) je me montrerais plus circonspect; gare aux injections SQL !

[yann123456]

[fodjio]

Merci à CinePhil et à yann (pour avoir posé la question )

J'avais le même problème que yann,
et grâce à vous j'ai découvert tout d'abord à quoi servaient les clé primaires et ensuite qu'on pouvait inclure plusieurs champs dans une clé primaire.

Code :

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

1
2
ALTER TABLE matable DROP PRIMARY KEY ,
ADD PRIMARY KEY ( champs1 , champs2 )