按下时没有响应
我是新来的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;
}
}
}
}
任何人都可以看到我做了什么错?
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);
}
}
是的,它现在正在工作,但现在出现了一个新问题。如果我使用任何操作(+, - ,/,*),我的应用程序会“强制关闭”。任何想法为什么? – 2014-10-10 15:42:46
我们不是在这里调试你的代码。你的原始问题已被回答。如果你有另一个研究得很好的问题,然后创建一个新的线程! – 2014-10-10 15:46:46
好的,我会尝试其他的概率。并给予你的答案是正确的。 Tnx很多。 :) – 2014-10-10 15:48:25
您需要实例监听器的实例,目前它为空:
btnClick = new ButtonClickListener();
我没有看到你有btnClick
随时随地初始化。你应该有这样的事情
btnClick = new ButtonClickListener();
地方之前设置的听众。
以上是正确的答案,但你并不需要创建一个扩展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
}
}
只是为了显示你的所有可能的方式来解决这个问题。我更喜欢在类上实现监听器,因此您不存储变量,但可以根据自己的情况采取最佳措施。
Tnx。我会在完成我的项目后尝试。 :) – 2014-10-10 15:53:24
太多无用的代码!将您的代码降低到可以再现您问题的最低版本!见:http://sscce.org/ – 2014-10-10 15:34:10
正如haywire所提到的,这里肯定有太多的代码。将来,尽量减少到最低限度*,我们需要开始。如有需要,我们随时可以提出更多要求。我已经为您删除了不必要的代码,因为它可能会吸引downvotes并可能为您投票。这也使得这个问题更易于阅读。 – codeMagic 2014-10-10 15:41:56