Need assistance

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

п.с. это к тому что у себя оттестил твой сорс - работает и заданию удовлетворяет. но валидатор не зря не пропускает...

спасибо за ответ!) дело в том, что я так же объявлял статик переменные в предыдущей похожей задаче, и все прокатило

они же примитивы, так что передавать их в метод - не сработает) так что я голосую за вариант перенести все в main :)

Точно! )

спасибо за ответ понятно что можно всю задачу решить только в методе main но я бы хотел все-таки оставить свой второй метод op() объясните пожалуйста фразу "Объявите их в методе main и передавайте в метод в качестве аргументов" этим двум переменным ("p" & "o") присваивается результат работы метода op, а аргументом метода выступает введенное нами число если сделать переменные аргументами метода, не представляю как будет выглядеть тело метода

"Объявите их в методе main и передавайте в метод в качестве аргументов" - это я глупость сказал, так как метод будет работать с копиями переменных... Если сильно хотите сохранить метод, то можно попробовать так: в мейне создать массив int на два числа, вторым аркументом в метод op передавать этот массив( массив это уже объект, поэтому предаваться будет ссылка на него, и изменяться будет именно он), в первую ячейку массива положительные/отрицательные числа, в другую - другие, в мейне вывести в консоль данные из массива.

Иван, спасибо за помощь) массивы я еще не проходил, поэтому пользуюсь пока достпуными мне знаниями) я решил задачу и все-таки сохранил свой метод, но пришлось создавать экземпляр класса, и новый класс. пока не могу отсортировать у себя в голове где писать this.переменная, где имя_класса.переменная, а где объект.переменная... public static void main(String[] args) throws Exception { BufferedReader reader = new BufferedReader(new InputStreamReader(System.in)); int x1 = Integer.parseInt(reader.readLine()); int x2 = Integer.parseInt(reader.readLine()); int x3 = Integer.parseInt(reader.readLine()); Solution2 sol = new Solution2(); sol.op(x1); sol.op(x2); sol.op(x3); System.out.println("количество отрицательных чисел: " + sol.o); System.out.println("количество положительных чисел: " + sol.p); } public static class Solution2 { public int p = 0; public int o = 0; public void op(int a) { if (a > 0) this.p++; else { if (a < 0) this.o++; } } } }

нуу, это круто! как для 4го уровня)) не думал, что до этого дойдет... но раз решил через классы и объекты - значит можно рассказывать тебе больше, чем ты пока прошел) можно было еще чтобы твой метод возвращал boolean значение. например true если больше 0 и false если меньше. но тогда еще по ифу пришлось бы добавить в мейн, так что это кардинально картину не поменяло бы. вообще, твой первый вариант тоже весьма неплох. ты вынес повторяющиеся действия в отдельный метод, это правильно. и менял значения переменных, которые были бы доступны из двух методов (main() и op()), это тоже правильно. и поскольку оба эти метода static - то и переменные тоже такими и должны быть) жаль, что валидатор не рассчитывал, что на твоем уровне люди будут уметь так делать поэтому твое решение и запорол) но оно очень даже неплохое!) можно было бы сделать переменные о и р не примитивами, а ссылочными типами данных (например Integer). но поскольку Integer - неизменяемый тип, то это тоже не дало бы никаких преимуществ. согласись, код который прошел проверку (с внутренним классом и созданием объектов этого класса) выглядит значительно более громоздко и сложно. вариант со статиками выглядел проще и понятнее. а насчет this - то его тут можно было и не использовать. р++ или о++ было бы достаточно, потому что у тебя там нет других переменных с этим же именем. если бы у тебя была переменная р уровня класса, и еще ты объявил бы переменную с таким же именем еще и внутри метода - тогда да, та переменная р из метода "затенила" бы переменную р уровня класса. и вот тогда тебе пришлось бы явно указывать this.p чтобы обратиться к переменной на уровне класса. в общем this.переменная говорит, что надо использовать переменную из этого объекта имя_класса.переменная - обычно используется извне класса, когда нужно получить доступ к статической переменной класса объект.переменная - тоже извне, но к не статической переменной. вот и вся разница.

ах, да! все хорошо, кроме названий методов и переменных) старайтесь потратить на секунду больше времени на придумывания понятного осмысленного имени :) как для переменных, так и для методов (и для классов в дальнейшем). хотя бы moreThan0, lessThan0 и checkIfMoreOrLess0() ну это так, навскидку) потому что когда я впервые увидел ваш код - подумал: "лол, что еще за оп? это он так в обе переменные пытается положить значение? оно хоть компилируется?" и только потом увидел этот метод))

ставил, та же проблема