2014-10-10 57 views
0

我是新來的android開發,我正在做一個簡單的計算器。一切工作正常,沒有錯誤顯示,但Buttons不工作,只要我按他們。按下時沒有響應

我的目標SDK是API 19: Android 4.4(Kitkat)並編譯爲API 20: Android 4.4(KitKat Wear)。這裏是我的代碼,

MainActivity.java

package edu.shihank.mycalcv2; 

import android.support.v7.app.ActionBarActivity; 
import android.os.Bundle; 
import android.view.Menu; 
import android.view.MenuItem; 
import android.view.View; 
import android.view.View.OnClickListener; 
import android.widget.Button; 
import android.widget.TextView; 


public class MainActivity extends ActionBarActivity { 
public TextView tv; 
public float numbBf; 
public String Operation; 
public ButtonClickListener btnClick; 

@Override 
protected void onCreate(Bundle savedInstanceState) { 
    super.onCreate(savedInstanceState); 
    setContentView(R.layout.activity_main); 
    tv = (TextView)findViewById(R.id.tv); 
    int idList[] = {R.id.zero, R.id.one, R.id.two, R.id.three, R.id.four, 
      R.id.five, R.id.six, R.id.seven, R.id.eight, R.id.nine, R.id.add, 
      R.id.sub, R.id.div, R.id.multi, R.id.equal, R.id.clear, R.id.history}; 

    for(int id:idList){ 
     View v = (View)findViewById(id); 
     v.setOnClickListener(btnClick); 
    } 
} 


@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; 
} 

@Override 
public boolean onOptionsItemSelected(MenuItem item) { 
    // Handle action bar item clicks here. The action bar will 
    // automatically handle clicks on the Home/Up button, so long 
    // as you specify a parent activity in AndroidManifest.xml. 
    int id = item.getItemId(); 
    if (id == R.id.action_settings) { 
     return true; 
    } 
    return super.onOptionsItemSelected(item); 
} 

public void getKeyboard(String str) { 
    String tvNow = tv.getText().toString(); 
    tvNow += str; 
    tv.setText(tvNow); 
} 

public void theResult() { 
    float numbAf = Float.parseFloat(tv.getText().toString()); 
    float result = 0; 
    if(Operation.equals("+")){ 
     result = numbAf + numbBf; 
    } 
    if(Operation.equals("-")){ 
     result = numbAf - numbBf; 
    } 
    if(Operation.equals("*")){ 
     result = numbAf * numbBf; 
    } 
    if(Operation.equals("/")){ 
     result = numbAf/numbBf; 
    } 
    tv.setText(String.valueOf(result)); 
} 

public void theMath(String str) { 
    numbBf = Float.parseFloat(tv.getText().toString()); 
    Operation = str; 
    tv.setText("0"); 
} 

private class ButtonClickListener implements OnClickListener{ 
    @Override 
    public void onClick(View v) {  
     switch(v.getId()){ 
     case R.id.clear: 
      tv.setText("0"); 
      numbBf = 0; 
      Operation = ""; 
      break; 
     case R.id.add: 
      theMath("+"); 
      break; 
     case R.id.sub: 
      theMath("-"); 
      break; 
     case R.id.div: 
      theMath("/"); 
      break; 
     case R.id.multi: 
      theMath("*"); 
      break; 
     case R.id.equal: 
      theResult(); 
      break; 
     default: 
      String numb = ((Button) v).getText().toString(); 
      getKeyboard(numb); 
      break; 
     } 

    } 
} 
} 

任何人都可以看到我做了什麼錯?

+2

太多無用的代碼!將您的代碼降低到可以再現您問題的最低版本!見:http://sscce.org/ – 2014-10-10 15:34:10

+1

正如haywire所提到的,這裏肯定有太多的代碼。將來,儘量減少到最低限度*,我們需要開始。如有需要,我們隨時可以提出更多要求。我已經爲您刪除了不必要的代碼,因爲它可能會吸引downvotes並可能爲您投票。這也使得這個問題更易於閱讀。 – codeMagic 2014-10-10 15:41:56

回答

1

btnClick尚未初始化!試試這個:

@Override 
protected void onCreate(Bundle savedInstanceState) { 
    super.onCreate(savedInstanceState); 
    setContentView(R.layout.activity_main); 
    tv = (TextView)findViewById(R.id.tv); 
    int idList[] = {R.id.zero, R.id.one, R.id.two, R.id.three, R.id.four, 
     R.id.five, R.id.six, R.id.seven, R.id.eight, R.id.nine, R.id.add, 
     R.id.sub, R.id.div, R.id.multi, R.id.equal, R.id.clear, R.id.history}; 

    btnClick = new ButtonClickListener(); // add this line 

    for(int id:idList){ 
     View v = (View)findViewById(id); 
     v.setOnClickListener(btnClick); 
    } 
} 
+0

是的,它現在正在工作,但現在出現了一個新問題。如果我使用任何操作(+, - ,/,*),我的應用程序會「強制關閉」。任何想法爲什麼? – 2014-10-10 15:42:46

+1

我們不是在這裏調試你的代碼。你的原始問題已被回答。如果你有另一個研究得很好的問題,然後創建一個新的線程! – 2014-10-10 15:46:46

+0

好的,我會嘗試其他的概率。並給予你的答案是正確的。 Tnx很多。 :) – 2014-10-10 15:48:25

1

您需要實例監聽器的實例,目前它爲空:

btnClick = new ButtonClickListener(); 
2

我沒有看到你有btnClick隨時隨地初始化。你應該有這樣的事情

btnClick = new ButtonClickListener(); 

地方之前設置的聽衆。

1

以上是正確的答案,但你並不需要創建一個擴展OnClickListener如果不大幅修改類的新ButtonClickListener。下面是一個替代方法

private OnClickListener btnCLick = new OnClickListener() { 
    @Override 
    public onClick(View v) { 
     // Switch statement here 
    } 
} 

現在你實際上並不需要聲明btnClick = new ButtonClickListener()。而只是爲了好玩,另一種方式是您的主要活動課可以implement View.OnClickListner然後設置button.setOnClickListener(this);

public class MainActivity extends Activity implements View.OnClickListener { 

    @Override 
    public void onCreate(Bundle savedInstanceState) { 
     super.onCreate(savedInstanceState); 
     // Other stuff 
     btn.setOnClickListener(this); 
    } 

    @Override 
    public void onClick(View v) { 
     // Switch statement here 
    } 
} 

只是爲了顯示你的所有可能的方式來解決這個問題。我更喜歡在類上實現監聽器,因此您不存儲變量,但可以根據自己的情況採取最佳措施。

+0

Tnx。我會在完成我的項目後嘗試。 :) – 2014-10-10 15:53:24