Discussion :

[dymezac]

Bonjour,

je soliccite votre aide pour une requête qui me pose problème :

P.S. :
- (je travaille avec du C# et Sql Serveur 2005, pour le code ci dessous je simplifie pas mal)
- COALESCE sert à remplacer les null par des 0

je voudrais éviter de faire deux requêtes (avec transaction serializable) :
1) récupérer le dernier id de ma table +1
2) insérer mon enregistrement avec l'id que j'ai récupérer

Code :

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

nouvelID = "SELECT (COALESCE(MAX(maTable_id),0)+1) FROM maTable;"

Code :

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

INSERT INTO maTable (maTable_id, maTable_lib) VALUES (nouvelID, 'du texte');

en une seule :
1) insérer mon enregistrement avec mon nouvel Id en même temps :

Code :

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

1
2
INSERT INTO maTable (maTable_id, maTable_lib) 
VALUES ((SELECT (COALESCE(MAX(maTable_id),0)+1) FROM maTable), 'du texte');

Mais je n'y arrive pas..
Sql Server me dit que je ne peux pas faire de sous requête dans ce contexte et que seules les expressions scalaires sont permises.

Alors j'ai essayé de mettre seulement COALESCE(MAX(maTable_id)) mais là il me dit que les noms de colonnes ne sont pas autorisées dans ce contexte..

Donc j'ai l'impression que ce n'est pas possible d'insérer un nouvel enregistrement en sélectionnant le (max des ID +1) dans la même requête..

Est ce que je ne peux vraiment pas le faire par aucun moyen et est ce que je suis vraiment obligé de faire deux requêtes pour insérer mes données ?
si oui je trouve ça domage mais s'il le faut il le faut..

d'avance merci beaucoup.

[SQLpro]

Lisez ceci :
http://sqlpro.developpez.com/cours/clefs/

Cela vous évitera au passage d'énormes bétises.

Intéressez vous aussi à la propriété IDENTITY que peut revêtir une colonne.

A +

[dymezac]

bonjour SqlPro et merci pour votre réponse,

en fait je dois travailler avec des tables dont les clés primaires ne sont pas auto-incrémentés et je dois le faire par moi même..

Et j'ai déjà tout lu sur le lien que vous m'avez donner (en passant, cet article m'a beaucoup appris)

Dans la mesure ou mes transactions sont 'SERIALIZABLE' je pense (? à tort ??) que ça ne pose pas de problème si je fais un Select MAX pour avoir un nouvel identifiant de clé primaire..

finalement, pour le problème que j'ai posé sur ce post, j'ai été obligé de passer par deux requêtes (en faisant mon select max)..

à moins que vous connaissiez un moyen de faire ces deux requêtes en une seule ?

encore merci !

[WOLO Laurent]

Vous pouvez créer une fonction UDF et la jouer dans votre commande insert et ca marche.
Merci.

[dymezac]

d'accord merci beaucoup, je vais me renseigner tout de suite sur les UDF (je ne connaissais pas). merci

[SQLpro]

Vous êtes obligés de faire deux requêtes et en mode SERIALIZABLE... Mais savez vous ce qui se passe dans le mode SERIALIZABLE ? Tout simplement un verrou exclusif est posé sur les ressources traitées. Aucun autre process ne peut donc ni lire ni écrire sur les ressources vérrouillées. Or en demandant une requête SELECT MAX(...) ... les ressources à considérer sont la table entière !!!
Au cours du développement au début de vos test, lorsqu'il y a peu d'utilisateurs et peu de données, cette méthode va donner de bons résultats dans des temps acceptable. Mais réfléchissez 30 secondes en essayant d'analyser ce qui va se passer lorsque vous aurez une table de 50 000 lignes et 200 utilisateurs simultanés... vous aurez des temps de réponse catastrophique, du aux bloquages imposés par les verrous et très certainement des interbloquages (deadlock, étreinte fatale, verrous mortel...). Bref si vous voulez livrer une application des plus pourries, continuez comme cela ! Mieux, utilisez des fichiers COBOL ce sera plus propre...

De plus je ne comprend pas votre souhait de ne pas utiliser l'auto incrément de SQL Server. Il est normalisé depuis la version SQL:2003 de la norme ISO de SQL. Il peut aussi être débranché pour des insertion avec des valeurs de clefs forcées. Je ne voit donc aucune incompatibilité pour l'utiliser dans toutes les circonstances...

Seul un manque de culture reflète votre incompréhension...

A +