Skip to content

Conversation

@pratikbhatia199
Copy link

@pratikbhatia199 pratikbhatia199 commented May 4, 2016

Requested changes

  1. A customer can transfer between their accounts
  2. Change Maxi-Savings accounts to have an interest rate of 5% assuming no withdrawals in the past 10 days otherwise 0.1%
  3. Interest rates should accrue daily (incl. weekends), rates above are per-annum

Additional changes:

  1. Bank can have account of customer with unique name, done because it becomes O(1) operation to retrieve customer by name
  2. Customer has unique accounts, done because accounts of same type are not really needed. It also becomes O(1) operation to retrieve account of a given type.

Note: Could've added test cases for account but I don't know Nose really well.
Update: I added the test cases after a break to think clearly, took only half hour.

Thanks for giving me the opportunity to interview, this was a really fun activity, I hope to hear from you soon!

Remove the account statement not required

Added test cases for all accounts
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