This is making me crazy, and it should be so simple!

[Alex Bruhart]

Here's a snippet of some of my code:

  listArray = listCandyJL.getSelectedIndices();
  for (int i = 0; i  < listArray.length; i++)
  {        
  if (namesArray[listArray[i]].equals("Chocolate Cheese: $10"));
        {
        indCost = 10;
        }
        if (namesArray[listArray[i]].equals("Gum Drops: $5"));
        {
        indCost = 5;
        }
        if (namesArray[listArray[i]].equals("Peppermint Patty: $15"));
        {
        indCost = 15;
        }
        if (namesArray[listArray[i]].equals("NECCO Wafers: $1"));
        {
        indCost = 1;
        }
        if (namesArray[listArray[i]].equals("Milky Way: $100"));
        {
        indCost = 100;
        }
        if (namesArray[listArray[i]].equals("Baby Ruth: $50"));
        {
        indCost = 50;
        }
        if (namesArray[listArray[i]].equals("Snickers: $2"));
        {
        indCost = 2;
        }
        if (namesArray[listArray[i]].equals("M&Ms: $3"));
        {
        indCost = 3;
        }
        if (namesArray[listArray[i]].equals("Charleston Chew: $50"));
        {
        indCost = 50;
        }
        if (namesArray[listArray[i]].equals("Candy Apples: $5"));
        {
        indCost = 5;
        }
        if (namesArray[listArray[i]].equals("Lowfat Butter: $10"));
        {
        indCost = 10;
        }
        if (namesArray[listArray[i]].equals("Land 'O' Lakes Butter: $7"));
        {
        indCost = 7;
        }
        if (namesArray[listArray[i]].equals("Blue Bonnet: $6"));
        {
        indCost = 6;
        }
        if (namesArray[listArray[i]].equals("Margarine: $875"));
        {
        indCost = 875;
        }
        subTot = subTot + indCost;
 
  }  
 

Now, all it does is keep adding (compounding) 875 dollars, which is the value in the last "if" statement. Why? Shouldn't it be adding the value of the other options when they're selected? Why isn't it?

This is making me crazy!

Thanks for your help.

[ June 15, 2008: Message edited by: Alex Bruhart ]
[ June 15, 2008: Message edited by: Alex Bruhart ]

[Robert Hill]

Can you print out the array contents and display them here?

[Wirianto Djunaidi]

It is because you have semicolon at the end of if, so basically you are terminating the if without condition statement to be executed.

The curly braces after the if in your code is just treated as a scoping block.

You need to remove all the semicolon at the end of your if statement to let the compiler see the following braces block as the condition statement to be executed.

-DJ

[imaya Munusamy]

There is semicolon at the end of every "if" statement so the indCost = value; is not part of the if block.

[Peter Chase]

As previous posters have indicated, the main bug in the posted program is that there is a semicolon at the end of each "if" line, meaning that the following code is not controlled by the "if", but instead is always executed. To fix, remove those semicolons.

However, beyond that, there is plenty to improve in the posted code.

Rather than have a huge set of "if" statements, you could use a HashMap to map menu items (String) to cost (Integer).

Alternatively, as each menu item includes the price, you might be able to parse (read) the price within the menu item; look at the API for String.indexOf(), String.substring(), Integer.parseInt() etc. Or, if feeling brave, something similar but better can be achieved with a "regular expression"; see the Pattern and Matcher classes.
[ June 16, 2008: Message edited by: Peter Chase ]

[Ilja Preuss]

Also note that you can put arbitrary objects into a JList. If you have a class that has an appropriate toString method and, for example, a getPrice method, you could put those objects into the list, and use getSelectedValues to extract. Just iterate over them and ask them for their price - voila!

[Rob Spoor]

And if you don't want to override toString, or you can't, you can use a ListCellRenderer instead to do the rendering for you.

[Alex Bruhart]

Oh wow, call me stupid. I didn't even realize I put a semi-colon at the end of each if statement.

Thanks for the other tips too guys, I appreciate it.

[Campbell Ritchie]

Originally posted by Alex Bruhart:
I didn't even realize I put a semi-colon at the end of each if statement.

Everybody does that at some time, so don't worry about it.