[Java] Cross-calling Costruttori

[Premier]

Salve,
volevo sapere come faccio una cross-calling tra costruttori nel seguente caso

public class User {
	private String username,password;
	
	public User() { //costruttore con 0 argomenti
		String nome,pass;
		// leggo nome e pass
		this(nome,pass); //in questo punto vorrei chiamare il costruttore con 2 argomenti per completare l'istanza
	}
	
	public User(String Nome,String Pass) { //costruttore con 2 argomenti
		System.out.println("Creato l'utente "+Nome);
		username= Nome;
		password= Pass;
	}

purtroppo il compilatore mi dice che la chiamata a costruttori, fatta nel metodo User(), dev'essere la prima istruzione del metodo. Tuttavia se non leggo prima nome e pass non ha senso richiamare il costruttore con 2 argomenti.

Come posso fare?

LA MIA SOLUZIONE: avevo pensato ai parametri di default nel costruttore a 2 argomenti, e in esso, prima di fare le assegnazioni, controllo prima il valore contenuto nei parametri. Se questi sono i valori di default allora provvedo a fare la lettura e poi l'assegnazione. Posso fare solo in questo modo?

Grazie

[andbin]

Ci sono parecchie regolette relative ai costruttori. Tra queste vi è quella che un costruttore deve sempre e comunque iniziare esclusivamente o con un super() o con un this().

Anche quando ad esempio fai:

class XYZ
{
    public XYZ ()
    {
        System.out.println ("ciao");
    }
}

La chiamata a super() c'è lo stesso ..... viene infatti inserita dal compilatore!
In ogni caso, anche se usi this(), prima o poi dovrà esser fatta una super().

Torniamo al tuo problema:

public User() { //costruttore con 0 argomenti
	String nome,pass;
	// leggo nome e pass
	this(nome,pass); //in questo punto vorrei chiamare il costruttore con 2 argomenti per completare l'istanza
}

Tu leggi nome e pass da dove??? Presumo dallo standard input, inserito dall'utente .....
Premetto che un "design" del genere non è certamente molto bello, comunque c'è un'altra regoletta relativa ai costruttori: fino a quando il super-costruttore non è stato eseguito, il "this" non è utilizzabile nella classe. Ciò significa che fino a quando il super() non è stato eseguito, non puoi usare metodi/variabili di istanza ...... però puoi usare metodi/variabili statici!!

Ecco un esempio:

import java.util.*;

public class Prova
{
    private String nome;

    public Prova ()
    {
        this (inputNome ());
    }

    public Prova (String nome)
    {
        this.nome = nome;
    }

    private static String inputNome ()
    {
        Scanner sc = new Scanner (System.in);
        System.out.print ("Nome: ");
        return sc.nextLine ();
    }

    public static void main (String[] args)
    {
        Prova p = new Prova ();
    }
}

Nota che inputNome() è static, perciò lo puoi chiamare mentre passi il parametro a this().
Se una tale soluzione sia un design valido o meno ..... non lo so, non sono un "guru" di Java. Però dal mio punto di vista, fa schifo ....

[R@ve M@ster]

Direi che la tua soluzione dei valori di default è più che valida. Al massimo per evitare "valori fissi" di default potresti usare una stringa vuota.

[Premier]

Originariamente inviato da andbin
Se una tale soluzione sia un design valido o meno ..... non lo so, non sono un "guru" di Java. Però dal mio punto di vista, fa schifo ....

Beh nemmeno io sono un "guru", dato che esprimi giudizio, ti inviterei a consigliarmi una tua soluzione al problema, ossia predisporre un giusto costruttore se l'oggetto è istanziato con o senza passargli i parametri di inizializzazione.

[LeleFT]

Anche a me non piace molto quella soluzione.
Infatti, secondo la logica della programmazione ad oggetti, un oggetto viene costruito quando si hanno in mano tutti gli elementi per farlo.

Se devi leggere da qualche parte i dati per la costruzione dell'oggetto, la soluzione più ovvia, più banale e più sensata è quella di leggerli prima di costruirlo.

Considerando che qualsiasi applicazione parte da un contesto statico (il main), non vedo perchè si debba ricorrere ad una soluzione del genere, quando questa è più che sufficiente:

public class MiaClasse {
   public MiaClasse(String s1, String s2) {
      ...
   }

   ...

   public static void main(String [] args) {
      // Leggo i parametri da tastiera
      BufferedReader br = new BufferedReader( new InputStreamReader( System.in ) );
      String s1 = br.readLine();
      String s2 = br.readLine();

      // Ora costruisco l'oggetto:
      MiaClasse mc = new MiaClasse(s1, s2);
   }
}

Se c'è l'esigenza di avere un costruttore di default, questo dovrà semplicemente passare due valori "standard" (ad esempio, due stringhe vuote) e la classe dovrà mettere a disposizione un metodo per la valorizzazione post-creazione. (E' questo il caso di oggetti soggetti a serializzazione).

Casi più particolareggiati vengono risolti utilizzando campi di tipo transient.


Ciao.

[Premier]

Un grazie a tutti quelli che mi hanno aiutato, la soluzione che ho adottato è la seguente

public class User {

    private String username,password;



    public User() { //costruttore con 0 argomenti

        String nome,pass;

        // leggo nome e pass

        createUser(nome,pass);

    }



    public User(String Nome, String Pass) {

        createUser(Nome, Pass);

    }



    private createUser(String Nome,String Pass) {

        System.out.println("Creato l'utente "+Nome);

        username= Nome;

        password= Pass;

    }

}

Saluti.