Discussion :

[JohnNC]

Bonjour,

Afin de nettoyer un peu mes codes, j'ai activé quelques paramètres en "Error" ou "Warning", dans les "Windows","préférences","Java","Compiler","Warnings".
J'ai un peu de mal à me dépatouiller avec tous ces nouveaux messages, mais beaucoup de suggestions m'aident à corriger mes problèmes (un @throw manquant dans la javadoc par exemple).

Mais il y en a un qui n'a pas de suggestion, donc je me permet de poser la question :

Code :

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

1
2
3
4
5
/** Constructeur vide. Aucun traitement n'est déclenché à l'instanciation de la classe.
 * 
 */
private Reports() {
}

J'ai donc le message suivant : "Empty block should be documented".
Qu'est ce que ca veut dire, et comment le corriger ? Est-ce propre ?

Merci !

[adiGuba]

Salut,


Ce warning te dit simplement que tu utilises un bloc vide. Le problème étant de savoir si c'est voulu, s'il s'agit d'un oubli, ou d'un code à faire pour plus tard...

Bref eclipse t'indique un problème potentiel que tu peux régler d'un commentaire, par exemple :

Code :

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

1
2
3
private Reports() {
    // Constructeur vide
}

Tu peux également modifier la config d'eclipse pour qu'il ne t'indique pas ce genre de problème...

a++

[JohnNC]

Ha d'accord, merci !

J'avais justement modifié toutes ces options "Error/Warning" pour améliorer mon code : Celà fait 5 ans que je fais du JAVA 1.4, je voudrais passer en 1.5 (et bientot 1.7 ?), mais avant je voudrais apprendre gràce à tous les warnings comme développer "efficacement".

En dehors des FAQ que je vais donc commencer à lire sur ces sujets de "programmes propres", y-a-t des parties du forum spécialisées dans la construction de la javadoc (jentend par la rendre au mieu compréhensible les commentaires de classe, méthode ou autres ?)

Cette seconde question n'étant pas urgente, je met le tag Résolu, je l'enleverais si le débat le nécessite.

Encore merci !

[JohnNC]

Me voilà de retour ! Toujours dans le soucis de nettoyer mes classes, j'ai maintenant quelques petits messages de warnings qui m'embête :

Dans une classe "mère", j'ai une méthode qui peut être surchargée dans des classes filles ou non (d'où l'absence du "abstract") :

Code :

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

1
2
3
4
/** A surcharger : Création de la table ZJOURNAL pour AAAA. Inutile pour l'instant dans BBBBBB.
 * @throws NEcritureException 
 */
public void créerTableZJOURNAL(NTransaction transac) throws NEcritureException {}

Cette méthode est donc surchargée comme indiqué dans une seule des classes héritières. Mais j'ai le warnings suivant :

Code :

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

The parameter transac is never read

Code :

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

The declared exception NEcritureException is not actually thrown by the method créerTableZJOURNAL(NTransaction) from type MTPartitionnement

Bien sur, je sais que le plus simple serait de changer les affichage de warnings, mais j'voulais quand meme savoir si c'était "propre"