Skip to content

Chris Juelfs JavaScript-I completed!#325

Open
CheapCharlie wants to merge 2 commits intobloominstituteoftechnology:masterfrom
CheapCharlie:master
Open

Chris Juelfs JavaScript-I completed!#325
CheapCharlie wants to merge 2 commits intobloominstituteoftechnology:masterfrom
CheapCharlie:master

Conversation

@CheapCharlie
Copy link
Copy Markdown

No description provided.

@47analogy
Copy link
Copy Markdown

Great job on the project. Couple of things to note are that since arrays are 0 indexed, for Challenge#1 in arrays.js, car with id =33 is inventory[32]. For challenge#2, you could use a for loop and push the car models to a new array, which you could apply the array sort method. For Challenge#3 in objects.js, since you created an array of objects, you could add the speak method to the the interns array, i.e. interns.speak = function () {...

@CheapCharlie
Copy link
Copy Markdown
Author

Regarding you challenge#2 notes. wouldn't the way i did it be the simpler and/or more efficient way to do it though?

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