onClick和setOnClickListener()导致我的应用程序不幸停止

时间:2014-02-12 16:44:39

标签: java android

这是Java代码

package com.example.getyourhandsoffmy;

import android.media.MediaPlayer;
import android.os.Bundle;
import android.app.Activity;
import android.view.Menu;
import android.view.View;
import android.widget.Button;

public class MainActivity extends Activity {
    MediaPlayer mediaPlayer = MediaPlayer.create(this, R.raw.bells);
    final Button button = (Button) findViewById(R.id.button1);
    public void bells(View view) {
         mediaPlayer.start();
     }
    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);

    }

    @Override
    public boolean onCreateOptionsMenu(Menu menu) {
        // Inflate the menu; this adds items to the action bar if it is present.
        getMenuInflater().inflate(R.menu.main, menu);
        return true;
    }

}

这是按钮XML

<Button
        android:id="@+id/button1"
        android:layout_width="fill_parent"
        android:layout_height="wrap_content"
        android:layout_alignParentTop="true"
        android:layout_centerHorizontal="true"
        android:layout_marginTop="60dp"
        android:text="Button" 
        android:onClick="bells"/>

所以问题是,只要我将onClick函数添加到我的代码或setOnClickListener并尝试编译并运行代码,它就会告诉我,不幸的是我的应用程序已停止

3 个答案:

答案 0 :(得分:4)

你在这里做得太早了:

MediaPlayer mediaPlayer = MediaPlayer.create(this, R.raw.bells);
final Button button = (Button) findViewById(R.id.button1);

初始化成员变量时,您无法将活动this用作Context。这会导致MediaPlayer.create()崩溃。

findViewById()此处将返回null,因为它在setContentView()之前执行。但是,这并不重要,因为您在所发布的代码中的任何位置都没有使用button变量。

解决方案:将两次初始化移至onCreate()

答案 1 :(得分:1)

你需要在setContentView

之后的onCreate里面
MediaPlayer mediaPlayer = MediaPlayer.create(this, R.raw.bells);
final Button button = (Button) findViewById(R.id.button1);

findViewById在当前膨胀的布局中查找视图。

答案 2 :(得分:1)

考虑Android官方文档: http://developer.android.com/guide/topics/ui/controls/button.html

  

你在android:onClick属性中声明的方法必须有一个   签名完全如上所示。具体来说,该方法必须:

     

公开

     

返回无效

     

将视图定义为唯一参数(这将是   是被点击的视图

1)在onCreate()中的setContentView之后初始化你的按钮和MediaPlayer;

2)添加此方法:

public void bells(View view) {}