Не выполняется условие корректного подсчета суммы

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

Обсуждается

Не понимаю, что не так. Вроде правильно считает, -1 в сумме учитывает, на отрицательные числа реагирует адекватно. Заранее спасибо!

Вводить с клавиатуры числа.
Если пользователь ввел -1, вывести на экран сумму всех введенных чисел и завершить программу.
-1 должно учитываться в сумме.

Подсказка: один из вариантов решения этой задачи, использовать конструкцию:

while (true) {
    int number = считываем число;
    if (проверяем, что number -1)
        break;
}

Требования:

Программа должна считывать числа c клавиатуры.
Если пользователь ввел -1, программа должна вывести сумму всех введенных чисел на экран и завершиться.
Программа должна посчитать сумму введенных чисел и вывести её на экран.
В программе должен использоваться цикл for, while или do-while.

package com.javarush.task.task04.task0442; /* Суммирование */ import java.io.*; import java.util.*; public class Solution { public static int sum = 0; public static void main(String[] args) throws Exception { //напишите тут ваш код BufferedReader reader = new BufferedReader(new InputStreamReader(System.in)); int a=0; while(a!=-1) { a = Integer.parseInt(reader.readLine()); Solution.sum = Solution.sum+a; } if (a==-1) { System.out.println(Solution.sum); } } }

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

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

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

Justinian Judge в Mega City One

Master

21 июля 2019, 11:27

public static int sum = 0;

убери эту статическую переменную, в поля класса мы выделяем значимые характеристики класса и порождающихся объектов. Sum в нашем случае это простая временная переменная, ей место в методе main. Валидатор недолюблюет также статические поля (в задачах, где им не место) по ряду причин Твой код:

while(a!=-1)  {
  a = Integer.parseInt(reader.readLine());
  Solution.sum = Solution.sum+a;
}
  if (a==-1) {
 System.out.println(Solution.sum);
 }

Проверка if (a==-1) здесь не нужна (после цикла), если мы вышли из цикла, значит у нас a гарантированно -1, все что надо просто вывести на печать

Ekaterina

Уровень 20

21 июля 2019, 11:34

Спасибо! Можно где-то почитать, почему валидатор недолюбливает статические переменные? Или это эмпирическим путем выясняется?

Justinian Judge в Mega City One

Master

21 июля 2019, 11:59

Для начала нужно понять, что он прав.

class Cat {
 String name;
}

Это класс Cat. У него есть поле Имя. Это важная и значимая характеристика класса, что за котэ без имени... При взаимодействии объектов имя отыгрывает важную роль. Теперь к твоему коду. Solution { static int sum; } Зачем кому-то доступ к Solution.sum, ну вот у меня есть класс

class CatTransformingRay {
   public void performCatTransformation() {
    ....
    int a = Solution.sum;
 }

Какой ценностью может обладать эта переменная? Кому она может понадобится и при каких обстоятельствах? Играет ли она роль при взаимодействии КотэПреобразователя и класса Солюшн? А почему валидатор ругается - потому что статическая переменная существует одна для любого количества объектов или вызовов методов, а значит грубо говоря, при тестах может быть такое, что она не будет обнуляться, тесты многократно вызывают метод, и будут просто увеличивать значение суммы, которое будет сохранять предыдущие значения. Статические переменные мы используем в задачах, но либо нам будут даны изначально, либо по условиям будет сказано, либо из логики программы следует что нужно их применять. Но в этом случае, они неоправданы. У всего должна быть цель, в данной задаче это обычная временная переменная, которой место в методе.

Vitalii

Уровень 22

21 июля 2019, 11:24

System.out.println(Solution.sum); без проверки if`ом, это просто безсмысленно. Так как эта строчка всегда должна отработать после цикла.