Почему не работает код java a^b?

Question

[enabl3]

Всем привет.
Подскажите пожалуйста, что делаю не так?
Есть задание, написать метод, который принимает 2 целых числа больше нуля и должен вернуть целое число в котором первое число возводится в степень второго.
Делаю это так

import java.util.Scanner;

public class testing {
    public static void main(String[] args) {
        System.out.print("Введи число (а): ");
        Scanner get_input = new Scanner(System.in);
        int a = 0;
        while (!get_input.hasNextInt() || ((get_input.nextInt()) <= 0))  {
            System.out.print("Введи число больше 0: ");
            get_input.next();
        }
        a = get_input.nextInt();
        System.out.print("Введи число (b): ");
        int b = 0;
        while (!get_input.hasNextInt()) {
            System.out.print("Введи число больше 0: ");
            get_input.next();
        }
        b = get_input.nextInt();
        int c = getDegree(a, b);
        System.out.println("'A' в степени 'B' = " + c);
    }

    public static int getDegree(int a, int b) {
        int res = (int) Math.pow(a, b);
        return res;
    }
}

Но проблема в том, что при вводе числа, если оно больше 0 и не текст, не сразу спрашивает число Б, что я не так сделал?

Comments

[сергей кузьмин]

отладка программы - это процесс осознания программистом того, что компьютер делает именно то, что его попросили

[Денис Загаевский]

И не работает, конечно, степень. Смешно.

[Орхан Гасанлы]

Добрый день.
Могу предположить, что у вас есть проблемы в ветке if. Рекомендую сделать дебаг и найти проблему самому. Проверьте насколько корректно срабатывает логика. Хочу обратить внимание, что если пользователь ввел текст, то в выбрасывается исключение InputMismatchException.

(!get_input.hasNextInt() || ((get_input.nextInt()) <= 0))

Также у вас есть проблема в следующей ветке:
while (!get_input.hasNextInt()) {

А так, вот, один из вариантов решения данной задачи:

public static void main(String[] args) {
        Scanner scanner = new Scanner(System.in);
        System.out.print("Введите число (а): ");
        int a = 0;
        boolean isATrue = true;
        while (isATrue)  {
            a = scanner.nextInt();
            if (a <= 0) {
                System.out.print("Введите число больше 0: ");
            } else {
                isATrue = false;
            }
        }
        System.out.print("Введите число (b): ");
        int b = 0;
        boolean isBTrue = true;
        while (isBTrue) {
            b = scanner.nextInt();
            if (b <= 0) {
                System.out.print("Введите число больше 0: ");
            } else {
                isBTrue = false;
            }
        }
        System.out.println("'A' в степени 'B' = " + pow(a, b));
    }

    public static int pow(int a, int b) {
        return  (int) Math.pow(a, b);
    }

Можно написать более лаконичный код, но я не стал сильно заморачиваться.

Сделал код ревью. Вот, небольшие советы:
1) Название классов должны быть с большой буквы. Прочитайте про naming convention.
2) В названиях переменных слова отделяются при помощи camelCase. Соответственно, get_input -> getInput

В общем, дебаг наше все)

[enabl3]

Орхан Гасанлы, Спасибо большое. Сделал так:

import java.util.InputMismatchException;
import java.util.Scanner;
public class testing {
    public static void main(String[] args) {
        Scanner getInput = new Scanner(System.in);
        System.out.print("Введи число (а): ");
        int a = 0;
        while (true) {
            try {
                a = getInput.nextInt();
                if (a <= 0) {
                    System.out.print("!!!Число должно быть больше 0!!!\nВведи повторно число (а): ");
                    getInput.hasNextInt();
                } else {
                    System.out.println("Число А = " + a);
                    break;
                }
            } catch (InputMismatchException ignore) {
                System.out.print("!!!Нужно вводить только число!!!\nВведи повторно число (а): ");
                getInput.next();
            }
        }
        System.out.print("Введи число (b): ");
        int b = 0;
        while (true) {
            try {
                b = getInput.nextInt();
                if (b <= 0) {
                    System.out.print("!!!Число должно быть больше 0!!!\nВведи повторно число (b): ");
                    getInput.hasNextInt();
                } else {
                    System.out.println("Число B = " + b);
                    break;
                }
            } catch (InputMismatchException ignore) {
                System.out.print("!!!Нужно вводить только число!!!\nВведи повторно число (b): ");
                getInput.next();
            }
        }
        int c = getDegree(a, b);
        System.out.println("'A' в степени 'B' = " + c);
    }
    public static int getDegree(int a, int b) {
        int res = (int) Math.pow(a, b);
        return res;
    }
}

Задачу решает, но не идеально. Для первого раза, думаю, не плохо)

[Орхан Гасанлы]

enabl3, не за что. Кстати, вы можете вынести дублирующийся код в отдельный метод.

[enabl3]

Орхан Гасанлы, тоже думал об этом. Нужно будет еще поэкспериментировать ))