Question

I cannot see what I am doing wrong here. Here is the code I am having trouble with :

String tempSummaryString = "SUMMARY:";
for(int z = 0; z<attributeList.size() ; z++)
{
    System.out.println(attributeList.get(z).substring(0,tempSummaryString.length()));
    if(attributeList.get(z).length() > tempSummaryString.length() && 
       attributeList.get(z).substring(0,tempSummaryString.length() == tempSummaryString)
    {
            event.setTitle(attributeList.get(z).substring(tempSummaryString.length(),attributeList.get(z).length()));
    }
}

Now my problem is that the program never goes into the if (does not execute the event.setTitle method). When I print the value of

attributeList.get(z).substring(0,tempSummaryString.length()) 

I get the following:

SUMMARY:

So I am stumped about why it is not entering the if! I don’t get it!

Hopefully someone can point out a stupid mistake I am making because I really dont know what else to do

Answer 1

You’ve fallen for the old == vs equals() problem. You are using ==, which unlike javascript, does an identity comparison (ie are these the same objects).

Try this:

attributeList.get(z).substring(0,tempSummaryString.length())
    .equals(tempSummaryString) // equals() not ==

Also, you should consider using the foreach syntax for your loop:

for (String attribute : attributeList) {
    if (attribute.substring... // forget about attributeList.get(z) and even z
}