-
Notifications
You must be signed in to change notification settings - Fork 14
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Convert object struct to Object class #8
Comments
might have bit off more than I can chew but I'm going to attempt this. I'll probably need your help on testing this. |
Yeah objects are going to be a bitch. I scanned the work you've pushed up and it looks good so far. Hopefully I'll have some time available next week. |
I finished most of it last night and have a working object class with lots of refactoring done. I'll check it in when I get home. Going to do a couple of dungeon crawls when I get home today and try to cause some crashes. I'm not really sure how to test this well--do you have any better ideas? |
Seems to be working okay. I was able to test various objects, cursed/blessed items, money, and activate-able, and unique items. I'm pretty busy this weekend, but this object should definitely be broken into abstract classes and virtual methods by various object types. This of course would be much harder than what I've done here. |
Nice, I'm going to review #15 shortly. As far as testing, there is a wizard system in there that was presumably added to make testing easier. I need to review how it works and possibly revamp it. IIRC your Unix user name has to match the hardcoded name string - not ideal. |
Oh yeah, you bash the top left corner of Rampart and you can enter "Wizard" mode. That's what I generally use to test what I can. Even with Wizard mode, there are still situations where even being Omega God cannot help you with, such as this. It's feels nearly impossible to fully test something this huge. To your credit Monsters is much bigger ^^ |
No description provided.
The text was updated successfully, but these errors were encountered: