Einfache Java-Rechner

Erstens dies ist keine Hausaufgaben Frage. Ich bin üben meine Kenntnisse über java. Ich dachte, ein guter Weg, dies zu tun ist, zu schreiben, ein einfaches Programm ohne Hilfe. Leider hat mein compiler sagt mir Fehler, die ich nicht wissen, wie Sie zu beheben. Ohne änderung viel Logik und den code, könnte mir jemand freundlicherweise zeigen, wo einige meiner Fehler? Dank

import java.lang.*;
import java.util.*;

public class Calculator
{
    private int solution;
    private int x;
    private int y;
    private char operators;

    public Calculator()
    {
        solution = 0;
        Scanner operators = new Scanner(System.in);
        Scanner operands = new Scanner(System.in);
    }

    public int addition(int x, int y)
    {
       return x + y;
    }
    public int subtraction(int x, int y)
    {
       return x - y;
    }
    public int multiplication(int x, int y)
    {    
       return x * y;
    }
    public int division(int x, int y)
    {
       solution = x / y;
       return solution;
    }
    public void main (String[] args)
    {
      System.out.println("What operation? ('+', '-', '*', '/')"); 

      System.out.println("Insert 2 numbers to be subtracted");
       System.out.println("operand 1: ");
       x = operands;
       System.out.println("operand 2: ");
       y = operands.next();
      switch(operators)
      {
          case('+'):
            addition(operands);
            operands.next();
            break;
          case('-'):
            subtraction(operands);
            operands.next();
            break;
          case('*'):
            multiplication(operands);
            operands.next();
            break;
          case('/'):
            division(operands);
            operands.next();
            break;
       }
  }
}

Welche Fehler sind Sie immer?
Vielen Dank für die Veröffentlichung Ihres Codes. Jedoch, es hilft auch, eine Menge, wenn man den text posten der Fehlermeldungen, die Sie bekommen, von der compiler - dies macht es einfacher für Menschen, um schnell das problem zu identifizieren (ohne Lesen Sie den gesamten code oder kompilieren Sie es selbst).

InformationsquelleAutor dukevin | 2010-04-29

calculator java

package org.com;

import java.lang.*; 
import java.util.*; 

public class Calculator 
{ 
    private int solution; 
    private static int x; 
    private static int y; 
    private char operators; 

    public Calculator() 
    { 
        solution = 0; 
        Scanner operators = new Scanner(System.in); 
        Scanner operands = new Scanner(System.in); 
    } 

    public int addition(int x, int y) 
    { 
       return x + y; 
    } 
    public int subtraction(int x, int y) 
    { 
       return x - y; 
    } 
    public int multiplication(int x, int y) 
    {     
       return x * y; 
    } 
    public int division(int x, int y) 
    { 
       solution = x / y; 
       return solution; 
    } 

    public void calc(int ops){
         x = 4; 
         System.out.println("operand 2: "); 
         y = 5; 

         switch(ops) 
         { 
             case(1): 
               System.out.println(addition(x, y)); 

           //   operands.next(); 
               break; 
             case(2): 
                 System.out.println(subtraction(x, y)); 
              //operands.next(); 
               break; 
             case(3): 
                 System.out.println(multiplication(x, y)); 
             // operands.next(); 
               break; 
             case(4): 
                 System.out.println(division(x, y));
             // operands.next(); 
               break; 
          } 
    }
    public static void main (String[] args) 
    { 
      System.out.println("What operation? ('+', '-', '*', '/')");  
      System.out.println(" Enter 1 for Addition");
      System.out.println(" Enter 2 for Subtraction");
      System.out.println(" Enter 3 for Multiplication");
      System.out.println(" Enter 4 for Division");

       Calculator calc = new Calculator();
       calc.calc(1);


  } 
}

Diese arbeiten

InformationsquelleAutor gmhk

4

operands und operators sind out-of-scope für main. Deklarieren Sie lokale Variablen im Konstruktor, so dass, wenn Sie verlassen Sie den ctor Sie sind dazu berechtigt, die für die GC gegangen.

Haben Sie Kompilierungsfehler - 10.

Würden Sie vorschlagen, eine neudeklaration in Haupt -, wie: dieser.Operanden; dieser.Operatoren; oder ist diese für einen anderen Zweck?
Duke: Sie können keine Felder hinzufügen einer Klasse zur Laufzeit in der Sprache. Sie müssen angeben, dass Sie in der Taschenrechner, wie Sie angegeben Lösung und Betreiber. Main auch nicht "dieses", wie es muss statisch sein.

InformationsquelleAutor duffymo
2

Weiteres Problem ist die Zeile
```
y = operands.next();
```
ist der Versuch auf der Stelle ein String zurück von Scanner.next() in der eine variable y die als ein Typ deklariert ist int.

Den Scanner.nextInt() Methode kann verwendet werden, um zu versuchen, wieder eine int.

InformationsquelleAutor coobird

package com.abc;

import java.util.Scanner;

public class Calculator {
    private static final String pos = "+";
    private static final String neg = "-";
    private static final String mult = "*";
    private static final String div = "/";

    private enum operation {
        pos, neg, mult, div
    };
    private int solution;
    private int x;
    public int getX() {
        return x;
    }

    public void setX(int x) {
        this.x = x;
    }

    public int getY() {
        return y;
    }

    public void setY(int y) {
        this.y = y;
    }

    private int y;



    static Scanner operators;

    public Calculator() {
        solution = 0;
        operators = new Scanner(System.in);

    }

    public int addition(int x, int y) {
        return x + y;
    }

    public int subtraction(int x, int y) {
        return x - y;
    }

    public int multiplication(int x, int y) {
        return x * y;
    }

    public int division(int x, int y) {
        solution = x / y;
        return solution;
    }

    public static void main(String[] args) {
        Calculator calc = new Calculator();

        System.out.println("Insert 2 numbers");

        System.out.println("operand 1: ");

        calc.setX(Integer.parseInt(operators.next()));

        System.out.println("operand 2: ");
        calc.setY(Integer.parseInt(operators.next()));

        System.out.println("What operation? ('pos', 'neg', 'mult', 'div')");
        operation ttt = operation.valueOf(operators.next());
        int output = 0 ;
        switch(ttt){
        case pos:
            output = calc.addition(calc.getX(), calc.getY());

            break;
          case neg:
              output = calc.subtraction(calc.getX(), calc.getY());

            break;
          case mult:
              output = calc.multiplication(calc.getX(), calc.getY());

            break;
          case div:
              output = calc.division(calc.getX(), calc.getY());

            break;
        }
        System.out.println("output ="+output);
    }
}

InformationsquelleAutor Shashank T

1

Zusätzlich zu den anderen Antworten, Ihre main () - Methode muss statisch sein, um ein Programm zu sein, Einstiegspunkt. In main (), die Sie benötigen, um erstellen Sie Ihre eigene Rechner-Objekt und rufen Methoden auf, die.

InformationsquelleAutor Yann Ramin

Dies ist alles toll, aber welches Programm verwenden Sie, um schreiben Sie Ihre java?
Vielleicht solltest du erwägen Sie die Verwendung einer IDE wie Eclipse, so kann es erkennt Fehler automatisch und fügt auch die Importe. (Ich bin mir nicht sicher, wenn Ihr dies tut) Es sagt Ihnen auch, was das problem mit Ihrem Programm "in Englisch".
Überlegen Sie auch, diese Klasse als vielleicht ein einfacher und unkomplizierter Weg, das zu tun, ein Taschenrechner:

public class Calculator {
public static void main(String[] args) {
    Scanner sc = new Scanner(System.in);
    System.out.print("Enter an Operator: ");
    String in = sc.next();
    char oper = in.charAt(0);

    System.out.print("Enter a number: ");
    in = sc.next();
    double num1 = Double.parseDouble(in);

    System.out.print("Enter another number: ");
    in = sc.next();
    double num2 = Double.parseDouble(in);

    if(oper == '+') {
        double result = num1 + num2;
        System.out.println(result);
    } else if(oper == '-') {
        double result = num1 - num2;
        System.out.println(result);
    } else if(oper == 'x') {
        double result = num1 * num2;
        System.out.println(result);
    } else if(oper == '/') {
        double result = num1 / num2;
        System.out.println(result);
    } else {
        double result = num1 % num2;
        System.out.println(result);
    }
        System.out.println("Hope this helped your mathmatical troubles!");
}

}

Und als eine Sache der Gewohnheit, anstatt das zu tun:

import java.util.*;

ist es besser zu tun:

import java.util.Scanner;

Dies ist wahrscheinlich nicht viel Unterschied hier, aber wenn Sie ein viel größeres Programm zu importieren das ganze von java.util wird erheblich verlangsamen Ihr Programm.

Hoffe, das hilft!

Dein letzter Punkt ist falsch. Importieren von wildcard-Pakete hat keinen Einfluss auf die Laufzeit-performance. Es ist nur zur compile-Zeit symbol Auflösung
Tut mir Leid, dass. Es ist nur, ich habe gehört, Tonnen von Menschen, die sagen, wie der Import der gesamten java-Bibliotheken verlangsamt das Programm. Oder habe ich mishear?
Ich vermute, Sie hat nicht mishear-es gibt nur eine Tonne von Fehlinformationen Umgebung. Werfen Sie einen Blick auf diese Frage.

InformationsquelleAutor Dan299

0

Ihre main-Methode deklariert werden muss, wie diese:
```
public static void main(String[] args) {..}
```
Außerdem, es scheint, dass Sie nur versorgt ein argument für Sie, alle Ihre arithmetischen Methoden (addition, Subtraktion usw.), obwohl Sie benötigen zwei.
```
public int addition(int x, int y);
```
Kann nicht aufgerufen werden, mit addition(operands), das ist nur ein argument, und das argument den falschen Typ (die Methode muss zwei int Sie geben ihm eine Scanner). Das gleiche gilt für alle diese Methoden. Müssen Sie zum extrahieren der ints aus der Scanner. Sie können tun, dass mit Scanner.nextInt().

InformationsquelleAutor Lars Andren

import java.lang.*;

import java.util.*;


public class Calculator
{
    private int solution;
    private int x;
    private int y;
 private char operators;

    public Calculator()
    {
        solution = 0;
        Scanner operators = new Scanner(System.in);
        Scanner operands = new Scanner(System.in);
    }

    public int addition(int x, int y)
    {
       return x + y;
    }
    public int subtraction(int x, int y)
    {
       return x - y;
    }
    public int multiplication(int x, int y)
    {    
       return x * y;
    }
    public int division(int x, int y)
    {
       solution = x / y;
       return solution;
    }
    public void main (String[] args)
    {
      System.out.println("What operation? ('+', '-', '*', '/')"); 

      System.out.println("Insert 2 numbers to be subtracted");
       System.out.println("operand 1: ");
       x = operands;
       System.out.println("operand 2: ");
       y = operands.next();
      switch(operators)
      {
          case('+'):
            addition(operands);
            operands.next();
            break;
          case('-'):
            subtraction(operands);
            operands.next();
            break;
          case('*'):
            multiplication(operands);
            operands.next();
            break;
          case('/'):
            division(operands);
            operands.next();
            break;
       }
  }
}

InformationsquelleAutor Waruna Wishwanath Usgoda

-1

Sie fordern den Benutzer zur Eingabe in ganzen zahlen, sondern setzen Sie die Anweisung operands.next(); wie die Eingabe. Versuchen Sie, konsistent mit Ihren Variablen und Benutzereingaben, so ändern Sie ihn auf operands.nextInt() helfen würde.

InformationsquelleAutor Lopes_CP_113013
-2

Nur als Tipp, es ist in der Regel nicht eine gute Idee zu Beginn werfen
```
import java.util.*;
```
in Ihrem Programm, da es macht das Programm unnötig groß und langsam. Alles was Sie dazu brauchen, ist
```
import java.util.Scanner;
```
Wenn ich richtig bin, die meisten, wenn nicht alles in java.lang ist bereits importiert für Sie.

Das ist falsch. Importieren von wildcard-Pakete hat keinen Einfluss auf die Laufzeit-performance (oder die Größe des Programms, für diese Angelegenheit). Es ist nur zur compile-Zeit symbol Auflösung.

InformationsquelleAutor Jazzy Josh

Schreibe einen Kommentar

Du musst angemeldet sein, um einen Kommentar abzugeben.