I laughed (and, tbh, it still deserves the Best Practice vote XD )

Man I'm surprised that someone actually replied to a 6 year old solution from when I was trolling around on Codewars :)

Aren't you supposed to implement this method on your own?

• The methods should be static
• There're already many palindrome-related katas, and this one brings nothing new or interesting

Hi,
I made similar solution, but instead of 'n' as number of rotation, I made n%data.lenght.
As a beginner my question is- does pefrormance improve in that situation? Program does not to f.e 12004 rotations, but has to make a division and get rest of that.

I'd prefer using static methods or a constructor to pass the value as the default constructor looks useless in this case.

# Before
assertTrue(new PalindromeGenerator().isPalindrome("otto"));
assertEquals("otto", new PalindromeGenerator().generateString("otto"));

# After
assertTrue(PalindromeGenerator.isPalindrome("otto"));
assertEquals("otto", PalindromeGenerator.generateString("otto"));

OR

var palindrome = new PalindromeGenerator("otto");
assertTrue(palindrome.isPalindrome());
assertEquals("otto", palindrome.generateString());

Yup, that's why I'm asking and in order to improve this Kata enough to get it published.

JavaScript translation available. Please review and approve. I clarified the description and added more test cases, to include random tests.

The test cases are much simpler- no spaces and no ASCII characters.

Needs random tests

Resolving this so it's more about random tests

It's already there

It's already there

It's already there

It's already there