Skip to content

Nathan culver#1

Open
Nathan-Culver wants to merge 2 commits into
masterfrom
Nathan-Culver
Open

Nathan culver#1
Nathan-Culver wants to merge 2 commits into
masterfrom
Nathan-Culver

Conversation

@Nathan-Culver

Copy link
Copy Markdown
Owner

Init pull

@GannonDetroit GannonDetroit left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

You did a good job but what happened to the given code in the prototype file? How are you missing the swordsman and archer? Without them the console,logs don't run/dont' have the answers.

Comment thread assignments/prototypes.js
@@ -14,15 +14,29 @@
* dimensions (These represent the character's size in the video game)

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

good job here

Comment thread assignments/prototypes.js
* takeDamage() // prototype method -> returns the string '<object name> took damage.'
* should inherit destroy() from GameObject's prototype
*/
function CharacterStats (Attributes) {

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

good job

Comment thread assignments/prototypes.js
* should inherit takeDamage() from CharacterStats
*/

function Humanoid (Attributes) {

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

good job

Comment thread assignments/prototypes.js
@@ -102,7 +126,7 @@
console.log(archer.greet()); // Lilith offers a greeting in Elvish.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

what happen to the swordsman and archer???

Comment thread assignments/prototypes.js
@@ -102,7 +126,7 @@
console.log(archer.greet()); // Lilith offers a greeting in Elvish.
console.log(mage.takeDamage()); // Bruce took damage.
console.log(swordsman.destroy()); // Sir Mustachio was removed from the game.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

several of these are not working because you're missing code.

Comment thread assignments/this.js
@@ -1,10 +1,10 @@
/* The for principles of "this";
* in your own words. explain the four principle for the "this" keyword below.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

very good! short and concise answers that are correct and to the point. Love it.

Comment thread assignments/this.js
@@ -13,14 +13,83 @@

// code example for Window Binding

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

good job

Comment thread assignments/this.js
sayAge();

// Principle 2

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

good job, I like that you took the time to make multiple examples.

Comment thread assignments/this.js
nathan.josh.sayName();
nathan.steven.sayName();

// Principle 3

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

good job

Comment thread assignments/this.js

console.log(mike);

// Principle 4

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

good job

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.

2 participants