На IDEA всё нормально проходит, а здесь нет (( Посмотрите, пожалуйста, что не так

Вопрос по задаче

Обсуждается

Ввести с клавиатуры положительное число. Определить, сколько в введенном числе четных цифр, а сколько нечетных.
Если число делится без остатка на 2 (т. е. остаток равен нулю), значит оно четное.
Увеличиваем на 1 счетчик четных цифр (статическая переменная even).
Иначе число нечетное, увеличиваем счетчик нечетных цифр (статическая переменная odd).
Вывести на экран сообщение: "Even: а Odd: b", где а - количество четных цифр, b - количество нечетных цифр.

Пример для числа 4445:
Even: 3 Odd: 1

Требования:

Программа должна считывать данные с клавиатуры.
Метод main должен посчитать сколько четных цифр во веденном числе и записать это количество в переменную even.
Метод main должен посчитать сколько нечетных цифр во веденном числе и записать это количество в переменную odd.
Программа должна выводить текст на экран.
Выведенный текст должен соответствовать заданию.

package com.javarush.task.task06.task0606; import java.io.BufferedReader; import java.io.IOException; import java.io.InputStreamReader; /* Чётные и нечётные циферки */ public class Solution { public static int even; public static int odd; public static void main(String[] args) throws IOException { //напишите тут ваш код BufferedReader reader = new BufferedReader(new InputStreamReader(System.in)); String text = reader.readLine(); int number = Integer.parseInt(text); int x = text.length(); int pow = (Pow(10, x))/10; int per = 1; for (int i=0; i<x; i++){ int m = number % per; if ((m/pow)%2==0) even++; else odd++; per = pow; pow = pow/10; } System.out.println("Even: " + even + " Odd: " + odd); } public static int Pow(int a, int b){ int x = 1; for (int i=0; i<b; i++){ x = x*a; } return x; } }

Комментарии (5)

популярные
новые
старые

Для того, чтобы оставить комментарий Вы должны авторизоваться

catalust

Уровень 19

5 ноября 2021, 12:50

Ты хоть сам понял что написал?) Мне было бы проще удалить все и начать заново. Для числа 1:

int number = Integer.parseInt(text); // number = 1
int x = text.length(); // x = 1
int pow = (Pow(10, x))/10; // pow = (10/10) = 1
int per = 1;

for (int i=0; i<x; i++){ // 1ый проход цикла - i = 0
    int m = number % per; // m = 1 % 1 = 0
    if ((m/pow)%2==0) // (0/1)%2 = 0 == 0 - true
        even++; // инкрементируется четный счетчик, что неверно - число то "1"
    else
        odd++;
    per = pow;
    pow = pow/10;
}

Для числа 12:

int number = Integer.parseInt(text); // number = 12
int x = text.length(); // x = 2
int pow = (Pow(10, x))/10; // pow = (100/10) = 10
int per = 1;

for (int i=0; i<x; i++){ // 1ый проход цикла - i = 0
    int m = number % per; // m = 12 % 1 = 0
    if ((m/pow)%2==0) // (0/10)%2 = 0 == 0 - true
        even++; // инкрементируется четный счетчик, что неверно - первая цифра то "1"
    else
        odd++;
    per = pow; // per = 10
    pow = pow/10; // pow = 10/10 = 1
}

for (int i=0; i<x; i++){ // 2ой проход цикла - i = 1
    int m = number % per; // m = 12 % 10 = 2
    if ((m/pow)%2==0) // (2/1)%2 = 0 == 0 - true
        even++; // инкрементируется четный счетчик, что для второй цифры верно
    else
        odd++; // если бы второй цифрой была единица то инкрементировался бы  odd
}

Как исправить твой код, чтобы первую цифру в числе верно определяло - уже голова отказывается думать.

catalust

Уровень 19

5 ноября 2021, 13:04

Для числа 4445:

int number = Integer.parseInt(text); // number = 4445
int x = text.length(); // x = 4
int pow = (Pow(10, x))/10; // pow = (10000/10) = 1000
int per = 1;

for (int i=0; i<x; i++){ // 1ый проход цикла - i = 0
    int m = number % per; // m = 4445 % 1 = 0
    if ((m/pow)%2==0) // (0/1000)%2 = 0 == 0 - true
        even++; // инкрементируется четный счетчик, что хоть и верно, но в логике кода ошибка и тебе просто повезло
    per = pow; // per = 1000
    pow = pow/10; // pow = 1000/10 = 100
}
for (int i=0; i<x; i++){ // 2ой проход цикла - i = 1
    int m = number % per; // m = 4445 % 1000 = 445
    if ((m/pow)%2==0) // (445/100)%2 = 0 == 0 - true
        even++; // инкрементируется четный счетчик
    per = pow; // per = 100
    pow = pow/10; // pow = 100/10 = 10
}
for (int i=0; i<x; i++){ // 3ий проход цикла - i = 2
    int m = number % per; // m = 4445 % 100 = 45
    if ((m/pow)%2==0) // (45/10)%2 = 0 == 0 - true
        even++; // инкрементируется четный счетчик
    per = pow; // per = 10
    pow = pow/10; // pow = 100/10 = 1
}
for (int i=0; i<x; i++){ // 4ый и последний проход цикла - i = 3
    int m = number % per; // m = 4445 % 10 = 5
    if ((m/pow)%2==0) // (5/1)%2 = 1 != 0 - false
        odd++; // инкрементируется нечетный счетчик
}

И получается вроде бы верный вывод - 3 четных и одно нечетное, но по логике код неверный, тк если бы у нас было число 3445 - был бы такой же вывод, а правильный был бы 2 четных и 2 нечетных.

catalust

Уровень 19

5 ноября 2021, 13:10

Кстати нашел решение именно для твоего кода который мне-лузеру кажется очень странным (не в обиду будет сказано) в строке 23 твоего кода:

int per = 1; // заменить на int per = pow*10;

С таким изменением четность/нечетность первой цифры в числе всегда будет верно определяться, а далее твой "per" станет как ты и задумывал в коде и правильно также будет определять последующие цифры числа.

Denis

Уровень 28

6 ноября 2021, 18:09

всё догнал, спасибо! я считал, что 12/1=12 и как раз 12 я остатком и считал. Теперь понял свою ошибку Ещё раз спасибо

Стас ПасинковSoftware Developer в Zipy

Master

6 ноября 2021, 20:33

если задача решена - пометьте вопрос как решенный, пожалуйста :)