Roman Numeral To Decimal

Question

Пытается внедрить очень простой конвертер Roman Numeral в Decimal, но не может показаться, что программа возвращает -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 в if()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

Не совсем ответ, просто немного веселья/альтернативный способ смотреть на проблему. Он решает проблему, если вы не планируете заказывать только добавление значений «цифр».

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; 
}