Все решено, но валидацию не проходит

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

Решен

Да, намудрил, но заодно повторил изученное раннее, все проверил работает, проблема в валидаторе?

Ввести с клавиатуры три целых числа. Определить, имеется ли среди них хотя бы одна пара равных между собой чисел.
Если такая пара существует, вывести на экран числа через пробел.
Если все три числа равны между собой, то вывести все три.

Примеры:
а) при вводе чисел
1
2
2
получим вывод
2 2

б) при вводе чисел
2
2
2
получим вывод
2 2 2

Требования:

Программа должна считывать числа c клавиатуры.
Программа должна содержать System.out.println() или System.out.print()
Если два числа равны между собой, необходимо вывести числа на экран.
Если все три числа равны между собой, необходимо вывести все три.
Если нет равных чисел, ничего не выводить.

package com.javarush.task.task04.task0417; /* Существует ли пара? */ import java.io.*; import java.util.*; public class Solution { public static void main(String[] args) throws Exception { Scanner scanner = new Scanner(System.in); int a = scanner.nextInt(), b = scanner.nextInt(), c = scanner.nextInt(); equals(a, b, c); } static short i = 0; public static void equals (int a, int b, int c){ int x = a, y = b, z = c; if(x == y) { System.out.print(x + " " + y); if(x == z) { System.out.println(" " + z); return; } else System.out.println(); } if(Solution.i != 1 && Solution.i != 2) { Solution.i++; equals(x, z, y); } if(Solution.i != 2) { Solution.i++; equals(y, z, x); } } }

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

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

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

Justinian Judge в Mega City One

Master

7 июля 2019, 14:37

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

Ihor Full Stack Developer

8 июля 2019, 10:48

вынести i в метод equals? при объявлении ей же нужно присвоить 0, а с рекурсией получается условные операторы добавляют по 1, чтобы выполнялось только по 1 разу, а потом при рекурсии i снова становится 0. Можно конечно без рекурсии решить задачу, просто в условных операторах перебрать все варианты, но я думал этот способ короче и читабельнее

Justinian Judge в Mega City One

Master

8 июля 2019, 11:37

В данном случае i это жесткий костыль, я понимаю что это удобней для тебя в данном случае было, но поэтому это костылем и называется, что вместо того чтобы прикрутить ножку к стулу, энциклопедии подставляют... Статические переменные и члены класса в принципе не совсем ООП, и если от методов отказаться еще сложно в ряде случаев, то статические переменные уж точно должны использоваться исключительно там где нужны по логике задания, это со временем понимание придет. Вариантов как решить очень много, можно с рекурсией, можно без рекурсии, можно перебором. Конкретно эта задачка просто на использование условных операторов, у меня в коде дополнительный метод который в два раза короче и раз в 10 читабельней, просто перебор условий - 4 строки с условиями, 4 строки с выводом. Я могу только поддержать твое мышление на самом деле, делать код универсальным это очень важный принцип, и хорошо что ты его понимаешь и применяешь, но в этой задаче можно и в лоб, а можно как ты, просто доведи решение до работоспособного варианта, который устроит валидатор.

Ihor Full Stack Developer

8 июля 2019, 19:46

Пошел другим способом (почти в лоб) и решил фактически в 2 строки:

public static void equals (int a, int b, int c){
     String s = (a == b && a == c) ? (a + " " + b + " " + c):
               a == c ? (a + " " + c):
              (a == b ) ? (a + " " + b) :
              b == c ? (b + " " + c) : "";
      System.out.print(s);
}

А можно посмотреть твое решение?