How to selection the elements in an ArrayList?

[MarMer]

Hi...
I make an arrayList to store my objects. And I want every time the user will put the new object, the program will selection the arrayList first. It will see whether the new objects are same with the object inside the arrayList or not. If it is same then that object will not be put into the arrayList and there will be a message. So every time the user press the submit button, it will make selection first. I also save that arrayList in file.
How to make it ?

[goldest]

How to make it ?

How are you making it at the moment? Or you want us to make it for you?

Anything attempted from your side?

Goldest

[Junky]

Use a Set instead as no duplicates are allowed.

[MarMer]

How are you making it at the moment? Or you want us to make it for you?

Anything attempted from your side?

Goldest

No...
I already try to make it...
Here is the code that I wrote...

ArrayList<Part> p = new ArrayList<Part>();

Data_File.read_object(p);
.
.
.
public boolean addParts(Object... objPris)
{
while(i < p.size()){
if(p.get(i).getId() == (String)objPris[2])
return false;
i++;
}

p.add(p1);
Data_File.write_object(p);
return true;

In the JFrame In the ActionPerformed(Action event e) ...
I write:
if(prisController.addParts(objPris))
JOptionPane.showMessageDialog(null, "Added Successfully");
else
JOptionPane.showMessageDialog(null, "ID have been assigned, Please change the ID");


But the result is always put the objects eventhough it's same ...

Thank you

[goldest]

if(p.get(i).getId() == (String)objPris[2])

Use equals method if you really want to compare Strings.

return false;
i++;

Anything after return is unreachable.

Is the entire code too big to post? This half code is confusing. And use code tags next time while posting code.

Goldest

[patriotsfan]

Use .equals() method instead of ==. The == looks if it refrences the same memory location while .equals() will compare value equality.

[MarMer]

[QUOTE=goldest;213967]Use equals method if you really want to compare Strings.

This is the solution, I just need to delete one 'equal' and my problem is solved :D

Thank you very much :)

Sorry, this is my first time to post with code


*Thank you all for answering my question

[goldest]

It's all right. Glad to know that you resolved your problem.

Come back again if you face any problems and the people here would be more than happy to help you out.

Goldest

[MarMer]

Thank you
Nice to meet you

[JosAH]

Allow me to repeat Junky's reply #3:

Use a Set instead as no duplicates are allowed.

kind regards,

Jos

[Junky]

Thanks Jos

I was beginning to think I was invisible too!

[MarMer]

Thanks Jos

I was beginning to think I was invisible too!

Thanks to you and to Jos who were willing to help me
Thank you for replied me

Nice to meet you all