Discussion :

[slimsamfr]

Bonjour à vous,

J'ai récupéré ce code pour supprimer les doublons et cela ne fonctionne pas, je suis sur SQL Server 2008 R2.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
DELETE 
FROM   SALARIES Ta 
WHERE  Ta.COLID < ANY (SELECT t2.COLID
                      FROM    SALARIES T2
                      WHERE  Ta.COLID <> T2.COLID
                        AND  Ta.NOM = T2.NOM
                        AND  Ta.PRENOM    = T2.PRENOM
                        AND  Ta.AGES = T2.AGES) 
 
ERREUR :
Msg 102, Niveau 15, État 1, Ligne 3
Syntaxe incorrecte vers 'Ta'

Merci.

[fflores]

Pas sur que tu puisses aliaser dans un delete
Essaie sans l'alias:

Code :

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

1
2
3
4
5
6
7
8
9
 
DELETE
FROM SALARIES
WHERE COLID < ANY (SELECT t2.COLID
FROM SALARIES T2
WHERE SALARIES.COLID <> T2.COLID
AND SALARIES.NOM = T2.NOM
AND SALARIES.PRENOM = T2.PRENOM
AND SALARIES.AGES = T2.AGES)

[Invité]

Pas sur que tu puisses aliaser dans un delete

Ça sort d'où cette légende urbaine ???
Un exemple rapide d'alias que j'ai sous le coude (mais sans rapport avec la demande initiale) :

Code :

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

1
2
3
4
5
6
7
8
-- supprime tous l'historique des jobs de PROD de plus de 6 mois qui ont fini en succès
delete DAJSJH
from AllDbs.db_all_jobs_sysjobhistory DAJSJH
join AllDbs.db_all_MASTER DAM
	on DAM.sname = DAJSJH.instance 
where DAJSJH.ts_day < dateadd ( month , -6 , @today )
	and DAJSJH.run_status = 1 -- SUCCESS
	and DAM.prod = 1 -- PROD

[fflores]

OK, je comprends maintenant pourquoi moi aussi je rencontrais des difficultés, j'avais mal analysé la syntaxe.
En fait, d'après ce que je vois dans cet exemple il faut ajouter l'alias juste après le delete, c'est ce qui manquait.
Il fallait écrire
DELETE Ta
FROM...

Il eut peut-être été plus sympa et plus constructif d'expliquer cela plutôt que de faire ce genre de remarque non?

[Rei Ichido]

Parce qu'il faut préciser l'alias après le delete.

Cette requête fonctionnerait :

Code :

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

1
2
3
4
5
6
7
8
9
 
DELETE Ta
FROM   SALARIES Ta 
WHERE  Ta.COLID < ANY (SELECT t2.COLID
                      FROM    SALARIES T2
                      WHERE  Ta.COLID <> T2.COLID
                        AND  Ta.NOM = T2.NOM
                        AND  Ta.PRENOM    = T2.PRENOM
                        AND  Ta.AGES = T2.AGES)

[Invité]

Il eut peut-être été plus sympa et plus constructif d'expliquer cela plutôt que de faire ce genre de remarque non?

Désolé mais pour la correction de syntaxe, j'ose imaginer que la première chose qui est faite, est d'aller la vérifier directement à la source sous MSDN.

[slimsamfr]

Merci Fabrice c'est ce que j'ai fait et sa marche.

Voici le code complet :

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
/****** AJOUTER UNE COLONNE QUI SERA UN CHAMP AVEC IDENTIFIANT UNIQUE ******/
ALTER TABLE dbo.SALARIES
      ADD COLID INT 
 
 
 
 
/****** ALIMENTER LA COLONNE CREER PRECEDAMENT AVEC DES VALEURS UNIQUE ******/
DECLARE @i INT
DECLARE @colid INT
 
SET @i = 1
 
DECLARE CURSEUR_DOUBLON CURSOR
FOR    
   SELECT COLID
   FROM   SALARIES
FOR UPDATE
 
OPEN CURSEUR_DOUBLON
 
FETCH CURSEUR_DOUBLON INTO @colid
 
WHILE @@FETCH_STATUS = 0
BEGIN
 
   UPDATE SALARIES
   SET    COLID = @i
   WHERE  CURRENT OF CURSEUR_DOUBLON
 
   SET @i = @i + 1
 
   FETCH CURSEUR_DOUBLON INTO @colid  
 
END
 
CLOSE CURSEUR_DOUBLON
 
DEALLOCATE CURSEUR_DOUBLON
 
 
 
/****** SUPPRIMER LES LIGNES EN DOUBLE ******/                            
 
DELETE FROM SALARIES
WHERE COLID NOT IN
(
	SELECT MIN(COLID) 
	FROM SALARIES T2
	WHERE SALARIES.NOM = T2.NOM
          AND  SALARIES.PRENOM    = T2.PRENOM
          AND  SALARIES.AGES = T2.AGES
	GROUP BY T2.NOM, T2.PRENOM, T2.AGES
);