Feedback on the collated code of A0139102U

[cmkumar87]

• you have a lot of SOP lines.
  System.out.println(taskToEditName);
  Are you using them for debugging? If so, try using the actual debugger which can show the variable values at run time; it is much easier to do this way. You can also use a logger to trace runtime logging.
• Not deleting lines likes this // TODO Auto-generated method stub is bad
• Unused code commented out.
  
  //if (!isValidEmail(email)) {
  
  // throw new IllegalValueException(MESSAGE_EMAIL_CONSTRAINTS);
  
  //}
  
• Not a lot of lines of code for v0.4
• Don't see any collated test code. Note that this is separately graded.
• Docs overall team needs to add design diagrams also for newly designed features or parts redesinged. Class and sequence diagrams are the minimum.