[Java] sovrascrivere metodo toString()

[VYCanisMajoris]

Salve ragazzi, ho una classe che ha 3 variabili esemplare e devo sovrascrivere il metodo toString.
le tre variabili di esemplare sono una Stringa, un double e un array di stringhe. Il problema è proprio su quest'ultimo di cui esce stampato male.
Mi sapete dire perche non funziona?

public String toString() {
		return getClass().getName() + "[nome="+nome + ", " + "altezza="+altezza + ", " + "cibiPreferiti="+cibiPreferiti.toString() + "]";
	}

Questa è tutta la classe per completezza ve la metto:

import java.util.Arrays;

public class Persona {
	private String nome;
	private double altezza;
	private String[] cibiPreferiti;

	public Persona(String nome, double altezza, String[] cibiPreferiti) {
		this.nome = nome;
		this.altezza = altezza;
		this.cibiPreferiti = cibiPreferiti;
	}

	public String getNome() {
		return nome;
	}

	public double getAltezza() {
		return altezza;
	}
	
	public String[] getCibiPreferiti() {
		return cibiPreferiti;
	}

	/* Sovrascrive metodo toString() della superclasse universale Object */
	public String toString() {
		return getClass().getName() + "[nome="+nome + ", " + "altezza="+altezza + ", " + "cibiPreferiti="+cibiPreferiti.toString() + "]";
	}
	
	/* Sovrascrive il metodo euals() della superclasse universale Object */
	public boolean equals(Object otherObject) {
		if(otherObject == null) return false;
		if(getClass() != otherObject.getClass()) return false;
		
		Persona p = (Persona)otherObject; //cast
		String[]pCp = p.getCibiPreferiti();
		if( nome.equals( p.getNome() ) && altezza == p.getAltezza() ) {
			for(int i=0 ; i<cibiPreferiti.length ; i++) {
				if( !( cibiPreferiti[i].equals(pCp[i]) ) ) return false;
			}
			return true;
		}
		else return false;
	}
	
	/* Sovrascrive metodo clone() della supeclasse Object */
	public Object clone() {
		String n = new String(nome); 										//copio il campo nome...
		double a = altezza;				 									//... il campo altezza...
		String[] cP = Arrays.copyOf(cibiPreferiti, cibiPreferiti.length); 	//...il campo cibiPreferiti che è un array ed uso, per copiarlo, il metodo appropriato
		
		Persona clone = new Persona(n, a, cP); 								//utilizzo il costruttore di Persona, per creare un clone
		return clone;
	}
}

[Who am I]

Il problema è proprio su quest'ultimo di cui esce stampato male.

Cosa significa questo? Sei in grado di spiegare qual'è il problema?

[Alex'87]

Non puoi "stampare" un array direttamente: devi interare ogni elemento e aggiungerlo ad una stringa singolarmente.

edit: in alternativa potresti provare ad usare Arrays.toString()

[Scara95]

Originariamente inviato da Alex'87
Non puoi "stampare" un array direttamente: devi interare ogni elemento e aggiungerlo ad una stringa singolarmente.

edit: in alternativa potresti provare ad usare Arrays.toString()

Non si capisce neanche che errore ottiene in output e nemmeno cosa vorrebbe ottenere (in che formato)...

[Alex'87]

Originariamente inviato da Scara95
Non si capisce neanche che errore ottiene in output e nemmeno cosa vorrebbe ottenere (in che formato)...

public String toString() {
    return getClass().getName() 
        + "[nome="
        + nome 
        + ", " 
        + "altezza="
        + altezza 
        + ", " 
        + "cibiPreferiti="
        + cibiPreferiti.toString()
        + "]";
}

private String[] cibiPreferiti;

cibiPreferiti è un array. Chiamare cibiPreferiti.toString() stamperà una cosa del tipo @String[]a34sda2 (più o meno). Immagino che lui invece voglia una stringa con il contenuto dell'array

edit: comunque Java avrebbe una sua sezione dedicata

[Who am I]

public String toString() 
{
    String result=getName()+"[nome="+nome + ", " + "altezza="+altezza;
    for(int i=0;i<cibiPreferiti.length;i++)
        result+=cibiPreferiti[i];
    return result;
}

[VYCanisMajoris]

Perfetto ragazzi vi ringrazio tutti e scusate se ho postato qui invece che nella sezione giusta, farò piu attenzione!
Comunque ho corretto il codice come da suggerimento vostro ed ora funziona bene

public String toString() {
		String result = 
			  getClass().getName()
			+ "["
			+ "nome=" + nome
			+ ", "
			+ "altezza=" + altezza
			+ ", "
			+ "cibiPreferiti=";
		for(int i=0 ; i<cibiPreferiti.length ; i++) {
			result = result + cibiPreferiti[i];
			if(i < cibiPreferiti.length - 1) result = result+", ";
		}
		return result+"]";
	}

Alex87 potresti precisare meglio quello che dici? che significa che potrei usare Arrays.toString() ?

[Alex'87]

Originariamente inviato da VYCanisMajoris
Alex87 potresti precisare meglio quello che dici? che significa che potrei usare Arrays.toString() ?

Se hai il seguente array

String[] myArray = new String[] { "hello", "world", "!" };

puoi stamparlo "velocemente" facendo

System.out.println(Arrays.toString(myArray));

Sulla console apparirà ciascun elemento, separato dalla virgola, il tutto tra parentesi quadre:

[hello, world, !]

Se non hai esigenze particolari di formattazione, può bastare

[VYCanisMajoris]

Assolutamente va bene cosi, nessun esigenza particolare, che stupido è proprio come Arrays.copyOf(tipo[] arrayOriginale, int newLength);
Solo un ultimo dubbio, questi sono metodi statici vero??

[Scara95]

Si sono metodi static