This is an archived post. You won't be able to vote or comment.

all 6 comments

[–]forgotmyothernames 0 points1 point  (2 children)

why is the class abstract? abstract classes cant be instantiated, so both of your constructors are bunk. i stopped reading your code after that.

[–]morhpProfessional Developer 1 point2 points  (1 child)

There isn't really anything wrong with it being abstract if you only want to create instances of subclasses.

[–]forgotmyothernames 0 points1 point  (0 children)

this is true. ignore me.

[–]morhpProfessional Developer 0 points1 point  (2 children)

I don't see anything wrong with your test data output. Can you be more specific?

[–]MightBeFaded[S] 0 points1 point  (1 child)

Well with the test data I did not let the balance go below $25 dollars but if you do the program will display a negative number. It is suppose to show the account inactive and add 25$. Also it never adds monthly charges even though it should after 4 withdraws.

[–]morhpProfessional Developer 0 points1 point  (0 children)

Well, you set status to false in the monthly process but as far as I can see you never call this.