[C#] Difficulté avec les catch

Version imprimable

Bonjour à tous,

J'ai de la difficulté à utiliser les catch, j'ai toujours des erreur à la compilation qui m'empêche d'utiliser les résultat.

Exemple :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
		public string getQuestion()
		{
			SqlConnection SqlCnx = new SqlConnection("workstation id=SERVER;packet size=4096;user id=xxx;data source=\"SERVER\\SQLSERVER\";persist security info=False;initial catalog=infosalvage;password=xxx");
			string SQL = "SELECT SecretQ FROM cust WHERE EmailWork = '"+usrMail.Text+"'";
			SqlCommand myCommand = new SqlCommand(SQL, SqlCnx);
			SqlCnx.Open();
			SqlDataReader SqlData = myCommand.ExecuteReader();
			SqlData.Read();
			string db_SecretQ = Convert.ToString(SqlData.GetValue(0));
			return db_SecretQ.ToString();
            SqlData.Close();
            SqlCnx.Close();
		}

Cependant si la personne n'a pas entré sont email cela lève un exception se qui est normal. Quand j'essaie de traiter cette exception avec un try catch de l'exception sql cela me donne un erreur.

Mon catch est comme ceci :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
 
		public string getQuestion()
		{
            try
            {
                SqlConnection SqlCnx = new SqlConnection("workstation id=SERVER;packet size=4096;user id=xxx;data source=\"SERVER\\SQLSERVER\";persist security info=False;initial catalog=infosalvage;password=xxx");
                string SQL = "SELECT SecretQ FROM cust WHERE EmailWork = '" + usrMail.Text + "'";
                SqlCommand myCommand = new SqlCommand(SQL, SqlCnx);
                SqlCnx.Open();
                SqlDataReader SqlData = myCommand.ExecuteReader();
                SqlData.Read();
                string db_SecretQ = Convert.ToString(SqlData.GetValue(0));
                return db_SecretQ.ToString();
                SqlData.Close();
                SqlCnx.Close();
            }
            catch (SqlException ex)
            {
                System.Diagnostics.Trace.WriteLine("[getQuestion] Exception " + ex.Message);
            }
		}

J'ai esseyer avec un if pour voir la différence mais cela me donne la même erreur.

Voici L'erreur :

Code:

1
2
 
'members.Login.getQuestion()': not all code paths return a value

J'ai souvent se genre de problème avec les catch, il doit y avoir quelque chose que je n'ai pas bien comprit.

Merci pour votre aide.

31/01/2006, 17h07
ddoc

re
pour les try catch

ceci fonctionne de cette manière
Code:

1 2 3 4 5 6 try{ ton code }catch(typeexception nomvariable){ }catch(typeexception nomvariable){ }
Faut faire de l'exception la plus specifique à la moins specifique.
Tu peux mettre autant de catch que tu veux.
Ainsi en dernier on termine par le type exception.

ddoc
31/01/2006, 17h08
lutecefalco

Si tu rentres dans le Catch, ta fonction ne retourne rien donc ça rale ;)
31/01/2006, 17h11
Roach-

Citation:

Envoyé par lutecefalco

Si tu rentres dans le Catch, ta fonction ne retourne rien donc ça rale ;)

Ok donc faudrait que dans le catch je retourne un valeur bidon.
31/01/2006, 17h28
Roach-

Bon j'ai retourner la valeur vide si un exception est lever, plus loin dans le code j'analyse le contenu de la fonction getQuestion() comme sa si elle est vide il avertie le client d'entrer un email.

Merci, J,ai comprit le fonctionnement.
31/01/2006, 17h30
slopera

Afin de bien gérer tes exceptions, il vaudrait mieux te créer ton propre jeu d'exceptions et faire toi même tes vérifications.
Remarque : attention, tu fais ton return avant tes close() !!
31/01/2006, 17h32
lutecefalco

Citation:

Envoyé par slopera

Remarque : attention, tu fais ton return avant tes close() !!

+1
31/01/2006, 17h49
Roach-

D'accord je vais mettre le return après les close.

Petite question qu'est ce que cela peut causé.
31/01/2006, 17h53
lutecefalco

Citation:

Envoyé par Roach-

D'accord je vais mettre le return après les close.

Petite question qu'est ce que cela peut causé.

La fonction arrete son exécution sur le Return donc tout ce qu'il y a après n'est pas exécuté. Ici, tu fermais pas tes connexions
31/01/2006, 17h57
Roach-

ahhhhhhh la je sais pourquoi j'ai des petit chose qui ne fonctionne pas correctement.

Meci à tous vous ête super