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

Fixed simple interest calculation #103

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

lrtraviteja
Copy link

Your checklist for this pull request

  • I have committed only those files that are required.
  • I have tried commit messages concise and reflective of the code committed.
  • I have used a branch name reflective of the issue.
  • If there is relavant issue for the PR, I have used closes #<issue number> to auto-close the issue once PR is merged.

Detailed description

The final amount calculated is divided by 100 so we can calculate the interest for every 100 in the principal amount

Test plan (required)
Example :
For principal amount of 10,000 Rs
With interest rate of 10%
For a time period of 1 year

 The simple Interest for 10,000Rs with 10% interest in a one year time period is 1000Rs.

Closing issues

closes #3

Multiplication is being processed instead of addition
Instead of getting the GCD we are getting sum of given two numbers hence used the GCD function to solve the issue.
we have to calculate the total for every 100 Rs so we divide the total by 100 and print the result
@lrtraviteja lrtraviteja reopened this Feb 19, 2023
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.

Simple Interest Calculator gives wrong output
1 participant