1

Аудиоплеер. Нажатие кнопки - запускает поток, а в нем музыку. Но как эту музыку остановить другой кнопкой stopButton?

public class MainActivity extends AppCompatActivity {

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);
       Thread myThread;

        Button playButton = (Button) findViewById(R.id.play);
        Button stopButton = (Button) findViewById(R.id.stop);

        final Runnable runaudio =  new Runnable() {
            public void run() {
                MediaPlayer media = new MediaPlayer();
                media = MediaPlayer.create(MainActivity.this, R.raw.arena);
                media.start();}};


        playButton.setOnClickListener(new View.OnClickListener() {
            public void onClick(View view){

                Thread myThread = new Thread(runaudio);

                myThread.start();
            }
        });

        stopButton.setOnClickListener(new View.OnClickListener() {
            public void onClick(View view){

            }
        });

    }
}

1 ответ 1

2

У вас не правильный подход к написанию плеера изначально. Проигрывание музыки должно выноситься в Service с пометкой foreground() и соответствующим уведомлением, иначе при нехватке ресурсов ваш плеер просто будет подчищен осью а пользователь будет негодовать.

И что уж говорить о том, что во время входящего вызова\каком либо другом действии плеер будет постоянно проигрываться. Если сейчас вы включите стандартный плеер, и ваш одновременно - они будут играть тоже одновременно.

Такого быть не должно.

Используйте сервис и данная проблема сама отпадёт.

5
  • а если в такой ситуации , но вместо плеера будет что другое, и понадобится так завершить поток, как мне можно будет это сделать?
    – PeteGr
    10 авг 2016 в 6:48
  • Это так же плохая практика, есть метод просящий поток прекратить свою работу - но это совершенно не значит, что он обязательно это сделает. Посему рассчитывать на это не стоит, и я бы посоветовал вам искать другое решение.
    – Silento
    10 авг 2016 в 7:41
  • создайте volatile static флаг в котором храните состояние плеера (проигрывать/не проигрывать). И проверяйте этот флаг.
    – DimXenon
    10 авг 2016 в 7:43
  • @DimXenon только не static. 10 авг 2016 в 7:56
  • На уровне класса флаг, почему бы и нет? Если это будет именно плеер, с именно одним потоком проигрывания и одним - управления - вполне себе вариант. Если потоков много - можно коллекцию потокобезопасную использовать с флагами для каждого потока (карта с инструкциями и ключами-идентификаторами потоков). В индивидуальном volatile флаге в каждом экземпляре плеера (для текущей задачи) я не вижу необходимости.
    – DimXenon
    10 авг 2016 в 8:22

Ваш ответ

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

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