search

Gloor / lma

MIT License
0 stars 1 forks source link

DataModel Change? #14

Open Gloor opened 11 years ago

Gloor commented 11 years ago

I saw that the model is basically the Profile and the Data classes. Whereas the Data items resemble either an item in the week table or an item representing a month. I think it would be better to just store the comments, likes and other items as lists in the profile. In the view you can then display them using lambda queries to iterate. This would have the advantage that the same data can also be used for the charts and it would easily allow future add on such as the display of the comment with the most likes.. What do you think?

maysam commented 11 years ago

I'd say let's leave refactoring to when the prototype is finished

maysam commented 11 years ago

I think this can be done in the next phase of the project

Gloor commented 11 years ago

I disagree. I think it would make it a lot easier to change this. I think that should not be such a big thing. E.g. with data tables or even just classes, this should be fairly easy when using linq and lmbda expressions.

maysam commented 11 years ago

yes, but calculating the percentages requires all this precalculations