5

Пишу приложение, в котором помимо основного потока есть второстепенный.
Задача второго потока - держать запущенным цикл while.
Переменные объявлены глобально:

private volatile int lineNumber;
volatile public static ArrayList<CodeString> applicationSourceCode = new ArrayList<CodeString>();

applicationSourceCode объявлен в абстрактном классе WorkData.
Цикл в потоке прописан следующим образом:

while (lineNumber < WorkData.applicationSourceCode.size()) {
    if (isLineComplete) {
        isLineComplete = false;
        runOnUiThread(runnable);
    }
}

В самом цикле же отрабатывается метод (содержится в Runnable), который запускается в основном потоке, а не в текущем, и который работает с lineNumber. В конце метода lineNumber увеличивается на 1.
Также есть флаг isLineComplete, который устанавливается в true в конце этого самого метода:

line = WorkData.applicationSourceCode.get(lineNumber);
...
isLineComplete = true;
lineNumber++;

Проблема в том, что даже если lineNumber явно уйдет дальше, чем описано в условии цикла, то содержимое цикла всё равно выполнено. Что приведет к выходу за границы массива.

В чем заключается проблема?

8
  • volatile? Мне кажется в нем проблема. Уберите его с инициализации lineNumber. А вообще попробуйте выводить в логи эту переменную.
    – user194625
    19 янв 2017 в 5:48
  • Добавьте if (lineNumber >= WorkData.applicationSourceCode.size()) return; перед line = WorkData.applicationSourceCode.get(lineNumber);, и не будет выходов за границы списка.
    – Regent
    19 янв 2017 в 5:58
  • И ещё: у вас в while нет Thread.sleep? Если это так, то это крайне не хорошо: грузить процессор на ровном месте.
    – Regent
    19 янв 2017 в 6:00
  • @Regent , поставить условие в цикле конечно можно, но выполнение кода в таком цикле происходить не должно. Поэтому это не решение проблемы, а решение ее последствий. Sleep в коде нет.
    – matMoteX
    19 янв 2017 в 6:13
  • @СергейГрушин переменную наблюдал и в трассировке (мониторил именно в теле цикла), и в логе - она везде уходит за границы. Избавления от модификатора volatile проблему никак не решили.
    – matMoteX
    19 янв 2017 в 6:13

1 ответ 1

0

Вопрос откровенно говоря странный.

Если нужно ждать результата выполнения runOnUiThread чтобы потом заинкрементить lineNumber, то смысл создавать отдельный Runnable? Либо мы скармливаем каждую строку отдельному потоку и ждём когда все потоки отработают, либо бы анализируем строки в цикле не заморачиваясь с отдельными Runnable. Или имеет место быть грубая недосказанность в условии.

Всё что касается счетчиков в многопоточности, то лучше использовать Atomic (если конечно приложение не HighLoad).

Всё что касается дешёвых операций, таких как создание потоков в цикле, то лучше завернуть в synchronized или реализовать через singleton;

Очень смущает static в applicationSourceCode

Непонятно что представляет из себя isLineComplete (boolean или AtomicBoolean)

Пример ниже, не должен привести к выходу за границы массива.

public class Analyzer {

    public void verify(ArrayList<String> applicationSourceCode) {
        AtomicInteger line = new AtomicInteger(0);
        while (line.get() < applicationSourceCode.size()) {
            runOnUiThread(applicationSourceCode.get(line.getAndIncrement()));
        }
    }

    private void runOnUiThread(String line) {
        //создаём новый объект Runnable и скармливаем потоку line.
        //не задерживаясь выходим.
    }
}

Для ответов на такие вопросы лучше использовать Java VisualVM или на крайний случай debug в Intellij Idea.

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.