search

jfroelich / rss-reader

A simple Chrome extension for viewing RSS feeds
Other
10 stars 0 forks source link

db and cdb should be object oriented #728

Closed jfroelich closed 5 years ago

jfroelich commented 5 years ago

I've gone back and forth a few times. However, I think the biggest clue really is that 99% of methods in these modules share the same first parameter. One of the reasons I did not like it was because I really like function-oriented design. However in this case after observing the code for quite a while I am beginning to think the OO makes sense. Here at least.

jfroelich commented 5 years ago

Created Entry and Feed. However, still need to consider Resource, and also consider Db and ChanneledDB.

jfroelich commented 5 years ago

done