Skip to content

Jennifer King - JavaScript I#275

Open
pghrogue wants to merge 3 commits intobloominstituteoftechnology:masterfrom
pghrogue:master
Open

Jennifer King - JavaScript I#275
pghrogue wants to merge 3 commits intobloominstituteoftechnology:masterfrom
pghrogue:master

Conversation

@pghrogue
Copy link
Copy Markdown

@pghrogue pghrogue commented Aug 15, 2018

Objects

  • Complete each challenge presented before moving on to Arrays.

Arrays

  • The arrays.js assignment takes us through a large data set of used cars. You have been asked to help a used car business with some customer requests based on their inventory. Use for loops and arrays to solve their problems.

  • Utilize the the array inventory to complete your challenges

  • You are not permitted to use map, reduce, or filter to solve these problems. Only use a basic for loop.

  • Complete each challenge presented before moving on to stretch.

Stretch

  • Move on to tomorrow's content and start studying callbacks, write a few of your own to get the hang of it.
  • Look at array methods like .map(), .reduce(), .filter(). use them on the data in the arrays assignment to accomplish the same things you did with the ES5 for loop.

Copy link
Copy Markdown

@kiachoi kiachoi left a comment

Choose a reason for hiding this comment

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

✔️ What Went Well:

  • Great work on arrays.js and objects.js
  • Clean commit messages and code

🎗 Recommendation(s):

  • I noted one about nesting the objects into one.


// Write your intern objects here:

const mitzi = {
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

You have this written as separate objects, could you think of another way where these objects are nested into one??

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