Discussion :

[bpy1401]

Bonjour à tous

j'ai bien peut que se soit une question de débutant.
j'ai un warning de compilation que je ne comprend pas sur le code suivant:

Code :

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

1
2
3
4
5
6
7
8
9
10
 
  private String getModuleName(String value) {
    File file = new File(value);
    if (file != null) {
      String fileName = file.getName();
      String moduleName = fileName.replace(".xml", "");
      return moduleName;
    }
    return null;     <-- Warning dead code
  }

Est ce que cela signifie que

Code :

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

File file = new File(value);

ne peut jamais retourner null ?

pour supprimer ce warning, j'ai remplacer le code par celui ci

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
private String getModuleName(String value) {
    try {
      File file = new File(value);
      String fileName = file.getName();
      String moduleName = fileName.replace(".xml", "");
      return moduleName;
    } catch (NullPointerException ex) {
      return null;
    }
  }

Est ce mieux ?

Cordialement

[oneagaindoguys]

Quel est le message du Warning exactement ?

Pour savoir si new File(value) peut renvoyer null, regarder avec ton éditeur le code le la classe File.

Sinon, ce que tu peux faire c'est retourner "" si file == null

[thelvin]

Est ce que cela signifie que

Code :

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

File file = new File(value);

ne peut jamais retourner null ?

Oui. Un constructeur ne peut jamais produire un pointeur null.
Si quelque chose se passe mal, il peut lancer une exception. Mais s'il ne le fait pas, alors il produit forcément un objet instancié du type du constructeur.

pour supprimer ce warning, j'ai remplacer le code par celui ci

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
private String getModuleName(String value) {
    try {
      File file = new File(value);
      String fileName = file.getName();
      String moduleName = fileName.replace(".xml", "");
      return moduleName;
    } catch (NullPointerException ex) {
      return null;
    }
  }

Est ce mieux ?

Possiblement, mais un NullPointerException ne peut arriver que si value est lui-même null.

Je verrais plutôt quelque chose comme ça :

Code :

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

1
2
3
4
5
6
7
8
9
private String getModuleName(String value) {
  if(value == null) {
    return null;
  }
  File file = new File(value);
  String fileName = file.getName();
  String moduleName = fileName.replace(".xml", "");
  return moduleName;
}

Après, tout dépend quelles vérifications sont à faire, si value contenait un path terminé par un /, par exemple.

[bpy1401]

Merci a tous les deux.

la solution est ici

Oui. Un constructeur ne peut jamais produire un pointeur null.