search

PrycilaCerqueira / P7_UrbanRead

UrbanRead is an online library for those who enjoy a good time!
0 stars 0 forks source link

ActiveBook class #1

Closed f-rakete closed 1 year ago

f-rakete commented 1 year ago

https://github.com/PrycilaCerqueira/P7_UrbanRead/blob/38d970e1da099dbc6dc8da4493944a1c1901311e/P7_UrbanRead/P7_UrbanRead/ActiveBook.cs#L11-L19

Not too sure what the purpose of this class is? Is it the current book somebody is reading?

also:

Why are fields of the book type called "title" and "cover"?

why is there a person field called wishlist 🙃

I'll assume that this is the book somebody is currently reading, so your Person class could just have a public Book ActiveBook, no?

PrycilaCerqueira commented 1 year ago

@f-rakete Yes, it is. This class has the purpose of presenting a Reading Dashboard to each user. It presents the books they are reading and their progress. It has the WhisList available for them to edit their book list.

The title is the title of the book, and the Cover will be the picture of the cover.

The person field called wishlist is because I have this field under Person (private List _WishList);

I am not sure how to reorganize it, because under ActiveBook I am collecting: private int _PageAt; private ReadStatus _ReadingStatus;

f-rakete commented 1 year ago

ok so if i get that right i think the most sane order would be:

user has a property list of active book active book class has properties: Book active book (contains all the book data like title total pages etc) int currentPAge enum? readstatus

PrycilaCerqueira commented 1 year ago

@f-rakete, thank you! Code adjusted. The wishlist field is for the user to create their personal list of desired books.