bounswe / bounswe2016group8

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

Assignment 5 - Peer Review for Group 11 #26

Open barisucakturk opened 8 years ago

barisucakturk commented 8 years ago

Use Cases:

Overall, the group members understand what the use case is and prepare it very well. At first glance, use case diagram may be pretty small, which is why I couldn't read properly. However, all parts of cases is explained clearly and thoroughly. I think it is a good job in general.

Sequence Diagram:

I think that the keyword should be explained in order to prevent from the misunderstanding. Again, all diagrams are very small and I couldn't see properly. So, I cannot comment anything about this situation. Another problem is that the statements at some parts are not enough but it is okey generally.

Activity Diagram:

First of all, all diagrams are visible and big enough, that is very important because flows of the chart should be traceable. I think that all diagrams are well drawn and well organized. Then, putting important note at beginning of diagram is crucial because it explains how the chart flows, which enforces understandability.

Class Diagram:

Diagram is prepared pretty good. There are some little mistakes in diagram. For example, rate should not return integer since the rate could be double. I think it is a good job generally, again.