Skip to content
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

Update calculator.c to display full calculation. #2249

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

yogirajbshinde21
Copy link

@yogirajbshinde21 yogirajbshinde21 commented May 7, 2023

In this contribution, I added more descriptive output to the calculation results. Now, the program will display the full calculation entered by the user along with the result. This makes the program more informative and user-friendly. Additionally, I added an extra check to the '%' operator case to prevent division by zero errors.

Pull Request checklist -

  • This pull request fixes #issue_reference (Create an issue if not already present.)
  • The directory structure is of the form (directory_name) (If directory doesn't exist, create a new.)
  • Don't duplicate stuff.

Make sure all the above conditions are met before a PR is created.

In this contribution, I added more descriptive output to the calculation results. Now, the program will display the full calculation entered by the user along with the result. This makes the program more informative and user-friendly. Additionally, I added an extra check to the '%' operator case to prevent division by zero errors.
Copy link

@Sidharthzidzid Sidharthzidzid left a comment

Choose a reason for hiding this comment

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

The update is good

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.

None yet

2 participants