Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

HW-ZIXUAN XU #16

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

zxuanxu
Copy link

@zxuanxu zxuanxu commented Feb 8, 2019

No description provided.

else if (num % 5 === 0) {
console.log('Buzz')
} else console.log(num)
}
Copy link
Member

Choose a reason for hiding this comment

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

This does not log anything to the console. You need to initiate the loop.


var result = num.reduce((accumulator, currentValue) => {
return accumulator + currentValue
})
Copy link
Member

Choose a reason for hiding this comment

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

I think you got the right answer... but it's hard to tell based on the console log

image


console.log('Which games are owned by both Ross and Nathan', query7);

/* =====================
Which games are exclusive to collections? In other words, only owned by either Ross or Nathan.
===================== */

var query8;
var query8 = _.difference(rossGameList,nathanGameList);
Copy link
Member

Choose a reason for hiding this comment

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

difference is insufficient - it only works in one direction. This would not capture the games nathan has that I do not have.

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