From: Erik V. <eri...@hc...> - 2008-11-02 20:07:00
|
Feeling a bit better (and having had thinking time), I have fixed the remaining share price bug, and also reorganised the price methods as follows: - PublicCompany now has 2 new methods: getStartPrice() and getMarketPrice() which return the integer price values. PublicCompany internally knows whether the start price is just the market price or a separate par price. All code only needing the price value now use these methods. - The existing methods getParPrice() and getCurrentPrice() have been renamed to getStartSpace() and getCurrentSpace() because these return StockSpace objects. Usage of these methods is now limited to cases where these objects are really needed for their other attributes. For consistency I have also renamed setParPrice and setCurrentPrice to setParSpace and setCurrentSpace. - The redundant (and confusing) method PublicCertificate.getCertificatePrice() has been removed. Using this method had caused the aforementioned bug. Everything now seems to work correctly, although I haven't tested all games, so I keep fingers crossed.... Erik Vos |