Discussion :

[audrey1912]

Bonjour,

voilà mon problème,

Je dois faire une requête SQL en VB pour aller chercher des données dans un formulaire pour la création d'un agent. Et le navigateur m'affiche ce message :
Microsoft SQL Native Client erreur '80040e14'
Syntaxe incorrecte vers 'agent'

Voici mon code :

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
<%
Dim app_name, app_firstname, app_id_geode, nouvel_agent
 
	app_id_geode = Request.QueryString("id_geode")
 
	Dim Conn, RsUser
	Set Conn = Server.CreateObject("ADODB.Connection")
	Set RsUser = CreateObject("ADODB.Recordset")
	verif_connexion bdd_sql, Conn
 
	nouvel_agent = "INSERT INTO agent(matricule, id_geode, civilite, nom, prenom) " _
				 + "VALUES agent.matricule='"& id_rh_agent_creation  &"', agent.id_geode='"& id_rh_agent_creation  &"', agent.civilite='"& civilite_agent_creation  &"', " _
				 + "agent.nom='"& nom_agent_creation  &"', agent.prenom='"& prenom_agent_creation  &"' " 
 
	RsUser.open nouvel_agent, Conn
 
		id_rh_agent_creation 			= Request.Form ("document.frm_agents_creation.txtBxIdRh")
		civilite_agent_creation  		= Request.Form ("document.frm_agents_creation.txtBxCivilite")
		nom_agent_creation  			= Request.Form ("document.frm_agents_creation.txtBxNom")
		prenom_agent_creation 			= Request.Form ("document.frm_agents_creation.txtBxPrenom")
 
%>

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

Merci d'avance.

[audrey1912]

Sachant que le navigateur me dit que l'erreur se situe vers le ligne suivante :

Code :

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

RsUser.open nouvel_agent, Conn

[darKStein]

Il me semble que la syntaxe pour un INSERT...VALUES serait plutôt :

Code :

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

1
2
3
4
 
nouvel_agent = "INSERT INTO agent(matricule, id_geode, civilite, nom, prenom) " _
    & "VALUES ('" & id_rh_agent_creation & "', '" & id_rh_agent_creation & "', '" _
    & civilite_agent_creation & "', '" & nom_agent_creation & "', '" & prenom_agent_creation &"') "

Après, attention si nom_agent_creation ou prenom_agent_creation contiennent des apostrophes, cela peut poser problème, penser à les doubler.

[iberserk]

nouvel_agent = "INSERT INTO agent(matricule, id_geode, civilite, nom, prenom) " _
& "VALUES ('" & id_rh_agent_creation & "', '" & id_rh_agent_creation & "', '" _
& civilite_agent_creation & "', '" & nom_agent_creation & "', '" & prenom_agent_creation &"') "

Attention, cette manière de faire (création dynamique de requête à partir de données saisies par l'utilisateur) vous expose aux risques d'attaque par injection SQL.

Pensez à remplacer les parties dynamiques par des paramètres.

Le mieux étant de passer par une procédure stockée...

[audrey1912]

Bonjour,

Je voudrais répondre à iberserk en premier. En effet, je sais que la création dynamique de requête à partir de données saisies par l'utilisateur n'est pas l'idéal. Je le fait comme cela dans un premier temps pour un essai entre guillemets. Dans un second temps, je passerai par une procédure stockée qui fera cet insert.

Pour répondre à darKStein, merci, en effet j'ai un petit souci de parenthèses et il y a pleins de choses en trop, je me demande comment je ne m'en suis pas aperçue plutôt... En ce qui concerne les variables en questions, elles ne peuvent pas contenir d'apostrophe, donc il n'y a pas de souci.

Merci à vous deux!

[audrey1912]

Bon ben au final j'ai trouvé plus simple et cela marche parfaitement, je le poste au cas ou.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
Dim Conn, RsUser
Set Conn = Server.CreateObject("ADODB.Connection")
Set RsUser = CreateObject("ADODB.Recordset")
verif_connexion bdd_sql, Conn
 
nouvel_agent = "INSERT INTO agent(id_geode, civilite, nom, prenom) " _
			+ "VALUES ('"& Request("txtBxIdRh") &"', " _
			+ " '"& Request("txtBxCivilite") &"', " _
			+ " '"& Request("txtBxNom") &"', " _
			+ " '"& Request("txtBxPrenom") &"', 	"
 
RsUser.open nouvel_agent, Conn

Encore merci à vous deux!!