Skip to content

Commit

Permalink
fix my copy and paste badness
Browse files Browse the repository at this point in the history
  • Loading branch information
quii committed Jul 10, 2019
1 parent ad22865 commit 9cdc8a0
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions roman-numerals.md
Expand Up @@ -216,7 +216,7 @@ The code looks better to me and describes the domain _as we know it right now_.

### The Romans were into DRY too...

Things start getting more complicated now. The Romans in their wisdom thought repeating characters would become hard to read and count. So a rule with Roman Numerals is you cant have the same character repeated 3 times in a row.
Things start getting more complicated now. The Romans in their wisdom thought repeating characters would become hard to read and count. So a rule with Roman Numerals is you cant have the same character repeated more than 3 times in a row.

Instead you take the next highest symbol and then "subtract" by putting a symbol to the left of it. Not all symbols can be used as subtractors; only (1), X (10), C (100) and M (1,000).

Expand Down Expand Up @@ -444,7 +444,7 @@ Here are some test cases, try and make them pass.
{"40 gets converted to XL", 40, "XL"},
{"47 gets converted to XLVII", 47, "XLVII"},
{"49 gets converted to XLIX", 49, "XLIX"},
{"50 gets converted to XLIX", 50, "L"},
{"50 gets converted to L", 50, "L"},
```

If you're a cheater, all you needed to add to the `RomanNumerals` array is
Expand Down
2 changes: 1 addition & 1 deletion roman-numerals/v8/numeral_test.go
Expand Up @@ -28,7 +28,7 @@ func TestRomanNumerals(t *testing.T) {
{"40 gets converted to XL", 40, "XL"},
{"47 gets converted to XLVII", 47, "XLVII"},
{"49 gets converted to XLIX", 49, "XLIX"},
{"50 gets converted to XLIX", 50, "L"},
{"50 gets converted to L", 50, "L"},
}

for _, test := range cases {
Expand Down

0 comments on commit 9cdc8a0

Please sign in to comment.