search

AY2122S2-CS2113-F10-3 / tp

0 stars 4 forks source link

Code Review: @bbawj #26

Closed xseh closed 2 years ago

xseh commented 2 years ago

@bbawj

src/main/java/seedu/duke/Parser.java:

  1. Line 35: Consider removing the comment since it looks self-explanatory
  2. Line 25, 28, 31, 34, 37, 40, 43, 46, 49: Avoid magic literals

Screenshots:

  1. image
  2. image

src/main/java/seedu/duke/TravelPackage.java:

  1. Avoid complicated expressions

Screenshots:

  1. image