Römische Zahl zu Dezimal

Question

Der Versuch, eine sehr einfache römische Ziffer zu Dezimal-Konverter zu implementieren, aber scheint nicht, einen Weg für das Programm zu finden, um -1 zurückzugeben, wenn nicht römische Ziffern in der Zeichenfolge sind. Das habe ich bisher.

#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")); 
} 

Der erste sollte 17 zurückgeben und der zweite sollte -1 zurückgeben. Beide geben jedoch -1 zurück, aber wenn ich die else-Anweisung entferne, gibt die erste 17 zurück, und die zweite gibt 100 zurück.

Jede Hilfe wird geschätzt.

Answer 1

ändern if()if()if()else zu 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

nicht wirklich eine Antwort, nur ein bisschen Spaß/alternative Art und Weise auf das Problem zu betrachten. Es löst das Problem, wenn Sie nicht in Erwägung ziehen, nur "Ziffern" -Werte hinzuzufügen.

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