Skip to content

Comments

Anagram solution#30

Open
usmailaabdoul wants to merge 3 commits intojsacademyio:masterfrom
usmailaabdoul:assignment3
Open

Anagram solution#30
usmailaabdoul wants to merge 3 commits intojsacademyio:masterfrom
usmailaabdoul:assignment3

Conversation

@usmailaabdoul
Copy link
Contributor

This is the solution to the Anagram assignment #13

}
let anagramsLength = anagrams.length;

return {
Copy link
Member

Choose a reason for hiding this comment

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

Like your attempt but this is not the return type. Like Glenn mentioned yesterday getting the right return type is what determines if your solution is correct.

}

function Anagrams(string) {
var i, j, anagrams = [];
Copy link
Member

Choose a reason for hiding this comment

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

Try to avoid using var now that you can use let and const
You can read more about why here

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