New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Enhance ui to improve user experience #90
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
These are the available topics: | ||
OOP | ||
These are the available topics and the number of available questions in each: | ||
OOP [1] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cool idea to add the number of questions
The number of quiz questions must be more than 1! | ||
------------------------------------------------------------------- | ||
------------------------------------------------------------------- | ||
There is not enough questions in the topic for the quiz! | ||
------------------------------------------------------------------- | ||
------------------------------------------------------------------- | ||
Start of quiz: | ||
You have chosen to complete 1 question. | ||
The topic chosen is: OOP | ||
------------------------------------------------------------------- | ||
1. What is encapsulation? | ||
|
||
1) A | ||
2) B | ||
3) C | ||
4) D | ||
------------------------------------------------------------------- | ||
------------------------------------------------------------------- | ||
Please choose the answer using its index (1,2,3 or 4)! | ||
------------------------------------------------------------------- | ||
------------------------------------------------------------------- | ||
Please choose only options 1, 2, 3 or 4 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice, good to check our exceptions
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good overall, seems like the interactivity has been taken up which is really good! Solid enhancement.
+ System.lineSeparator() | ||
+ "quiz t/<topic> n/<number of questions>"; | ||
+ System.lineSeparator() + "quiz t/<topic> n/<number of questions>" + System.lineSeparator() | ||
+ "(Topics available can be found using the \"topics\" command)"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh thank you for adjusting the formatting!
return new IncorrectCommand("Please choose only 1, 2, 3 or 4"); | ||
return new IncorrectCommand(ERROR_QUIZ_ANSWER_INDEX_OUT_OF_BOUNDS); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh sorry I missed this out!
private static final String MESSAGE_ANSWER_CORRECT = "Congrats! This answer is correct! Well Done!"; | ||
private static final String MESSAGE_ANSWER_CORRECT = "Great Job! That is the correct answer! Keep it up!"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh, it sounds a lot nicer!
Fixes #89