Почему этот код вызывает исключение «недопустимое начало выражения»?

Эта ветка может быть полезна. Это подсказки для перехода от «описания проблемы» к «фактическому кодированию».

По вашим конкретным вопросам:

Как я могу дать пользователю три возможности ввести действительный номер для КАЖДОГО числа? (Мне не нужен «ответ», просто некоторые идеи / подсказки.

Как бы вы это сделали, если бы в реальной жизни были кассиром (с плохой памятью)? И я имею в виду очень-очень плохую память, так много, что вы не можете вспомнить, отклоняете ли вы клиента в первый раз или во второй?

Я записывал это на листе бумаги и каждый раз, когда отвергал клиента, я добавлял "|"

"| | |"

Когда в газете будет три строчки, я позвоню в полицию. .. :)

Здесь то же самое. Если у вас нет чего- то , что может отличаться, вы попадете в бесконечный цикл ;)

Является validNumCheck, чтобы вернуть значение И ЗАТЕМ выйти в основном методе void

Нет

... или я могу просто выйти из программы в этом основном методе?

да.

В сигнатуре метода (это будет " public static int validNumCheck(num1,num2,num3) ") вы должны объявить типы формальных параметров . " public static int validNumCheck(int num1, int num2, int num3) " должно сработать.

Однако лучше было бы сделать так, чтобы он validNumCheck принимал только один параметр, а затем вы вызывали бы его с каждым из трех чисел.

Мое следующее предложение (после просмотра вашего обновленного кода) - получить достойную среду IDE. Я только что загрузил его в NetBeans и обнаружил ряд ошибок. В частности, «недопустимое начало выражения» происходит из-за того, что вы забыли } в цикле while, что IDE немедленно отметила бы. После того, как вы закончите «Hello world», Блокнот больше его не сокращает.

Я не буду перечислять поправки для каждой ошибки, но имейте в виду , что int[], int, String[], и String все разные. Кажется, вы используете их как взаимозаменяемые (вероятно, из-за количества изменений, внесенных вами в свой код). Опять же, IDE отметит все эти проблемы.

Ответ на ваш новейший код (версия 12): вы приближаетесь. Похоже, вы использовали MAX_TRIES две разные цели: три числа для ввода и три шанса для каждого числа. Хотя эти два числа одинаковы, лучше не использовать одну и ту же константу для обоих. NUM_INPUTS и MAX_TRIES я бы их назвал.

И вы все еще не добавили недостающее } для цикла while.

Следующее, что нужно сделать после их исправления, - это посмотреть if (getNumber == false) . getNumber является a String[], поэтому это сравнение недопустимо. Вы должны получить возвращаемое значение validNumCheck переменной, например:

boolean isValidNum = validNumCheck(getNumber[0]);

Кроме того, нет причин getNumber быть массивом. Вам нужно только по одному, String не так ли?

Каждому параметру в определении метода нужен тип:

public static int validNumCheck(int num1, int num2, int num3){

но мне интересно, почему вы передаете все три числа, когда вы проверяете только одно за раз. И вы хотите вернуть, если число истинно или нет, поэтому возвращаемое значение должно быть логическим:

public static boolean validNumCheck(int num){
    // test and return true or false

Кроме того, если пользователь вводит «abc», вы получите исключение из метода «Intger.pareInt (String)». Возможно, вы хотите сохранить введенный текст как строку и передать его в validNumCheck, попробуйте преобразовать его и проверить, находится ли он в диапазоне от 0 до 200.

public static boolean isValidNumber(String num){
try {
    int number = Integer.parseInt(num);
    return number >= 0 && number <= 200;        
}catch(NumberFormatException e){
    return false;
}
} 

РЕДАКТИРОВАТЬ 1: для трех попыток вам понадобится цикл, который выполняется до тех пор, пока число не станет действительным или не будут выполнены три попытки. Просто посчитайте попытки в int.

Кстати, вы уверены, что вам нужно использовать JOptionPane и JMessageDialog? Это графический интерфейс, который только усложняет эту дыру. Вы также можете читать и записывать текст в консоль, используя System.out и System.in.

РЕДАКТИРОВАТЬ 2: Еще один совет: когда вы создаете массив int с заданной длиной, каждое место в массиве заполняется 0, поэтому вы можете написать:

int[] count= new int [3];

и не нужно этого:

count[0]=0;
count[1]=0;
count[2]=0;

И если вы хотите использовать значения, отличные от 0, вы можете использовать более короткую форму, например:

int[] count = {1, 5, 2}

РЕДАКТИРОВАТЬ 3: Одна вещь, которую вы должны сделать / изучить, заключается в следующем: не пишите все это и в конечном итоге получите много ошибок и неработающую программу. Немного кодируйте, немного тестируйте . Сделайте одну часть этого, посмотрите, как это работает, и вы будете счастливы.

Эта штука когда-нибудь запускалась? Я имею в виду, вы уже видели, что JOptionPane-InputDialog вы пытаетесь использовать? Если нет, сделайте сначала только это: (и запустите!)

public class Assignment3 {

    public static void main (String[] args){

        int[] numbers = new int[3];

        for (int i = 0; i < numbers.length; i++) {
            JOptionPane.showInputDialog("enter something");
        }
    }
}

Есть, чтобы запустить? Большой! Теперь попробуйте обернуть эту строку JOptionPane в цикл, который выполняется до тех пор, пока пользователь не введет действительное число (используя метод checkValidNum, который у вас уже есть) или не использует свои три попытки.

Когда вы считаете (в реальной жизни), вам не нужно хранить числа в коробках, вам просто нужно изменить значение счета и установить предел.

Например, вы можете заменить это:

        int[] count= new int[2];

С этим:

        int attemtps = 0 ; // or count 
        int maxTries = 3;

Вы не можете присвоить массив целых чисел массиву строк.

       String[] getNumber= new int [2];

Использование массива строк может работать, но вам нужно назначить массив строк

       String[] numbers = new String[2];

Или вы можете использовать и массив int:

       int [] numbers = new int [2]; 

Выбери один.

Помните, что массивы похожи на коробки, в которых вы можете хранить значения.