Open
Conversation
…, instead of just outputting it for the user.
…than 1 for their initial balance, otherwise an ArgumentError will be thrown.
…ready in their account.
…one, email, and address attributes.
…sh to create the instance variables
…file as an array of arrays, then iterates through and creates hashes from that, then puts those hashes into a new array which will be returned whenever the method is accessed.
…s what the requirements were asking for
…ed as an Account instance, THEN shovels into the account_info_array
… comments explaining my dilemma with self.find in plain old English
… bunch of old useless comments
…ted the SavingsAccount class with a super'd initializer to account for the different minimum balance requirements.
… to withdraw past the minimum balance
…e SavingsAccount and CheckingAccount withdrawal methods from doing show_balance twice thanks to super
…...withdraw_using_checks gets all messed up so I'm reorganizing what functionality needs to be in each withdraw method
… up each time you call the method, and takes away a transaction fee when you try to use more than 3 checks
…od when only the transaction fee sends it over
…t class, then linked child classes to that. Cleaned up code and deleted all superfluous comments and methods.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
I left some instances of various account types down at the bottom of the code that you can tinker with if you'd like to play around with it in IRB.
Not really any optionals this time around, sadly... I basically had to restructure all my withdraw methods from scratch on the last day to get everything working the right way, so I spent all my remaining time on that.
(I even needed to draw out a TRIPLE VENN DIAGRAM on my scratch paper to figure out which methods should contain what, like middle school science class or something! It helped.)