SELECT dans UPDATE

Bonjour,

Voilà, je cherche désespérément à faire un SELECT pour une valeur de mon UPDATE.
En fait, je vous explique brièvement le pourquoi du comment : j'ai un formulaire, et la personne qui saisit rentre le nom de l'activité de la personne concernée en clair, et quand je récupère cela, il faut que j'écrive dans le table agent le numéro correspondant à l'intitulé de l'activité qu'a rentré la personne qui saisie...

Voici ce que j'ai essayé :

Code:

---

1 2 3 4 5 6 7 8 9 10

MODIF = "UPDATE agent " _ + "SET id_geode = '"& Request("txtBxIdRh") &"', " _ + "civilite = '"& Request("txtBxCivilite") &"', " _ + "nom = '"& Request("txtBxNom") &"', " _ + "prenom = '"& Request("txtBxPrenom") &"', " _ + "id_fonction = (SELECT agent.id_fonction, fonction.id_fonction, fonction.intitule_fonction " _ + " FROM agent, fonction " _ + " WHERE fonction.intitule_fonction = '"& Request("txtBxId_Fonction") &"' AND agent.id_fonction = fonction.id_fonction), " _ + "type_acces = "& acces &" " _ + "WHERE id_geode = '"& Request("txtBxIdRh") &"' "

---

Si je n'ai pas été assez claire...

Toutes remarques, tout conseils... seront là bienvenue.

Audrey

Bonjour

Vous essayez d'assigner à une colonne (id_fonction) le résultat d'une requête renvoyant plusieurs colonnes... SQL Server ne va pas choisir à votre place la bonne valeur !

En plus votre requête retourne plusieurs lignes, une par agent. Vous n'avez pas besoin de jointure sur agent dans votre sous requête, dont le but est uniquement de trouver l'id d'une fonction à partir de son intitulé :

à partir de votre requête, vous pouvez faire :

Code:

---

1 2 3 4 5 6 7 8 9 10 11

MODIF = "UPDATE agent " _ + "SET id_geode = '"& Request("txtBxIdRh") &"', " _ + "civilite = '"& Request("txtBxCivilite") &"', " _ + "nom = '"& Request("txtBxNom") &"', " _ + "prenom = '"& Request("txtBxPrenom") &"', " _ + "id_fonction = (SELECT fonction.intitule_fonction " _ + " FROM fonction " _ + " WHERE intitule_fonction = '"& Request("txtBxId_Fonction") &"' AND agent.id_fonction = fonction.id_fonction), " _ + "type_acces = "& acces &" " _ + "WHERE id_geode = '"& Request("txtBxIdRh") &"'

---

Il reste à s'assurer que "intitulé fonction" est unique, pour cela, ajoutez (si elle n'existe pas déjà) une contrainte d'unicité sur cette colonne.

Mais vous pouvez faire plus simplement avec un UPDATE ... FROM

Quelque chose comme :

Code:

---

1 2 3 4 5 6 7 8

UPDATE Agent SET id_geode = ... ... id_fonction = F.id_fonction FROM fonction F WHERE F.intitule_fonction = '...' AND id_geode = ...

---

Bonjour,

Quelque chose m'échappe : j'ai réécrit votre requête comme suit :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

UPDATE dbo.Agent SET id_geode = @_txtBxIdRh , civilite = @_txtBxCivilite , nom = @_txtBxNom , prenom = @_txtBxPrenom , id_fonction = F.id_fonction , type_acces = @_acces FROM dbo.Agent AS A INNER JOIN dbo.Fonction AS F ON A.id_fonction = F.id_fonction WHERE A.id_geode = @_txtBxIdRh AND F.intitule_fonction = @_txtBxId_Fonction

---

Si je comprend bien un Agent peut avoir plusieurs fonctions ...
Sans cela l'id_fonction restera identique puisque la jointure se fait sur l'id_fonction ...

Aujourd'hui, au travail comme sur le forum, j'ai vu combien il est compliqué et lourd d'écrire une requête dans le code d'une classe, alors qu'un appel à une procédure stockée est si simple ...

Code:

---

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

CREATE PROCEDURE maProcedure @_txtBxIdRh [type] , @_txtBxCivilite [type] , @_txtBxNom [type] , @_txtBxPrenom [type] , @_acces [type] , @_txtBxIdRh [type] , @_txtBxId_Fonction [type] AS BEGIN SET NOCOUNT ON   UPDATE dbo.Agent SET id_geode = @_txtBxIdRh , civilite = @_txtBxCivilite , nom = @_txtBxNom , prenom = @_txtBxPrenom , id_fonction = F.id_fonction , type_acces = @_acces FROM dbo.Agent AS A INNER JOIN dbo.Fonction AS F ON A.id_fonction = F.id_fonction WHERE A.id_geode = @_txtBxIdRh AND F.intitule_fonction = @_txtBxId_Fonction END

---

Il suffit ensuite d'appeler la procédure stockée, ce qui doit bien faire un paquet de lignes de code en moins dans votre classe ;)

@++ ;)

Citation:

---

MODIF = "UPDATE agent " _
+ "SET id_geode = '"& Request("txtBxIdRh") &"', " _
+ "civilite = '"& Request("txtBxCivilite") &"', " _
+ "nom = '"& Request("txtBxNom") &"', " _
+ "prenom = '"& Request("txtBxPrenom") &"', " _
+ "id_fonction = (SELECT agent.id_fonction, fonction.id_fonction, fonction.intitule_fonction " _
+ " FROM agent, fonction " _
+ " WHERE fonction.intitule_fonction = '"& Request("txtBxId_Fonction") &"' AND agent.id_fonction = fonction.id_fonction), " _
+ "type_acces = "& acces &" " _
+ "WHERE id_geode = '"& Request("txtBxIdRh") &"'

---

O la belle requête toute pleine d'injection SQL :-)

Pensez au SqlParameters pour protéger vos requêtes.

Bonjour,

Tout d'abord, désolée d'avoir mis autant de temps à répondre.
Merci à vous tous pour vos réponses!! Cela fonctionne très bien.

En revanche, j'aimerais savoir, iberserk, ce que sont les SqlParameters ?

Merci d'avance.

Audrey

Google est toujours votre ami:
http://msdn.microsoft.com/fr-fr/libr...(v=vs.80).aspx

En effet, pas pensé à chercher, absorber pas autre chose...

Merci encore à vous trois.