Warning de compilation sur File

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:

---

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:

---

File file = new File(value);

---

ne peut jamais retourner null ?

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

Code:

---

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

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

Citation:

---

Envoyé par bpy1401

Est ce que cela signifie que

Code:

---

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.

Citation:

---

Envoyé par bpy1401

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

Code:

---

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:

---

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.

Merci a tous les deux.

la solution est ici

Citation:

---

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

---