What if i want to make a function out of it to be called later?

Question

So given the array of names we should get all the names strrting with 'S' in a new array. 
What if i want to leave the whole code as a function to be called later when i need to get names with 'S'? 
The only problem i encountered is that when i call the function it gives me not only the array with the names with 'S' but also and 'undefined' afterwards. Where does that 'undefined' come from? Thanks for help
```javascript
const names = ['Selma', 'Ted', 'Mike', 'Sam', 'Sharon', 'Marvin'];
    // Result: ['Selma', 'Sam', 'Sharon'];
let namesWithS = [];
function namesWithSfunction() { 
  names.forEach((name) => {
    if (name.charAt(0) === 'S') {
      namesWithS.push(name);
    }
  });
  console.log(namesWithS);
};
console.log(namesWithSfunction());
```
And here's what i get in the console
```javascript
treehouse:~/workspace$ node iteration.js                                                                                
[ 'Selma', 'Sam', 'Sharon' ]                                                                                            
undefined                   
```

Steven Parker · Accepted Answer

This function does not "return" anything, so when you call it inside a "console.log", the log displays "undefined".
You can easily convert it to return the list instead of logging it internally:
javascript
  console.log(namesWithS);  // replace this line...
  return namesWithS;        // with this

Welcome to the Treehouse Community

Looking to learn something new?

RODRIGO MARTINEZ

RODRIGO MARTINEZ

What if i want to make a function out of it to be called later?

1 Answer

Steven Parker

Steven Parker

RODRIGO MARTINEZ

RODRIGO MARTINEZ