Java-Синтаксична помилка на токені "! =", Ім'я очікується після цього токену

Question

Java-Синтаксична помилка на токені "! =", Ім'я очікується після цього токену

Я намагаюся зробити простий калькулятор і продовжувати отримувати помилку в заголовку, коли намагаюся показати "помилку", якщо користувач не вводить жодний з вказаних типів операцій.

import java.util.Scanner;

public class experiments {
    public static void main(String args[]) {
        Scanner userInput = new Scanner(System.in);
        String operation;
        double fNum, sNum, ans;

        //select type of operation
        System.out.println("Type addition, subtraction, multiplication, or division, then press enter");
        operation = userInput.nextLine();
        if (operation!=("addition","subtraction","multiplication","division")) {
            System.out.println("error");
        }

        //enter numbers
        System.out.println("Enter first number");
        fNum = userInput.nextDouble();
        System.out.println("Enter second number");
        sNum = userInput.nextDouble();

        //calculate
        if (operation.equals("addition")) {
            ans=fNum + sNum;
        }
        else if (operation.equals("subtraction")) { 
            ans=fNum - sNum;
        }
        else if (operation.equals("multiplication")){
            ans=fNum * sNum;
        }
        else if (operation.equals("division")) {
            ans=fNum/sNum;
        }
        //print answer
        System.out.println("The answer is ");
        System.out.println(ans);    
    }
}

0

додано 27 червень 2013 при 06:57 Автор user2526522 Змінено 29 вересень 2014 при 05:35

Перегляди: 1

джерело

ro nl ja ru fr es pt de zh hi bn kk uz be tr

7 Відповіді

Автор Henry Keiter · Answer 1 · 2013-06-27T07:00Z

Ви не можете порівнювати речі з групою таких об'єктів:

operation!=("addition","subtraction","multiplication","division")

Напевно, що ви хочете, це "якщо operation не є однією з цих чотирьох речей". У вас є кілька варіантів. Той, хто найбільше любить те, що ви зараз маєте, створити новий ArrayList (скажімо, legalOperations ), що містить чотири юридичні операції, а потім використовуйте legalOperations.contains (operation) .

Однак чистим способом, який є "кращою Java", було б зробити enum і використовувати його для порівняння.

public enum LegalOperations {
    ADDITION,
    SUBTRACTION,
    MULTIPLICATION,
    DIVISION
}

Тоді ви можете зробити свої порівняння з вашим перерахуванням (можливо, ви надали переписку конструктору, щоб він мав чітке значення для кожного константа переліку, а метод isLegalOperation , наприклад).

Автор MadProgrammer · Answer 2 · 2013-06-27T07:00Z

Java не може зрозуміти цю команду ...

if (operation!=("addition","subtraction","multiplication","division")) {

Натомість потрібно перевірити кожен індивідуально ...

if (!"addition".equals(operation) &&
    !"subtraction".equals(operation) &&
    !"multiplication".equals(operation) &&
    !"division".equals(operation)) {
   //Handle error...
}

1

додано 27 червень 2013 при 07:00 Автор MadProgrammer

джерело

Любіть подвійні негативи: P

додано 27 червень 2013 при 07:08, Автор MadProgrammer, джерело

@Kevin я міг би просто обернути його в єдиний (...) блок: P

додано 27 червень 2013 при 07:14, Автор MadProgrammer, джерело

@ user2526522: Див також De Morgan's_laws .

додано 30 червень 2013 при 02:34, Автор trashgod, джерело

Я вважаю, що ви хочете && , інакше один з перших двох гарантовано буде правдою.

додано 27 червень 2013 при 07:03, Автор Kevin, джерело

Дякую вам за допомогу всім, я виявив, що це найпростіший, оскільки я ще не вивчив масиви, тому дякую вам за це дуже легко зрозуміти код

додано 27 червень 2013 при 05:11, Автор user2526522, джерело

Автор Alex · Answer 3 · 2013-06-27T07:00Z

You should create ArrayList and put value there.
Then you can check if the value exists in the ArrayList.
Like that:
How Arrays.asList(...).contains(...) works?

1

додано 27 червень 2013 при 07:00 Автор Alex

джерело

+1 Для нульової ідеї

додано 27 червень 2013 при 07:00, Автор MadProgrammer, джерело

Крім того, я думаю, що це буде List # contains , не потрібно використовувати масиви , оскільки це вже є List : P

додано 27 червень 2013 при 07:13, Автор MadProgrammer, джерело

@MadProgrammer thx

додано 27 червень 2013 при 07:03, Автор Alex, джерело

@MadProgrammer Ви маєте рацію. Але, можливо, автор спочатку бажає створити список з масиву. :)

додано 27 червень 2013 при 07:18, Автор Alex, джерело

Автор Prasanth · Answer 4 · 2013-06-27T07:02Z

Ви можете використовувати метод містить, як це:

String[] operations = {"addition","subtraction","multiplication","division"};
if(!operations.contains(operation)) {
    ...

Автор Harshavardhan Konakanchi · Answer 5 · 2013-06-27T07:02Z

Я думаю, ви можете переписати те саме, що і в

if (!(    operation.equals("addition") || operation.equals("subtraction") ||
          operation.equals("multiplication") || operation.equals("division"))) {
                System.out.println("error");
        }

0

додано 27 червень 2013 при 07:02 Автор Harshavardhan Konakanchi

джерело

Чи не буде ця помилка друку, навіть якщо ці умови є істинними?

додано 27 червень 2013 при 07:06, Автор Amal Murali, джерело

@AmalMurali, навіть якщо одне з цих умов є істинним, остаточний результат до statements if є помилковим. Якщо вказаний ввід не є одним із умов, то кінцевий результат до if є tru і він друкує помилку

додано 27 червень 2013 при 07:16, Автор Harshavardhan Konakanchi, джерело

Автор Usman Saleem · Answer 6 · 2013-06-27T07:00Z

if (operation!=("addition","subtraction","multiplication","division")) is incorrect comparison in Java. Use switch statement if you are using Java 7 or string.equals method with multiple if statements.

Автор Ashrith Sheshan · Answer 7 · 2013-06-27T07:07Z

Існує ще один Легкий спосіб зробити це, що передбачає менше написання

як у вас є масив рядків, оголосити його як масив

String [] operations = {"addition", "subtraction", "multiplication", "division"};

потім використовуйте це для перевірки

 if(!Arrays.asList(operations).contains(operation)) {
   System.out.println("error");
 }