Project Part 4 Feedback - lagecamp/CMPUT301W13T02 GitHub Wiki

Feedback

Code base of Prototype -- Excellent (3)

  • The classes are well organized.
  • Requirement has been completed.
  • The code is clean and is of good quality.

Tangible Demo -- Excellent (3)

  • The demo is organized. You work as a cohesive unit to make the presentation, which is good.
  • The demo is a bit short. But in general, it's good.

Code Documentation -- Good (2)

  • The Javadoc looks better.
  • Some inconsistency exists. For example, you explain param serverID for some methods, but for the other methods, there is no description for serverID. It's better to make all the description consistent. It's not easy to take everything into account though...

Tese Cases -- Good (2)

  • Test cases exist and run.
  • Some inconsistency exists. The same problem as what I mentioned in Part 3.

OOD -- Excellent (2)

  • The sequence diagram explains non-trivial scenario of your app.
  • Class diagram looks good as well.

Release Planning -- Excellent (1)

Addressing Feedback -- Excellent (1)

Refactoring -- Excellent (2)

  • Refactoring has been documented.
  • The explanation is clear.

Requirement Specification -- Excellent (3)

  • The new requirement has been thoroughly documented.

Video Demo -- Good (1)

Manager's Best Pick -- (1)

Summary -- (21) -> capped to (20)

  • Congrats!