Android Media Player 搜索栏在停止后启动时不更新

Question

我很难处理这个问题。

我有一个媒体播放器，我可以播放、暂停、停止、暂停后再次播放、停止......等等。 现在我想要一个 SeekBar 来提供可视组件。我的问题是： 当我第一次启动播放器时，一切正常。播放音乐并查找酒吧更新。当我暂停时也有效。

现在，如果我停止播放器并再次启动它，播放器将启动，run() 方法将执行，但搜索栏不会更新，很快应用程序就会给出无响应错误。 我在这里缺少什么？

run 方法是 Runnable 接口的实现，通过简单的日志，我可以看到它正在执行，即使在停止/播放情况之后也是如此。唯一似乎不起作用的是seek.setProgress(...)。 请帮忙？ :)

这是我的代码：

public class MediaPlayerTestingActivity extends Activity 
               implements OnClickListener, OnPreparedListener, Runnable {

private MediaPlayer mp;
private boolean paused = false;
private SeekBar seek;
private boolean threadStarted = false;;
private Thread thread = new Thread(this);


/** Called when the activity is first created. */
@Override
public void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    setContentView(R.layout.main);

    seek = (SeekBar)findViewById(R.id.seek);
    mp = new MediaPlayer();

    Button start = (Button)findViewById(R.id.start);
    start.setOnClickListener(this);
    Button pause = (Button)findViewById(R.id.pause);
    pause.setOnClickListener(this);
    Button stop = (Button)findViewById(R.id.stop);
    stop.setOnClickListener(this);

}

//click handlers
public void onClick(View v) 
{
    int buttonId = v.getId();

    switch (buttonId)
    {
        case R.id.start:
            if(mp != null)
            {
                if(!mp.isPlaying() && !paused)
                    prepareTostartPlayer();
                else
                    mp.start(); //if it was just paused
            }
            else
            {
                mp = new MediaPlayer();
                prepareTostartPlayer();
            }
            break;
        case R.id.pause:
            if(mp.!= null && mp.isPlaying())
            {   
                mp.pause();
                paused = true;
            }
            break;

        case R.id.stop:
            if(mp != null && mp.isPlaying())
            {
                mp.stop();
                mp.release();
                mp = null;
            }
            break;

        default:
            break;
    }
}

//When the player is ready to play
public void onPrepared(MediaPlayer arg0) 
{
    seek.setMax(mp.getDuration());
    mp.start(); 
    if(!threadStarted)
        thread.start();
    else
        thread.run();
}

//Method to prepare to start the player
private void prepareTostartPlayer()
{
    mp.setAudioStreamType(AudioManager.STREAM_MUSIC);
    try {
        mp.setDataSource("http://feedproxy.google.com/~r/rc-cadernetadecromos/~3/JH1kfZCmP3M/cdc_190112_2.mp3");
        mp.prepareAsync();
        mp.setOnPreparedListener(this);
    } catch (IllegalArgumentException e) {
        e.printStackTrace();
    } catch (IllegalStateException e) {
        e.printStackTrace();
    } catch (IOException e) {
        e.printStackTrace();
    }
}

public void run() 
{
    threadStarted   = true;
    int current = 0;
    while(mp != null && current < mp.getDuration())
    {
        try {
            current = mp.getCurrentPosition();
            seek.setProgress(current);
            Thread.sleep(1000);
        } catch (InterruptedException e) {
            e.printStackTrace();
        }
    }       
}

}

Answer 1

我在这里看到三个问题：

首先，这部分：

public void onPrepared(MediaPlayer arg0) 
{
    seek.setMax(mp.getDuration());
    mp.start(); 
    if(!threadStarted)
        thread.start();
    else
        thread.run();
}

第一次运行线程时，您在线程代码中将 threadStarted 设置为 true，但是当线程结束时，你永远不会将其设置回false。因此，在第一次运行后，它将永远为 true，因此上面的 else 将始终执行，按顺序运行播放器代码，从而阻塞您的用户界面。

第二，您正在从与 UI 线程不同的线程更新用户界面（SeekBar）。请参阅此相关问题，了解在这种情况下如何正确更新 UI：从线程更新 UI

<第三，这可能只是一个建议，但由于您的 threadStarted 变量在线程之间修改，因此最好将其声明为 易失性 以防止某些优化破坏你的代码：

private volatile boolean threadStarted = false;

Answer 2

根据我的经验以及 docs，Android 中的线程仅在程序的纯算法部分工作。任何在活动中使用它们的尝试，或者任何与 UI 连接的东西，都会出错。这样根本行不通。 永远你无法修复它！

请使用 AsyncTask（或子 Activity/Fragment 来实现更大的事情）。