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（true），那么我想将这四个连续值（例如IIII）更改为适当的值（例如IV ）。

问题是，它不能正常工作，check()函数总是返回0（false），即使我输入的数字如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);
}

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

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

1 个答案: