Logic should be correct, but failing verification

Question about the task

Do we have a pair?

Java Syntax, Level 4, Lesson 4

Resolved

Seems to be an issue here: else if((a==b)&&(b==c)){ System.out.println(a+" "+b+" "+c); }

Use the keyboard to enter three integers. Determine whether there is at least one pair of equal numbers among them.
If such a pair exists, display the numbers separated by a space.
If all three numbers are equal, then display all three.

Here are some examples:
a) if you enter the numbers
1
2
2
then we display
2 2

b) if you enter the numbers
2
2
2
then we display
2 2 2

Requirements:

The program should read the numbers from the keyboard.
The program should contain System.out.println() or System.out.print().
If two numbers are equal to each other, you must display them on the screen.
If all three numbers are equal, then display all three.
If there are no equal numbers, don't display anything.

package com.codegym.task.task04.task0417; /* Do we have a pair? */ import java.io.*; import java.util.Scanner; public class Solution { public static void main(String[] args) throws Exception { //write your code here Scanner in=new Scanner(System.in); int a=in.nextInt(); int b=in.nextInt(); int c=in.nextInt(); if((a==b)||(b==c)){ System.out.println(b+" "+b); } else if(c==a){ System.out.println(c+" "+c); } else if((a==b)&&(b==c)){ System.out.println(a+" "+b+" "+c); } else{ } } }

Comments (3)

Popular
New
Old

You must be signed in to leave a comment

Denis Lupanov

Level 9 , Minsk, Belarus

11 March 2019, 14:08

Try this code: public static void main(String[] args) throws Exception { //write your code here Scanner in=new Scanner(System.in); while (true) { // add this cycle to check how code is working. Comment it before verify int a = in.nextInt(); int b = in.nextInt(); int c = in.nextInt(); if ((a == b || b == c) && a != c) { // != means "a" not equal "c" System.out.println(b + " " + b); } /*else*/ if (c == a && a != b) { // you don't need this "else" System.out.println(c + " " + c); } /*else*/ if ((a == b) && (b == c)) { // you don't need this "else" System.out.println(a + " " + b + " " + c); } /*else { //thi "else" body si empty, thus you don't need this }*/ } // add this cycle to check how code is working. Comment it before verify }

Jeff Jeffy

Level 15 , Irvine, United States

11 March 2019, 00:41

And no, I do not need to put c==a, because if a=b, and b=c, then a=c.

Guadalupe Gagnon

Level 37 , Tampa, United States

11 March 2019, 14:15

solution

If you enter 3 identical numbers then the code only prints 2 of them when it should print 3. This is because your first if statement "if((a==b)||(b==c))", when it evaluates to true, will be the only statement run because every other one is an 'else' statement. Logically, you want the to capture the largest pull first (when all are the same. When this is not true then you can check each pair for matches in the else blocks, so the if-else blocks should look like this: #1 check if all three are the same, print result #2 check if value 1 is equal to value 2, print result #3 check if value 1 is equal to value 3, print result #4 check if value 2 is equal to value 3, print result (numbers 2-4 can be in any order)