C代码进入无限循环

Question

代码进入无限循环。 如果我删除for循环内部的所有行并替换为printf()它将正常工作。 但是保留这些行，就会进入无限循环。

如何解决这个问题？ ＃包括

#include <string.h>

void itoa(int, char[]);
void reverse(char[]);

int main()
{

    int i,j;
    for(i = 0;i<= 4; i++)
    {
        for (j = 0; j <= 9; j++)
        {
            char fileName[10]="A";
            char append[1];

                itoa(i,append);
            strcat(fileName,append);

            itoa(j,append);
            strcat(fileName,append);

            printf("i=%d j=%d\n", i,j);
            printf("%s\n", fileName);

        }
    }
}

void itoa(int n, char s[])
{
    int i, sign;

    if ((sign = n) < 0)  /* record sign */
        n = -n;          /* make n positive */
    i = 0;
    do {       /* generate digits in reverse order */
        s[i++] = n % 10 + '0';   /* get next digit */
    }
    while ((n /= 10) > 0);     /* delete it */
    if (sign < 0)
    s[i++] = '-';
    s[i] = '\0';
    reverse(s);
}
void reverse(char s[])
{
    int i, j;
    char c;

    for (i = 0, j = strlen(s)-1; i<j; i++, j--) {
        c = s[i];
        s[i] = s[j];
        s[j] = c;
    }
}

Answer 1

该程序使用以下代码来调用正式称为未定义行为的行为 ：

    char append[1];
    itoa(i,append);

itoa()在append中编写了多个元素。 可能是append[] （即append[1] ）后面的一个字节是导致无限循环的循环变量之一（ main::i或main::j ）的一部分。

Answer 2

使用sprintf进行这种事情要简单很多，例如

char fileName[32];
sprintf( filename, "A%d%d", i, j );
printf("%s\n", fileName);

注意，始终使字符串缓冲区大于所需的大小。