2014-02-17 59 views
2

我被要求打開一個新問題。button1.setOnClickListener(this);導致空指針

我有一個滑動菜單顯示片段。其中一個片段是一組按鈕,它們在按下時播放音頻。

按鈕的OnClickListener導致logcat中的NullPointer錯誤,並導致應用程序崩潰。誰能告訴我爲什麼?這些按鈕正在通過正確的XML相關的片段引用,所以我不確定是什麼導致了錯誤。

這些行導致問題。

button1.setOnClickListener(this); 
button2.setOnClickListener(this); 

SoundsFragment.java具有託管在它的按鈕和代碼。

import android.app.Fragment; 
import android.os.Bundle; 
import android.view.LayoutInflater; 
import android.view.View; 
import android.view.ViewGroup; 
import android.app.Activity; 
import android.media.AudioManager; 
import android.media.MediaPlayer; 
import android.view.Menu; 
import android.view.View.OnClickListener; 
import android.widget.Button; 


public class SoundsFragment extends Fragment implements OnClickListener{ 

    public SoundsFragment(){} 

    @Override 
    public View onCreateView(LayoutInflater inflater, ViewGroup container, 
      Bundle savedInstanceState) { 

     View rootView = inflater.inflate(R.layout.fragment_sounds, container, false); 

     return rootView; 
    } 

    private MediaPlayer mp; 
    @Override 
    public void onCreate(Bundle savedInstanceState) { 
    super.onCreate(savedInstanceState); 
    setContentView(R.layout.fragment_sounds); 
    setVolumeControlStream(AudioManager.STREAM_MUSIC); 
    Button button1=(Button)findViewById(R.id.button_1); 
    Button button2=(Button)findViewById(R.id.button_2); 
    button1.setOnClickListener(this); 
    button2.setOnClickListener(this); 
    } 

    private Button findViewById(int button1) { 
     // TODO Auto-generated method stub 
     return null; 
    } 

    private void setVolumeControlStream(int streamMusic) { 
     // TODO Auto-generated method stub 

    } 

    private void setContentView(int activityMain) { 
     // TODO Auto-generated method stub 

    } 

    public void onClick(View v) { 
    int resId; 
    switch (v.getId()) { 
    case R.id.button_1: 
     resId = R.raw.a; 
     break; 
    case R.id.button_2: 
     resId = R.raw.b; 
     break; 
    default: 
     resId = R.raw.a; 
     break; 
    } 
    // Release any resources from previous MediaPlayer 
    if (mp != null) { 
     mp.release(); 
    } 
    // Create a new MediaPlayer to play this sound 
    mp = MediaPlayer.create(getActivity(), resId); 
    mp.start(); 
    } 


    @Override 
    public void onDestroy() { 
     if(null!=mp){ 
    mp.release(); 
     } 
     super.onDestroy(); 
    } 
} 

fragment_sounds.xml它具有按鈕和ID的。

<?xml version="1.0" encoding="utf-8"?> 
<RelativeLayout xmlns:android="http://schemas.android.com/apk/res/android" 
    android:layout_width="match_parent" 
    android:layout_height="match_parent" 
    android:background="@color/sounds_fragment"> 

    <Button 
     android:id="@+id/button_1" 
     android:layout_width="wrap_content" 
     android:layout_height="wrap_content" 
     android:layout_alignParentTop="true" 
     android:layout_marginRight="22dp" 
     android:layout_marginTop="36dp" 
     android:text="Play audio a.mp3" /> 

    <Button 
     android:id="@+id/button_2" 
     android:layout_width="wrap_content" 
     android:layout_height="wrap_content" 
     android:layout_alignLeft="@+id/button_1" 
     android:layout_below="@+id/button_1" 
     android:text="Play audio b.mp3" /> 

</RelativeLayout> 
+0

他們被蝕婚姻不幸福我的代碼後加。 – rossd

回答

3

更改爲

@Override 
public View onCreateView(LayoutInflater inflater, ViewGroup container, 
     Bundle savedInstanceState) { 

    View rootView = inflater.inflate(R.layout.fragment_sounds, container, false); 
    Button button1=(Button)rootView.findViewById(R.id.button_1); 
    Button button2=(Button)rootView.findViewById(R.id.button_2); 
    button1.setOnClickListener(this); 
    button2.setOnClickListener(this); 
    return rootView; 
} 

刪除此

private Button findViewById(int button1) { 
    // TODO Auto-generated method stub 
    return null; 
    } 

而且

private void setContentView(int activityMain) { 
    // TODO Auto-generated method stub 

} 

而且

setContentView(R.layout.fragment_sounds); 

onCreate

+0

完美謝謝。我有私人按鈕和私人空位,因爲eclipse不滿意我提供的代碼並希望添加它們。改變我的代碼後,你提供的工作正常。當我被允許時,將選擇你的答案爲正確的。 – rossd

+0

@rossd確認答案已被接受,如果它通過點擊答案旁邊的勾號來幫助 – Raghunandan

+0

將在約6分鐘內完成!你回答得太快了我必須等待:) – rossd

0

是,button1爲空,這就是爲什麼你會得到一個NullPointerException

0

你忘了@OverrideonClick(View v)前添加符號...

@Override 
public void onClick(View v) {