[Java] Cercare una stringa in un ArrayList di stringhe

[angelo.digiacomantonio]

Ciao ragazzi,
spero possiate aiutarmi a risolvere questo problema:

Codice: Seleziona tutto

public boolean cercaLibro(String libro){
	boolean trovato=false;
	for(int i=0; i<libri.size(); i++){
		if(libri.get(i).equals(libri)){
			trovato=true;
			return trovato;
		}
	}
	return trovato!=true? false:null;
}

Praticamente questo è un metodo che, in un ArrayList di stringhe, dovrebbe cercare una stringa presa in input e restituire true se c'è, altrimenti false;
il mio problema è che mi restituisce sempre falseee ...mi ci sono spremuto il cervello ma non sono proprio riuscito a trovare una soluzione ??? ...qualcuno può darmi qualche dritta?
Grazie mille!

[ele_girasole]

Codice: Seleziona tutto

if(libri.get(i).equals(libri))

Ciao,presumo che la stringa che devi cercare è libro giusto?

Codice: Seleziona tutto

public boolean cercaLibro(String libro)

invece tu nella condizione metti libri che da quanto ho capito è l'array,devi fare così (ma forse lo sai ti solo distratto)

Codice: Seleziona tutto

libri.get(i).equals(libro)   //libro=stringa

[Zoff]

Perché non usare il metodo contains?

http://docs.oracle.com/javase/7/docs/ap ... .Object%29

[angelo.digiacomantonio]

Codice: Seleziona tutto

if(libri.get(i).equals(libri))

Ciao,presumo che la stringa che devi cercare è libro giusto?

Codice: Seleziona tutto

public boolean cercaLibro(String libro)

invece tu nella condizione metti libri che da quanto ho capito è l'array,devi fare così (ma forse lo sai ti solo distratto)

Codice: Seleziona tutto

libri.get(i).equals(libro)   //libro=stringa

Codice: Seleziona tutto

libri.get(i).equals(libro)   //libro=stringa

in questo codice volevo intendere che se l'elemento "i" dell'array list  è uguale alla stringa libro (che contiene il nome di un libro), restituisce true, altrimenti false; il problema è che mi restituisce sempre false...credo il problema sia in questa riga...

[angelo.digiacomantonio]

Ragazzi grazie millee!!!!
grazie a "ele girasole" per avermi fatto notare l'errore di distrazione e grazie a zoff per avermi suggerito il contains! :)
grandissimi come sempree!!

[elsco]

Ciao,
considerando che la cosa più giusta da fare te l'ha consigliata zoff, ti dico comunque la mia:
secondo me potevi implementare lo stesso metodo in questo modo:

Codice: Seleziona tutto

public boolean cercaLibro(String libro){
	for(String l : libri){
		if(libro.equals(l)){
			return true;
		}   
	}	
	return false;
}