search

bounswe / bounswe2016group8

CmpE 352/451 Group Project
Apache License 2.0
1 stars 1 forks source link

Assignment 5 - Peer Review for Group 11 #29

Open H-ymn opened 8 years ago

H-ymn commented 8 years ago

Use Case

I liked the use case diagram and the scenarios in terms of organization and content. They are well designed and group 11 did a very good job explaining the cases thoroughly. Though the diagram could have been a little bit larger.

Class Diagram

I think its okay for users to only rate integer values, but it would be better if topics and posts had average(double) ratings. Also I did not quite understand why functions like createTag and gradeAPost (of "RegisteredUser" class) don't take a string / integer as parameters.

Activity Diagram

I think everything looks alright. The diagrams are clear and precise. It's also a good features for diagrams to be large and visible since that makes the diagrams easier to understand.

Sequence Diagram

I very much liked the fact that group11 has drawn precise and well informing diagrams that are easy to understand. It makes it easier for an outsider to understand the concept.

SaitTalhaNisanci commented 8 years ago

review for group 8 ?

H-ymn commented 8 years ago

my bad... :D