search

CSC207-2023Y-UofT / course-project-tindar

course-project-tindar created by GitHub Classroom
0 stars 0 forks source link

[Task 9] Address potentially redundant classes #55

Open navarhontes opened 1 year ago

navarhontes commented 1 year ago

We have some classes that may be redundant but that are ostensibly there for CA. We should either confirm with the TA that this is okay, or try to reduce these redundancies.

navarhontes commented 1 year ago 
          Might be worth explaining the meta reasons why this class exists. Honestly every confused about why this exists, because the only thing this class currently does is hold a likeListDsGateway object, call that object's methods, and do logic on it.

_Originally posted by @navarhontes in https://github.com/CSC207-2023Y-UofT/course-project-tindar/pull/26#discussion_r1279926460_

navarhontes commented 1 year ago 
          Can you elaborate on this and put it in the javadoc? Because right now it looks like this connects front end to a LikeListInputBoundary, and a LikeListInputBoundary connects to a LikeListDsGateway, and the LikeListDsGateway connects to the actual database.

_Originally posted by @navarhontes in https://github.com/CSC207-2023Y-UofT/course-project-tindar/pull/26#discussion_r1280584902_

kresimirgotovac commented 1 year ago

I think we can close this issue given our conversation with the TA during our most recent lab