Wie kann ich diese If-Anweisung mit allem arbeiten lassen?

Question

Für mein Programm versuche ich, einen einfachen Rechner zu machen, der zwei Zahlen Addition, Subtraktion, Multiplikation und Division macht, aber ich treffe eine Straßensperre mit meiner if-Anweisung. Wie mache ich diese Anweisung mit meiner Auswahl-Zeichenfolge arbeiten, sondern auch mit meiner Addition, Subtraktion, Multiplikation und Division String? Ich verwende Java

import javax.swing.JOptionPane; 

public class Calculator 
{ 

public static void main (String args []) 
{ 
    String MainMenu = " Calculator Options \n\n1. Addition\n2" + 
    " Subtraction\n3. Multiplication\n4. Division\n5. Exit"; 

    menu (MainMenu); 
} 

public static String menu (String info) 
{ 
    String choice = JOptionPane.showInputDialog (info); 
    return choice; 

} 


{ 
if (choice.equals ("1")) 
    Add(); 
    else 
     if (choice.equals ("2")) 
     Sub(); 
      else 
      if (choice.equals ("3")) 
      Mult(); 
       else 
       if (choice.equals ("4")) 
       Div(); 
} 


public static int Addition (String Add) 
{ 
    String add1 = JOptionPane.showInputDialog ("Number 1 is?"); 
    int numadd1 = Integer.parseInt (add1); 

    String add2 = JOptionPane.showInputDialog ("Number 2 is?"); 
    int numadd2 = Integer.parseInt (add2); 

    int addEquals = numadd1 + numadd2; 

    JOptionPane.showMessageDialog (null, "The Answer Is: " + addEquals); 

    return addEquals; 
} 

public static int Subtraction (String Sub) 
{ 
    String sub1 = JOptionPane.showInputDialog ("Number 1 is?"); 
    int numsub1 = Integer.parseInt (sub1); 

    String sub2 = JOptionPane.showInputDialog ("Number 2 is?"); 
    int numsub2 = Integer.parseInt (sub2); 

    int subEquals = numsub1 - numsub2; 

    JOptionPane.showMessageDialog (null, "The Answer Is: " + subEquals); 

    return subEquals; 
} 

public static int Multiplication (String Mult) 
{ 
    String mult1 = JOptionPane.showInputDialog ("Number 1 is?"); 
    int nummult1 = Integer.parseInt (mult1); 

    String mult2 = JOptionPane.showInputDialog ("Number 2 is?"); 
    int nummult2 = Integer.parseInt (mult2); 

    int multEquals = nummult1 * nummult2; 

    JOptionPane.showMessageDialog (null, "The Answer Is: " + multEquals); 

    return multEquals; 
} 

public static int Divistion (String Div) 
{ 
    String div1 = JOptionPane.showInputDialog ("Number 1 is?"); 
    int numdiv1 = Integer.parseInt (div1); 

    String div2 = JOptionPane.showInputDialog ("Number 2 is?"); 
    int numdiv2 = Integer.parseInt (div2); 

    int divEquals = numdiv1/numdiv2; 

    JOptionPane.showMessageDialog (null, "The Answer Is: " + divEquals); 

    return divEquals; 
} 
} 

Answer 1

Zunächst einmal in Betracht ziehen, eine switch-Anweisung anstelle dieser geschachtelten ifs zu verwenden. Wenn Sie Ihre Operationsauswahl a, b, c, d vorgenommen haben, könnte der Standardfall eine Zahl annehmen und versuchen, sie zu parsen.

Answer 2

Es ist eine Weile her, seit ich Java geschrieben habe, aber ich denke, Sie haben ein paar Probleme mit dem Code wie er ist. Erstens, Ihre if-Anweisung befindet sich in einem anonymen Block. Zweitens rufen Sie das Methodenmenü auf, das einen Wert zurückgibt, aber Sie setzen diesen Wert auf nichts. D, du hast die "Division" falsch geschrieben. Schließlich wird der Code nicht fehlschlagen, aber Sie verwenden Großbuchstaben für den ersten Buchstaben der Methoden. Ich glaube, die normale Art, Java-Anwendungen zu schreiben, besteht darin, Klassen zu Großbuchstaben zu machen, aber Methoden sind normalerweise unter dem ersten Buchstaben, CamelCase für den Rest (Pakete sind in der Regel auch Unterfall/CamelCase). Jemand korrigiert mich bitte, wenn ich falsch liege.

Bearbeiten: PS. Ich habe vergessen, dass Sie die Methoden anscheinend falsch aufrufen (Sie können die Methode "Addition" nicht mit Add()) aufrufen und wie bei der Menümethode setzen Sie den Wert, der von der Methode zurückgegeben wird, nicht auf irgendetwas .

Answer 3

Es gibt eine Menge Probleme mit Ihrem Code, so dass hier eine überarbeitete Version:

public static void main(String[] args) { 
    String mainMenu = " Calculator Options \n\n1. Addition\n2" + 
      " Subtraction\n3. Multiplication\n4. Division\n5. Exit"; 

    String choice = JOptionPane.showInputDialog(mainMenu); 

    int num1 = Integer.parseInt(JOptionPane.showInputDialog("Number 1 is?")); 
    int num2 = Integer.parseInt(JOptionPane.showInputDialog("Number 2 is?")); 

    int result; 
    if (choice.equals("1")) { 
     result = num1 + num2; 
    } else if (choice.equals("2")) { 
     result = num1 - num2; 
    } else if (choice.equals("3")) { 
     result = num1 * num2; 
    } else if (choice.equals("4")) { 
     result = num1/num2; 
    } else { 
     return; 
    } 

    JOptionPane.showMessageDialog(null, "The Answer Is: " + result); 
} 

Dieses etwas sauberer mit einem switch sein würde, aber ich habe if s pro Ihre Anforderung verwendet.

Answer 4

Ok hier ist das, was ich tat, um es zu beheben:

1. Bewegen Sie den, wenn Block in das Hauptverfahren: wie es ist, kann es nicht kompilieren
2. ändern Methode ruft Methodennamen übereinstimmen
3. Remove method parmeter - sie werden nicht benötigt, da Sie die Werte aus dem JOptionPane erhalten
4. Ändern Sie Main zu Return und Int.

Nizza App Sie dorthin gehen ~ Prost

import javax.swing.JOptionPane; 

public class Calculator { 

    public static void main (String args []) { 
     String MainMenu = " Calculator Options \n\n1. Addition\n2" + 
       " Subtraction\n3. Multiplication\n4. Division\n5. Exit"; 
     menu (MainMenu); 
    } 

    public static int menu (String info) { 
     String choice = JOptionPane.showInputDialog (info); 

     if (choice.equals ("1")) 
      return Addition(); 
     else if (choice.equals ("2")) 
      return Subtraction(); 
     else if (choice.equals ("3")) 
      return Multiplication(); 
     else if (choice.equals ("4")) 
      return Division(); 
     else 
      return 0; 
    } 

    public static int Addition() { 
     String add1 = JOptionPane.showInputDialog ("Number 1 is?"); 
     int numadd1 = Integer.parseInt (add1); 

     String add2 = JOptionPane.showInputDialog ("Number 2 is?"); 
     int numadd2 = Integer.parseInt (add2); 

     int addEquals = numadd1 + numadd2; 

     JOptionPane.showMessageDialog (null, "The Answer Is: " + addEquals); 

     return addEquals; 
    } 

    public static int Subtraction() 
    { 
     String sub1 = JOptionPane.showInputDialog ("Number 1 is?"); 
     int numsub1 = Integer.parseInt (sub1); 

     String sub2 = JOptionPane.showInputDialog ("Number 2 is?"); 
     int numsub2 = Integer.parseInt (sub2); 

     int subEquals = numsub1 - numsub2; 

     JOptionPane.showMessageDialog (null, "The Answer Is: " + subEquals); 

     return subEquals; 
    } 

    public static int Multiplication() 
    { 
     String mult1 = JOptionPane.showInputDialog ("Number 1 is?"); 
     int nummult1 = Integer.parseInt (mult1); 

     String mult2 = JOptionPane.showInputDialog ("Number 2 is?"); 
     int nummult2 = Integer.parseInt (mult2); 

     int multEquals = nummult1 * nummult2; 

     JOptionPane.showMessageDialog (null, "The Answer Is: " + multEquals); 

     return multEquals; 
    } 

    public static int Division() 
    { 
     String div1 = JOptionPane.showInputDialog ("Number 1 is?"); 
     int numdiv1 = Integer.parseInt (div1); 

     String div2 = JOptionPane.showInputDialog ("Number 2 is?"); 
     int numdiv2 = Integer.parseInt (div2); 

     int divEquals = numdiv1/numdiv2; 

     JOptionPane.showMessageDialog (null, "The Answer Is: " + divEquals); 

     return divEquals; 
    } 
}