What the problem?

OK, do like this. public class Solution { public static void main(String[] args) throws Exception { ArrayList<String> list = new ArrayList<String>(); list.add("rose"); // 0 list.add("measure"); // 0 list.add("love"); // 1 list.add("lyre"); // 2 "rose", "measure", "love", "lyre", "wade", "bark". list.add("wade"); // 2 "rose", "measure", "love", "lyre", "wade", "bark". list.add("bark"); // 2 "rose", "measure", "love", "lyre", "wade", "bark". list = fix(list); for (String s : list) { System.out.println(s); } } public static ArrayList<String> fix(ArrayList<String> list) { for (int i = 0; i < list.size(); i++) { if (list.get(i).contains("r") && !list.get(i).contains("l")) { list.remove(i); if (i > 0) { i--; } } else if (list.get(i).contains("l") && !(list.get(i).contains("r"))) { list.add(i,list.get(i)); i =i+1; } else if (list.get(i).contains("r") && list.get(i).contains("l")) { i++; } else { i++; } } return list; } }

and it give me error, but when I do it on the IntelliJ it give me measure love love lyre wade What wrong I doing?

remove the bold: public static ArrayList<String> fix(ArrayList<String> list) { for (int i = 0; i < list.size(); i++) { if (list.get(i).contains("r") && !list.get(i).contains("l")) { list.remove(i); if (i > 0) { i--; } } else if (list.get(i).contains("l") && !(list.get(i).contains("r"))) { list.add(i,list.get(i)); i =i+1; } else if (list.get(i).contains("r") && list.get(i).contains("l")) { i++; } else { i++; } } return list; }

It don't work.

You will also have to remove: if(i > 0){ i--; }

Thanks bud still it don't work.

im sorry, i meant to remove the if statement, you still need to leave the i--. This should be the code: public static ArrayList<String> fix(ArrayList<String> list) { for (int i = 0; i < list.size(); i++) { if (list.get(i).contains("r") && !list.get(i).contains("l")) { list.remove(i); i--; } else if (list.get(i).contains("l") && !(list.get(i).contains("r"))) { list.add(i, list.get(i)); i = i + 1; } } return list; }

Thank you very much. It was hard for me. But with your help, I do it...Thanks man. You are the best. If some one need the right code. public class Solution { public static void main(String[] args) throws Exception { ArrayList<String> list = new ArrayList<String>(); list.add("rose"); // 0 list.add("love"); // 1 list.add("lyre"); // 2 list.add("measure"); // 3 list.add("wade"); // 4 list.add("bulk"); list = fix(list); for (String s : list) { System.out.println(s); } } public static ArrayList<String> fix(ArrayList<String> list) { for (int i = 0; i < list.size(); i++) { if (list.get(i).contains("r") && !list.get(i).contains("l")) { list.remove(i); i--; } else if (list.get(i).contains("l") && !(list.get(i).contains("r"))) { list.add(i, list.get(i)); i = i + 1; } } return list; } }

But why i-- ??

Because when you remove the word that space becomes null, so you need to return to the previous space. So you reuse the empty as the end of list otherwise you will end up with a null space in your list.

And why i=i+1;

Thanks Guadalupe Gagnon, Evghenii