0

我在为数组动态分配内存时遇到问题。该程序只是简单地将第一行与第二行交换，将第三行与第四行交换。我得到了奇怪的结果，例如：

输入字符串：你好

输入字符串：你好吗

输入字符串：我很好，谢谢

输入字符串：再见

输入字符串：bai

输入字符串：xx

==========================

你好吗

！我很好谢谢

你好

！你好吗

再见

!拜

我很好谢谢

！再见

白

!xx

    int count = 0;
    char *lines[MAX_LINES];
    char *tmp[50]; 


    printf("Enter string: ");
    fgets(tmp, 50, stdin);
    lines[count] = (char *) malloc((strlen(tmp)+1) * sizeof(char));
    strcpy(lines[count], tmp); 

    while(strcmp("xx\n", lines[count])){
            count++;
            printf("Enter string: ");
            fgets(tmp, 50, stdin); 
            lines[count] = (char *) malloc((strlen(tmp)+1)* sizeof(char));

            strcpy(lines[count], tmp); 
    }

void exchange(char * records[])
{
    char * temp;
    temp = records[0];
    records[0] = records[1];
    records[1] = temp;

    temp = records[2];
    records[2] = records[3];
    records[3] = temp; 

}

void printArray(char * inputs[], int row, int col)
{
    int i, j;

    for(i = 0; i < row; i++){
        for(j = 0; j < col; j++){
            printf("%c", inputs[i][j]);
        }

    }
}
4

1 回答 1

0

情况不妙：

char *tmp[50];

你打算这样做：

char tmp[50];

奇怪的是，它会起作用，但你的编译器应该到处抛出警告。

我认为您的主要问题是您的printArray函数，它不检查字符串中的 NULL 终止符。所以它会在他们中的大多数人结束时运行。

不要逐个字符地打印，而是这样做：

void printArray(char * inputs[], int length)
{
    int i;
    for(i = 0; i < length; i++){
        printf("%s", inputs[i]);
    }
}

这里有一个使用技巧malloc。不要转换结果，如果您为char值保留空间，请不要使用sizeof(char)- 它始终为 1。

lines[count] = malloc( strlen(tmp) + 1 );
于 2013-07-03T23:26:30.560 回答