Open
Conversation
The information files have been updated to describe the current state of the repository. Repository owner information added to README.md
Status.txt now contains the output for the git status command.
answers.txt is an empty file
answer.txt has all the answers to part 3 of the assignment
Some of the hitpoints and armor ignoring is incorrect, copied from commonspear
Added the simplehammer and crazyrandomsword calls for getWeapon
main.cpp now creates the simplehammer and crazyrandomsword and simulates their use
CrazyRandomSword hitpoints is now a random number between 7 and 100
If armor is below 30, armor is ignored
Damage now ignores a random amount of integer armor points, ranging from to 2 to a third of the armor the weapon hits
LightSaber has either 1 or 100 hitpoints
Main now calls LightSaber and simulates its use.
Either they know the ways of the force and get minimal damage, or they don't and they get a ton of damage.
Weaponfactory now includes the three new weapons in the include"" header
Lightsaber now simplifies the random number generation
Author
|
complete |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Dev complete