Skip to content

Jess Harrison Javascript-II - #259

Open
jessharrison83 wants to merge 2 commits into
bloominstituteoftechnology:masterfrom
jessharrison83:master
Open

Jess Harrison Javascript-II#259
jessharrison83 wants to merge 2 commits into
bloominstituteoftechnology:masterfrom
jessharrison83:master

Conversation

@jessharrison83

Copy link
Copy Markdown

Initial commit - array-methods.js and callbacks.js completed

@clarakosi

array-methods.js and callbacks.js completed
completed closure.js with stretch goal, function-conversion.js

@clarakosi clarakosi 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.

Great job, Jess! All of your code looks great 💯

let largeShirts = [];
const sizeFiltered = function(___, index, arr) {
if (arr[index].shirt_size === "L" || arr[index].shirt_size === 'l') {
return true;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Great use of filter here!

Comment thread assignments/callbacks.js
firstItem(items, itemZero);

function sayLength(length) {
console.log(length);

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I love that you wrote your own cb functions 👍

Comment thread assignments/closure.js
// Return an object that has two methods called `increment` and `decrement`.
// `increment` should increment a counter variable in closure scope and return it.
// `decrement` should decrement the counter variable and return it.
let count = 0;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Fantastic 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