Анализ доступа к ArrayList

[hex1r0]

я не знаю что у тебя этот класс делает и что хранится _normalSkillTree и где используется/изменяется, если там статика, а ты достаешь оттуда лист и потом его модифицируешь, конечно будет конкурентка и еще куча левых багов

[ALF.]

Сообщение от hex1r0

я не знаю что у тебя этот класс делает и что хранится _normalSkillTree и где используется/изменяется, если там статика, а ты достаешь оттуда лист и потом его модифицируешь, конечно будет конкурентка и еще куча левых багов

	private static TIntObjectHashMap<List<SkillLearn>> _normalSkillTree = new TIntObjectHashMap<List<SkillLearn>>();

[hex1r0]

сделай как я выше написал, должно помочь

[ALF.]

В общем всем спасибо.
Психонул, сделал ConcurrentHashMap

[hex1r0]

при чем там хешмап, если ошибка в листе

[VISTALL]

оффтоп:

Свернуть ↑

Сообщение от hex1r0

при чем там хешмап, если ошибка в листе

без очков невидит

Свернуть ↑Развернуть ↓

[bloodshed]

Сообщение от hex1r0

тут видимо ошибка в алгоритме (это при условии что _normalSkillTree должна хранить/хранит статические данные)

PHP код:

public Collection<SkillLearn> getAvailableAllSkills(Player player)
    {
        final Collection<SkillLearn> skills;
        // тут ошибка видимо
        // skills = _normalSkillTree.get(player.getActiveClassId());
        skills = new ArrayList<>(_normalSkillTree.get(player.getActiveClassId()));
        if (player.isAwaking()) 


тут как раз то, что тебе нужно.
у тебя же ошибка:
ты берёшь ArrayList из _normalSkillTree, он у тебя один для всех игроков с одним classId по ходу и зачем то туда добавляешь потом скилы, по этому и выходит: тут ты добавляешь, а в другом потоке бежишь по коллекции и у тебя вылазит ConcurrentModificationException. Создавай для таких манипуляций отдельный список.

[hex1r0]

причем конкурентка тут еще не самое страшное, а страшное то, что у персов будет по 100500 скилоф

[ALF.]

Про ConcurrentHashMap - протупил)
Просто в другой сборке такое увидел, вот она меня и сбила с толку)

Точнее там FastList.setShared

Добавлено через 2 минуты

Сообщение от bloodshed

тут как раз то, что тебе нужно.
у тебя же ошибка:
ты берёшь ArrayList из _normalSkillTree, он у тебя один для всех игроков с одним classId по ходу и зачем то туда добавляешь потом скилы, по этому и выходит: тут ты добавляешь, а в другом потоке бежишь по коллекции и у тебя вылазит ConcurrentModificationException. Создавай для таких манипуляций отдельный список.

Благодарю)

[n3k0nation]

Сообщение от ALFOS

Про ConcurrentHashMap - протупил)
Просто в другой сборке такое увидел, вот она меня и сбила с толку)

Точнее там FastList.setShared

Javadoc используемых фреймворков читать не пробовали?
P.S: ваша версия javolution безбожно устарела, обновитесь.