AY2122S2-CS2113T-T09-3 / tp

0 stars 5 forks source link

[Code review] @wraineflores #174

Closed slewyh closed 2 years ago

slewyh commented 2 years ago
  1. Use standard words to name your methods and variables. createArray.. is a low-level method name. How about displayPatientList() since the code prints out a lists of patients and their details in tabular form?

    Screenshot 2022-04-05 at 3 07 28 PM Screenshot 2022-04-05 at 3 07 53 PM
  2. Consider refactoring the following repeated code for better code quality.

    Screenshot 2022-04-05 at 3 14 56 PM