Discussion :

[candidat]

bonjour ,
j ai un pas pu inserer des donnees a ma base au début j'ai voulu recupere le resultat d'une requete sql dans une valiable pour l utiliser dans un second temps dans ma requete insert mai j ai pa arriver a faire ceci donc j'ai essayé d'ecrire ma requete select dans la requete insert comme ca

Code :

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

1
2
3
4
5
cmd = new SqlCommand("insert into [diplome_obt] values (select id from [diplome] 
where (([filiere] = '" + ddlfiliere.SelectedValue + "') AND 
([nature_dip] = '" + ddlnaturedip.SelectedValue + "') AND 
([nom_etab] = '" + ddletablissement.SelectedValue + "') AND 
([univ] = '" + ddluniversite.SelectedValue + "')),cin.Text,txtdateobten.Text)", con);

pouvez vous m 'aider soit a corriger cette requete soit a recuperer le resultat du select dans une variable
merci

[popo]

Puisque tu utilise un SqlCommand, je pars du principe que tu utilise SQL Server.
Si ce n'est pas le cas, pense à préciser la prochaine fois.

Déjà ta requête n'est pas bonne au niveau de la syntaxe.
En remplacer toutes les données tirées de l'écran par des valeur voilà ce que donne ta requête (j'ai volontairement sauté des lignes pour que ça soit plus parlant)

Code SQL :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
insert into [diplome_obt] 
values 
(
	select id from [diplome] 
	where 
	(
		([filiere] = 'Filiere') 
		AND 
                ([nature_dip] = 'Nature') 
		AND 
		([nom_etab] = 'Etablissement') 
		AND 
		([univ] = 'Université')
	),'CIN','txtdateobten'
)

- Tu as le mot clé "VALUES" en trop car tu fait une requête de sélection pour remplir la table.
- "CIN" et "txtdateobten" devraient être derrière l'id (ou alors je ne sais pas ce que tu cherches à faire et il faudra préciser).

Voici ta requête en syntaxe correcte

Code SQL :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
insert into [diplome_obt] 
(
	select id, 'CIN', 'txtdateobten' from [diplome] 
	where 
	(
		([filiere] = 'Filiere') 
		AND 
                ([nature_dip] = 'Nature') 
		AND 
		([nom_etab] = 'Etablissement') 
		AND 
		([univ] = 'Université')
	)
)

Prends l'habitude de faire des requêtes paramétrées afin de sécuriser un peu plus le code (et accessoirement de ne pas te prendre la tête lorsque ces données seront des dates).
Voila ce que ça donnerai.

Code C# :

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
using (SqlConnection connection = new SqlConnection(connectionString))
{
	String query = "INSERT INTO diplome_obt"
	  + " SELECT ID FROM diplome"
          + " WHERE filiere=@Filiere AND nature_dip=@NatureDip AND nom_etab=@NomEtab AND univ=@Universite";
 
	using (SqlCommand command = new SqlCommand())
        {
        	/* CA2100
         	 * Review SQL queries for security vulnerabilities */
        	command.Connection = connection;
 
		command.Parameters.Add("@Filiere", SqlDbType.VarChar).Value = ddlnaturedip.SelectedValue;
        	command.Parameters.Add("@NatureDip", SqlDbType.VarChar).Value = ddlnaturedip.SelectedValue;
        	command.Parameters.Add("@NomEtab", SqlDbType.VarChar).Value = ddletablissement.SelectedValue;
        	command.Parameters.Add("@Universite", SqlDbType.VarChar).Value = ddluniversite.SelectedValue
 
        	command.CommandText = query;
 
        	connection.Open();
        	command.ExecuteNonQuery(); 
        }
}