Skip to content

some work is done#205

Closed
hillal20 wants to merge 5 commits intobloominstituteoftechnology:masterfrom
hillal20:master
Closed

some work is done#205
hillal20 wants to merge 5 commits intobloominstituteoftechnology:masterfrom
hillal20:master

Conversation

@hillal20
Copy link
Copy Markdown

No description provided.

Copy link
Copy Markdown

@indifferentghost indifferentghost left a comment

Choose a reason for hiding this comment

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

Great

Your commits have greatly improved! Your syntax and whitespace are awesome! The in depth answers for Answers.md is fantastic.

Requested Improvements

Small things mentioned in comments.

Questions

None

Rating: {1-3}

3





Copy link
Copy Markdown

Choose a reason for hiding this comment

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

These are in depth and awesome!

### ** closure is the inside scope of a function. Nothing from the inside of the function will be handed outside unless the function is invoked and returns somthing. example : const bill = function(){
### return () => { return 'hello world'}
### };
### bill(); ======the the content of the inner function can not be rached unless we invoke the bill function and we invoke the inner function as well to get the 'hellow world'
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

We'd need to do something like bill()() to access hello world.

// newCounter(); // 2
let f = 0;
let newCounter = function () {
console.log(b)
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Is there a reason we're logging b?

@mixelpixel mixelpixel closed this Apr 20, 2018
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.

3 participants