Android: Unfortunately, Sample has stopped

Question

I’m creating an application but in case it gives problem when running to a certain point, I want to transform geoi = 0; and Mati = 0; , and when I add any number in edittext it will put that number inside the geoi and Mati int which in the case is equal to 0. What is the error in my code ? Why are you giving Unfortunately ?

package com.gustavo.sample;

import android.os.Bundle;
import android.support.v7.app.ActionBarActivity;
import android.view.View;
import android.widget.Button;
import android.widget.CheckBox;
import android.widget.EditText;
import android.widget.TextView;


public class MainActivity extends ActionBarActivity {

    CheckBox g;
    CheckBox m;
    Button send;
    TextView say;
    EditText num;

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);

        bacon();

        send.setOnClickListener(new View.OnClickListener() {

            @Override
            public void onClick(View v) {

                String counter = num.getText().toString();

                int counterAsInt = Integer.parseInt(counter);

                int geoi = 0;
                int mati = 0;           



                  if(g.isChecked()){
                      geoi += counterAsInt;
                      say.setText("Geo" + geoi);
                  }
                  else if(m.isChecked()){  
                    mati += counterAsInt;
                    say.setText("Math" + mati);
                  }

            }
        });
    }
    public void bacon() {
        g = (CheckBox)findViewById(R.id.checkBox1);
        m = (CheckBox)findViewById(R.id.checkBox2);
        send = (Button)findViewById(R.id.button1);
        say = (TextView)findViewById(R.id.textView1);
        num = (EditText)findViewById(R.id.editText1);   
    }

}

Answer 1

By Stacktrace posted, the problem is that it is converting a String in format float for a int.

I don’t know if your intention is to treat the field value as integer or decimal.

To treat as integer number you must do:

int counterAsInt = (int) Float.parseFloat(counter);

With that, the value String of counter will be converted to decimal number (float) and then converted into one casting, with this it will discard the decimal part.

If you want to treat as decimal number you should ignore casting and use a variable of type float, thus:

float counterAsFloat = Float.parseFloat(counter);

And then change the type of the other variables int used in the calculation for float also:

float geof = 0f;
float matf = 0f;           

if(g.isChecked()){
    geof += counterAsFloat;
    say.setText("Geo" + Float.toString(geof));
} else if(m.isChecked()){  
    matf += counterAsFloat;
    say.setText("Math" + Float.toString(matf));
}

The use of Float.toString is optional, but makes it clear that it is converting a variable of type float for its representation in String.