Ist es in Ordnung, statische Attribute zur Laufzeit zu manipulieren?

Question

Ich stelle Einstellungen meines JAVA-Projekts in einer Prefs.java-Klasse mit statischen Attributen und statischen Methoden bereit. Das Token für OAuth2 muss jedoch zur Laufzeit zugewiesen werden. Ist das ein guter Weg zu gehen ...?

Answer 1

Ich würde gegen solche Design raten. Dieser Typ von statischen Variablen ist nicht besser als globale Variablen. This page gibt ein paar Gründe, warum Sie sie vermeiden sollten. Hier sind ein paar von ihnen.

• Nichtlokalität
• keine Zugriffskontrolle oder Constraint-Prüfung
• Implizite Kopplung
• Concurrency gibt
• Testing und Confinement

Aber das Token für OAuth2 muss sein zur Laufzeit zugewiesen. Ist das ein guter Weg zu gehen ...?

Hier scheint es mir wirklich so, als würden Sie ein solches Token an den Konstruktor des Prefs-Objekts übergeben.

Answer 2

In der Regel werden globale Konfigurationsdateien zur Laufzeit aus Eigenschaftendateien gelesen. Sie können verschiedene Konfigurationsdateien für die Entwicklungs-, QS- und Produktionsumgebungen verwenden.

Sie müssen sich nur darüber im Klaren sein, dass Sie in einer Webanwendung, wenn Sie statische Variablen aus einer Webanfrage setzen, die Daten munge, wenn Sie den Setter nicht synchronisieren. Wenn Sie nur lesen, dann sollte es Ihnen gut gehen.

Eine Alternative zu dem, was Sie tun, ist warum nicht den OAuth-Schlüssel in einen Dienst injizieren, der die Authentifizierungsanliegen Ihres Systems behandelt? Sie könnten tun, was Sie mit einer statischen Ressource machen müssen, aber in diesem Fall sollten Sie sich bewusst sein, dass Sie keine statische Variable benötigen, um den Schlüssel zu halten.

Answer 3

Statische Variablen sind objektorientierte Substitute für globale Variablen in C. Versuchen Sie diese nach Möglichkeit zu vermeiden.

Oftmals benötigen Sie nur ein Objekt, in Ihrem Fall ist es das Prefs-Objekt.

public class Prefs { 

    //known before runtime 
    public final String SERVER_BASE_URL ="http://api.mycompany.com/"; 

    //needs to be set on startup through the setter method 
    private String token; 


    public String getToken() { 
    return token; 
    } 

    public void setToken(String token) { 
    Prefs.token = token; 
    } 

    public String getXyEndpointUrl() { 
    return SERVER_BASE_URL + "/xy"; 
    } 

} 

public class Program { 

    protected Prefs prefs; 

    protected Other prefsAware; 

    public Program() { 
    prefs = new Prefs(); 
    prefsAware = new Other(prefs); 
    } 

    // or even (if you don't like constructor mediated passing of prefs) 
    public Prefs getPrefs() { 
    return prefs; 
    } 

}