로마 숫자를 10 진수로

Question

매우 간단한 로마 숫자를 10 진수로 변환하려고 시도하지만 로마자가 아닌 숫자 문자가 문자열에있는 경우 프로그램이 -1을 반환하는 방법을 찾지 못하는 것 같습니다. 이것은 내가 지금까지 가지고있는 것이다.

#include <stdio.h> 
#include <ctype.h> 

int convertFromRoman(const char *s) 
{ 

int i = 0; 
int total = 0; 

while (s[i] != '\0') { 

    if (isalpha(s[i]) == 0) { 
     return -1; 
    } 

    if (toupper(s[i]) == 'I') { 
     total += 1; 
    } 

    if (toupper(s[i]) == 'V') { 
     total += 5; 
    } 

    if (toupper(s[i]) == 'X') { 
     total += 10; 
    } 

    if (toupper(s[i]) == 'L') { 
     total += 50; 
    } 

    if (toupper(s[i]) == 'C') { 
     total += 100; 
    } 

    if (toupper(s[i]) == 'D') { 
     total += 500; 
    } 

    if (toupper(s[i]) == 'M') { 
     total += 1000; 
    } else { 
     return -1; 
    } 

    i++; 
} 

if (total == 0) { 
    return -1; 
} 

return total; 
} 



int main() 
{ 
    printf("%d\n", convertFromRoman("XVII")); 
    printf("%d\n", convertFromRoman("ABC")); 
} 

첫 번째 것은 17을 반환하고 두 번째 것은 -1을 반환해야합니다. 그러나 둘 다 -1을 반환하지만 else 문을 제거하면 첫 번째 문은 17을 반환하고 두 번째 문단은 100을 반환합니다.

도움이 되었습니까?

Answer 1

변경 if()if()if()else 답변, 문제를 보는 재미/다른 방법의 조금별로 else if()else if()else

if (toupper(s[i]) == 'I') { 
     total += 1; 
    } 

    else if (toupper(s[i]) == 'V') { 
     total += 5; 
    } 

    else if (toupper(s[i]) == 'X') { 
     total += 10; 
    } 

    .... 

    else if (toupper(s[i]) == 'M') { 
     total += 1000; 
    } else { 
     return -1; 
    } 

Answer 2

if()합니다. "숫자"값을 추가하기 만하면 주문을 고려하지 않으면 문제가 해결됩니다.

char *romanNumerals = "IVXLCDM"; 
int values[] = { 1, 5, 10, 50, 100, 500, 1000 }; 

int convertFromRoman(const char *s) { 
    int val = 0; 
    for (int i = 0; s[i]; i++) { 
     char *idx; 
     if (NULL == (idx = strchr(romanNumerals, toupper(s[i])))) { 
      return -1; 
     } 
     val += values[idx - romanNumerals]; 
    } 
    return val; 
}