This a section of my Java program I've taken out and simplified to test. The task is to compare two integers from an ArrayList and state whether they are equal.
The below code works for numbers <128 but any number >128 and the code will not work.
Any help would be really great,
thanks.
import java.util.*;
public class test
{
public static void main (String[] args)
{
Integer seat1Store = 128;
Integer seat2Store = 128;
Integer seat3Store = 0;
Integer seat4Store = 0;
Integer seat5Store = 0;
ArrayList<Integer> proceedArray = new ArrayList<Integer>();
if (seat1Store !=0)
{
proceedArray.add(seat1Store);
}
if (seat2Store !=0)
{
proceedArray.add(seat2Store);
}
if (seat3Store !=0)
{
proceedArray.add(seat3Store);
}
if (seat4Store !=0)
{
proceedArray.add(seat4Store);
}
if (seat5Store !=0)
{
proceedArray.add(seat5Store);
}
System.out.println("ArrayList = " + proceedArray);
boolean proceed = false;
for(int i = 0; i<proceedArray.size();i++)
{
for(int p=0; p<proceedArray.size(); p++)
{
if(i != p)
{
if(proceedArray.get(i) == proceedArray.get(p))
{
System.out.println("DUPLICATE");
System.exit(0);
}
}
}
proceed = true;
}
if (proceed == true)
{
System.out.println("PROCEEDED");
}
}
}
Best Answer
Yes, that's expected. You should not compare object references with
==
or!=
. You should use.equals(..)
instead, or better - use the primitiveint
rather thanInteger
.The thing is, values up to 128 are cached, and the JVM gives you the same objects (hence the reference comparison works). Above 128 it creates a new instance. Look at the javadoc of
Integer.valueOf(int)
(which is what happens behind the scene)