Warmup

Part A: Department Bank Accounts

In bank.py, you’ll find an implementation of a BankAccount class. The class is relatively simple: there is a class owner and a current account balance. There is also the ability to specify the type of account (e.g. Checking or Savings). The class writer has documented what each method should do in a docstring.

They also worked on testing the class functionality in main(). However, there are a lot of bugs –at least five of them!– throughout both the class implementation and the test code.

Your job: help out the class writer by fixing both the implementation and the test code! Make sure to document the fixes in WARMUP.md.

Reminder

After you fix and document each bug, be sure to commit and push your changes!

Part B: A Basic Move Class

Similar to Part A, your task in this part of the Warmup will be to debug an existing class implementation. You’ll extend this class in Part 3 of the lab assignment, so make sure you’ve understood it here before moving on!

For both this lab and Lab 8 you’ll be making use of Move classes. The simplest of the move classes is the RepeatMove class, which is currently defined in repeat.py.

An instance of the RepeatMove class will store a sequence of values, and whenever asked, will return the next value in the sequence. Each value in the sequence represents a possible movement direction. When it gets to the end of its sequence, it loops back to the beginning and starts over. In other words, a RepeatMove object has two attributes:

  • self.moves contains the list of values that the RepeatMove will go through
  • self.current contains the index of the element that the RepeatMove will return next time it is asked (this should start at 0)

The constructor for a RepeatMove object takes in one parameter moves, so that the user can specify what sequence of values (representing moves) the object should use.

A RepeatMove object has one method, next(), which returns the element of self.moves at index self.current. If self.moves is the empty list, next() should return None. When self.current reaches len(self.moves), self.current should be reset to 0. Otherwise, self.current should increment every time next() is called.

There is a single error in the class definition for RepeatMove that you need to identify! There are no errors in the test code in the main implementation. Remember to document the change in WARMUP.md.

The expected correct output should be:

None
None
None
None
None
None
None
None
None
None
1
2
3
4
5
1
2
3
4
5

Reminder

Be sure to commit and push your changes after you find and document the bug!