Skip to content
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

Tier 1 loot #582

Closed
wants to merge 1 commit into from
Closed

Tier 1 loot #582

wants to merge 1 commit into from

Conversation

dgatewood
Copy link
Collaborator

WIP Tier 1 loot, new classes for missing loot item, additions to AceObject and WorldObject for variables that needed to be added, command added in commandHandler, etc. Much more work needs to be done to added all the necessary variables to WorldObject.

This should be discussed. I feel that my coding skills may not be on par with most others. I felt the million if/else statements and rnd numbers was excessive, but didn't know how else to do it since all items had different ranges for their stats, etc.

WIP Tier 1 loot, new class definitions for missing loot, change in commandHandler, etc.
default:
return new GenericObject(aceO);
}
}

public static WorldObject CreateCorpse(Creature cs)
{
////Console.WriteLine("Inside CreateCorpse");
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Corpse creation should probably be moved to its own WO class that is derived from Container, instead of being a method within the WorldObjectFactory class.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants