Discussion on: Daily Challenge #34 - WeIrD StRiNg CaSe

View post

Replies for: Here goes a simple oneliner: toWeirdCase=(s)=>[...s].map((e,i)=>i%2?e.toLowerCase():e.toUpperCase()).join(''); And the result: toWeir...

I believe case should be printed as CaSe as first letter (index 0) should be always capitalized.

Indeed! Didn't catch that! Here is a little fix for that:

toWeirdCase=(a)=>a.split` `.map(s=>[...s].map((e,i)=>i%2?e.toLowerCase():e.toUpperCase()).join``).join` `;

which output is:

toWeirdCase('String'); // "StRiNg"
toWeirdCase('Weird string case'); // "WeIrD StRiNg CaSe"

Ken Bellows • Aug 6 '19

Index 0 refers to the first letter of the entire string; we don't need to consider each word separately, just the string as a whole

Sung M. Kim • Aug 6 '19

Sorry about that.
I meant it as index 0 of each word.

the example shows the result should be

to_weird_case('Weird string case') # => returns 'WeIrD StRiNg CaSe'

Ken Bellows • Aug 6 '19 • Edited

Ah, you're right, that's interesting. I hadn't noticed that discrepancy with the capital 'C' in 'CaSe'; that makes this a more interesting challenge!

The original post should probably call that out a little more clearly, because it raises additional questions: Should punctuation count as a word separator? e.g., should 'word-other' become 'WoRd-oThEr' or 'WoRd-OtHeR'? Or should we only worry about letters and spaces? What characters are we considering?

@thepracticaldev , any help here?

Sung M. Kim • Aug 6 '19

For some reason, CodeWars isn't loading for me so won't be able to check the edge cases... 🤔