non va bene perchè prima aggiungi la nuova matricola e poi fai il controllo, devi fare il contrario.
puoi anche usare il metodo contains
codice:
ArrayList<String> matricole=new ArrayList<String>();
System.out.println("Inserisci una matricola per studente: ");
String matr=in.nextLine();
if(matricole.contains(matr)) {
System.out.println("Errore gia presente");
} else {
matricole.add(matr);
}
ancora meglio puoi usare un Set al posto di una List, che non accetta valori duplicati
codice:
Set<String> matricole=new HashSet<String>();
System.out.println("Inserisci una matricola per studente: ");
String matr=in.nextLine();
if(matricole.add(matr)) { //add ritorna false se l'elemento esiste già
System.out.println("Errore gia presente");
}
ah mi aspetto che
codice:
ArrayList<String> matricole=new ArrayList<String>();
lo fai in un altra parte del codice altrimenti crei una lista nuova ogni volta