[Vb.Net] - Function - RETURN

J'ai un "avertissement" dans VB 2005 qui se lit comme ceci :

>La fonction "ChangerDateCaractere" ne retourne pas une valeur pour tous les chemins de code. Une exception de référence Null peut se produire au moment de l'exécution lorsque le résultat est utilisé>

Voici cette fonction....

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19

Private Function ChangerCaractereDate(ByVal strCodeDivers As String) As String Dim strDateCarac As String = String.Empty Dim intCompteur As Integer = 0 Try Do While intCompteur < 8 Select Case strCodeDivers.Substring(intCompteur, 1) Case "a" strDateCarac = strDateCarac & "0" Case "b" strDateCarac = strDateCarac & "1" Case "c" strDateCarac = strDateCarac & "2" End Select intCompteur += 1 Loop Return strDateCarac Catch objException As Exception End Try End Function

---

Quel est le problème de RETURN ????? par rapport à l'avertissement ????

Merci de vos réponses!

Pense aux balise code pour rentre ton message plus lisible.
Ton problème est que si tu as une exception, tu n'auras pas de valeur de retournée.

Citation:

---

Envoyé par AP

Pense aux balise code pour rentre ton message plus lisible.
Ton problème est que si tu as une exception, tu n'auras pas de valeur de retournée.

---

+1, ajouter une clause finally pour retourner une valeur.

Ok! J'ai parfaitement bien compris mon erreur!

Il faut que je place "Return" juste avant "End Function"

Ça fonctionne très bien!

Merci!

non ce n'est pas forcément la solution

déjà si tu ne mets rien dans le catch ca ne sert à rien de mettre un try catch !
en général dans un catch on met soit quelque chose qui enregistre les erreurs soit un message
faire le return dans le finally c'est pas top, mettre un return dans le catch serait éventuellement mieux

donc soit dans le catch tu fais un return d'une valeur "en cas de problème"
soit tu retire le try catch pour que l'appelant sache qu'il y a eut un problème (à lui d'avoir un try catch et d'arreter son traitement si c'est critique)

sinon d'une manière générale ce genre d'avertissement ne doit pas etre pris en compte, enfin quand on sait ce qu'on fait

Dans le Try Catch, j'ai toujours un message d'erreur qui récupère mes erreurs et les mets dans un fichier texte.
Mais pour l'explication de mon problème, j'ai raccourci mon code pour ne pas qu'il soit trop long!

Je ne laisse jamais un Try catch vide, veuillez m'excusez!

mais comment pouvoir mettre un "Return" un peu partout dans ma fonction avec un "Exit Function" ???
Devrais-je plutôt utiliser "ChangerDateCaractere = strDateCarac" ????

Merci de vos réponses!

la question à ce poser est ce légitime de répondre une valeur en cas d'erreur
si ca a planté c'est que ca c'est pas passé comme prévu
renvoyer une chaine à moitié remplie ne me semble pas ce qu'il y a de mieux


sinon comme je disais

Citation:

---

donc soit dans le catch tu fais un return d'une valeur "en cas de problème"
soit tu retire le try catch pour que l'appelant sache qu'il y a eut un problème (à lui d'avoir un try catch et d'arreter son traitement si c'est critique)

sinon d'une manière générale ce genre d'avertissement ne doit pas etre pris en compte, enfin quand on sait ce qu'on fait

---

soit tu fais throw ex dans le catch pour que l'erreur se reproduise après l'avoir loggé et l'appelant fait en fonction


sinon "ChangerDateCaractere = strDateCarac & "0" " est valide

Merci pour tous ces conseils!

Citation:

---

Envoyé par sperot51

faire le return dans le finally c'est pas top, mettre un return dans le catch serait éventuellement mieux

---

De toute façon, ce n'est pas possible de mettre un return dans un finally.

Je viens de me rendre compte de ma bétise :oops:.

mais comment pouvoir mettre un "Return" un peu partout dans ma fonction avec un "Exit Function" ???
Devrais-je plutôt utiliser "ChangerDateCaractere = strDateCarac" ????

Merci de vos réponses!

pourquoi mettre des return un peu partout ?
c'est pas parce que visual studio te signales ce qui pourrait etre une erreur de programmation que c'en est une !!!
c'est la 3eme fois que je le dis
ta fonction est bien comme ca ormis le fait qu'en cas d'erreur c'est abérant de retourner quelque chose justement ! laisse l'erreur se propager si tu n'a pas de valeur par défaut à donner


return "blabla"

fait la meme chose que

ChangerDateCaractere = "blabla"
exit function