"einfache" If-Abfrage

[clou]

Hallo,
ich bin absoluter Anfänger und lerne Java in einem Distanzkurs auf Schwedisch, das sind drei Probleme auf einmal

Ich habe folgenden Programminhalt:

int i=0;

//if (i==0) {
String sTal2 = JOptionPane.showInputDialog("Skriv in det andra talet");

String sOperator = JOptionPane.showInputDialog("Skriv in vilken operator ni vill använda\n + - * / %");

float tal1 = Integer.parseInt(sTal1);
float tal2 = Integer.parseInt(sTal2);

	if (sOperator.equals(operator[3])) {
		if (tal2==0) {

		System.out.println("sOperator: " + sOperator);
		i=0;
		}
	else i=1;
	}
i=1;
//}

Soweit so gut, abgesehen von der umständlichen Programmierung, denke ich. Aber es funzt und ich bin glücklich. Wenn ich jetzt aber die Kommentarzeichen, also die "//" in Zeile 4 und 21 entferne, funzt es nicht mehr. Warum? Es ist doch eine ganz einfache if-Abfrage, die nur ausgeführt werden soll, wenn i==0 ist. Was mache ich falsch?

Oder hilft es euch weiter, wenn ich den kompletten Code hier reinsetze? Aber meine Frage betrifft eigentlich nur den angesprochenen Teil.

LG
Clou

 

[Gast]

Was funktioniert denn nicht mehr?

 

[1mp]

Kann kein Syntax fehler erkennen nur ein kleinen Logik fehler aber der hat nichts mit der if abfrage zu tun!

Hab das programm gerade bei mir mal getestet! Also funktioniert einwandfrei hab zwar die berechnungen ausgeklammert aber die if abfrage funktioniert!

Zum kleine Logik fehler

   if (sOperator.equals(operator[3])) {
      if (tal2==0) {

      System.out.println("sOperator: " + sOperator);
      i=0;
      }
   else i=1;
   }
i=1;

du setzt i immer auf 1 am ende aber fragst ab wenn was passiert soll er es auf 0 setzten sonst auf 1 und nach dem setzt du es immer auf 1

MfG

 

[clou]

Hallo ihr zwei und Danke für eure Hilfe.

Was nicht funktioniert ist, dass das Prog nicht kompiliert wird. Nachdem bei 1mp alles funktioniert, habe ich jetzt genau diesen Teil des Progs in eine leere Datei kopiert. Funzt. Aaaaber: das Eigenartige ist, dass im Gesamtprogramm genau diese beiden Zeilen den Ärger machen. Ist die If-Abfrage drin (Zeile 20 und 40), wird das Prog nicht kompiliert. Ich setze mal den gesamten Code rein und hoffe auf eure Hilfe, für die ich mich schon jetzt bedanke.

//ett program som fungerar som en enkel miniräknare
import javax.swing.JOptionPane;

class Test01 {


public static void main(String [] args) {

	String [] operator = {"+", "-", "*", "/", "%"};

//genererar meddelande rutan
	JOptionPane.showMessageDialog(null,"Välkommen till min miniräknare");

//genererar inmatnings rutan
	String sTal1 = JOptionPane.showInputDialog("Skriv in det första talet");

	int i=0;

	if (i==0) {
//genererar inmatnings rutan
		String sTal2 = JOptionPane.showInputDialog("Skriv in det andra talet");

//genererar inmatnings rutan
		String sOperator = JOptionPane.showInputDialog("Skriv in vilken operator ni vill använda\n + - * / %");

//konverterar från String till int
		float tal1 = Integer.parseInt(sTal1);
		float tal2 = Integer.parseInt(sTal2);

			if (sOperator.equals(operator[3])) {
				if (tal2==0) {

//String sOperator = JOptionPane.showInputDialog("Kan inte dividera med 0. Skriv in vilken operator ni vill använda\n + - * / %");
				System.out.println("sOperator: " + sOperator);
				i=0;
				}
				else i=1;
			}
	}

float summa=0;
if (sOperator.equals(operator[0]))
	{
	summa=tal1+tal2;

	JOptionPane.showMessageDialog(null,tal1 + " " + operator[0]+ " " + tal2 + " = " + summa);//genererar meddelande rutan
	}
else if (sOperator.equals(operator[1]))
	{
	summa=tal1-tal2;

	JOptionPane.showMessageDialog(null,tal1 + " " + operator[1]+ " " + tal2 + " = " + summa);//genererar meddelande rutan

	}
else if (sOperator.equals(operator[2]))
	{
	summa=tal1*tal2;

	JOptionPane.showMessageDialog(null,tal1 + " " + operator[2]+ " " + tal2 + " = " + summa);//genererar meddelande rutan

	}
else if (sOperator.equals(operator[3]))
	{
	summa=tal1/tal2;

	JOptionPane.showMessageDialog(null,tal1 + " " + operator[3]+ " " + tal2 + " = " + summa);//genererar meddelande rutan

	}
else if (sOperator.equals(operator[4]))
	{
	summa=tal1%tal2;

	JOptionPane.showMessageDialog(null,tal1 + " " + operator[4]+ " " + tal2 + " = " + summa);//genererar meddelande rutan

	}

System.out.println("tal 1: " + tal1);
System.out.println("tal 2: " + tal2);
System.out.println("operator: " + sOperator);
System.out.println("summa: " + summa);
System.out.println("i: " + i);

System.exit(0);

	}
}

Was will ich überhaupt machen? Das Prog soll ein Minirechner sein. Man gibt zwei Zahlen ein und einen Operator. Ist die zweite Zahl==0 und der Operator ein "/" dann soll was passieren. Am Liebsten hätte ich, dass er eine Fehlermitteilung macht und zum Eingabefeld der zweiten Zahl zurück springt. Das hab ich nicht geschafft also versuchte ich die If-Abfrage und dann eben eine Fehlermeldung, dass die Eingabe so nicht geschehen darf und der Anwender nochmal den Operator eingeben soll (hier noch auf Kommentar gesetzt, Zeile 34, so lange die If nicht funzt).

Also, Leute: Ich hoffe, Ihr könnt mir weiterhelfen. Wieso funzt die If-Abfrage allein aber nicht im Gesamtprogramm?
Noch etwas: Brauch ich Zeile 84 überhaupt?

LG
Clou

P.S.: Der Logik-Fehler war in der Tat dämlich Danke dir 1mp.

 

[Landei]

1) Nimm:
float tal1 = Float.parseFloat(sTal1);

2) Hilft jetzt zwar nicht mit deinem Problem, aber hier ein kleiner Tipp: Nimm für die Operatoren enums:

class Test01 {
  enum Operator {
      PLUS, MINUS, MAL, DURCH, MOD;
      public String toString() {
         switch(this) {
          case PLUS: return "+"; 
          case MINUS: return "-"; 
          case MAL: return "*"; 
          case DURCH: return "/"; 
          case MOD: return "%";
         }  
      }
      public float summa(float a, float b) {
         switch(this) {
          case PLUS: return a + b; 
          case MINUS: return a - b; 
          case MAL: return a * b; 
          case DURCH: return a / c; 
          case MOD: return a % c;
         }  
      } 
  }
  ...
  
  for(Operator op : Operator.values()) {
     if(op.toString().equals(sOperator)) {
        summa=op.summa(tal1,tal2); 
        JOptionPane.showMessageDialog(null, tal1 + " " + op+ " " + tal2 + " = " + summa);
        break; 
     }
  } 
  ...

 

[Murray]

Ist die If-Abfrage drin (Zeile 20 und 40), wird das Prog nicht kompiliert.

In solchen Fällen wäre es hilfreich, die Fehlermeldung des Compilers zu posten.
Aber ich tippe mal ( zum Ausprobieren habe ich jetzt keine Lust): im Code-Block, der durch die if-Bedingung eingerahmt wird, werden Variablen deklariert; diese Variablen sind lediglich in diesem Block sichtbar - Du verwendest sie aber auch weiter unten, und das geht nicht.
Abhilfe: Variablen vor dem Block deklarieren

Noch etwas: Brauch ich Zeile 84 überhaupt?

Nein

 

[clou]

Landei: Vielen Dank. Allerdings kann ich das hier noch nicht anwenden, weil ich das noch nicht gelernt habe. Aber ich werde auf alle Fälle deine Ratschläge beherzigen, um mich weiterzuentwickeln.

Murray: Du bist der KÖÖÖÖNIIG der Java-Könige

Das war es, die Variabeln mussten vor der if deklariert werden. Jetzt funktioniert auch meine eigentlich geplante do while Schleife.

Danke, danke, danke euch allen bei der Hilfe. Aber keine Sorge: Ich komm wieder ;-)

 

[Gast]

ich finde auch zeile 19 etwas merkwürdig:

int i=0;

if (i==0)

?

 

[clou]

ich finde auch zeile 19 etwas merkwürdig:

int i=0;

if (i==0)

das sollte nur testen, ob die if funzt. Oder was meinst du kann ich besser machen? Ich habe das nun auch im fertigen Programm.

Das fertige Prog inkl. schwedischer Kommentare sieht nun so aus:

//ett program som fungerar som en enkel miniräknare

import javax.swing.JOptionPane;

class Projekt06 {

public static void main(String [] args) {

//deklarerar variabler
	String [] operator = {"+", "-", "*", "/", "%"};
	String sTal1;
	String sTal2;
	float tal1;
	float tal2;
	float summa=0;
	String sOperator;
	int i=0;

//genererar meddelande rutan
	JOptionPane.showMessageDialog(null,"Välkommen till min miniräknare");

//genererar inmatnings rutan
	sTal1 = JOptionPane.showInputDialog("Skriv in det första talet");

//slinga för att upprepa inmatnings rutorna
do {
//sätta i=0 igen för upprepningen av slingan
i=0;
//genererar inmatnings rutan
	sTal2 = JOptionPane.showInputDialog("Skriv in det andra talet");

//konverterar från String till int
	tal1 = Integer.parseInt(sTal1);
	tal2 = Integer.parseInt(sTal2);

//genererar inmatnings rutan
	sOperator = JOptionPane.showInputDialog("Skriv in vilken operator ni vill använda\n + - * / %");

//vilkorssats för att kontrollera andra talet=0 och operator="/"
	if (i==0) {
		if (sOperator.equals(operator[3])) {
			if (tal2==0) {

			JOptionPane.showMessageDialog(null,"Kan inte dividera med 0!");
			i=1;
			}
			else i=0;
		}
	}
//upprepar slingan bara när i==1
} while (i==1);

//vilkorssatser för att räkna och visa resultatet
if (sOperator.equals(operator[0]))
	{
	summa=tal1+tal2;

	JOptionPane.showMessageDialog(null,tal1 + " " + operator[0]+ " " + tal2 + " = " + summa);//genererar meddelande rutan
	}
else if (sOperator.equals(operator[1]))
	{
	summa=tal1-tal2;

	JOptionPane.showMessageDialog(null,tal1 + " " + operator[1]+ " " + tal2 + " = " + summa);//genererar meddelande rutan

	}
else if (sOperator.equals(operator[2]))
	{
	summa=tal1*tal2;

	JOptionPane.showMessageDialog(null,tal1 + " " + operator[2]+ " " + tal2 + " = " + summa);//genererar meddelande rutan

	}
else if (sOperator.equals(operator[3]))
	{
	summa=tal1/tal2;

	JOptionPane.showMessageDialog(null,tal1 + " " + operator[3]+ " " + tal2 + " = " + summa);//genererar meddelande rutan

	}
else if (sOperator.equals(operator[4]))
	{
	summa=tal1%tal2;

	JOptionPane.showMessageDialog(null,tal1 + " " + operator[4]+ " " + tal2 + " = " + summa);//genererar meddelande rutan

	}
}
}

Ich weiß, es ist nicht sauber, aber für Vorschläge bin ich offen. Erstmal bin ich froh, dass es mit eurer Hilfe funktioniert. Die vielen if-else-Abfragen am Schluss kann man doch bestimmt sauberer lösen?

 

[1mp]

Du kannst dein JOptionpane

JOptionPane.showMessageDialog(null,tal1 + " " + operator[4]+ " " + tal2 + " = " + summa);//genererar meddelande rutan

Was du in jeder else if anweisung hast unten drunter schreiben nimmst einfach statt operator[4] dann sOperator Dann wäre dein Code um einiges kürzer!

Und für die ganzen if else gibt nen netten befehl der da heißt switch Einfach mal googeln da findest du zu 100% genug tutorials die zeigen wie das funktioniert mit switch zu arbeiten!

mfg

 

[clou]

Hallo 1mp,

danke für deinen Tipp. Jedoch gab es eine Aufgabe zu lösen und in dieser musste der Code

JOptionPane.showMessageDialog(null,tal1 + " " + operator[4]+ " " + tal2 + " = " + summa);

verwendet werden. Deshalb fällt dein sOperator-Vorschlag hier weg. Oder gibt es eine schönere Lösung?

Und das mit Switch habe ich versucht, ich war aber zu dämlich um das hinzukriegen :-(
Ich habe zum Testen diesen Code verwendet:

switch (operator) {
        case '+':	System.out.println("operator är +");
            break;

        case '-':	System.out.println("operator är -");
            break;

        case '*':	System.out.println("operator är *");
            break;

        case '/':	System.out.println("operator är /");
            break;

        case '%':	System.out.println("operator är %");
            break;

        default:	System.out.println("Eine der Möglichkeiten du Dummbeutel");
     }

Als Fehlermeldung kam dann diese:

G:\Java\Kurs\Projekt07.java:49: incompatible types
found : java.lang.String[]
required: int
switch (operator) {
^
1 error

Tool completed with exit code 1

 

[Landei]

Die switch-Variante ist fast richtig, aber du brauchst char anstatt String, und zwar in diesem Fall den ersten char im String, also die Position 0.

switch(sOperator.charAt(0)) {
...
}

 

[clou]

coool - vielen Dank Landei

 

[clou]

Hallo Landei,

vielen Dank nochmal, jetzt hatte ich Zeit, das auch umzusetzen und es funktioniert prima. Wenn du Lust und Zeit hast...

... warum funktioniert hier nicht

switch (operator.charAt(0)) {

sondern nur

switch (sOperator.charAt(0)) {

?

 

[SlaterB]

charAt ist eine Methode von String,
operator dagegen nach dem, was du nicht sagst und man sich auf vorherigen Postings zusammen sammeln darf, ein Array,

ein Array ist kein String

 

[clou]

danke dir und entschuldige, dass ich nicht ausführlicher war.