Skip to content

Tim Zulf#158

Open
Tulf wants to merge 6 commits intobloominstituteoftechnology:masterfrom
Tulf:master
Open

Tim Zulf#158
Tulf wants to merge 6 commits intobloominstituteoftechnology:masterfrom
Tulf:master

Conversation

@Tulf
Copy link
Copy Markdown

@Tulf Tulf commented Jun 26, 2018 •
edited
Loading

Function Conversion:

  • converted all functions

Closure

  • Challenge 1
  • Challenge 2
  • Challenge 3

Array Methods

  • forEach
  • map
  • filter
  • reduce
  • creativity!

@groov1234
Copy link
Copy Markdown

groov1234 commented Jun 26, 2018

Hey, Tim! Looks like you're having some difficulty converting to ES6 arrow syntax. Here is a good example of an ES5 to ES6 conversion.

  'Hydrogen',
  'Helium',
  'Lithium',
  'Beryllium'
];

elements.map(function(element ) { 
  return element.length; 
}); // [8, 6, 7, 9]

elements.map(element => {
  return element.length;
}); // [8, 6, 7, 9]

elements.map(({ length }) => length); // [8, 6, 7, 9]```

@TERR1E
Copy link
Copy Markdown

TERR1E commented Jun 27, 2018

Hi Tim, here are some helpful links you can check out:
https://codeburst.io/javascript-map-vs-foreach-f38111822c0f

https://medium.com/@JeffLombardJr/understanding-foreach-map-filter-and-find-in-javascript-f91da93b9f2c

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Tulf @groov1234 @TERR1E