Discussion :

[Claude Robin]

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 VB.NET :

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
 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!

[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.

[Mehdi Feki]

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.

[Claude Robin]

Ok! J'ai parfaitement bien compris mon erreur!

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

Ça fonctionne très bien!

Merci!

[Pol63]

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

[Claude Robin]

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!

[Pol63]

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

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

[Claude Robin]

Merci pour tous ces conseils!

[Mehdi Feki]

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 .

[Claude Robin]

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!

[Pol63]

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