Proprietà java: esporre o non esporre?

Question

Questo potrebbe essere un problema di età avanzata e sono sicuro che ognuno ha i propri modi. Supponiamo che io sono alcune proprietà definite come

secret.user.id=user 
secret.password=password 
website.url=http://stackoverflow.com 

Supponiamo che io sono 100 diverse classi e posti in cui ho bisogno di utilizzare queste proprietà. Quale è buono (1) Creo una classe Util che caricherà tutte le proprietà e le servirà utilizzando una costante di chiave Ad esempio: Util è un singleton che carica tutte le proprietà e continua con la chiamata getInstance().

Util myUtil = Util.getInstance(); 
String user = myUtil.getConfigByKey(Constants.SECRET_USER_ID); 
String password = myUtil.getConfigByKey(Constants.SECRET_PASSWORD); 
.. 
//getConfigByKey() - inturns invokes properties.get(..) 
doSomething(user, password) 

Quindi ovunque io abbia bisogno di queste proprietà, posso eseguire i passaggi sopra.

(2) Creo una classe significativa per rappresentare queste proprietà; per esempio, ApplicationConfig e fornire getter per ottenere proprietà specifiche. così sopra il codice può apparire come:

ApplicationConfig config = ApplicationConfig.getInstance(); 
doSomething(config.getSecretUserId(), config.getPassword()); 
//ApplicationConfig would have instance variables that are initialized during 
// getInstance() after loading from properties file. 

Nota: il file delle proprietà in quanto tale avrà solo piccole modifiche in futuro.

La mia scelta personale è (2) - fammi sentire alcuni commenti?

Answer 1

trovo il primo approccio ad essere più dettagliato del necessario. (Soprattutto se non ci si aspetta che le proprietà cambino molto.) Inoltre, utilizzando il secondo approccio è possibile gestire i problemi di casting/tipo quando le proprietà vengono caricate invece di quando vengono utilizzate.

Answer 2

Credo che la mia prima domanda è perché si desidera creare un'istanza di qualcosa che si sta dicendo è un singleton (hai menzionato usando il codice come Util.getInstance()). Un singleton ha solo 1 istanza, quindi non dovresti provare a istanziare più copie nel tuo codice.

Se i dati sono statici (come questo sembra essere) creerei un singleton e recupererò i valori da esso.

Answer 3

Non penso ci sia alcun vantaggio significativo di un metodo rispetto all'altro e non penso che la soluzione (1) sia più sicura, solo perché fornisce una chiave di proprietà invece di un getter java per ottenere le password .

Se dovessi sceglierne uno, prenderei l'opzione (2).

Answer 4

farlo nel modo più semplice (una classe con valori statici):

package com.domain.packagename 

public class Properties { 
    private static String hostName; 
    public static getHostName() { return hostName; } 
    private static int port; 
    public static int getPort() { return port; } 

    public static void load() { 
     //do IO stuff, probably 
     hostName = ??; 
     port = ??; 
     //etc 
    } 
} 

Answer 5

L'opzione (2) per mantenere i getter specifici delle applicazioni sembra migliore e pulita. Le chiavi finali statiche pubbliche di un'interfaccia sono state un cattivo progetto in Java per anni.