EditText und Fehler beim parsen

Timee · 6. November 2012

Hallo Gemeinde,

ich versuche aus 2 EditText Feldern die Inhalte auszuleden, in Integer umzuwandeln, zu berechnen un danschließemd als TOAST auszugeben.
Leider bleibt das Ergebnis beim berechnen immer 0.

EditText

Java

&lt;EditText
    	android:id="@+id/tf_angebot"
    	android:layout_width="wrap_content"
    	android:layout_height="wrap_content"
    	android:layout_below="@+id/txt_info_handelsrechner"
    	android:layout_centerHorizontal="true"
    	android:layout_marginTop="30dp"
    	android:ems="10"
    	android:inputType="number"
    	android:digits="0123456789"
    	android:text="@string/tf_angebot" &gt;
		&lt;requestFocus /&gt;
	&lt;/EditText&gt;


	&lt;EditText
    	android:id="@+id/tf_kosten"
    	android:layout_width="wrap_content"
    	android:layout_height="wrap_content"
    	android:layout_alignLeft="@+id/txt_angebot"
    	android:layout_below="@+id/txt_angebot"
    	android:layout_marginTop="20dp"
    	android:ems="10"
    	android:inputType="number"
    	android:digits="0123456789"
    	android:text="@string/tf_kosten" /&gt;

Alles anzeigen

Methode HandelBerechnen()

Java

public void HandelBerechnen(View view) {
		
		TextView textView1;
		TextView textView2;
		int Berechnung = 0;
		String Angebot;
		String Kosten;
		String Ergebnis = "";
		
		// Eingabefelder auslesen		
		textView1 = (TextView) findViewById(R.id.tf_angebot);
		textView2 = (TextView) findViewById(R.id.tf_kosten);
		Angebot = textView1.getText().toString();
		Kosten  = textView2.getText().toString();
		
		
		// Handel auf 400er Stacks berechnen
		Berechnung = (Integer.parseInt(Kosten) / Integer.parseInt(Angebot)) * 400;
		
		// Wenn kein Fehler bein Stringparsen, als TOAST ausgeben
		try {
			Ergebnis = String.valueOf(Berechnung);
			//Ergebnis = "Angebot: " + Angebot + "\nKosten: " + Kosten;
			Toast toast = Toast.makeText(this, Ergebnis, Toast.LENGTH_LONG);
			//Toast toast = Toast.makeText(this, "Ein 400er Stack kostet dich: " + Ergebnis, Toast.LENGTH_LONG);
			toast.setGravity(Gravity.CENTER, 0, 0);
			toast.show();
		}
		// Wenn Fehler, Fehlermeldung als TOAST ausgeben
		catch (Exception e) {
			Toast toast = Toast.makeText(this, "Es ist ein Fehler aufgetreten!", Toast.LENGTH_LONG);
			toast.setGravity(Gravity.CENTER, 0, 0);
			toast.show();
		}
	}

Alles anzeigen

Danke für die Hilfe,

Gruß Timee

killphil75 · 6. November 2012

Naja wenn du Integer durch Integer teilst kommt auch Integer raus.

10 / 100 = 0.1 reale Welt

10/ 100 = 0 Integer Welt

Ich glaube du willst eher Double benutzen wegen der Kommastellen.

titus · 7. November 2012

Hi Timee,

noch als Tipp:

1.) in Java hat sich die Naming Conventions durchgesetzt das Variablen (zbsp. String Objekte oder int Variablen) mit einem kleinen Buchstaben beginnen. Wenn man sich an diese Convention hält ist es vorallem für andere leichter deinen Code zu lesen bzw zu vestehen.

2.) im try/catch solltest du dir immer aber aufjedenfall wärend der Entwicklung den Stacktrace oder die Message ausgeben lassen. EIn einfacher Toast mit "Es ist ein Fehler aufgetreten" wird dir bei Fehlersuchen wenig helfen.

wie gesagt sind nur Tipps.

Mfg Titus

Timee · 7. November 2012

Danke für die Hilfe und die Tipps.
Tipps werden gerne angenommen und auch versucht umzusetzten.
Mit double statt int habe ich meine Berechnungen hinbekommen.

Gruß
Timee

P.S. anbei noch das Ergebnis zum Nachschlagen oder zum Verbessern

Java

public class HandelsrechnerActivity extends Activity {


	@Override
	public void onCreate(Bundle savedInstanceState) {
    	super.onCreate(savedInstanceState);
    	setContentView(R.layout.activity_handelsrechner);
	}
	
	public boolean Zahlenprüfung (String s) {
		// Prüfen ob String LEER ist
		if (s.equals("")) {
			return false;
		}
		else {
			return true;
		}
	}
	
	public void HandelBerechnen(View view) {
		
		TextView textView1;
		TextView textView2;
		double berechnung = 0;
		String angebot;
		String kosten;
		
		// Eingabefelder auslesen		
		textView1 = (TextView) findViewById(R.id.tf_angebot);
		textView2 = (TextView) findViewById(R.id.tf_kosten);
		angebot = textView1.getText().toString();
		kosten  = textView2.getText().toString();
		
		// Prüfung auf NICHT LEER
		if (Zahlenprüfung(angebot) == true && Zahlenprüfung(kosten) == true) {
			try {
				// Handel auf 400er Stacks berechnen
        		berechnung = (Double.parseDouble(kosten) / Double.parseDouble(angebot)) * 400;
        		// Runden auf 2 Nachkommastellen
        		berechnung = Math.round(berechnung*100)/100.0;
        		
        		// Werte &lt;= 0 nicht anzeigen, da sinnlos
        		if (berechnung &lt;= 0) {
        			Toast toast = Toast.makeText(this, "Das Ergebnis ist &lt;= 0 und somit nutzlos!", Toast.LENGTH_SHORT);
        			toast.setGravity(Gravity.CENTER, 0, 0);
        			toast.show();
        		}
        		else {
        			// Ausgabe der Berechnung
        			Toast toast = Toast.makeText(this, "Ein 400er Stack kosten dich " + String.valueOf(berechnung), Toast.LENGTH_LONG);
        			toast.setGravity(Gravity.CENTER, 0, 0);
        			toast.show();
        		}
			}
			catch (Exception e) {
    			Toast toast = Toast.makeText(this, "Bei der Berechnung ist ein Fehler aufgetreten!", Toast.LENGTH_SHORT);
    			toast.setGravity(Gravity.CENTER, 0, 0);
    			toast.show();
			}
		}
		else {
			Toast toast = Toast.makeText(this, "Eines oder beide Felder sind nicht ausgefüllt!", Toast.LENGTH_SHORT);
			toast.setGravity(Gravity.CENTER, 0, 0);
			toast.show();
		}
	}


}

Alles anzeigen

killphil75 · 7. November 2012

Wenn du lernwillig bist, dann noch ein kleiner Tipp.

Jedes mal wenn du jetzt eine Berechnung ausführst, musst du dir die Views (Textfeld) ect holen.
Bei 2 Feldern ist das auch noch nicht so performance relevant, aber wenn es mehr werden kann das schon bremsen.

Deswegen hol dir doch die Views in deiner OnCreate Methode (das layout hat ja auch so lange Bestehen),
packe es in eine Variable und benutze die... das spart dir ein paar Aufrufe von findView

Java

private TextView textView1;
 private TextView textView2;


	@Override
	public void onCreate(Bundle savedInstanceState) {
    	super.onCreate(savedInstanceState);
    	setContentView(R.layout.activity_handelsrechner);




		// Eingabefelder auslesen		
		textView1 = (TextView) findViewById(R.id.tf_angebot);
		textView2 = (TextView) findViewById(R.id.tf_kosten);
 }

Alles anzeigen

Timee · 8. November 2012

Danke killphil75,

hab deinen Tipp berücksichtigt und die Views "ausgelagert".

EditText und Fehler beim parsen

Jetzt mitmachen!

Teilen

Tags

Benutzer online in diesem Thema