Skip to content

Assignment Complete - #502

Open
jpwigdor wants to merge 1 commit into
bloominstituteoftechnology:masterfrom
jpwigdor:master
Open

Assignment Complete#502
jpwigdor wants to merge 1 commit into
bloominstituteoftechnology:masterfrom
jpwigdor:master

Conversation

@jpwigdor

Copy link
Copy Markdown

No description provided.

@jpwigdor

Copy link
Copy Markdown
Author

@TheBrockstar

@TheBrockstar TheBrockstar left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Objectives

With some basic JavaScript principles in hand, we can now expand our skills out even further by exploring callback functions, array methods, and closure. Finish each task in order as the concepts build on one another.

MVP

Task 1: Set Up The Project With Git

  • Fork the project into your GitHub user account
  • Clone the forked project into a directory on your machine
  • You are now ready to build this project with your preferred IDE
  • To test your console.log() statements, open up the index.html file found in the assignments folder and use the developer tools to view the console.

Task 2: Callbacks

  • Review the contents of the callbacks.js file. Notice you are given an array at the top of the page. Use that array to aid you with your callback functions.
  • Write out each function using the ES5 function keyword syntax.
  • Solve the problems listed. Save the stretch problems until you have completed Tasks 1-4.

Task 3: Array Methods

  • Review the contents of the array-methods.js file.
  • Complete the problems provided to you
  • Notice the last three problems are up to you to create and solve. This is an awesome opportunity for you to push your critical thinking about array methods, have fun with it.

Task 4: Closures

  • Review the contents of the closure.js file.
  • Solve the problems listed. Save the stretch problems until you have completed Tasks 1-4.
  • Once you have completed this task please submit a pull request against the original fork.

Stretch

  • Various Stretch Problems

Needs Improvement

James, most of this is great. However, you didn't finish MVP so I have to give you a 1 on this. Please let me know if you run into a blocker on a project so that I can help you get past it!

If this was a mistake, let me know. That said, keep in mind that this won't really reflect badly on you in any way. It's a metric that we use to look for trends, and to give students a means to self evaluate. I'm very confident in your abilities. Ask for help if you need it.

Requests for Improvement

  • Finish MVP
  • Better Commits

Commits

Only one commit was made on this. I know that you can do better than this and I'd like to see it in the future. Let me know if you're running into blockers.

Rating: 1

let ticketPriceTotal = runners.reduce((a, b) => a + b.donation, 0);
console.log(ticketPriceTotal);

// ==== Challenge 5: Be Creative ====

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This is a great opportunity to practice using helpful comments. Your comments here could have told me, or any future user, what problems you decided to solve. This is a valuable skill as it makes your code readable for future developers (and for any manager/supervisor/Team Lead whose job it is to review your code).


// Problem 3 No newline at end of file
// Problem 3
runners.forEach(x => console.log(`${x.first_name} ${x.last_name} wears a size ${x.shirt_size} shirt`));

@TheBrockstar TheBrockstar Oct 24, 2018

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This is a great idea. Well done!

Comment thread assignments/closure.js
}
})();

// ==== Challenge 2: Create a counter function ====

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

What blocked you from being able to complete this? Let me know if you need any help!

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