• Sign Up
    Time to claim your honor
  • Training
  • Practice
    Complete challenging Kata to earn honor and ranks. Re-train to hone technique
  • Freestyle Sparring
    Take turns remixing and refactoring others code through Kumite
  • Community
  • Leaderboards
    Achieve honor and move up the global leaderboards
  • Chat
    Join our Discord server and chat with your fellow code warriors
  • Discussions
    View our Github Discussions board to discuss general Codewars topics
  • About
  • Docs
    Learn about all of the different aspects of Codewars
  • Blog
    Read the latest news from Codewars and the community
  • Log In
  • Sign Up
mmabraham Avatar
Name:Menachem Abraham
Clan:Unknown
Member Since:Jan 2017
Last Seen:Mar 2024
Profiles:
Following:2
Followers:2
Allies:1
View Profile Badges
  • Stats
  • Kata
  • Collections
  • Kumite
  • Social
  • Discourse
  • Conversations (7)
  • Replies
  • Authored
  • Needs Resolution
  • Custom User Avatar
    • ltzyp
    • commented on "Roman Numerals Decoder" ruby solution
    • 10 months ago

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

  • Custom User Avatar
    • mmabraham
    • commented on "Categorize New Member" csharp solution
    • 17 months ago

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

  • Custom User Avatar
    • Vadlusk
    • commented on "Roman Numerals Decoder" ruby solution
    • 7 years ago

    I thought this was really cool, down to two lines, but it doesn't work for cases where single numerals need to be treated as such. For example, XIX returns 21 with this solution instead of 19 like it's supposed to.

  • Custom User Avatar
    • JohnnyNiu
    • commented on "Roman Numerals Decoder" ruby solution
    • 8 years ago

    Wrong solution, should not compare the element with the accumulator, but the previous element

  • Custom User Avatar
    • mmabraham
    • commented on "Roman Numerals Decoder" kata
    • 9 years ago

    I too submitted a solution that didn't work for irregular subtractive expressions (such as "MCDXLIII").

  • Custom User Avatar
    • mmabraham
    • commented on "Roman Numerals Decoder" ruby solution
    • 9 years ago

    Thanks for finding this issue! I used #reduce incorrectly

  • Custom User Avatar
    • lucianoq
    • commented on "Roman Numerals Decoder" ruby solution
    • 9 years ago

    It's a broken solution!
    MCDXLIII is 1443 but your
    solution("MCDXLIII") is 1663
    because while reducing, (1000+100)>500 while you thought that 100<500

  • © 2025 Codewars
  • About
  • API
  • Blog
  • Privacy
  • Terms
  • Code of Conduct
  • Contact

Confirm

  • Cancel
  • Confirm

Collect: undefined

Loading collection data...