Discussion :

[namrekan]

Bonjour,

Je viens vous soumettre mon nouveau problème pour un plugin wordpress après avoir farfouillé un peu partout.
Je voudrais écrire dans une table uniquement si l'entrée n'existe pas déjà.
J'ai donc fait la requête suivante :

Code php :

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

1
2
3
4
5
6
7
8
9
10
11
global $wpdb;
$user_id = get_current_user_id();
 
$wpdb->query("
        CASE
		WHEN user_id != $user_id THEN
			INSERT INTO ma_table (user_id, utilisateur_data)
			VALUES ($user_id, 'Madonnee')
		ELSE 'rien'
	END
");

Ma table comporte les colonnes user_id et _utilisateur_data. Je ne sais pas quoi mettre dans le ELSE pour que ça n'exécute aucune action.
Mais ça me retourne une erreur de syntaxe.
Savez-vous comment je peux m'y prendre ? J'ai testé aussi avec des WHERE NOT EXISTS...

Bonne journée !

[vanagreg]

Bonjour,

Tu peux utiliser un NOT EXISTS en effet:

Code :

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

1
2
3
INSERT INTO ma_table (user_id, utilisateur_data)
SELECT $user_id, 'Madonnee'
WHERE NOT EXISTS (select null from ma_table t2 where t2.user_id != $user_id)

[namrekan]

Merci !
Par contre ça me retourne une erreur de syntaxe SQL :

Erreur de la base de données WordPress : [You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near 'WHERE NOT EXISTS (select null from ma_table t2 where t2.user_id != 1)' at line 3]

Je ne sais pas quoi faire haha...
Et à quoi sert le t2 ?

Encore merci pour ton explication.
Bonne journée

[vanagreg]

C'est du MariaDB, peut-être qu'utiliser la table DUAL fonctionne alors:

Code :

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

1
2
3
INSERT INTO ma_table (user_id, utilisateur_data)
SELECT $user_id, 'Madonnee' FROM DUAL
WHERE NOT EXISTS (select null from ma_table t2 where t2.user_id != $user_id)

t2 c'est juste l'alias de la table ma_table qui sert à vérifier l'existence de la ligne dans ta table

[escartefigue]

Bonjour,

La restriction du test d'existence utilise l'opérateur "différent de" qui n'est pas sargable, du coup cette recherche sera très lente si la table est volumineuse.
Pourquoi ne pas simplement faire l'INSERT et ignorer le message d'erreur si clef en double ?

Attention aussi : SELECT NULL n'est pas applicable sur tous les SGBD (SELECT 1 par exemple est applicable sur tous les SGBD)

Par ailleurs, la syntaxe du case est invalide (ELSE 'RIEN' n'est pas du SQL !)

[namrekan]

Merci à vous deux pour ces précisions. Je n'avais aucune notion de ce FROM DUAL masi ça semble résoudre mon problème.
Ma requête est au final :

Code php :

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

1
2
3
4
5
$wpdb->query("
	INSERT INTO ecn_users_items (user_id, utilisateur_data)
	SELECT $user_id, 'Madonnee' FROM DUAL
	WHERE NOT EXISTS (select * from ma_table where user_id = $user_id)
");

J'ai des petites questions si ça ne vous embête pas : quel est l'intérêt ici de mettre un alias de table ?
@escartefigue : vu que mon test est une égalité c'est bon pour le temps de recherche ? Aurais-je pu mettre dans mon CASE un INSERT IGNORE ?

Et une dernière : j'ai vu qu'il fallait préparer la requête pour empêcher une attaque, est-ce le bon code ?

Code php :

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

1
2
3
4
5
6
7
8
9
$wpdb->query(
        $wpdb->prepare(
        "
	INSERT INTO ecn_users_items (user_id, utilisateur_data)
	SELECT %d, 'Madonnee' FROM DUAL
	WHERE NOT EXISTS (select * from ma_table where user_id = $user_id)
        ",$user_id
        )
);

Je vous remercie pour votre temps.
Bonne journée

[escartefigue]

J'ai des petites questions si ça ne vous embête pas : quel est l'intérêt ici de mettre un alias de table ?

Quand il n'y a ni jointure, ni requête corrélée, un alias de table n'a pas d'intérêt

@escartefigue : vu que mon test est une égalité c'est bon pour le temps de recherche ?

Oui, l'opérateur "égal" est sargable, sous réserve que la variable qui est comparée soit du même type et de la même longueur que la colonne

Aurais-je pu mettre dans mon CASE un INSERT IGNORE ?

Oui

Et une dernière : j'ai vu qu'il fallait préparer la requête pour empêcher une attaque, est-ce le bon code ?

C'est surtout qu'il s'agit de SQL dynamique (requête paramétrée), l'ordre "prepare" est donc requis pour assembler les morceaux de la requête, avant d'exécuter la requête avec l'ordre "execute"

[namrekan]

Un très grand merci !