search

ui-cs383 / Class-Diagrams

PlantUML Class Diagrams
0 stars 7 forks source link

Resources, Taxation and Spending #7

Closed hallister closed 11 years ago

hallister commented 11 years ago

Team discussion for the Resources, Taxation and Spending

Current Team

@leit7193 @ztgentle0910 @chishaung

jleithart commented 11 years ago

Shaung, Tao and I will take this one since I think it's decent size and we have 3 people

hallister commented 11 years ago

The one thing I'm wondering is if the unit cost charts are necessary. I'm thinking that we could add a method to the class called UnitCost(UnitID:integer) to each of the two types of Unit Purchasing and remove the cost class altogether. The cost class is more of a data store than a class in of itself.

I'd be happy to change this myself, but what do you guys think?

jleithart commented 11 years ago

That sounds great to me. I put it in there because it was pertinent information, but it doesn't necessarily have to be seen by the user.

hallister commented 11 years ago

Cool I'll fix this and push the changes then.

hallister commented 11 years ago

Fixed the issue in the last PR, and I'm pretty happy with this (unless someone see's something I missed). Closing this as complete.