How to selection the elements in an ArrayList?

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 ?:(sweat):

Quote:

---

Originally Posted by MarMer

How to make it ?:(sweat):

---

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

Anything attempted from your side?

Goldest

Use a Set instead as no duplicates are allowed.

Quote:

---

Originally Posted by goldest

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

Anything attempted from your side?

Goldest

---

No... :(shake):
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 :)-:

Quote:

---

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

---

Use equals method if you really want to compare Strings.

Quote:

---

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

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

[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 :(blush):


*Thank you all for answering my question :)-::(happy):

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. :(grin):

Goldest

Thank you :D:
Nice to meet you :(hi):

Allow me to repeat Junky's reply #3:

Quote:

---

Originally Posted by Junky

Use a Set instead as no duplicates are allowed.

---

kind regards,

Jos

Thanks Jos

I was beginning to think I was invisible too!

Quote:

---

Originally Posted by Junky

Thanks Jos

I was beginning to think I was invisible too!

---

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

Nice to meet you all :D: