Telegram bot Java ConcurrentModificationException [duplicate]

Question

I want that after writing any message, the user gets into the list. The first time it works, but as soon as another user writes, for some reason it shows null.

@Override
public void onUpdateReceived(Update update) {
    if (update.hasMessage() && update.getMessage().hasText()) {
        getLogs(update);
        addUser(update.getMessage().getChat().getUserName(), update.getMessage().getChatId());
    }
}

private void addUser(String name, long chatId) {
    Buddy buddy;
    if (!users.isEmpty()) {
        for (Buddy user : users) {
            if (name.equals(user.getName()) && chatId == user.getNameId()) {
                System.out.println("Exists");
            } else {
                buddy = new Buddy(name, chatId);
                users.add(buddy);
            }
        }
    } else {
        users.add(new Buddy(name, chatId));
    }
    System.out.println(users.toString());
}

I expected that after writing from any user, a unique Buddy would be added to the list, but for some reason this did not happen.

Catching such a mistake: java.util.ConcurrentModificationException: null

Off topic, you are adding a new Buddy for each existing item in users. — tgdavies
– tgdavies, Commented Feb 12, 2023 at 22:25

Bill Mair · Accepted Answer · 2023-02-12 22:56:27Z

1

You are iterating through users:

for (Buddy user : users) {

while changing the users list:

buddy = new Buddy(name, chatId);
users.add(buddy);

== ConcurrentModificationException

You have to add the user, if not found, after you have finished the for loop.

answered Feb 12, 2023 at 22:56

Bill Mair

1,1618 silver badges16 bronze badges

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

1 Comment

Mark Rotteveel Over a year ago

The alternative is to explicitly use the list iterator and add it through the list iterator.

Collectives™ on Stack Overflow

Telegram bot Java ConcurrentModificationException [duplicate]

1 Answer 1

1 Comment

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

1 Comment

Linked

Related