Skip to content

Update calculateCompoundInterest.py #476

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

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

Conversation

Om149
Copy link

@Om149 Om149 commented Aug 4, 2025

Refactored the code into clear, modular functions: get_user_input(), calculate_future_value(), and main().

Replaced repetitive prints with emoji-enhanced UI prompts for better readability.

Added basic input validation and error handling.

Applied consistent formatting and added inline comments to clarify logic.

Removed unnecessary conditions from loop.

PR Title

Give a title to your PR

Summary

summary...

Description

The changes are as follows:

  • Change 1
  • Change 2

Checks

in the repository

  • Made no changes that degrades the functioning of the repository
  • Gave each commit a better title (unlike updated README.md)

in the PR

  • Followed the format of the pull_request_template
  • Made the Pull Request in a small level (for the creator's wellfare)
  • Tested the changes you made

Thank You,

[Your Name]

Refactored the code into clear, modular functions: get_user_input(), calculate_future_value(), and main().

Replaced repetitive prints with emoji-enhanced UI prompts for better readability.

Added basic input validation and error handling.

Applied consistent formatting and added inline comments to clarify logic.

Removed unnecessary conditions from loop.
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.

1 participant