Mit @SuppressWarnings("unused") - annotation Erklärung Abschnitt

Ich habe eine grundlegende Frage in Bezug auf die @SuppressWarnings("unused") annotation. Grundsätzlich würde ich gerne unterdrücken, ungenutztes Abmahnungen code-block, der in meinem Deklarationen von meiner Klasse. Ich würde es vorziehen, nicht zu unterdrücken, nicht verwendete Warnhinweise, die sich auf die gesamte Klasse.

Derzeit bin ich, dies zu tun:

public class MyClass {
    //Constants
    private final String ACCEPT = "Yes";
    @SuppressWarnings("unused")
    private final String DENY = "No";            //Not currently used
    private final String TENTATIVE = "Maybe";
    @SuppressWarnings("unused")
    private final String POSTPONE = "Later";     //Not currently used

    //Variables
    private int counter;
    ....

Finde ich den oben genannten code ein wenig unübersichtlich mit den Anmerkungen geworfen, wie zufällig, die. Was ich tun möchte ist etwas wie:

//Constants
@SuppressWarnings("unused")
{
  private final String ACCEPT = "Yes";
  private final String DENY = "No";            //Not currently used
  private final String TENTATIVE = "Maybe";
  private final String POSTPONE = "Later";     //Not currently used
}

//Variables
private int counter;
....

Dies ist natürlich illegal in Java als ein block kann nicht einfach gemacht wie dieser. Es ist nicht eine große Sache oder etwas, aber ich bin neugierig, wenn ich bin fehlt eine offensichtliche und elegante Lösung.

Irgendein besonderer Grund, warum Sie machen nicht, Sie static?
Guter Punkt Betoverse. Ich könnte Sie statisch. Ich habe gerade warf diese zusammen als ein Beispiel. Aber mein compiler immer noch werfen Warnungen zu unbenutzten statische Felder.

InformationsquelleAutor Cody | 2012-09-28

java

Wenn Sie reeeeealy behalten möchten alle diejenigen, die nicht verwendete Konstanten, eine quick-and-dirty trick wäre;

public class MyClass {
//Variables
    private int counter;

@SuppressWarnings("unused")
    private static final class UnusedConstants{
        private final String ACCEPT = "Yes";
        private final String DENY = "No";            
        private final String TENTATIVE = "Maybe";
        private final String POSTPONE = "Later";     
   }
}

Und ich würde Sie auch statisch, sondern Ihren Anruf.

BEARBEITEN:
Oder, was ich denke, ist viel schöner:

public class MyClass {

    private enum Answer {

        ACCEPT ("Yes"),
        DENY ("No"),
        TENTATIVE("Maybe"),
        POSTPONE("Later");

        private final String answer;

        private Answer(String answer) {
        this.answer = answer;
        }

        public String toString() {
            return answer;
        }
    }
}

Diese Weise bekommt man nicht die unbenutzten Warnung, solange Sie mindestens einen der Werte.

Ich denke, das ist in etwa so nah wie das, was ich hätte suchen sollen. Der Grund für das einfügen nicht verwendete Konstanten ist für die Lesbarkeit und die Tatsache, dass Sie verwendet werden, in der Fernen Zukunft. Ich bin mir nicht sicher, ob dies verbessert die Lesbarkeit, aber es sagt mir, vielleicht sollte ich umziehen meine Konstanten in einer eigenen Konstanten-Klasse. Das ist vielleicht die beste design-Entscheidung. Danke!
Hinzugefügt edit, etwas besseres in den Sinn gekommen 🙂
Ah, ich mag diese Lösung. Reiniger für sicher.

InformationsquelleAutor Chirlo

Schreibe einen Kommentar

Du musst angemeldet sein, um einen Kommentar abzugeben.