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>