Ne pas répéter Try Catch

Version imprimable

Bonjour à tous,

J'ai besoin d'une petite astuce pour éviter de répéter plusieurs fois des Try Catch, voici un bout de code :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
If condition1 Then
                        If condition2  Then
                            str = currentLine.Split(New Char() {"|"c})
                            maVariable1.Add(countDictionary, str(1).Replace(" ", ""))
 
                            Try
                                maVariable1.Add(countDictionary, Double.Parse(str(2).Replace(" ", ""), ciClone))
                            Catch ex As Exception
                                maVariable1.Add(countDictionary, Nothing)
                            End Try
 
                            Try
                                maVariable2.Add(countDictionary, Double.Parse(str(3).Replace(" ", ""), ciClone))
                            Catch ex As Exception
                                maVariable2.Add(countDictionary, Nothing)
                            End Try
 
                            Try
                                maVariable3.Add(countDictionary, Double.Parse(str(4).Replace(" ", ""), ciClone))
                            Catch ex As Exception
                                maVariable3.Add(countDictionary, Nothing)
                            End Try
 
                            countDictionary += 1
                            Continue While
                        End If                      
End If

Ce qui se passe c'est que je veux relever une valeur sauf qu'elle peut être absente et dans ce cas le Parse ne fonctionne pas.
Du coup j'utilisr le try catch sauf que je vais devoir le répéter beaucoup de fois :?

Quelqu'un a une idée ?

Merci,

Jah

07/02/2013, 16h07
youtpout978

Il y a une méthode tryparse qui retourne un booléen qui indique si la conversion a réussi.
TryParse
07/02/2013, 17h13
Jah73

@youtpout978 : J'ai pensé au TryParse, sauf que ça m'obligerai à faire des If Else ?

Bonjour,

Utilise une procédure ou une fonction qui remplace ton bloc

Code:

1
2
3
4
5
                            Try
                                maVariable1.Add(countDictionary, Double.Parse(str(2).Replace(" ", ""), ciClone))
                            Catch ex As Exception
                                maVariable1.Add(countDictionary, Nothing)
                            End Try

Comme cela le try catch ne sera que dans ta procédure ou fonction.

A+, Hervé.

07/02/2013, 17h44
Sokhz

Il existe également la fonction Trim pour supprimer les espaces au lieu de faire des Replace(" ","").

Exemple de procedure

Code:

1
2
3
4
5
6
7
    Sub Ajoute(ByRef maVariable As TonType, ByVal countdictionary As Integer, ByVal Str As String, ByVal ns As System.Globalization.NumberStyles)
        Try
            maVariable.Add(countdictionary, Double.Parse(Str.Replace(" ", ""), ns))
        Catch ex As Exception
            maVariable.Add(countdictionary, Nothing)
        End Try
    End Sub

Utilisation

Code:

1
2
3
4
5
6
7
8
                    If condition1 Then
                        If condition2  Then
                            str = currentLine.Split(New Char() {"|"c})
                            maVariable1.Add(countDictionary, str(1).Replace(" ", ""))
 
                            Ajoute(maVariable1,countDictionary,str(2),ciclone)
                            Ajoute(maVariable2,countDictionary,str(3),ciclone)
    ' ...

08/02/2013, 09h31
Jah73

Tout simplement ... :mur:

Je cherchais à faire compliqué pour rien :?

Merci beaucoup pour l'aide :ccool:

Je pensais plus à ça (par contre c'est en c#):

Code:

1
2
3
4
5
6
            double nb = 0;
            double? nbVide = null;
            for (int i = 0; i < str.Length; i++)
            {
                dico.Add(i, double.TryParse(str[i].Replace(" ",""), out nb) ? nb : nbVide);
            }

Il vaut mieux éviter la répétition de try catch parce que le temps de récupération d'une erreur est assez couteux.

08/02/2013, 10h27
sinople

Je tiens à préciser qu'un bloque Try...Catch ne doit jamais être utilisé pour récupérer une erreur provenant d'un cas d'utilisation identifié et contournable.

En effet les exceptions sont très couteuse en terme de temps d'execution. Si on peut éviter d'en lancer une à l'aide d'un IF conditionfoireuse THEN traitementspécifique, il faut le faire !

Ceci est d'autant plus vrai si le traitement est dans une boucle ou très souvent utiliser. Vous risquez sinon de voir apparaître des problèmes de performance !

Merci pour vos réponses :ccool:

Est-ce une bonne méthode si je fais :
Code:

1 2 3 4 5 6 7 If (TryParse...){ maVariable.Add(countdictionary, Double.Parse(Str.Replace(" ", ""), ns)) 'Si true } Else{ maVariable.Add(countdictionary, Nothing) }
Le tout dans une procédure comme me l'a recommandé rv26t

08/02/2013, 11h09
youtpout978

oui tu peux faire comme ça mais si tu fais un tryparse tu n'as pas besoin de refaire un parse derrière.

C'est à dire ?

J'enlève le Parse et je mets juste le Str ?

Code:

1
2
3
4
5
6
7
If (TryParse...){
      maVariable.Add(countdictionary, Str.Replace(" ", ""), ns) 'Si true
}
 
Else{
      maVariable.Add(countdictionary, Nothing)
}

le tryparse te donne le résultat de la conversion tu as juste à utiliser ce résultat:

Code:

1
2
3
4
5
6
7
8
9
            double nb = 0;
            if (double.TryParse(Str.Replace(" ", ""), out nb))
            {
                maVariable.Add(countdictionary, nb);
            }
            else
            {
                maVariable.Add(countdictionary, null);
            }

08/02/2013, 13h39
Jah73

Effectivement !

Merci beaucoup :ccool: