Having trouble with iterating through an ArrayList (Java)

Question

I don't understand why the following if statement never hits, I've verified through the log that movie and films.get(i).getTitle() are the same string:

List<Film> films = new ArrayList<Film>();
films = filmDB.getAllFilms();



for(int i = 0; i < films.size(); i++)

                if(movie == films.get(i).getTitle())
                {
                    Log.d("TITLEMOVIE", films.get(i).getTitle());
                    ;
                }
            else
                {
                    Log.d("NOMATIC", films.get(i).getTitle());
                }

You need to use String's equals method to compare content of strings. == tests whether the two references are either both null or both point to the same object. — Patricia Shanahan, Apr 03 '16 at 13:56
I don't how I managed to miss that I needed to use .equals for two strings, thanks! — Se O, Apr 03 '16 at 14:28

Jorn Vernee · Accepted Answer · 2016-04-03T14:08:01.610

0

== for anything but primitives compares the memory addresses of the two objects. use .equals to compare strings and other objects.

i.e.;

String str1 = "bla";
// Use String::new to make sure 'str1' and 'str2' don't reference the same literal
String str2 = new String("bla");

System.out.println(str1 == str1); // true
System.out.println(str1 == str2); // false
System.out.println(str1.equals(str2)); // true

edited Apr 03 '16 at 14:08

answered Apr 03 '16 at 13:57

Jorn Vernee

31,735
4
76
93

Having trouble with iterating through an ArrayList (Java)

1 Answers1