Discussion :

[solo190]

bonjour a tous,
j'ai ecris une methode pour executer une procedure stockée sur une base de donnée sql serveur 2000, lorsque je met les variables en paramètre sa ne marche pas par contre lorsque je lè met en dur tout ce passe bien, j'aimerais que vous me donner un coup de main dans la formation dynamique de ma chaine.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
public void execute()
        {
            // exeution de la procedure stockée 
            SqlConnection connexion = new SqlConnection("Data Source=("+Serveur+"); Initial Catalog=("+Nom_Bd+");" +
                                                          "Integrated Security=SSPI;");
            //SqlConnection connexion = new SqlConnection("Data Source=ICS-DEV1;  Initial Catalog=SMS;"+
            //                                         "Integrated Security=SSPI;");
            connexion.Open();
            SqlCommand cmdBD = new SqlCommand(Nom_Procedure, connexion);
            cmdBD.CommandType = CommandType.StoredProcedure;
            cmdBD.Parameters.Add("@Nom_utilisateur", SqlDbType.NVarChar).Value = Parametre1;
            cmdBD.Parameters.Add("@Passwords", SqlDbType.NVarChar).Value = Parametre2;
	    cmdBD.ExecuteNonQuery();
        }

Merci

[kheironn]

Quel est le problème exactement ?

[Bluedeep]

Poste le DDL header de ta procédure stockée.

[solo190]

lorsque les parametres sont passés a la procedure, connexion.Open() genere une erreur, tandis que lorsqu'ils sont mis en dure comme dans la partie en commentaire, tout ce passe bien.

[calagan99]

Le mieux serait peut-être de stocker ces informations dans un fichier de configuration, en dur, et de récupérer cette chaîne via la classe ConfigurationManager.
Cela éviterait les concaténations hasardeuses.
Regarde ce tutoriel

[Guulh]

lorsque les parametres sont passés a la procedure, connexion.Open() genere une erreur, tandis que lorsqu'ils sont mis en dure comme dans la partie en commentaire, tout ce passe bien.

Dans un cas t'as mis des parenthèses, dans l'autre non.

"Data Source=("+Serveur+"); Initial Catalog=("+Nom_Bd+");"

[solo190]

la partie en commentaire qui n'a pas de parenthèse est celle qui marche mes avec des parametres en dur.

[calagan99]

Ce que tu dis Guulh est bon :
- dans la version en dur, tu n'as pas de parenthèses et ta chaîne est correcte
- dans la version concaténée, tu as des parenthèses et ta chaîne n'est pas bonne.

Concatène tes paramètres sans parenthèses :

Code :

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

SqlConnection connexion = new SqlConnection("Data Source="+Serveur+"; Initial Catalog="+Nom_Bd+";" + "Integrated Security=SSPI;");

Cela dit, je persiste, tu n'aurais pas ce problème en stockant ta chaine dans un fichier de configuration.

[solo190]

avec le tutoriel que ma proposé calagan99,
j'ai trouvé la solution au problème.
voici mon fichier de config

Code :

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

1
2
3
4
5
6
<?xml version="1.0" encoding="utf-8" ?>
<configuration>
  <appSettings>
    <add key="MaConnection" value= "Data Source=ICS-DEV1; Initial Catalog=SMS; Integrated Security=SSPI"/>
  </appSettings>
  </configuration>

et voici comment je recupere la chaine de connexion

Code :

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

1
2
3
string chaine_connexion = null;
            chaine_connexion = ConfigurationManager.AppSettings["MaConnection"];
            SqlConnection connexion = new SqlConnection(chaine_connexion);

[calagan99]

Voila une solution qui me paraît tout à fait adaptée

Tu peux passer le topic en résolu si tu as eu la réponse à ta question.

[hegros]

Remarque que pour construire des connexionstring il existe la classe SqlConnectionStringBuilder
http://msdn.microsoft.com/fr-fr/libr...ngbuilder.aspx

Cela évite de faire des concaténations 'hasardeuses'

[calagan99]

Remarque que pour construire des connexionstring il existe la classe SqlConnectionStringBuilder
http://msdn.microsoft.com/fr-fr/libr...ngbuilder.aspx

Cela évite de faire des concaténations 'hasardeuses'

Bien vu, c'était aussi une alternative crédible

[kheironn]

Plutôt que d'utiliser les clefs de config, je préfère passer par les settings du programme et faire appel à mon élémet avec un nom fort :

Code :

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

Properties.Settings.Default.[le nom de la chaine]

. Tu peux en plus définir une portée et typer en connexion string.