Skip to content

Add interactive menu with options for greeting and exit - #5

Merged
NachoOFC merged 1 commit into
mainfrom
NachoOFC-patch-5
Sep 14, 2026
Merged

NachoOFC merged 1 commit into
mainfrom
NachoOFC-patch-5

Conversation

@NachoOFC

Copy link
Copy Markdown
Member

No description provided.

Copilot AI lite review requested due to automatic review settings September 14, 2026 14:07
@NachoOFC
NachoOFC merged commit 0f7e3cc into main Sep 14, 2026
1 check passed

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 Changes recommended

The advertised date/time option and malformed-input handling remain unresolved.

Get a fresh assessment by requesting another Copilot review.

Pull request overview

Adds a standalone Java interactive console menu for greeting, status/date-time display, invalid choices, and exit.

Changes:

  • Adds a Scanner-based menu loop.
  • Dispatches options with a switch.
  • Handles greeting, status, exit, and invalid numeric options.
File summaries
File Summary Findings
menu_interactivo.java Implements the interactive menu. Moderate (3 votes): Option 2 does not display a date/time. Moderate (1 vote): non-numeric input can terminate the program.
Review details

Suppressed comments (1)

menu_interactivo.java:14

  • nextInt() throws InputMismatchException for any non-numeric entry, terminating the menu before the default case can report an invalid option. Read and parse the input defensively so users can recover from malformed input without restarting the program.
            opcion = sc.nextInt();
  • Files reviewed: 1/1 changed files
  • Comments generated: 1
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread menu_interactivo.java
System.out.println("¡Hola! Bienvenido al sistema.");
break;
case 2:
System.out.println("Entorno de desarrollo activo en UST.");
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants