Ad
  • Custom User Avatar

    This kata has no useful directions. Can we please get rid of it? Surely, there are more sensible variations readily available.

  • Custom User Avatar

    This is a fun solution

  • Custom User Avatar

    I think I'm in love <3

  • Custom User Avatar

    Thx! Good old days of fun programming.

  • Custom User Avatar

    I'm digging your check4uniqueness helper fxn!

  • Custom User Avatar

    There's no such thing as self-documenting code.

    Come down from your ivory tower. Code can be elegant, but a single function is not itself art.

    A function, or even a library of functions, is a tool. Comments and documentation are how we explain to future users how best to implement whatever tool we've created.

  • Custom User Avatar
  • Custom User Avatar

    so MoOse and moose would be treated the same way

    Test for "moose" expects false. Test for "moOse" expects false. Relevant assertions in tests are:

    assert.strictEqual( isIsogram("moose"), false );
    assert.strictEqual( isIsogram("moOse"), false, "same chars may not be same case" );
    

    Doesn't this mean they both behave the same way? What is different about them?

  • Custom User Avatar

    The tests expects false, I don't know what you're complaining about, the test is fine, you're wrong.

    assert.equal( isIsogram("moOse"), false, "same chars may not be same case" );
    

    That is the test, if it was be case-sensitive as you claim, it would expect true because o and O would be considered different letters, but it isn't.

  • Custom User Avatar

    Please reread what I wrote and re-check your tests. As many other users have pointed out, that test fails for them, as well. Case insensitization would mean that the case wouod be ignored, so MoOse and moose would be treated the same way. That isn't happening as it should be; it's not working as intended. Check your work.

  • Custom User Avatar

    Please read the definition of case insensitive here: https://en.wikipedia.org/wiki/Case_sensitivity

    In case-insensitive, uppercase and lowercase letters are the same, and that's what the test asks.

  • Custom User Avatar

    I would appreciate some explanation from you why you consider my answers wrong, and what is the error in tests you are complaining about, because I think I am missing something:

    • The word moose is not an isogram, because the letter o occurs twice.
    • The word mOose is not an isogram, because the letter o occurs twice, with differend case (lowercase and uppercase).

    The occurrence of letters should be done in a case-insensitive way, i.e. both lowercase letters and uppercase letters contribute to the count of occurrences. And this is how it works in JavaScript translation. So, what exactly you think is wrong?

  • Custom User Avatar

    There are absolutely ways to ignore casing here. Unfortunately, one of the tests written requires case sensitivity, so your function might be perfectly correct and still fail to clear this kata.

  • Custom User Avatar

    The instructions state that the function should be case INSENSITIVE, but one of the tests requires case sensitivity (i.e.: "MoOse === moose" // false;).

  • Custom User Avatar

    This comment is hidden because it contains spoiler information about the solution

  • Loading more items...