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 |
From: Mark S. <mar...@gm...> - 2008-11-02 20:44:30
|
The StockRound Class now generates 19 "Cannot Find Symbol" errors. cert.getCertificatePrice () comp.getParPrice () comp.getCurrentPrice () stockSpace = company.getCurrentPrice () Did you check in the latest Public Company, and PublicCertificate Clases? Or am I still having issues getting the latest greatest code updates? Mark On Sun, Nov 2, 2008 at 3:06 PM, Erik Vos <eri...@hc...> wrote: > 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 > > > ------------------------------------------------------------------------- > This SF.Net email is sponsored by the Moblin Your Move Developer's > challenge > Build the coolest Linux based applications with Moblin SDK & win great > prizes > Grand prize is a trip for two to an Open Source event anywhere in the world > http://moblin-contest.org/redirect.php?banner_id=100&url=/ > _______________________________________________ > Rails-devel mailing list > Rai...@li... > https://lists.sourceforge.net/lists/listinfo/rails-devel > |
From: Mark S. <mar...@gm...> - 2008-11-02 20:55:58
|
Futz... it is my Code Checkout/Update process that is getting in the way. I got a good update and it compiles fine. Sorry. On Sun, Nov 2, 2008 at 3:44 PM, Mark Smith <mar...@gm...> wrote: > The StockRound Class now generates 19 "Cannot Find Symbol" errors. > > cert.getCertificatePrice () > comp.getParPrice () > comp.getCurrentPrice () > stockSpace = company.getCurrentPrice () > > Did you check in the latest Public Company, and PublicCertificate Clases? > > Or am I still having issues getting the latest greatest code updates? > > Mark > > > On Sun, Nov 2, 2008 at 3:06 PM, Erik Vos <eri...@hc...> wrote: > >> 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 >> >> >> ------------------------------------------------------------------------- >> This SF.Net email is sponsored by the Moblin Your Move Developer's >> challenge >> Build the coolest Linux based applications with Moblin SDK & win great >> prizes >> Grand prize is a trip for two to an Open Source event anywhere in the >> world >> http://moblin-contest.org/redirect.php?banner_id=100&url=/ >> _______________________________________________ >> Rails-devel mailing list >> Rai...@li... >> https://lists.sourceforge.net/lists/listinfo/rails-devel >> > > |