Dubbio su un metodo

[Markus85]

Ciao

In un esercizio che mi è stato dato, mi si chiede di scrivere un metodo rimuovi(E elem) che rimuove da un contenitore dati l'elemento ricevuto come parametro e nel caso non sia presente restituisce false .

Io l'ho scritto in questo modo:

public boolean rimuovi(E elem) {
		Iterator<E> it = this.iterator();
		while(it.hasNext()){
			E elemento = it.next();
			if(elemento.equals(elem))
			it.remove();
			return true;
			
		}
		return false;
	}

nella soluzione però il metodo era scritto cosi :

public boolean rimuovi(E elem) {
		Iterator<E> it = this.iterator();
		while(it.hasNext()){
			E elemento = it.next();
			if(!elemento.equals(elem)) continue;
			it.remove();
			return true;
			
		}
		return false;
	}

A me sembra che quello che ho scritto io sia pure corretto.
E cosi?
Ciao

[valia]

analizza un po il codice e capisci l'errore (e usa le parentesi per non confonderti)
Hai testato il tuo codice? io dico di no.
In ogni caso tu dici: se l'elemento è uguale rimuovilo, ma in ogni caso esci.
Il prof ti dice, se l'elemento non è uguale, controlla il successivo.
A parer mio un modo che aumenta la leggibilità del metodo è questo

public boolean rimuovi(E elem) {
           boolean ret = false; 
		Iterator<E> it = this.iterator();
		while(it.hasNext()){
			E elemento = it.next();
			if(elemento.equals(elem)){

          			it.remove();
	         		ret = true;
                                break;
			}
		}
		return ret;
	}

se l'elemento è diverso, continui a controllare il successivo. Se è uguale rimuovi ed esci (sempre che nella tua collezione non ci siano doppioni).
Infine considera che io ho messo un solo return, un solo punto di ingresso e di uscita, utile in progetti più complessi
Nota anche l'uso delle parentesi (e prendi l'abitudine di metterle sempre!!!)

[andbin]

Originariamente inviato da Markus85

			if(elemento.equals(elem))

			if(!elemento.equals(elem)) continue;

A me sembra che quello che ho scritto io sia pure corretto.
E cosi?

Sono uguali come concetto/risultato finale. Cambia solo la logica per fare o no il remove.
Nel tuo: se trovato, allora rimuove e ritorna
Nell'altro: se non trovato, passa ad eseguire subito un nuovo ciclo (saltando remove e return)

[Markus85]

Ok Andrea.
Grazie
Ciao

[valia]

Originariamente inviato da andbin
Sono uguali come concetto/risultato finale. Cambia solo la logica per fare o no il remove.
Nel tuo: se trovato, allora rimuove e ritorna
Nell'altro: se non trovato, passa ad eseguire subito un nuovo ciclo (saltando remove e return)

attento che nel primo caso sia se fa la remove, sia se non la fa esce con true
Occhio alle parentesi!!!

[andbin]

Originariamente inviato da valia
attento che nel primo caso sia se fa la remove, sia se non la fa esce con true
Occhio alle parentesi!!!

Cavoli, ero davvero di fretta, ho quotato solo i due equals, ragionando solo su di essi e non ho fatto caso alle questioni sulla struttura.

Il codice di Markus85 sarebbe equivalente all'altro SE remove e return fossero racchiusi da un blocco { } del if.

La prossima volta starò più attento. Però voi cercate di scrivere sempre il codice ben indentato e usando { } anche se if/for/ecc... hanno solo 1 istruzione. Perché a quel punto la struttura e logica del codice sarebbe evidente e inequivocabile!!