Как синхронизировать метод?

Question

[Артем]

Имею задачу синхронизировать метод. Пробовал с wait()/notify() и семафорами. В итоге, блокировка срабатывает как надо, но при этом метод onCompletion() не вызывается и, как следствие, блокировка не снимается. Как быть?

Object lock = new Object();

    public void playSound(String path) {
        Uri myUri = Uri.parse(path);
        MediaPlayer mp = MediaPlayer.create(context, myUri);
        mp.setOnCompletionListener(new MediaPlayer.OnCompletionListener() {
            @Override
            public void onCompletion(MediaPlayer mp) {
                mp.stop();
                mp.release();
                lock.notifyAll();
            }
        });
        mp.start();
        
            try {
                lock.wait();
            } catch (InterruptedException e) {
                e.printStackTrace();
            }
    }

Answer 1

[LeoCcoder]

Документация:
«In order to receive the respective callback associated with these listeners, applications are required to create MediaPlayer objects on a thread with its own Looper running»

Если в этом предложении есть незнакомые слова — советую все изучить, тогда будет понятно почему не работает.

Скорее всего lock.wait(); блокирует поток и сообщения больше не обрабатываются, поэтому никаких колбеков бы больше не получаете (если у вас вообще есть Looper в потоке, покажите в каком потоке вы создаете плеер). Если у вас нет Looper'a в потоке, то даже без блокировок никаких колбеков вы не получите назад.

Answer 2

[AR1ES]

Хм, если речь идет именно о синхронизации метода, то просто объявление его как synchronized не подходит?

Comments

[Артем]

В таком случае он просто не ждет окончания проигрывания звукового файла.

[AR1ES]

Ну да, сначала я как-то особо не смотрел исходники, ответ дал чисто из описания вопроса.
Такс, ну вообще ведь возможно возникновение ошибки, поэтому надо определить еще: mp.setOnErrorListener… Хотя это вряд ли решит проблему, как мне кажется решит проблему.

[AR1ES]

Ну написал… Называется, не отрывайся от работы. Оборвал на полуслове и потом решил дописать.

[Артем]

Нет, setOnErrorListener также не вызывается.

[AR1ES]

Гугление дало 2 возможных варианта:
1. Определить отдельно класс, реализующий интерфейс OnCompletionListener, и работать уже с ним.
2. Если юзается эмулятор, то проблема может быть в нем, и тогда стоит попробовать запустить на реальном устройстве.

[Артем]

Проверил оба вариант и оба не работают :(

Answer 3

[Артем]

Если у вас нет Looper'a в потоке, то даже без блокировок никаких колбеков вы не получите назад

Looper есть и без блокировки OnCompletionListener() работает нормально.

Скорее всего lock.wait(); блокирует поток и сообщения больше не обрабатываются, поэтому никаких колбеков бы больше не получаете

Очень на то похоже. А как это исправить?

Comments

[LeoCcoder]

использовать другой способ синхронизации, который не будет блокировать поток, в котором вы создали MediaPlayer. Решение зависит от того, что вам надо.

Пусть, например, блокируются другие потоки, которые хотят, дать задание плееру (что-то проиграть). И ждут, пока плеер не освободится.

Можно сделать очередь и складывать туда Uri, того, что надо проиграть, а плеер по мере освобождения будет брать задания из очереди и играть.

Answer 4

[Артем]

Со списком Uri мне не подходит: между звуками могут вызываться другие методы.
Моя же задача состоит в том, что мне нужно отследить конец проигрывания мелодии и только после этого производить дальнейшие действия. Метод этот вызывается с помощью Method.invoke(). Соответственно, ждать нужно либо внутри метода либо сразу после invoke(), других вариантов нет. Но это один и тот же поток ведь…