Discussion :

[ThibaultCollin]

Bonjour,
J'ai un soucis avec ON DUPLICATE KEYS.

Voici la structure de mes tables :
TABLE T_mouvement:


TABLE T_histo :


Et enfin, voici la requête SQL que j'essaye d'executer :

Code :

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

1
2
3
4
5
6
7
 
INSERT INTO T_histo(periode, compte, somme_depuis_periode)
SELECT imputation, depuis, montant
FROM `T_mouvement` 
GROUP BY depuis, imputation
ORDER BY id_mouvement ASC
ON DUPLICATE KEYS UPDATE T_histo.periode=T_mouvement.imputation, T_histo.compte=T_mouvement.depuis

Le problème, c'est que mySQL ne reconnait pas le mot "DUPLICATE" de la fonction "ON DUPLICATE KEYS UPDATE".
Voici les erreurs :

Analyse statique :

15 erreurs trouvées lors de l'analyse.

Mot clé non reconnu. (near "KEYS" at position 186)
Un nouvel énoncé a été trouvé, mais il n'y a aucun délimiteur entre celui-ci et le précédent. (near "UPDATE" at position 191)
Jeton inattendu. (near "UPDATE" at position 191)
Jeton inattendu. (near "=" at position 213)
Jeton inattendu. (near "T_mouvement" at position 214)
Jeton inattendu. (near "." at position 225)
Jeton inattendu. (near "imputation" at position 226)
Jeton inattendu. (near "," at position 236)
Jeton inattendu. (near "T_histo" at position 238)
Jeton inattendu. (near "." at position 245)
Jeton inattendu. (near "compte" at position 246)
Jeton inattendu. (near "=" at position 252)
Jeton inattendu. (near "T_mouvement" at position 253)
Jeton inattendu. (near "." at position 264)
Jeton inattendu. (near "depuis" at position 265)

Voici également la configuration de mon serveur :


PS : J'effectue ces requêtes à l'aide de phpMyAdmin.

Quelqu'un a-t-il une idée de ce qui cloche ?

Merci par avance,
Thibault

[manzeki]

La bonne syntaxe serait :

Code :

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

ON DUPLICATE KEY UPDATE

et non

Code :

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

ON DUPLICATE KEYS UPDATE

Par contre, je ne sais pas si MySQL implemente ON DUPLICATE KEY avec un SELECT

[ThibaultCollin]

Super, merci ! Erreur bête...
Concernant le SELECT, la seule façon de le savoir, c'est de résoudre la prochaine erreur

#1054 - Champ 'T_mouvement.imputation' inconnu dans field list

Ce champ existe pourtant bien... Promis, j'ai regardé l'orthographe cette fois !

[manzeki]

Après test les versions recentes de MySQL implémentent bien un INSERT .... SELECT ... ON DUPLICATE KEY

Code :

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

1
2
3
4
5
6
7
 
INSERT INTO T_histo(periode, compte, somme_depuis_periode)
SELECT imputation, depuis, montant
FROM T_mouvement
GROUP BY depuis, imputation
ORDER BY id_mouvement ASC
ON DUPLICATE KEY UPDATE T_histo.periode = T_mouvement.imputation, T_histo.compte = T_mouvement.depuis

[ThibaultCollin]

Super, merci d'avoir pris le temps de tester.
Et pour le fait que le champ ne soit pas reconnu, as-tu une idée ?
L'idée étant que s'il existe déjà une ligne où T_histo.periode = T_mouvement.imputation et T_histo.compte = T_mouvement.depuis, il la mette à jour. Sinon, il en insert une nouvelle.

[escartefigue]

Bonsoir

Ce n'est pas l'objet de votre question, mais je constate que le typage des colonnes est très mauvais :
- les montants dans un type double auront une mauvaise précision décimale, il faut utiliser un type decimal pour préserver la précision
- les colonnes alphanumériques sont de type texte avec un collation UTF8MB4. C'est un type très encombrant et une collation également encombrante, vu le nom des colonnes, je doute que ces choix soient judicieux.
il faudrait plus probablement du char fixe si la longueur maxi est faible (cas du compte comptable par exemple) ou du varchar si elle est plus importante et sujette à variation selon les lignes.

À vérifier

[ThibaultCollin]

Bonsoir escartefigue,
Merci pour ces conseils. Il est vrai que je début dans cette BDD, et n'ai pas encore vraiment cherché à l'optimiser.
Je vais modifier ça.

Bonne soirée

[isabelle.letrong]

Et pour le fait que le champ ne soit pas reconnu, as-tu une idée ?

Bonjour,

En préambule, dans votre requête la clause 'group by' ne sert à rien puisqu'elle est faite pour travailler sur des agrégats (vous n'agrégez pas des données pour en faire une somme, une moyenne, etc..).
En second lieu, concernant le champ non reconnu : dans votre clause insert, la sous-requête SELECT produit une table interne QUI N'EST PAS LA TABLE 'T_mouvement'.
Si vous voulez vraiment préfixer 'imputation' et 'depuis' par un nom de table, utilisez un alias pour nommer la table interne issue du SELECT (Donc après la clause ORDER et avant 'ON DUPLICATE...' introduisez 'AS T1' ce qui vous permettra de référencer les colonnes sources respectivement par T1.imputation et T1.depuis. Sinon, plus simplement, comme il n'y a pas de confusion possible (pas d'homonymie de colonnes) vous pouvez juste référencer les colonnes de cette table interne par 'imputation' et 'depuis'.