Inutile @SuppressWarnings(“neuf”)

Je reçois un avertissement du compilateur pour le @SuppressWarnings annotation dans eclipse pour le code:

@Override
public boolean doSomething(@SuppressWarnings("unused") String whatever) throws AnException {
    throw new AnException("I'm still in bed and can't do anything until I've had a shower!");
}

Il ressemble à un jaune squiggle sous le mot "neuf" et de la souris passe dessus je reçois l'info-bulle Unnecessary @SuppressWarnings("unused").

Je pense qu'un autre développeur est invité à mettre dans ces annotations par eclipse et je suis à la base avoir été invité à le faire sortir. Comment puis-je configurer eclipse pour me demander de mettre la @SuppressWarnings annotation au lieu de s'en plaindre?

Si quelqu'un voudrais faire un commentaire sur les meilleures pratiques ici puis que serait le bienvenu.

  • Cela dépend vraiment si vous aimez voir squggily lignes dans votre code. Il me pousse à la folie.
InformationsquelleAutor Edd | 2012-03-02
  1. 52

    Dans le code dans votre question, la @SuppressWarnings("unused") annotation est inutile parce que la méthode est soit substituant une autre méthode d'une super-classe ou de la mise en œuvre d'une interface. Même si vous n'avez pas fait d'utiliser l' whatever paramètre, il est obligatoire de le déclarer, sinon le @Override annotation produira une erreur (vous devez être en train de changer la signature de la méthode de remplacement si vous avez supprimé le paramètre.)

    Dans certaines anciennes versions d'Eclipse, le code n'aurait pas un avertissement, mais dans les dernières versions, il n'. Je crois que c'est une bonne avertissement, et je préfère retirer la @SuppressWarnings("unused") dans ce cas.

    • En outre, dans les versions plus récentes de l'état "inutilisé" à l'avertissement est devenu plus raffiné - si il y a une annotation avec l'exécution de stratégie de rétention attaché à la méthode, le solde non utilisé d'avertissement est supprimée (comme il se doit).
    InformationsquelleAutor Óscar López
  2. 23

    Aller à

    Fenêtre → Préférences → Java → Compilateur → Les Erreurs/Avertissements → Annotations.

    Et sélectionnez Ignorer pour Inutilisé " @SuppressWarnings` token.

    InformationsquelleAutor aioobe
  3. 5

    Alternativement, si vous pensez qu'il est plus correct de supprimer le SuppressWarnings annotation:

    Fenêtre -> Préférences -> Java -> Compiler -> Erreurs/Avertissements -> Inutile de code -> Valeur de paramètre n'est pas utilisé

    et sélectionnez Ignorer dans le remplacement et la mise en œuvre de méthodes

    InformationsquelleAutor David Carboni
  4. 5

    Dans mon code il n'y a pas d'héritage de définir les 3 méthodes avec @SuppressWarnings("neuf")

    Ce code donne " Inutiles @SuppressWarnings("neuf") " dans Eclipse Juno (dernière version), mais si je supprime le @SuppressWarnings("neuf"), j'obtiens un "Constructeur/Méthode n'est jamais utilisé" mises en garde dans IntelliJ IDEA 11.1.3

    Les méthodes ne sont pas directement utilisés dans le projet, que par la 3ème partie des produits Jackson, JAXB & GSON, de sorte que l'Ide est de droite, je dirais ...

    public class EmailUnsendable extends SkjemaError {

    private NestedCommand command;        //Can't be Command (interface) because of GSON!

    @SuppressWarnings("unused")            //Used by Jackson/JAXB/GSON
    public EmailUnsendable() {
    }

    public EmailUnsendable(String referenceNumber, String stackTrace, NestedCommand command) {
        super(referenceNumber, stackTrace);
        this.command = command;
    }

    @SuppressWarnings("unused")            //Used by Jackson/JAXB/GSON
    public NestedCommand getCommand() {
        return command;
    }

    @SuppressWarnings("unused")            //Used by Jackson/JAXB/GSON
    public void setCommand(NestedCommand command) {
        this.command = command;
    }
}

    Je crois que c'est une erreur dans Eclipse.

    • Doh, j'ai essayé d'envoyer un rapport de bogue sur eclipse.org mais le site est trop compliqué - l'espoir que quelqu'un leur donne ce rapport de bug
    • J'ai trouvé cela hilarant. Avez-vous passé à IntelliJ encore?
    • Je dirais que l'Éclipse est de droit. L'accès public sans-arg constructeurs via la réflexion est un motif fréquent en Java, comme ici, en fait.
    • En effet, c'est l'Ide qui est cassé.
    • Je suis passé à IntelliJ à WMdata à Århus en 2003, et nous avons essayé de ne jamais utiliser quoi que ce soit d'autre ... même quand le client m'a demandé de le faire 😉
    InformationsquelleAutor Torben Vesterager