checkpoint

kristofer · kristofer · commit fc147330d45b · 2024-06-26T13:57:26.000-04:00
diff --git a/reader/ReadingJava/Programs/Bookstore.java b/reader/ReadingJava/Programs/Bookstore.java
@@ -1,4 +1,12 @@
 
+// Make Bookstore class a "collection" class. It should have an array of Books.
+// Add a method to add a Book to the array.
+// Add a method to search for the price of a book given its title.
+// Add a method to search for the titles of all books written by a given author.
+// Add a method to search for the titles of all books with a price below a given amount.
+
+// Make a Main class that creates a Bookstore object and tests all the methods.
+
 class Author 
 {
 	private String fName;
@@ -93,4 +101,8 @@ public static void main(String[] args)
 		Book2.setPrice(735);
 		System.out.println("Title : " + Book2.getTitle() + "\nAuthor : " + Book2.getAuthor() + "\nPrice : Rs " + Book2.getPrice());
 	}
-}
+}
+
+// Change public class Bookstore to just a class Bookstore
+// Make Main a `public class` with a main method that creates a Bookstore object and tests all the methods.
+
diff --git a/reader/ReadingJava/Programs/Deal.java b/reader/ReadingJava/Programs/Deal.java
@@ -25,12 +25,23 @@
  *
  ******************************************************************************/
 
+
+
+ // This needs to be Re-Factor'd
+ // This is a simple program that deals 5-card hands at random to the given number of players.
+ // It needs to be generalized, so that it can be used for any number of players and any number of cards per player.
+ // Also the blocks of code in `main` should be broken down into methods.
+// the code is not well-organized, and it is not clear what the code does.
+// the SUITS and RANKS arrays should be declared as constants. And outside of the main method.
+
+
 public class Deal {
     public static void main(String[] args) {
         int CARDS_PER_PLAYER = 5;
 
         // number of players
         int PLAYERS = Integer.parseInt(args[0]);
+        // what does this ^^^^ mean?
 
         String[] SUITS = {
             "Clubs", "Diamonds", "Hearts", "Spades"
