Im confused. Everything seems like in requirements but doesn't pass.

Question about the task

CashMachine (part 9)

Java Collections, Level 9, Lesson 15

Under discussion

1st and 2nd requirements. What else can I do?:)

Today we're going to tackle ExitCommand.
1. Implement the following logic in ExitCommand:
1.1. Ask whether the user really wants to exit. Provide the following options: yes (y) or no (n).
1.2. If the user confirms, then say goodbye.
1.3. If the user does not confirm, then don't say goodbye — simply exit the program.

This is all good, but sometimes we need to urgently interrupt an operation, for example, if the user chose the wrong operation.
This is why we have InterruptedOperationException.
2. Implement the following logic:
2.1. If at any time the user enters "EXIT", regardless of case, throw an InterruptedOperationException.
2.2. Find the only correct place to put this logic. Implement the functionality in this one method.

3. Wrap the body of the main method in a try-catch and handle InterruptedOperationExceptions.
Use ConsoleHelper to say goodbye to the user in the catch block.

Requirements:

The ExitCommand class's execute() method, ask the user whether he or she really wants to exit. Provide the following options: yes (y) or no (n). If the user confirms, then say goodbye. If not, then don't say goodbye — simply exit the program.
Implement the logic with the ability to interrupt operations (by throwing an InterruptedOperationException).
The main method should not throw exceptions.

package com.codegym.task.task26.task2613; import com.codegym.task.task26.task2613.command.CommandExecutor; import com.codegym.task.task26.task2613.exception.InterruptedOperationException; import java.util.Locale; public class CashMachine { public static void main(String[] args) { Locale.setDefault(Locale.ENGLISH); try { Operation operation; do { operation = ConsoleHelper.requestOperation(); CommandExecutor.execute(operation); } while (operation != Operation.EXIT); } catch (InterruptedOperationException e) { ConsoleHelper.writeMessage("Bye"); } } }

Comments (6)

Popular
New
Old

You must be signed in to leave a comment

JianQiu Hwang

Level 35 , Washington

21 May 2020, 12:17

I have passed the test, here is the code different:

class ExitCommand implements Command {
    @Override
    public void execute() throws InterruptedOperationException {
        while (true) {
            ConsoleHelper.writeMessage("Are you sure? yes (y) or no (n):");
            String s = ConsoleHelper.readString();
            if (s.equals("y")) {
                ConsoleHelper.writeMessage("GoodBye!");
                break;
            } else if (s.equals("n")) {
                break;
            }
        }
    }
}

Elvis Lee

Level 41 , Aurora, United States

2 March 2020, 01:39

Try to modify the message like this: "Are you sure to exit? Yes(y) or No(n)" You know the validator looks for some key words, Yes(y) or No(n), stated in the requirement

Elvis Lee

Level 41 , Aurora, United States

29 February 2020, 19:35

class ExitCommand implements Command { @Override public void execute() throws InterruptedOperationException { ConsoleHelper.writeMessage("Are you sure to exit? Yes(y) or No(n)"); String input = ConsoleHelper.readString(); if (input.equalsIgnoreCase("y")) { ConsoleHelper.writeMessage("Thank you for using ATM, goodbye!"); } } } Don't use System.exit()

Marcin Olech

Level 41 , Kraków, Poland

1 March 2020, 10:50

Unfortunately still not passing, everything works fine to me.

Elvis Lee

Level 41 , Aurora, United States

29 February 2020, 00:01

There's no need to catch and throw a new InterruptedOperationException as the method signature already includes the throw statement. It means the exception will be thrown automatically.

Marcin Olech

Level 41 , Kraków, Poland

29 February 2020, 12:29

You mean in ExitCommand.execute() method? I've corrected this but it's still not passing. Also tried with System.exit() but not working. I've spent about 4 hours now on this task:P