[Java] Mancata lettura variabile String

[skud]

Salve ragazzi,
per esercitarmi stavo scrivendo un programmino che, data una stringa, verifichi che il numero di caratteri maiuscoli sia uguale al numero di caratteri minuscoli.

Ecco come l'ho pensato:

public class Uguali {
	private String str;
	
	public Uguali(String str){
		this.str = str;
	}
	
	public void setStr(String str) {
		this.str = str;
	}
	
	public String getStr() {
		return str;
	}
	
//		Metodo controllo
	
	public Boolean isCaratteriUguali(){
		int countLower = 0;
		int countUpper = 0;
		for (int i = 0; i<this.str.length(); i++){
			Character c = ((Character)str.charAt(i));

			if(c.equals(c.toString().toUpperCase()))
				countUpper++;
			else 
				countLower++;

		}
		
		boolean valore = false;
		if(countUpper==countLower)
			valore = true;
		return valore;
		
		}

}

E la classe testUguali che esegue:

public class testUguali {
	public static void main (String[] args){
		
		Uguali stringa1 = new Uguali("R1_3j");
		System.out.println("L'operazione su "+stringa1+" risulta: "+stringa1.isCaratteriUguali());

		}
	
}

Per fare una prova ho istanziato la stringa1 dandogli come valore R1_3j. Una volta lanciato però invece di stamparmi la variabile, mi da il suo indirizzo e di conseguenza il metodo booleano isCaratteriUguali() sarà sempre false. Probabilmente l'errore sarà banale, ma non capisco perchè mi stampi l'indirizzo visto che la classe Uguali ha un costruttore definito correttamente. Dove ho sbagliato?

Grazie in anticipo!

[andbin]

Originariamente inviato da skud
però invece di stamparmi la variabile, mi da il suo indirizzo

Ma questo non centra con i tuoi conteggi/test. Hai usato la concatenazione delle stringhe e semplicemente viene invocato toString() sul tuo stringa1 (per ottenere un String da concatenare).
Ma nella tua classe Uguali NON hai ridefinito toString(), che rimane quindi quello ereditato da Object che ritorna quella forma particolare che hai visto.

Semplice: ridefinisci public String toString() { ....... }

Originariamente inviato da skud
e di conseguenza il metodo booleano isCaratteriUguali() sarà sempre false.

Ma questo per un motivo: hai un Character e ci invochi equals passandogli un String. Ed è sbagliato (per il equals di Character!)

Quindi o fai:

if (c.toString().equals(c.toString().toUpperCase())

oppure potresti anche evitare tutti i giri di conversioni (ne fai troppe, da char -> Character, poi al fondo da boolean -> Boolean). Evita.

E comunque ci sarebbe lo stesso un problema di fondo: 1 _ 3 li tratti come "upper" perché toUpperCase non li cambia.

[skud]

Originariamente inviato da andbin
Ma questo non centra con i tuoi conteggi/test. Hai usato la concatenazione delle stringhe e semplicemente viene invocato toString() sul tuo stringa1 (per ottenere un String da concatenare).
Ma nella tua classe Uguali NON hai ridefinito toString(), che rimane quindi quello ereditato da Object che ritorna quella forma particolare che hai visto.

Semplice: ridefinisci public String toString() { ....... }

Ma questo per un motivo: hai un Character e ci invochi equals passandogli un String. Ed è sbagliato (per il equals di Character!)

Quindi o fai:

if (c.toString().equals(c.toString().toUpperCase())

oppure potresti anche evitare tutti i giri di conversioni (ne fai troppe, da char -> Character, poi al fondo da boolean -> Boolean). Evita.

E comunque ci sarebbe lo stesso un problema di fondo: 1 _ 3 li tratti come "upper" perché toUpperCase non li cambia.

Ti ringrazio. Ho fatto l'override di toString() ed adesso funziona. Adesso penso a pulire il codice e gestire il problema degli interi e dei caratteri speciali che mi hai fatto notare.