Make 'Anagrams of string' fully functional

Gets rid of mutable `.push` (and param reassignment) inside `.map`
This commit is contained in:
Jussi Saurio
2017-12-12 22:23:44 +02:00
committed by GitHub
parent 292411a812
commit f0982a01a3

View File

@ -9,8 +9,7 @@ Base cases are for string `length` equal to `2` or `1`.
const anagrams = str => { const anagrams = str => {
if(str.length <= 2) return str.length === 2 ? [str, str[1] + str[0]] : [str]; if(str.length <= 2) return str.length === 2 ? [str, str[1] + str[0]] : [str];
return str.split('').reduce( (acc, letter, i) => { return str.split('').reduce( (acc, letter, i) => {
anagrams(str.slice(0, i) + str.slice(i + 1)).map( val => acc.push(letter + val) ); return acc.concat(anagrams(str.slice(0, i) + str.slice(i + 1)).map( val => letter + val ));
return acc;
}, []); }, []);
} }
// anagrams('abc') -> ['abc','acb','bac','bca','cab','cba'] // anagrams('abc') -> ['abc','acb','bac','bca','cab','cba']