Skip to content

Charlie Winslow - #139

Open
cwinslow22 wants to merge 9 commits into
bloominstituteoftechnology:masterfrom
cwinslow22:master
Open

Charlie Winslow#139
cwinslow22 wants to merge 9 commits into
bloominstituteoftechnology:masterfrom
cwinslow22:master

Conversation

@cwinslow22

Copy link
Copy Markdown

Here for the pull request

Comment thread assignments/closure.js
// Write a simple closure of your own creation. Keep it simple!

let simple = () => {let x = 5};

@Nate152 Nate152 Jun 27, 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.

I'm not sure how clearly this demonstrates that you understand closure. I can see how you could convince me, but without having that conversation this isn't super explicit. Josh's example was this:

let myName = "Josh";

function sayHello() {
  let example = "jim";
  // accessing myName outside this function
  return `Hello ${myName}`;
}
console.log(sayHello());

console.log(example);

Note how he defines the variable outside the scope of the function, but calls it from within. There are also variables defined within the function that we could talk about not being able to be called from outside. Just a bit more explicit in its demonstration of the concept.

@Nate152

Nate152 commented Jun 27, 2018

Copy link
Copy Markdown

Good job Charlie. I love to see the steady improvement over the course of the week. We will continue to cement the fundamentals as we go but I think you are doing a great job so far. This was day 2. Its all about perspective a lot of the time and it's super easy to lose sight of that. Especially when we are frustrated. Keep having a great attitude and we will keep making sure you have the tools to succeed.

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