È una cattiva pratica creare nuovi oggetti senza memorizzarli?

Ho visto oggetti creati nel codice Java senza memorizzare un riferimento all'oggetto. Ad esempio, in un plugin eclipse ho visto una shell SWT creata in questo modo:

new Shell();

Questo nuovo oggetto Shell non è memorizzato in una variabile, ma rimarrà referenziato fino a quando non verrà eliminata la finestra che [credo?] Si verifica per impostazione predefinita alla chiusura della finestra.

È una cattiva pratica creare oggetti come questo senza memorizzarne un riferimento? O la biblioteca è stata progettata male? Cosa succede se non ho bisogno di un riferimento, ma voglio solo gli "effetti collaterali" dell'oggetto? Devo comunque memorizzare un riferimento?

AGGIORNARE:

Devo ammettere che il mio esempio sopra è scarso. Mentre ho visto elementi dell'interfaccia utente creati in questo modo, creare una shell SWT come questa sarebbe probabilmente inutile perché è necessario chiamare il metodo open sull'istanza Shell. Ci sono esempi migliori forniti da aix come i seguenti dal tutorial sulla concorrenza Java :

(new HelloThread()).start();

Questa pratica è vista in molti contesti, quindi le domande rimangono. È buona pratica?

C'è un elemento di preferenza personale in questo, ma penso che non memorizzare il riferimento non sia necessariamente una cattiva pratica.

Considera il seguente esempio ipotetico:

new SingleFileProcessor().process(file);

Se è necessario creare un nuovo oggetto processore per ogni file e non è necessario dopo la process()chiamata, non ha senso memorizzare un riferimento ad esso.

Ecco un altro esempio, tratto dal tutorial sulla concorrenza Java :

(new HelloThread()).start();

Ho visto molti altri esempi quando il riferimento non è stato memorizzato e leggo perfettamente bene ai miei occhi, come:

String str = new StringBuilder().append(x).append(y).append(z).toString();

(L' StringBuilderoggetto non è mantenuto.)

Ci sono modelli simili che coinvolgono common.lang's HashCodeBuilderet al.

Se non hai bisogno di un riferimento all'oggetto creato, non trattenerlo. E 'così semplice.

In generale, è buona norma rilasciare riferimenti il ​​più rapidamente possibile. Non vedo alcuna differenza tra:

HelloThread thread = new HelloThread();
thread.start();
// where thread is never used for the rest of the method

e

(new HelloThread()).start();

Per quanto riguarda il codice, stai semplicemente evitando l'uso di un nome di variabile, che potrebbe essere una cosa positiva. Il compilatore JIT è in genere abbastanza intelligente da riconoscere che può spazzatura raccogliere il thread dopo il suo ultimo utilizzo, quindi probabilmente non c'è differenza dal punto di vista delle prestazioni.

L'unico vero problema da evitare è il codice nell'Anti-Pattern del costruttore , ma non sembra che sia quello che stai chiedendo.

Questo potrebbe essere negativo se stai usando SWT, perché lì devi ripulire dopo te stesso (chiamando il metodo dispose ()). Ma per altre classi (non SWT) va bene.

Ecco un articolo sulla gestione delle risorse del sistema operativo

Prima di tutto, infastidirai le persone che hanno imparato C o C ++ prima di Java. Lascio che sia un esercizio per il lettore decidere se è un pro o un contro.

Mentre ci sono certamente situazioni in cui gli oggetti sono progettati per essere di breve durata, il più delle volte se qualcosa è abbastanza complesso per creare un oggetto, è abbastanza complesso da avere buone ragioni per mantenere un riferimento, quindi non tenerlo dovrebbe il minimo è un avvertimento per ricontrollare se hai perso qualcosa.

Ad esempio, in un tutorial nessuno si preoccupa di mantenere un riferimento al thread, ma nella vita reale, se qualcosa impiega abbastanza tempo per generare un thread, di solito ci vuole abbastanza tempo per poter annullare il thread. O se hai una vita più breve, di solito stai generando un sacco di fili che vorrai joinprima di continuare. Oppure vuoi assicurarti che la creazione del tuo thread sia effettivamente riuscita. Oppure vuoi assicurarti che il thread termini in modo pulito prima di uscire. Ottieni l'immagine.