INTERBASE: DELETE avec sous requete conditionnelle [Résolu]

[Papino]

INTERBASE ne semble pas accepter la syntaxe suivante ( empruntée au cours de SQLPro ):

Code :

1
2
3
4
 
DELETE FROM T_PROSPECT
WHERE  (CLI_NOM, CLI_PRENOM) = (SELECT PRP_NOM, PRP_PRENOM
                                FROM   T_CLIENT)

avec le message d'erreur 'Token unknown , ' laissant supposer qu'il n'est pas possible de sélectionner plusieurs colonnes comme critère de sélection.
Comment contourner cet obstacle ( syntaxique ? ) pour supprimer les rangées d'une table existant dans une autre table ?

:

Merci de vos conseils

[Barbibulle]

Il faut le faire avec une requete corellée.

Code :

1
2
DELETE FROM T_PROSPECT TP
WHERE  EXISTS (SELECT * FROM   T_CLIENT TC WHERE TC.PRP_NOM = TP.CLI_NOM AND TC.PRP_PRENOM= TP.CLI_PRENOM)

Mais bon normalement dans la table T_PROSPECT on met une clé étrangère ce qui évite de devoir recopier le nom et prénom de votre client dans la table prospect et permet de retrouver plus rapidement les enregistrements relatifs à un client. Et aussi permet si vous le désirer de supprimer tous les enregistrements relatifs à votre client quand vous le supprimez.

[Fares BELHAOUAS]

Salam,

Je crois que cette requête fait l'affaire aussi:

Code :

1
2
DELETE FROM T_PROSPECT TP 
WHERE  (SELECT TC.PRP_NOM FROM   T_CLIENT TC WHERE TC.PRP_NOM = TP.CLI_NOM AND TC.PRP_PRENOM= TP.CLI_PRENOM) IS NOT NULL

Bon courage.

[Papino]

Les deux fonctionnent parfaitement.
Seul problème, mais de taille : la lenteur !
J'ai pensé arranger les choses en rajoutant des index sur les colonnes adéquates, sans grand résultat probant.
Mon but est d'insérer de nouvelles lignes et de remplacer ( ou de mettre à jour ) les lignes pré-existantes ( de même valeur d'index ) d'une table d'un millier de ligne dans une table qui en fait environ 120 000.

Merci encore de vos irremplaçables conseils.

[Benjamin GAGNEUX]

Salut,

J'ai pensé arranger les choses en rajoutant des index sur les colonnes adéquates, sans grand résultat probant.

Si tu fais un Delete, il vaut mieux supprimer tous les index/FK/PK avant et de les reconstruire après.

Ainsi, les ressources nécessaire à la suppression sont moins important et la reconstruction des index/FK/PK permet de les avoir optimiser sur le dernier 'état' des données.

A+

[Barbibulle]

Envoyé par yobenzen

Si tu fais un Delete, il vaut mieux supprimer tous les index/FK/PK avant et de les reconstruire après.

Ainsi, les ressources nécessaire à la suppression sont moins important et la reconstruction des index/FK/PK permet de les avoir optimiser sur le dernier 'état' des données.

Celà va dépendre de la proportion de suppression par rapport au nombre d'enregistrements de la table.
Or je ne pense pas qu'il veuille supprimer la majorité des enregistrements?

Il faut en effet créer 2 indexes
le premier sur la table T_PROSPECT (PRP_NOM, PRP_PRENOM)
et le second sur la table T_client (CLI_NOM, CLI_PRENOM).

A moins que pour la table prospect vous n'ayez construit un foreign key sur nom, prenom dans ce cas l'indexe exise déjà.

Ensuite une autre remarque plus général, si dans la table prospect vous aviez plutot mis une colonne FK_ID_CLI (représentant la clé primaire de la table client) de type integer à la place de NOM et PRENOM, ça serait certainement plus performant, car un integer est plus petit en taille et donc prend moins de place et les recherche/comparaison sont plus rapide et facile à faire sur un integer que sur des chaines.
Mais bon cette remarque est valable pour les gros volumes.

Qu'appelez vous lenteur du traitement ? Combien d'enregistrements dans votre table client, votre table prospect et combiens sont concernés par la supression ?

[jean-jacques varvenne]

attention que les index sont inutilisés dans les "sous-requêtes" et qu'il est infiniment plus rapide de construire une procédure stockée qui se charge de détruire les enregistrements

un exemple:

Code :

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
CREATE procedure deletemassive
AS
  declare variable db_key char(8);
begin
  FOR SELECT rdb$db_key FROM matable WHERE mescondistions
  INTO :db_key do
  begin
 
    DELETE matable WHERE rdb$db_key = :db_key;
 
  end
 
end

c'est le procédé le plus rapide, je n'est encore rien trouvé de mieux pour faire le ménage.

attention toutefois aux index qui sont désorganisés en cas de "delete" massif, dans ce cas, le plus sage et de les désactivés avant l'opération et de les réactiver après, enfin, si cela est possible.