Couldn't agree more! Best practice is about maintainability not clever looking solutions which are not necessarily more efficient (from most I have seen they are usually awful for performance)

I tried to do this without using strlen for a challenge and to optimise effciency, I believe this will fail if the string ends in say "baa" and the ending string is just "a" if anyone can advise on the change needed please advise.