nus-cs2113-AY2324S1 / forum

0 stars 0 forks source link

Level-8, Replacing String with LocalDate #23

Closed StanleyW00 closed 1 year ago

StanleyW00 commented 1 year ago

I'm wondering for this level, do we replace the variables holding from, to, and by, from string to localdate, so that it only accepts a specific date and time format?

okkhoy commented 1 year ago

sure! you can do that :-) we leave specific implementation details to you to decide. We specify the following: image However, I'm curious, if not the way you specify, what other approach have you thought about?

StanleyW00 commented 1 year ago

I'm thinking of making somekind of indicator to specify which type to use when creating a deadline or an event. I believe that the former is simpler and I think it should be faster to implement.

okkhoy commented 1 year ago

Think from the user's perspective. Would the user really care if you stored it as a string or a date? All the user would want is a way to store the date and time. 🙂

StanleyW00 commented 1 year ago

Ahhh okay then. Anyway, thank you for answering, prof!