2008 Exam answers
From CSSEMediaWiki
(Difference between revisions)
(→Question 1) |
|||
| Line 5: | Line 5: | ||
* Replace Stock class with CompanyStock and OwnedStock. These classes will not share a common super class as they differ greatly in what they represent. | * Replace Stock class with CompanyStock and OwnedStock. These classes will not share a common super class as they differ greatly in what they represent. | ||
**'''Violations:''' [[Single responsibility principle]] | **'''Violations:''' [[Single responsibility principle]] | ||
| − | * Broker and StockHolder should both hold a reference to a ContactDetails object. Alternatively they could both inherit from a Person class, but if we favor composition over inheritance this would seem the less preferred solution. | + | * Broker and StockHolder should both hold a reference to an Account object. This would replace the brokersAccount property of Broker. The Account class would manage Transactions and include a reference to the account holder (Broker or StockHolder). |
| + | **'''Violations:''' [[Single responsibility principle]] | ||
| + | * Broker and StockHolder should both hold a reference to ContactDetails object. Alternatively they could both inherit from a Person class, but if we favor composition over inheritance this would seem the less preferred solution. This also wouldn't be a good solution if a Broker can be a company as well as a person. This would allow personal details of Brokers and StockHolders to be stored. In the current design the notion of a Person is mixed with that of an Account. The improved design separates these two and would allow someone to have multiple accounts. | ||
**'''Violations:''' [[Single responsibility principle]] | **'''Violations:''' [[Single responsibility principle]] | ||
Revision as of 00:29, 17 July 2009
Question 1
Possible improvements to the design:
- Replace the Pair class with a Transaction class and two subclasses CreditTransaction and DebitTransaction. These classes will contain the information that was contained in Pair, and any other behavior related to transactions which is likely to have leaked out into the rest of the system.
- Violations: name unknown; something related to a class capturing behavior as well as holding data
- Replace Stock class with CompanyStock and OwnedStock. These classes will not share a common super class as they differ greatly in what they represent.
- Violations: Single responsibility principle
- Broker and StockHolder should both hold a reference to an Account object. This would replace the brokersAccount property of Broker. The Account class would manage Transactions and include a reference to the account holder (Broker or StockHolder).
- Violations: Single responsibility principle
- Broker and StockHolder should both hold a reference to ContactDetails object. Alternatively they could both inherit from a Person class, but if we favor composition over inheritance this would seem the less preferred solution. This also wouldn't be a good solution if a Broker can be a company as well as a person. This would allow personal details of Brokers and StockHolders to be stored. In the current design the notion of a Person is mixed with that of an Account. The improved design separates these two and would allow someone to have multiple accounts.
- Violations: Single responsibility principle
