Skip to content

Commit 6bdb109

Browse files
committed
Clarified the priority of a refactoring.
1 parent 6e43a11 commit 6bdb109

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

Diff for: documents/inbox.md

+5-5
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,10 @@
11
# Today's session plan
22

3+
- Clean up before moving on:
4+
- **Stop reusing a single instance of `PurchaseInProgress` to represent the purchase in progress. Maintaining the state has already become too complicated!**
5+
- The class and one of its properties have the same name. Improve this _somehow_.
6+
- Make the lifecycle of the Purchase in Progress more obvious in the API. For example: Purchase.begin() returns PurchaseInProgress; purchaseInProgress.complete() returns PurchaseInfo.
7+
- Can we compute `isPurchaseInProgress` without maintaining a separate state value?
38
- Print a receipt
49
- What happens if we have a total with no items? scenario
510
- Print the barcode with each item in the receipt
@@ -9,11 +14,6 @@
914
- Add a timestamp for the receipt
1015
- Line up all the decimal points
1116
- Defect: excessive whitespace when requesting a receipt for an empty Purchase.
12-
- Clean up before moving on:
13-
- **Stop reusing a single instance of `PurchaseInProgress` to represent the purchase in progress. Maintaining the state has already become too complicated!**
14-
- The class and one of its properties have the same name. Improve this _somehow_.
15-
- Make the lifecycle of the Purchase in Progress more obvious in the API. For example: Purchase.begin() returns PurchaseInProgress; purchaseInProgress.complete() returns PurchaseInfo.
16-
- Can we compute `isPurchaseInProgress` without maintaining a separate state value?
1717

1818
# Inbox
1919

0 commit comments

Comments
 (0)