| Section | Grade Check Num | Lab1 (10pts) | lab2 (10pts) | Lab2 Notes | Lab3 (10pts) | Lab3 Notes | Lab4 (20pts) | Lab4 Notes | Lab5 (20pts) | Lab5 Notes (2pts per method. 1pt per instance variable 1pt per object model. Remainder, it's on-time, it works, it has proper links, iv's inited properly, and it has javadocs.) | Lab6 (30pts) | Lab6 Notes (Constructors: 1pt each. Display method: 15pts. grandchildren method: 10pts. Other: 2pts.) | Lab7 (30pts) | Lab7 Notes | Lab Points Received | Lab Points Available | Lab Average | Midterm | Midterm+15 | Final Exam | Final +15 | Test Average |
| 7 | 77 | 10 | 5 | Applet not on page. | 4.6 | Late. Only two method types. | 18 | Late | 37.6 | 130 | 28.92 | 36.6 | 51.6 | 0 | 25.8 | |||||||
| 76 | 10 | 8 | No README or link | 10 | 19 | Failed to use the Math class. | 8.5 | Instance variables should be initialized in the init method, not in their declaration. Only one method defined. Late | 14 | display and grandchildren methods both need extensive work. | 26 | No object model. | 95.5 | 130 | 73.46 | 56.6 | 71.6 | 61.8 | 76.8 | 74.2 | ||
| 9 | 50 | 10 | 10 | 8 | APIinspect | 18.1 | Variables too localized. Variables should be at method scope so they can be used in both the if and the else. Declaring the same variable names in if and else blocks adds to overhead in method execution. No display of center point and top left corner with circle. Late. | 14.5 | Instance variables should be initialized in the init method, not in their declaration. Only two methods defined. No object models. | 0 | 60.6 | 130 | 46.62 | 65.5 | 80.5 | 0 | 40.25 | |||||
| 7 | 1 | 10 | 7.1 | Late / README link | 8 | APIinspect | 0 | Lab basically not done. FamilyTree.java non-existant but a FamilyRecords.java exists but is nowhere near complete. | 10 | Lab needs extensive work. | 35.1 | 130 | 27.00 | 61.1 | 76.1 | 61.8 | 76.8 | 76.45 | ||||
| 9 | 51 | 10 | 10 | 9 | Think about comments. Link to docs & source. | 20 | 0 | 49 | 130 | 37.69 | 38.8 | 53.8 | 0 | 26.9 | ||||||||
| 8 | 25 | 10 | 10 | 10 | 19.5 | Variables too localized. Variables should be at method scope so they can be used in both the if and the else. Declaring the same variable names in if and else blocks adds to overhead in method execution. | 16.5 | Instance variables should be initialized in the init method, not in their declaration. Only one method defined. | 30 | 20 | Lab does not work correctly. | 116 | 130 | 89.23 | 62.2 | 77.2 | 58.1 | 73.1 | 75.15 | |||
| 9 | 52 | 10 | 10 | 10 | 20 | 20 | 19 | Output mislabeled. No grandchildren method. | 25 | setNewGame method needs to reset the chosenColor array to zeros. This is why the getNextColor method does not return after the setNewGame method is called from the new game button | 114 | 130 | 87.69 | 68.8 | 83.8 | 70 | 85 | 84.4 | ||||
| 15 | 26 | 8 | 8 | 130 | 6.15 | W | W | W | W | W | ||||||||||||
| 8 | 27 | 10 | 10 | 8.5 | Only 3 method types. | 19.5 | Display top left corner? | 0 | 30 | 78 | 130 | 60.00 | 65.5 | 80.5 | 63.6 | 78.6 | 79.55 | |||||
| 9 | 53 | 10 | 10 | 10 | Think about comments. | 19.5 | Be careful of integer division. | 18.5 | Instance variables should be initialized in the init method, not in their declaration. Only two object models. | 30 | 0 | 98 | 130 | 75.38 | 77.7 | 92.7 | 79 | 94 | 93.35 | |||
| 8 | 28 | 10 | 10 | 10 | Think about comments. | 19.5 | Be careful of integer division. | 20 | Excellent. | 30 | 30 | Excellent | 129.5 | 130 | 99.62 | 82.2 | 97.2 | 86.3 | 101.3 | 99.25 | ||
| 15 | 2 | 10 | 10 | 130 | 7.69 | W | W | W | W | W | ||||||||||||
| 7 | 3 | 10 | 10 | 8 | No docs or docs link. | 19 | Variables too localized. Variables should be at method scope so they can be used in both the if and the else. Declaring the same variable names in if and else blocks adds to overhead in method execution. Display Top Left corner? | 17.5 | Only defined 2 methods. Instance variables need to be initialized in the init method, not in their declaration. | 30 | 94.5 | 130 | 72.69 | 75.5 | 90.5 | 76.3 | 91.3 | 90.9 | ||||
| 8 | 29 | 10 | 10 | 9.5 | Abstract art | 18.6 | Wrong area calculation. Wrong center point calculation. Late | 0 | 30 | 78.1 | 130 | 60.08 | 52.2 | 67.2 | 44.5 | 59.5 | 63.35 | |||||
| 8 | 30 | 10 | 5 | 10 | Think about comments. | 20 | 19.1 | Instance variables should be initialized in the init method, not in their declaration. Late. | 64.1 | 130 | 49.31 | 34.4 | 49.4 | 0 | 24.7 | |||||||
| 7 | 4 | 10 | 10 | 9.5 | Abstract art. | 19 | Variables too localized. Variables should be at method scope so they can be used in both the if and the else. Declaring the same variable names in if and else blocks adds to overhead in method execution. Display of Top Left corner? | 14 | Only defined 2 methods. Instance variables need to be initialized in the init method. Graphics g is declared in the paint method and need not be declared as an instance variable. Only one instance variable. No object models. | 15 | No null checks. No grandchildren method. | 15 | Needs extensive work. | 92.5 | 130 | 71.15 | 73.3 | 88.3 | 81.8 | 96.8 | 92.55 | |
| 8 | 31 | 10 | 0 | Way too late. | 0 | Way too late. | 0 | Way too late. | 0 | Way too late. | 0 | Lab does not work. No source code available. | 10 | 130 | 7.69 | 65.5 | 80.5 | 70.9 | 85.9 | 83.2 | ||
| 9 | 54 | 10 | 10 | 10 | Think about comments. | 19.5 | Be careful of integer division. | 19.5 | Instance variables should be initialized in the init method, not in their declaration. | 30 | 30 | 129 | 130 | 99.23 | 72.2 | 87.2 | 79 | 94 | 90.6 | |||
| 8 | 32 | 10 | 10 | 10 | Think about comments. | 19.5 | Variables too localized. Variables should be at method scope so they can be used in both the if and the else. Declaring the same variable names in if and else blocks adds to overhead in method execution. | 20 | 18 | display method needs null checks. No grandchildren method. | 30 | 117.5 | 130 | 90.38 | 72.2 | 87.2 | 79 | 94 | 90.6 | |||
| 9 | 55 | 10 | 8 | No README or link | 19.5 | Display top left corner? | 0 | 25 | Second click does not show a color unless it matches the first click. | 62.5 | 130 | 48.08 | 61.1 | 76.1 | 54.5 | 69.5 | 72.8 | |||||
| 9 | 56 | 10 | 10 | 10 | Arrays, oh my. | 20 | 18.4 | Late. Found your object models. | 30 | 20 | Lab does not work correctly. | 118.4 | 130 | 91.08 | 65.5 | 80.5 | 74.5 | 89.5 | 85 | |||
| 7 | 5 | 10 | 9.5 | No README link. | 10 | 20 | 0 | 49.5 | 130 | 38.08 | 62.2 | 77.2 | 43.6 | 58.6 | 67.9 | |||||||
| 9 | 57 | 10 | 0 | Late | 0 | Late | 0 | Late | 0 | Turned in way too late. | 0 | Late. | 0 | 10 | 130 | 7.69 | 63.3 | 78.3 | 81.8 | 96.8 | 87.55 | |
| 7 | 75 | 10 | 10 | 10 | 19.2 | Late. Be careful with integer division and casting. | 18 | No instance variables. | 28 | Null checks missing or incorrect. | 28.8 | Late. | 124 | 130 | 95.38 | 73.3 | 88.3 | 75.4 | 90.4 | 89.35 | ||
| 8 | 33 | 10 | 10 | 4.8 | Very late. | 24.8 | 130 | 19.08 | 68.8 | 83.8 | 0 | 41.9 | ||||||||||
| 7 | 6 | 10 | 10 | 10 | Good comments | 20 | Comments can be taken too far. | 19.5 | Instance variables should be initialized in the init method, not in their declaration. | 23 | grandchildren method needs work. | 15 | Needs extensive work. | 107.5 | 130 | 82.69 | 53.3 | 68.3 | 69 | 84 | 76.15 | |
| 9 | 58 | 10 | 10 | 9 | Link to docs & source. | 20 | 13 | Only defined 2 methods. No object models. No instance variables. | 62 | 130 | 47.69 | 42.2 | 57.2 | 34.5 | 49.5 | 53.35 | ||||||
| 9 | 59 | 10 | 10 | 10 | Think about comments. | 19 | Variables too localized. Variables should be at method scope so they can be used in both the if and the else. Declaring the same variable names in if and else blocks adds to overhead in method execution. Display of Top Left corner? Display Center Point on Circle? | 28 | Null checks missing or incorrect. | 26 | No object model. | 103 | 130 | 79.23 | 41.1 | 56.1 | 51.8 | 66.8 | 61.45 | |||
| 8 | 34 | 10 | 10 | 9.5 | Abstract art. | 19.5 | Variables too localized. Variables should be at method scope so they can be used in both the if and the else. Declaring the same variable names in if and else blocks adds to overhead in method execution. | 19 | Instance variables should be initialized in the init method, not in their declaration. The concept of indentation should be applied to your source code. | 28 | Null check on display and grandchildren needs to check array slot, not method call. | 0 | 96 | 130 | 73.85 | 78.8 | 93.8 | 61.8 | 76.8 | 85.3 | ||
| 15 | 7 | 0 | 130 | 0.00 | W | W | W | W | W | |||||||||||||
| 8 | 35 | 10 | 9.5 | Link to README | 9.5 | link to APIinspect | 19.5 | Be careful of integer division. | 18.5 | static does not declare an instance variable, it declares a class variable (a completely different concept). The init method should initialize instance variables. | 30 | Excellent. | 30 | 127 | 130 | 97.69 | 80 | 95 | 72.7 | 87.7 | 91.35 | |
| 9 | 60 | 10 | 10 | 10 | Good comments | 30 | 130 | 23.08 | 63.3 | 78.3 | 57.2 | 72.2 | 75.25 | |||||||||
| 9 | 61 | 10 | 10 | 10 | Think about comments. | 19.1 | Wrong area calculation. Late | 49.1 | 130 | 37.77 | 63.3 | 78.3 | 0 | 39.15 | ||||||||
| 9 | 62 | 10 | 10 | 10 | Think about comments. | 19 | Variables too localized. Variables should be at method scope so they can be used in both the if and the else. Declaring the same variable names in if and else blocks adds to overhead in method execution. Be careful of loss of precision when using integer division. | 8 | Instance variables should be initialized in the init method, not in their declaration. The applet does not show up. Source compiles but will not run. | 20 | Lab does not work correctly. | 77 | 130 | 59.23 | 75.5 | 90.5 | 72.7 | 87.7 | 89.1 | |||
| 8 | 36 | 10 | 10 | 8.5 | Only 3 method types. | 19.5 | Variables too localized. Variables should be at method scope so they can be used in both the if and the else. Declaring the same variable names in if and else blocks adds to overhead in method execution. | 19.5 | instance variables should be initialized in the init method, not in their declaration. | 18 | display method needs some work. No grandchildren method. | 30 | 115.5 | 130 | 88.85 | 55.5 | 70.5 | 57.2 | 72.2 | 71.35 | ||
| 9 | 63 | 10 | 10 | 7 | Only 2 method types. | 19 | Display top left corner? Display Center Point on circle? | 6.6 | Only defined 2 methods. No attempt to put colors in. Looks like there was about 15 – 20 minutes worth of work put into the lab. No object models. Late. | 14 | display and grandchildren methods both need extensive work. | 30 | 96.6 | 130 | 74.31 | 61.1 | 76.1 | 71.8 | 86.8 | 81.45 | ||
| 8 | 37 | 0 | 9.8 | Late | 9.8 | 130 | 7.54 | 66.6 | 81.6 | 0 | 40.8 | |||||||||||
| 8 | 38 | 10 | 9.8 | Late | 10 | 19.5 | Be careful of integer division. | 18.5 | static does not declare an instance variable, it declares a class variable (a completely different concept). The init method should initialize instance variables. | 0 | 67.8 | 130 | 52.15 | 75.5 | 90.5 | 71.8 | 86.8 | 88.65 | ||||
| 9 | 64 | 10 | 10 | 7 | 19.5 | Be careful of integer division. | 16.5 | Only defined 2 methods. Only declared one instance variable (not one that was needed). Instance variables need to be initialized in the init method. | 26 | Some of the null checks are not made. No grandchildren displayed. | 30 | 119 | 130 | 91.54 | 51.1 | 66.1 | 62.7 | 77.7 | 71.9 | |||
| 7 | 8 | 10 | 10 | 8 | APIinspect. Think about coments. | 19.5 | Variables too localized. Variables should be at method scope so they can be used in both the if and the else. Declaring the same variable names in if and else blocks adds to overhead in method execution. | 17.5 | Only defined 2 methods. Instance variables need to be initialized in the init method, not in their declaration. | 12 | display method needs extensive work. No grandchildren method. | 29.4 | Late | 106.4 | 130 | 81.85 | 55.5 | 70.5 | 53.6 | 68.6 | 69.55 | |
| 8 | 39 | 10 | 10 | 10 | 19.5 | Variables too localized. Variables should be at method scope so they can be used in both the if and the else. Declaring the same variable names in if and else blocks adds to overhead in method execution. | 20 | 18 | some null checks not made or made incorrectly. No grandchildren method. | 28 | Some game logic misfires. | 115.5 | 130 | 88.85 | 48.8 | 63.8 | 41.8 | 56.8 | 60.3 | |||
| 9 | 65 | 10 | 10 | 7 | Only 2 method types. | 20 | 20 | Excellent. | 26 | some of the null checks are not made. ArrayIndexOutOfBoundsException. | 30 | Excellent | 123 | 130 | 94.62 | 68.8 | 83.8 | 69 | 84 | 83.9 | ||
| 8 | 40 | 10 | 10 | 9.5 | Abstract art. | 19.5 | Try to keep variable scope as local as possible. If a variable is only going to be used in a method, it should have method scope. No need to use that temp variable like you did. | 16.5 | Only one defined method. Found your other object models. Instance variables need to be initialized in the init method, not their declaration. | 5 | Nothing done other than copy and paste from assignment web site. | 70.5 | 130 | 54.23 | 56.6 | 71.6 | 38.1 | 53.1 | 62.35 | |||
| 8 | 41 | 10 | 10 | 9 | Abstract art. No link to docs. | 20 | 19.5 | Instance variables should be initialized in the init method, not in their declaration. | 28 | Null check on display and grandchildren needs to check array slot, not method call. | 30 | 126.5 | 130 | 97.31 | 74.4 | 89.4 | 69 | 84 | 86.7 | |||
| 7 | 9 | 10 | 10 | 8 | APIinspect | 19.5 | Variables too localized. Variables should be at method scope so they can be used in both the if and the else. Declaring the same variable names in if and else blocks adds to overhead in method execution. | 47.5 | 130 | 36.54 | 68.8 | 83.8 | 86.3 | 101.3 | 92.55 | |||||||
| 9 | 66 | 10 | 9 | Late | 10 | Think about comments. | 12.8 | No API Inspect, no output statements. very late. | 41.8 | 130 | 32.15 | 48.8 | 63.8 | 0 | 31.9 | |||||||
| 7 | 79 | 10 | 9.5 | No README link. | 9.5 | Abstract art. | 9 | No use of the Math.PI constant. Very Late. | 13.6 | Only defined 2 methods. Late | 51.6 | 130 | 39.69 | 80 | 95 | 0 | 47.5 | |||||
| 7 | 10 | 10 | 10 | 9.2 | Late | 18.7 | Variables too localized. Variables should be at method scope so they can be used in both the if and the else. Declaring the same variable names in if and else blocks adds to overhead in method execution. Late. | 47.9 | 130 | 36.85 | 0 | 0 | 0 | 0 | ||||||||
| 7 | 11 | 10 | 10 | 10 | 20 | 19.5 | Instance variables should be initialized in the init method, not in their declaration. | 25 | grandchildren method needs significant work. | 30 | 124.5 | 130 | 95.77 | 76.6 | 91.6 | 80 | 95 | 93.3 | ||||
| 7 | 12 | 10 | 10 | 9.6 | Late | 0 | Turned in WAY too late. | 0 | Turned in way too late. | 29.6 | 130 | 22.77 | 60 | 75 | 47.2 | 62.2 | 68.6 | |||||
| 9 | 67 | 10 | 10 | 19.1 | Wrong area calculation. Late | 15.5 | Instance variables should be initialized in the init method, not in their declaration. Only one object model. Only two methods defined. Arrays and loops are cool, but make sure to meet the minimum specs first. | 0 | 54.6 | 130 | 42.00 | 38.8 | 53.8 | 30.9 | 45.9 | 49.85 | ||||||
| 9 | 68 | 0 | 130 | 0 | W | W | W | W | W | |||||||||||||
| 7 | 13 | 10 | 10 | 10 | Good comments | 19.5 | Variables too localized. Variables should be at method scope so they can be used in both the if and the else. Declaring the same variable names in if and else blocks adds to overhead in method execution. | 49.5 | 130 | 38.08 | 0 | 0 | 0 | 0 | ||||||||
| 7 | 14 | 10 | 10 | 9.5 | Abstract art. | 20 | 19.5 | Instance variables in an applet should be initialized in the init method, not in a constructor. Constructors should not be used in applets. | 30 | 30 | 129 | 130 | 99.23 | 72.2 | 87.2 | 67.2 | 82.2 | 84.7 | ||||
| 8 | 42 | 2 | 2 | 130 | 1.54 | 36.6 | 51.6 | 0 | 25.8 | |||||||||||||
| 7 | 15 | 10 | 10 | 10 | Good comments | 19.1 | Instance variables should be initialized in the init method, not in their declaration. Late. | 25 | grandchildren method needs significant work. | 26 | Pressing new game button needs to clear the screen. No lines between cells. | 100.1 | 130 | 77.00 | 62.2 | 77.2 | 58.1 | 73.1 | 75.15 | |||
| 9 | 69 | 10 | 10 | 10 | Think about comments. | 19 | Variables too localized. Variables should be at method scope so they can be used in both the if and the else. Declaring the same variable names in if and else blocks adds to overhead in method execution. Display of Top Left corner and Center Point for Circle? | 17.5 | Only defined 2 methods. Instance variables need to be initialized in the init method, not in their declaration. | 20 | no grandchildren method. | 30 | 116.5 | 130 | 89.62 | 58.8 | 73.8 | 53.6 | 68.6 | 71.2 | ||
| 9 | 70 | 10 | 10 | 10 | 19 | Variables too localized. Variables should be at method scope so they can be used in both the if and the else. Declaring the same variable names in if and else blocks adds to overhead in method execution. Display Top Left and Center Point for circle? | 14.2 | Only defined 2 methods. No object models. Late | 63.2 | 130 | 48.62 | 56.6 | 71.6 | 0 | 35.8 | |||||||
| 7 | 16 | 10 | 10 | 9.4 | Late | 19 | Variables too localized. Variables should be at method scope so they can be used in both the if and the else. Declaring the same variable names in if and else blocks adds to overhead in method execution. Be careful of loss of precision when using integer division. | 17 | static final does not declare an instance variable, it declares a class constant. The init method should initialize instance variables. Late | 5 | Everything needs extensive work. No constructors, display method way off, grandchildren method way off. | 10 | Needs extensive work. | 80.4 | 130 | 61.85 | 74.4 | 89.4 | 70.9 | 85.9 | 87.65 | |
| 7 | 17 | 20 | 16 | No instance variables. Only two methods defined. Only one of the three needed object models. | 36 | 130 | 27.69 | 51.1 | 66.1 | 45.4 | 60.4 | 63.25 | ||||||||||
| 8 | 43 | 10 | 7.4 | Late | 7 | Link to APIinspect, only two method types. Double check URL before submitting. | 18.5 | Variables too localized. Variables should be at method scope so they can be used in both the if and the else. Declaring the same variable names in if and else blocks adds to overhead in method execution. Display of all information a the same y coordinate. | 42.9 | 130 | 33.00 | 46.6 | 61.6 | 0 | 30.8 | |||||||
| 9 | 71 | 10 | 10 | 10 | 20 | 18.8 | Late | 30 | 26 | No object model. | 124.8 | 130 | 96.00 | 53.3 | 68.3 | 48.1 | 63.1 | 65.7 | ||||
| 7 | 18 | 10 | 10 | 8 | APIinspect | 19.5 | Variables too localized. Variables should be at method scope so they can be used in both the if and the else. Declaring the same variable names in if and else blocks adds to overhead in method execution. | 19.6 | Late | 23 | grandchildren method does not check lotschildren anywhere. | 90.1 | 130 | 69.31 | 62.2 | 77.2 | 77.2 | 92.2 | 84.7 | |||
| 7 | 19 | 10 | 10 | 9.5 | Link to docs. | 19.5 | Be careful of integer division. | 15.5 | Instance variables should be initialized in the init method, not in their declaration. Only one method defined. No object models. | 15 | Not nearly enough null checks. Both display and grandchildren need significant work. | 15 | No game logic. No object model. | 94.5 | 130 | 72.69 | 62.2 | 77.2 | 64.5 | 79.5 | 78.35 | |
| 9 | 72 | 10 | 10 | 4 | Applet does not show up on page. | 19.5 | Try to keep variable scope as local as possible. If a variable is only going to be used in a method, it should have method scope. No need to use separate variables for circle and rectangle. | 16 | Only defined one method. Instance variables need to be initialized in the init method not their declarations. Found your object models. | 18 | Null checks incorrect. No grandchildren method. | 15 | Needs extensive work. | 92.5 | 130 | 71.15 | 64.4 | 79.4 | 58.1 | 73.1 | 76.25 | |
| 8 | 44 | 10 | 10 | 10 | 20 | 5.8 | Late. No Object Models. | 30 | 30 | 115.8 | 130 | 89.08 | 68.8 | 83.8 | 70 | 85 | 84.4 | |||||
| 15 | 78 | 10 | 10 | 130 | 7.69 | no record | no record | W | ||||||||||||||
| 9 | 73 | 10 | 10 | 10 | 19.5 | Instance variables need to be declared at the top or bottom of a class, not between methods. And, all instance variables need to be declared in the same spot. Try to keep variable scope as local as possible. If a variable is only going to be used in a method, that should be its scope. | 5 | Lab is nowhere near complete. | 15 | Does not work. Needs extensive work. | 69.5 | 130 | 53.46 | 56.6 | 71.6 | 58.1 | 73.1 | 72.35 | ||||
| 7 | 20 | 10 | 5 | Come see me. | 0 | Could not find lab 3 | 15 | 130 | 11.54 | 0 | 0 | I | I | |||||||||
| 9 | 74 | 9 | 9.8 | Late | 18.8 | 130 | 14.46 | 36.6 | 51.6 | 0 | 25.8 | |||||||||||
| 8 | 45 | 10 | 10 | 9.4 | 4 | Very late. | 33.4 | 130 | 25.69 | 55.5 | 70.5 | 0 | 35.25 | |||||||||
| 8 | 46 | 10 | 5 | Applet not on page. | 9.5 | Abstract art | 10 | None of the required figures displayed on screen. No center point calculations. No link to source or javadocs. No javadocs created. | 10 | Lab did not work and evidently never compiled. | 44.5 | 130 | 34.23 | 58.8 | 73.8 | 49 | 64 | 68.9 | ||||
| 8 | 47 | 10 | 10 | 9.5 | Link to source. | 19.1 | Variables too localized. Variables should be at method scope so they can be used in both the if and the else. Declaring the same variable names in if and else blocks adds to overhead in method execution. Late. | 19.5 | Instance variables should be initialized in the init method, not in their declaration. | 30 | 30 | 128.1 | 130 | 98.54 | 68.8 | 83.8 | 60 | 75 | 79.4 | |||
| 7 | 21 | 5 | 0 | Could not find lab. | 5 | 130 | 3.85 | 57.7 | 72.7 | 52.7 | 67.7 | 70.2 | ||||||||||
| 7 | 22 | 10 | 10 | 8 | APIinspect | 19 | Variables too localized. Variables should be at method scope so they can be used in both the if and the else. Declaring the same variable names in if and else blocks adds to overhead in method execution. Be careful of loss of precision when using integer division. | 19.5 | Found your object models. The concept of indentation should be applied to your source code. | 30 | 30 | 126.5 | 130 | 97.31 | 66.6 | 81.6 | 77.2 | 92.2 | 86.9 | |||
| 8 | 48 | 10 | 9.5 | No README link. | 10 | 20 | 14 | Only defined one method. No instance variables. No object models. | 15 | No null checks in display. No grandchildren method. No bodies in constructors. | 30 | 108.5 | 130 | 83.46 | 57.7 | 72.7 | 43.6 | 58.6 | 65.65 | |||
| 7 | 23 | 10 | 10 | 9.5 | Abstract art | 19.5 | Variables too localized. Variables should be at method scope so they can be used in both the if and the else. Declaring the same variable names in if and else blocks adds to overhead in method execution. | 16.6 | No object models. Late. | 30 | 95.6 | 130 | 73.54 | 72.2 | 87.2 | 75.4 | 90.4 | 88.8 | ||||
| 7 | 24 | 10 | 10 | 10 | 19.5 | 14 | Only one defined method. No instance variables. No object models. | 12 | display method needs extensive work. No grandchildren method. | 15 | No object model. No lines between cells. Game logic does not work correctly. | 90.5 | 130 | 69.62 | 47.7 | 62.7 | 50.9 | 65.9 | 64.3 | |||
| 8 | 49 | 10 | 10 | 8 | APIinspect | 19.5 | Think about precision loss when casting. Consider what values need to be stored and choose variable types accordingly. | 11.4 | No object models. Only one method defined. Late. | 12 | display method needs extensive work. No grandchildren method. | 70.9 | 130 | 54.54 | 61.1 | 76.1 | 46.3 | 61.3 | 68.7 |