Umm...

How to remove a lot of`pos += 1`

and `pos < len_str`

```
class Solution:
def romanToInt(self, s: str) -> int:
if not s:
return 0
pos = 0
len_str = len(s)
numOfDigits = 0
symbols = ['dummy', ('I', 'V', 'X'), ('X', 'L', 'C'), ('C', 'D', 'M'), ('M')]
s = s[::-1]
value = 0
while pos < len_str:
num = 0
numOfDigits += 1
# ex. I ... III, VI ... VIII
if s[pos] == symbols[numOfDigits][0]:
num = 1
pos += 1
while pos < len_str and s[pos] == symbols[numOfDigits][0]:
num += 1
pos += 1
if pos < len_str and s[pos] == symbols[numOfDigits][1]:
num += 5
pos += 1
# ex. V, IV
elif s[pos] == symbols[numOfDigits][1]:
num = 5
pos += 1
if pos < len_str and s[pos] == symbols[numOfDigits][0]:
num -= 1
pos += 1
# ex. IX, XC, M
elif s[pos] == symbols[numOfDigits][2]:
num = 10
pos += 1
if pos < len_str and s[pos] == symbols[numOfDigits][0]:
num -= 1
pos += 1
value += num * (10 ** (numOfDigits - 1))
return value
```

## Top comments (4)

Besides

`pos += 1`

appearing multiple times in the code, I think there is a bigger problem with`s[pos] == symbols[numOfDigits][X]`

: it makes it difficult to understand what number each roman character ismappedto. Since each roman character is mapped to a decimal number, it makes sense putting them in a`dict`

like so:This makes things easy. Let's assume for a moment that we have a roman number without reverse-order subtractions (e.g: without

`IV`

,`IX`

).Than you could simply write:

This is very neat. And can even be written in one line:

`sum(ROMAN_TO_DECIMAL[c] for c in s)`

How to add the reverse-order subtractions now? Consider and try to implement this pseudo-code:

Wow! Thank you for the detailed explanation and code example!😍

I implemented:

I could not come up with the idea of

`ROMAN_TO_DECIMAL`

.😭When thinking about this problem, did you first think about

`ROMAN_TO_DECIMAL`

and then the logic?Or is the logic first?

If it's ok with you, could you tell me about your thought process?

I am sorry to disappoint you but I don't really know how I came up with this or in what order. I have solved this question a while ago.

`ROMAN_TO_DECIMAL`

makes sense becausefor eachroman numeral there isonecorresponding number. There is a one to onemappingbetween them, and`dict`

is python'smap.I think that the bottom line is experience. The more questions you solve on leetcode, see how other people's solutions look like, you develop more intuition as to what are the data-structures and algorithms suitable for each problem.

Also, you might find these books helpful, depends on what your goals are, and how much spare time you have:

Thank you for your helpful reply.

I thought that the key solved the problem is awareness of "each roman numeral there is one corresponding number".

I will continue to solve problems and gain experiences, with other people's solutions.

And thanks for introducing books!😍