Overall
Good job on the readability of the code, but be sure to do your javadocs and lint your code to ensure it complies with checkstyle standards.
In src/main/java/arcs/data/staff/Staff.java
[ ] Line 45, 48, 60, 63: Try to avoid using magic numbers, you may want to declare constants on the top of the file instead.
In src/main/java/arcs/parser/StaffParser.java
[ ] The code in prepareAddStaffCommand and parseFields is quite confusing, perhaps you can consider using regular expressions or commenting more on the behaviour of the code
@LI-XINYI1
Overall Good job on the readability of the code, but be sure to do your javadocs and lint your code to ensure it complies with checkstyle standards.
In
src/main/java/arcs/data/staff/Staff.javaIn
src/main/java/arcs/parser/StaffParser.javaprepareAddStaffCommandandparseFieldsis quite confusing, perhaps you can consider using regular expressions or commenting more on the behaviour of the code