New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

feat: added js solution (and tests) for challenge palindrome-number #87

Open

aleattene wants to merge 2 commits into tomorrowdevs-projects:main from aleattene:solution/palindrome-number

Contributor

aleattene commented Nov 17, 2021

Reviewers:
@micheleriva
@storrisi

I'm not sure it's the best solution.

An alternative but the less performing solution is for example this:

function isPalindrome(number) {
    let word = number.toString()
    let wordReversed = word.split("").reverse().join("")
    return (word === wordReversed)
}

Is possible to test the performance of both the solutions here:
https://replit.com/join/tvletostle-aleattene

Any other advice is always welcome.

aleattene added 2 commits

November 17, 2021 12:45


          feat: added js solution (and tests) for challenge palindrome-number

7139c14


          style: improved the code layout

abea037

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

Labels

None yet