如何更改罗马数字转换器中的 4 个重复数字？

Question

我正在编写一个程序，将 0 到 3999 之间的整数转换为等效的罗马数字。

目前，我实际执行转换的主要功能工作正常。它取一个数字，用罗马数字变量组成罗马数字数组，直到用户输入的整数变为0：

int main(int argc, const char * argv[])
{
    int num;
    char roman[10];
    int I = 1, V = 5, X = 10, L = 50, C = 100, D = 500, M = 1000;

    printf("Please input a number from 1 to 3999:\n");
    scanf("%d", &num);

    do {
        for (int j = 0; j < 10; j++) {
            if (num >= M) {
                num -= M;
                roman[j] = 'M';
            }
            else if (num >= D) {
                num -= D;
                roman[j] = 'D';
            }
            else if (num >= C) {
                num -= C;
                roman[j] = 'C';
            }
            else if (num >= L) {
                num -= L;
                roman[j] = 'L';
            }
            else if (num >= X) {
                num -= X;
                roman[j] = 'X';
            }
            else if (num >= V) {
                num -= V;
                roman[j] = 'V';
            }
            else if (num >= I) {
                num -= I;
                roman[j] = 'I';
            }
        }
    } while (num != 0);

    // check(roman, 'X');

        printf("\nYour number in roman numerals is %s\n", roman);
}

注释行是我编写的一个单独的函数（称为check()）。理想情况下，这个新函数应该检查任何重复 4 次的字母，如果有则返回 true，否则返回 false。它看起来像这样：

int check(char * numeralArray, char numeral)
{
    int counter = 0;
    int size = strlen(numeralArray);

    for (int i = 0; i < size; i++) {
        if(numeralArray[i] == numeral)     // check for each iteration if numeral is in numeralArray[i]
            counter++;     // if it is then counter increases by 1
        else
            counter = 0;    // this resets the value to 0 if its not consecutive.

        if(counter == 4)    // this stops the loop when 4 consecutive letters have been found.
            break;
    }

    if(counter >= 4)    // if counter is 4, it means 4 consecutive letters have been found
        return 1;   // true
    else
        return 0;   // false
}

所以我的想法是，例如，我将使用该函数并将roman 和I 作为参数传递给check()。然后它会检查I 是否连续出现 4 次。如果是，则该函数返回 true（由 1 表示）。在我的 main 函数中，如果从 check() 返回的值为 1（真），那么我想将这四个连续值（例如 IIII）更改为适当的值（例如 IV）。

问题是，它不能正常工作，check() 函数总是返回 0（假），即使我输入像 14 这样的数字，而不是 XIIII，它应该是 XIV .所以我不知道如何让我的函数在这种情况下返回 true，然后我必须做什么才能对数组进行所需的更改？另外，在我的 main 函数中，我是否一定需要在 do-while 循环中使用 for 循环？或者我可以摆脱 for 循环并在每次 do-while 循环运行时通过数组递增（例如j++）？

Answer 1

正确使用，您的check() 函数可以正常工作。这是一个测试它的最小main() 函数：

int main(void)
{
    char *str[] = { "IIII", "XIIII", "XIII", "IIIXIIII", "IIIXIII" };
    enum { NUM_STR = sizeof(str)/sizeof(str[0]) };

    for (int i = 0; i < NUM_STR; i++)
        printf("%s: %d\n", str[i], check(str[i], 'I'));

    return 0;
}

输出；

IIII: 1
XIIII: 1
XIII: 0
IIIXIIII: 1
IIIXIII: 0

所以，您遇到的问题不在于 check() 函数本身。

---

这是格式化罗马数字的另一种方法。它使用表格来确定什么是可接受的（并将数字范围限制为 1 .. 3999）。在我的主库中，该结构有两个额外的字段来帮助解析和验证罗马数字字符串，将它们转换为 int。

#include <ctype.h>
#include <errno.h>

/* Header "roman.h" */
extern int fmt_roman(int i, char *buffer, size_t buflen);
/* End "roman.h" */

typedef struct Roman
{
    int           value;
    const char    chars[3];
} Roman;

enum { MAX_PERMITTED_ROMAN = 4000 };

static const Roman numerals[] =
{
    {   1000,   "M",    },
    {    900,   "CM",   },
    {    500,   "D",    },
    {    400,   "CD",   },
    {    100,   "C",    },
    {     90,   "XC",   },
    {     50,   "L",    },
    {     40,   "XL",   },
    {     10,   "X",    },
    {      9,   "IX",   },
    {      5,   "V",    },
    {      4,   "IV",   },
    {      1,   "I",    },
};
enum { NUM_NUMERALS = sizeof(numerals) / sizeof(numerals[0]) };

int fmt_roman(int i, char *buffer, size_t buflen)
{
    int rc = -1;

    if (i <= 0 || i > MAX_PERMITTED_ROMAN)
        errno = EDOM;
    else if (buflen <= 1)
        errno = ENOSPC;
    else
    {
        char *end = buffer + buflen;
        char *dst = buffer;
        const Roman *e = numerals + NUM_NUMERALS;

        for (const Roman *r = numerals; i > 0 && r < e; r++)
        {
            while (i >= r->value)
            {
                const char *src = r->chars;
                char c;
                while ((c = *src++) != '\0' && dst < end)
                    *dst++ = c;
                if (dst >= end)
                    break;
                i -= r->value;
            }
        }
        if (dst < end)
        {
            *dst = '\0';
            rc = 0;
        }
        else
            errno = ENOSPC;
    }
    return(rc);
}

和往常一样，很多代码都与错误处理相关。