Je joue un peu avec PMD. Il est dit "Évitez d'utiliser une instruction de branchement comme dernier dans une boucle.".PMD: Évitez d'utiliser une instruction de branchement comme dernière en boucle
private static MyClass getObj(ResultSet rs) {
try {
while (rs.next()) {
MyClass obj = new MyClass(rs.getString("name"));
// do other stuff
return obj;
}
} catch (SQLException e) {
logAndShowException(e);
}
return null;
}
Comment réparer cet avertissement? J'ai pensé à supprimer la boucle while et j'ai juste écrit une instruction if car j'attends exactement un ResultSet.
Merci.
Sur quelle ligne obtenez-vous l'avertissement? – Juvanis
ne déclarez pas d'objets à l'intérieur de la boucle – Selva
@Selva Cela n'a pas vraiment d'importance quand ce n'est pas une boucle - il est garanti de ne jamais l'exécuter deux fois. Globalement, une boucle est juste la mauvaise chose à utiliser là-bas comme l'a dit Thiago dans sa réponse. – Thor84no