I'm having trouble extracting repeated numbers from a ArrayList , here's the snippet of code:
for( int i = 0 ; i < houses.size(); i++ ) {
for( int j = 1; j < houses.size(); j++ ) {
if(houses.get(i).equals(houses.get(j)) && i!=j){
houses.remove(j);
}
}
}
It correctly deletes most of the repeated elements, but in some cases deletes a few more, I really wanted to know what's wrong with this code, as I can not figure it out.
The data structure you are using is not ideal for what you want to do. I do not know the reason for using ArrayList, but if you need the structure does not accept repeated values it is best to use the Set so that you do not have to do additional treatments and leave the code cleaner.
Set<String> houses = new HashSet<>();
public void add (String house){
// nunca aceitará valor repetido
houses.add(house);
}
You can not remove items from the object being iterated because it passes through another structure during the iteration started. You have to create an auxiliary structure. If you want to do it on the same hand and do not use a HashSet :
ArrayList<String> aux = new ArrayList<String>();
for (String item : houses) {
if (!aux.contains(item)) {
aux.add(item);
}
}
houses = aux;
See working on ideone .