Java ArrayList issue

Question

I need some help with a line of code in a program that I am writing. I feel like it's going to be some stupid mistake but I just can't seem to figure out what it is...

ArrayList<Integer> knockSequence;               //Default knockSequence
ArrayList<ArrayList<Integer>> customSequences;  //used to store custom sequences for client after   first connection
ArrayList<ServerClient> connectedClients;       //List of connected clients
//...
public void giveNewSequence(ArrayList<Integer> newSequence, ServerClient client) //client MUST be in connectedClients in order for this to work
{
    customSequences.get(connectedClients.indexOf(client)) = newSequence;
}

Why won't the line "customSequences.get(......." work? The error I'm getting is saying that it is looking for a variable but a value is being found. Any feedback is appreciated

In the future, please post the exact error message.

Jon Skeet
– Jon Skeet

2013-12-12 18:16:20 +00:00
Commented Dec 12, 2013 at 18:16 — Jon Skeet
– Jon Skeet, Commented Dec 12, 2013 at 18:16
Sorry, first post here. Will remember to do that

Jcarosella10
– Jcarosella10

2013-12-12 18:19:15 +00:00
Commented Dec 12, 2013 at 18:19 — Jcarosella10
– Jcarosella10, Commented Dec 12, 2013 at 18:19

Jon Skeet · Accepted Answer · 2013-12-12 18:15:26Z

4

Why won't the line "customSequences.get(......." work?

You're trying to assign a value to the result of a method call. That's what doesn't work. You can only use the assignment operator with a variable.

I suspect you want:

customSequences.set(connectedClients.indexOf(client), newSequence);

You should also consider using a single collection with a composite type which contains the knock, custom sequence and connected client, rather than managing three separate collections where the values are related by index (which is what I suspect you've got here). You might want to use a map as well, rather than a list.

answered Dec 12, 2013 at 18:15

Jon Skeet

1.5m893 gold badges9.3k silver badges9.3k bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

Jcarosella10 Over a year ago

Thank you for the advice EDIT: the code you posted works, thanks a lot!

Collectives™ on Stack Overflow

Java ArrayList issue

1 Answer 1

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related