search

hornunjb / Weather-Wear

Group Project for IT3048C
0 stars 0 forks source link

Codereviewmeyer3cj #13

Open meyer3cj opened 3 years ago

meyer3cj commented 3 years ago

• Analysis of the program. The program was cleanly written. The program didn’t have much functionality yet. There were few problems with the program. • Was the program available in UC Github on time? Yes the program was available in plenty of time. • Is the program easy for you, as an outsider, to understand? I think the program is hard to understand. There are no comments. There is no documentation. I am not sure what they are trying to do. • Does the program compile? Yes. The program does compile.

  1. I learned about the best practices for java classes
  2. I learned how to restructure the main activity
  3. I learned that it is usually a good practice to put all classes in their own file.
discospiff commented 3 years ago

Several good concepts here that reduce technical debt. These are worth considering. I recommend the group review each change and decide which ones to harvest.