[JAVA/Swing] Registro di componenti. Idea buona o cattiva?

[lovaz]

A volte serve fare un update di un componente di cui non si ha
la referenza. Passare referenze in giro porta imho a un groviglio
orribile di dipendenze, quindi come fare?
Ho pensato a una classe singleton che mantiene un registro di
componenti di interesse generale, a cui tutte le classi possono
accedere, più o meno così:
MainPanel mp = ComponentRegistry.getComponent( "MAIN PANEL" );

Secondo voi c'e' qualche controindicazione?

[Oceans11]

Originariamente inviato da lovaz

A volte serve fare un update di un componente di cui non si ha
la referenza. Passare referenze in giro porta imho a un groviglio
orribile di dipendenze, quindi come fare?
Ho pensato a una classe singleton che mantiene un registro di
componenti di interesse generale, a cui tutte le classi possono
accedere, più o meno così:
MainPanel mp = ComponentRegistry.getComponent( "MAIN PANEL" );

Secondo voi c'e' qualche controindicazione?

Non so proprio come aiutarti...ma hai tutto il mio appoggio morale, visto che succede sempre che impreco per tutte le referenze che mi porto dietro!!!!
"certe volte" scrivo cose assurde!!!!


se riesci fammi sapere che ti faccio una statua!!!!!

[lovaz]

Come implementazione non è difficile, oggi o domani comincio il refactoring.

A presto!

[lovaz]

Una cosa di questo tipo:

public class ComponentRegistry
{
	private static final Map<String, Component> map =
		new HashMap<String, Component>();

	private ComponentRegistry() {}
	
	public static void bind(String key, Component component)
	{
		map.put( key, component );
	}
	
	public static Set<String> getBoundComponentNames()
	{
		return map.keySet();
	}
	
	public static Component getComponent(String name)
	{
		return map.get(name);
	}
}

Un componente si registra con bind:

ComponentRegistry.bind( "MAIN_WINDOW", frame );

e si recupera con getComponent:

JFrame main = (JFrame) ComponentRegistry.getComponent( "MAIN_WINDOW" );

Naturalmente vanno registrati solo i componenti di particolare interesse,
di uso "generale".

Se avete idee dite pure.

[thebol]

any time you do a singletone god kill a kitten.
Think to kitten

cmq http://en.wikipedia.org/wiki/Singleton_pattern per l'implementazione corretta in java.

potresti anche pensare di creare un oggetto e passarlo ai vari oggetti che ne hanno bisogno.

La cosa brutta imho del registry, e che rischi l'aliasing dei nomi. Inoltre di devi ricordare ogni volta che ne crei un componente di metterlo dentro. A meno che di far fare la creazione direttamente al registry...

Altra soluzione potresti nel tuo getInstance mettere un parametro per capire che vuoi il registro della finestra X, o di un gruppo X.

Se l'applicazione è semplice, o cmq non prevede sviluppi futuri lascia perdere queste seghe mentali

[lovaz]

Originariamente inviato da thebol

cmq http://en.wikipedia.org/wiki/Singleton_pattern per l'implementazione corretta in java.

Non è un singleton canonico, intendevo un aggeggio unico nella jvm,
che ottengo usando direttamente la classe "come fosse un oggetto".
Capisco che in fondo è una variabile globale, ma forse la cura
(non usarlo) è peggiore del male - a proposito, da dove viene la citazione?

potresti anche pensare di creare un oggetto e passarlo ai vari oggetti che ne hanno bisogno.

E' proprio quello che intendo evitare.

La cosa brutta imho del registry, e che rischi l'aliasing dei nomi. Inoltre di devi ricordare ogni volta che ne crei un componente di metterlo dentro. A meno che di far fare la creazione direttamente al registry...

L'applicazione non è gigantesca, ci saranno 3 o 4 componenti
che mi interessa registrare. Se il componente non è registrato
ci penserò, magari lancio un'eccezione.

Altra soluzione potresti nel tuo getInstance mettere un parametro per capire che vuoi il registro della finestra X, o di un gruppo X.

Dici una gerarchia di registri? Non vale la pena, l'applicazione non è grossa.

Se l'applicazione è semplice, o cmq non prevede sviluppi futuri lascia perdere queste seghe mentali

infatti...