Решение выдает приавильный ответ, исходя из вычислений, но проверку не проходит

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

Обсуждается

Почему нет? Решение полностью правильное, даже если подставить другое число

public static int sumDigitsInNumber(int number) {
       int num1 = number / 100;
       int num2 = (number - (num1 * 100))/10;
       int num3 = number - (num1 * 100+(num2*10));
       if (number%num1 == 0)
           return(num1);
       else
           number = (num1+num2+num3);
           return(number);

Так же ментор пишет следующее: Тебе нужно добавить к сумме остаток от деления числа number на 10. После этого присвоить числу number значение (number/10). Эту операцию повторить три раза. Не понимаю вообще смысла этих действий, т.к. задача уже выполняется верно

Напиши код метода sumDigitsInNumber(int number). Метод на вход принимает целое трехзначное число. Нужно посчитать сумму цифр этого числа, и вернуть эту сумму.

Пример:
Метод sumDigitsInNumber вызывается с параметром 546.

Пример вывода:
15

Требования:

Программа не должна считывать данные с клавиатуры.
Метод sumDigitsInNumber(int) должен быть публичным и статическим.
Метод sumDigitsInNumber должен возвращать значение типа int.
Метод sumDigitsInNumber не должен ничего выводить на экран.
Метод sumDigitsInNumber должен правильно возвращать сумму всех цифр в числе number.

package com.javarush.task.task01.task0132; /* Сумма цифр трехзначного числа */ public class Solution { public static void main(String[] args) { System.out.println(sumDigitsInNumber(546)); } public static int sumDigitsInNumber(int number) { int num1 = number / 100; int num2 = (number - (num1 * 100))/10; int num3 = number - (num1 * 100+(num2*10)); if (number%num1 == 0) return(num1); else number = number + (num1+num2+num3); return(number); } }

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

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

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

Мария Осипова

Уровень 20

4 июня 2020, 15:38

полезный

Объяснение почему тот метод, который предлагает ментор лучше: В нём можно проследить повторяющиеся действия и написать уже код для подсчёта суммы не только трёхзначных чисел, а также 4х значных, 5ти, 6ти и.т.д: public static void main(String[] args) { System.out.println(sumDigitsInNumber(546)); } public static int sumDigitsInNumber(int number) { int sum = 0; while (number != 0) { sum += number%10; number /= 10; } return sum; }

Мария Осипова

Уровень 20

4 июня 2020, 15:27

полезный

по поводу ментора, он тебе просто подсказывает другой ход решения, вот такой: public static void main(String[] args) { System.out.println(sumDigitsInNumber(546)); } public static int sumDigitsInNumber(int number) { int num1 = number%10; // тут остаток от деления будет равен 6 number = number/10; // здесь number стал равен 54 int num2 = number%10; // снова получаем остаток от деления, который равен 4 number = number/10; // здесь number стал равен 5 int num3 = number%10; // здесь тоже получаем остаток от деления, который равен 5, но можно просто написать int num3 = number return(num1 + num2 + num3); }

Shdbdbd Susjebj

Уровень 22

5 июня 2020, 06:17

я не до конца понял, что делает number%10

Мария Осипова

Уровень 20

5 июня 2020, 07:10

полезный

% - эта операция берет остаток от деления. Например 5/2 = 4 остаток 1 6/4 = 1 остаток 2 9/2 = 8 остаток 1 Так вот операция % вернет нам этот остаток: 5%2 = 1 6%4 = 2 9%2 = 1 Тоже самое и с 10кой 54/10 = 5 остаток 4 54%10 = 4 Так как у нас система счисления десятичная, то остаток от деления на 10 вернет нам нужные разряды чисел. 546%10 = 6 //третье число 546/10 = 54 54%10 = 4 //второе число 54/10 = 5 5%10 = 5 //первое число Пояснение: 5/10 = 0 остаток 5 Мы как бы собираем числа с конца.

Мария Осипова

Уровень 20

4 июня 2020, 15:15

полезный

Строки 16-19 лишние и результат ты возвращаешь number = number + (num1+num2+num3), к самому числу прибавляешь сумму его чисел, а нужно просто сумму чисел вернуть: public static void main(String[] args) { System.out.println(sumDigitsInNumber(546)); } public static int sumDigitsInNumber(int number) { int num1 = number / 100; int num2 = (number - (num1 * 100))/10; int num3 = number - (num1 * 100+(num2*10)); return(num1 + num2 + num3); }

Екатерина

Уровень 16

4 июня 2020, 14:15

полезный

Попробуйте ввести 999 - получите интересный результат.

Алексей Михайлов

Уровень 22

Expert

4 июня 2020, 14:13

зачем тебе проверка в 16 строке? у тебя всё правильно считает значит ты всегда возвращаешь первую цифру, а должен возвращать результат вычисления