[JAVA] Metodo equals

[marcoqwertyuiop]

Salve, sul libro fa questo esempio per usare il metodo equals:

class Quadrilatero {
	int latoA;
	int latoB;
	int latoC;
	int latoD;
	public Quadrilatero(int a, int b, int c, int d) {
		latoA = a;
		latoB = b;
		latoC = c;
		latoD = d;
	}
	public int getLato(int i) {
		switch (i) {
			case 0:
				return latoA;
				break;
			case 1:
				return latoB;
				break;
			case 2:
				return latoC;
				break;
			case 3:
				return latoD;
				break;
		}
		boolean equals(Object o) {
			Quadrilatero q = (Quadrilatero) o;
			if (getLato(0) == q.getLato(0) && getLato(1) == q.getLato(1) && getLato(2) == q.getLato(2) && getLato(3) == q.getLato(3)) return true;
			else {
				return false;
			}
		}
	}
	public static void main(String[] args) {
		Quadrilatero uno = new Quadrilatero(1, 1, 1, 1);
		Quadrilatero due = new Quadrilatero(1, 1, 1, 1);
		System.out.println(uno.equals(q2));
	}
}

Perchè non funziona? Cioè deve vdere se i lati del primo quadrilatero sono uguali ai lati del secondo quadrilatero.

[Amsirak]

perché q2 (che passi come argomento ad equals) non esiste

[marcoqwertyuiop]

Originariamente inviato da Amsirak

perché q2 (che passi come argomento ad equals) non esiste

Giusto, ma anche se correggo e metto due al posto di q2 mi da comunque errore:

[Guille]

Originariamente inviato da marcoqwertyuiop

Giusto, ma anche se correggo e metto due al posto di q2 mi da comunque errore:

Il problema dipende dal fatto che hai definito il metodo equals all'interno del metodo getLato.

[marcoqwertyuiop]

Originariamente inviato da Guille

Il problema dipende dal fatto che hai definito il metodo equals all'interno del metodo getLato.

Da comunque errore:

class Quadrilatero {
	int latoA;
	int latoB;
	int latoC;
	int latoD;
	public Quadrilatero(int a, int b, int c, int d) {
		latoA = a;
		latoB = b;
		latoC = c;
		latoD = d;
	}
	public int getLato(int i) {
		switch (i) {
			case 0:
				return latoA;
				break;
			case 1:
				return latoB;
				break;
			case 2:
				return latoC;
				break;
			case 3:
				return latoD;
				break;
		}
	}
	boolean equals(Object o) {
		Quadrilatero q = (Quadrilatero) o;
		if (getLato(0) == q.getLato(0) && getLato(1) == q.getLato(1) && getLato(2) == q.getLato(2) && getLato(3) == q.getLato(3)) return true;
		else {
			return false;
		}
	}
	public static void main(String[] args) {
		Quadrilatero uno = new Quadrilatero(1, 1, 1, 1);
		Quadrilatero due = new Quadrilatero(1, 1, 1, 1);
		System.out.println(uno.equals(due));
	}
}

[wingman87]

Sì ma leggi l'errore: "attempting to assign weaker access privileges; was public"
E' chiaro che il metodo deve essere public.

[marcoqwertyuiop]

Originariamente inviato da wingman87

Sì ma leggi l'errore: "attempting to assign weaker access privileges; was public"
E' chiaro che il metodo deve essere public.

Si, ma se metto il metodo public, mi esce l'errore che mi dice di torgliere i break nello swtch, li tolgo e mi esce quest'altro errore:

class Quadrilatero {
	int latoA;
	int latoB;
	int latoC;
	int latoD;
	public Quadrilatero(int a, int b, int c, int d) {
		latoA = a;
		latoB = b;
		latoC = c;
		latoD = d;
	}
	public int getLato(int i) {
		switch (i) {
			case 0:
				return latoA;
			case 1:
				return latoB;
			case 2:
				return latoC;
			case 3:
				return latoD;
		}
	}
	public boolean equals(Object o) {
		Quadrilatero q = (Quadrilatero) o;
		if (getLato(0) == q.getLato(0) && getLato(1) == q.getLato(1) && getLato(2) == q.getLato(2) && getLato(3) == q.getLato(3)) return true;
		else {
			return false;
		}
	}
	public static void main(String[] args) {
		Quadrilatero uno = new Quadrilatero(1, 1, 1, 1);
		Quadrilatero due = new Quadrilatero(1, 1, 1, 1);
		System.out.println(uno.equals(due));
	}
}

Ma alla riga 27, la } ci deve stare per chiudere il metodo getLato.

[wingman87]

Questo perché non hai messo un case default. Se ad esempio richiami getLato(5) cosa deve succedere?

[marcoqwertyuiop]

Originariamente inviato da wingman87

Questo perché non hai messo un case default. Se ad esempio richiami getLato(5) cosa deve succedere?

Ok, ora funziona. Grazie.

class Quadrilatero {
	int latoA;
	int latoB;
	int latoC;
	int latoD;
	public Quadrilatero(int a, int b, int c, int d) {
		latoA = a;
		latoB = b;
		latoC = c;
		latoD = d;
	}
	public int getLato(int i) {
		switch (i) {
			case 0:
				return latoA;
			case 1:
				return latoB;
			case 2:
				return latoC;
			case 3:
				return latoD;
			default:
				return 0;
		}
	}
	public boolean equals(Object o) {
		Quadrilatero q = (Quadrilatero) o;
		if (getLato(0) == q.getLato(0) && getLato(1) == q.getLato(1) && getLato(2) == q.getLato(2) && getLato(3) == q.getLato(3)) return true;
		else {
			return false;
		}
	}
	public static void main(String[] args) {
		Quadrilatero uno = new Quadrilatero(1, 1, 1, 1);
		Quadrilatero due = new Quadrilatero(1, 1, 1, 1);
		System.out.println(uno.equals(due));
	}
}