Refactor grade evaluation to use user input

[harshmendhe-arch]

Removed hardcoded grade and added user input for grade evaluation.

[Copilot]

🟡 Changes recommended

The moderate output-behavior change and requested header, variable-naming, and capitalization fixes remain unresolved.

Get a fresh assessment by requesting another Copilot review.

Pull request overview

This pull request refactors grade evaluation to use user input instead of a hardcoded value.

Changes:

• Adds interactive grade input and pass/fail branching.
• Preserves the passing threshold of 55.
• Changes output text and removes the standard lesson header.
• Introduces a/num variable aliases.

File summaries

File	Summary and final review comments
3-control-flow/12_grades.py	Implements user-input grade evaluation. Moderate (1 vote): retain the existing You passed. and You failed. output unless the behavior change is intentional. Nits: preserve the # Grades 💯 / # Codédex header (2 votes and 1 vote), use grade directly instead of a/num (2 votes and 1 vote), and capitalize the failure message (2 votes).

Review details

Suppressed comments (2)

3-control-flow/12_grades.py:3

• The refactor drops the standard lesson header and replaces the descriptive grade variable with the ambiguous a/num alias. Every other Python exercise retains the title plus # Codédex header, so please preserve that metadata and compare the input through one grade variable.

a = int(input("grade: "))
num = a
if num>=55:

3-control-flow/12_grades.py:6

• The PR description says this change replaces the hardcoded grade with user input, but these branches also change the existing output strings and capitalization. That changes the script's observable behavior and can break expected exercise output; retain You passed. and You failed. unless changing the output is intentional.

  print('You passed the exam')
else:
  print('you failed the exam')

• Files reviewed: 1/1 changed files
• Comments generated: 3
• Review effort level: Lite

---

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