Discussion :

[artatnas]

Bonjour a tous
On m a demande de blinder le code suivant
J'ai une vague idée du probleme mais je ne sais pas exactement quoi faire
Pouvez vous m aider?
Merci

// Fonctionnement attendu de la fonction :
// Retourne une valeur numérique valide (diff de NaN, +INF, -INF) en fonction de
// - la date "parDate" fournie en entrée
// - la valeur "parValue" fournie en entrée
// - la valeur du 1er élément présent dans la liste fournie en entrée
// Si un des paramètres fourni est invalide, la fonction doit retourner null

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
public Double retourneValeurEnFctDeDate(Calendar parDate, Double parValue, final List parValueList) {
    if ((parDate.get(Calendar.MONTH) == Calendar.APRIL) && (parDate.get(Calendar.DAY_OF_MONTH) == 1)) {
      parDate = getNewDate();
    }    
    if (parDate.get(Calendar.DAY_OF_WEEK) == Calendar.MONDAY) {
      return Double.valueOf((Double)parValueList.get(0) / parValue);
    } else {
      return Double.valueOf(2.0 * (Double)parValueList.get(0) / parValue);
    }       
  }
 
private Calendar getNewDate() { … }    // Fonction qui retourne une nouvelle date

[tchize_]

qu'est-ce que tu entends par "blinder" ?

[CheryBen]

Je pense que tu devrais commencer par tester si les paramètres sont null et vérifier que la liste ne soit pas vide. En faisant directement un get(0), tu risques de lever une IndexOutOfBoundsException.

[artatnas]

blinder = faire en sorte qu'à l'Execution le code ne provoque pas d'erreur
Merci pour les réponses

[tchize_]

le truc classique alors:

vérifie que tes valeurs ne sont pas null avant de les utiliser, vérifie les limites des tes listes avant d'y accéder, vérifie que les liste on bien les type de contenu attendu.

[Uther]

Vérifie également le type des objets contenus dans parValueList avant de les caster en Double.
Ou change son type en List<Double>

[Babyboystwo]

Bonjour à tous, est ce que quelqu'un peut donner une réponse à ce problème de blindage de code en java ? s'il vous plaît, j'ai le même problème. merci d'avance

[tchize_]

les réponses ont déjà été données

[Rastafire]

Il existe un super framework permettant de faire des tests unitaires nommé JUnit qui permet de vérifier plein de choses. Ca permet de bien

blinder

le code. Qui plus est il existe aussi des plugins de calcul de couverture de code par les tests, ça peut démontrer aux personnes pour qui tu dois faire ça que tu as bien couvert ton code par les tests unitaires(on estime qu'entre 50 et 70% de couverture de code on a un code qui fonctionne très bien).