Skip to content

Made changes to Options, RollNo. check, CGPA Check - #491

Open
Sumanth1997 wants to merge 1 commit into
souravjain540:mainfrom
Sumanth1997:main
Open

Made changes to Options, RollNo. check, CGPA Check#491
Sumanth1997 wants to merge 1 commit into
souravjain540:mainfrom
Sumanth1997:main

Conversation

@Sumanth1997

Copy link
Copy Markdown

I made the following changes,

  1. Now, when the code is run, the main function will be executed, which will first give the user options to select from.
  2. Added a check to see if the input for the roll number is an integer; if not, an error message will be printed.
  3. Added a check to see if the input for CGPA is a decimal number; if not, an error message will be printed.
  4. In the option "Remove student by RollNo", if the student is not found, an error message will be shown stating the same.
  5. In the option "View student by Name", if the student is not found, an error message will be shown stating the same.
  6. Added a new option, "Option 5", to view the list of all students.

@Ruhan-Saad-Dave Ruhan-Saad-Dave 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.

Looks good to me. But maybe consider adding documentation in each functions, as majority of programmers do that so others would understand better. Also if possible, try to optimise the code.

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