Discussion :

[#define]

L'IDE (NetBeans 8.2 en l'occurence) se plaint avec missing return statement alors qu'il existe bien dans le code (simplifié ici) :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
public static ArrayList<xxx> deserialize(String fileName) {
  //...
 
  try (www){
     //...
     return inputDataList;
  } 
 
  catch (IOException | ClassNotFoundException e1) {
     // ...
  }
}

Pourquoi l'IDE ne prend en compte le return dans le try ?
Que faut-il écrire pour supprimer cet avertissement ?
(L'IDE propose "return null;" avant l'accolade fermante de la méthode, mais je n'ai pas besoin de cette ligne...)

Merci pour vos conseils.

[joel.drigo]

Salut,

Une méthode qui a un type de retour est censée toujours retourner une valeur, dans tous les cas. Ce n'est pas que l'IDE ne tient pas compte du return dans le bloc try, c'est qu'il n'est pas suffisant : en effet, en cas d'exception attrapée par ton bloc catch, il n'y a pas de return, or il faut aussi retourner une valeur dans ce cas.

Soit,

1. tu repousses le traitement de l'exception à une méthode appelante, et tu fais un throws au lieu du catch (ou tu wrappes l'exception dans une RuntimeException )
2. soit tu retournes une valeur qui pourra être interprétée par le code appelant comme un echec de ta méthode, comme null comme te le propose l'IDE, ou une liste vide ou une liste remplie avec des éléments par défaut, comme ça t'arrange dans ton programme.

[#define]

@joel.drigo
Merci pour cette réponse.

Je vais opter pour le null dans un premier temps et regarderai la liste remplie avec des éléments par défaut (déjà créée) dès cette partie du programme finalisée.