Question

我正在编写一个程序，它接受两个字符串并将一个字符串输入另一个字符串，以便：

字符串1： abc
字符串2： 123
输出： a123b123c123

现在由于某种原因，我的输出字符串在中间变为垃圾： a123 = b123 = c123 。我不知道为什么，并希望得到一些帮助！

以下是代码：

#define _CRT_SECURE_NO_WARNINGS
#define N 80
#define ONE 1
#include <stdio.h> 
#include <stdlib.h>
#include <string.h>

void InputStr(char str[]);
char* CreateString(char str1[], char str2[]);
int main()
{
    char strA[N], strB[N], *strF;

    InputStr(strA);
    InputStr(strB);
    strF = CreateString(strA, strB);
    puts(strF);

}

void InputStr(char str[])
{

    printf("Please enter the string\n");
    scanf("%s", str);


}
char* CreateString(char str1[], char str2[])
{

    char* newstr;
    int len1, len2, size, i, j, b;
    len1 = strlen(str1);
    len2 = strlen(str2);
    size = len1*len2;
    newstr = (char*)malloc(size*sizeof(char) + 1);
    for (i = 0, b = 0; i<len1; i++, b++)
    {
        newstr[b] = str1[i];
        b++;
        for (j = 0; j<len2; j++, b++)
            newstr[b] = str2[j];


    }
    newstr[b + ONE] = 0;
    printf("test\n");
    return newstr;


}

Answer 1

您的问题

您正在将b变量递增2次：

for (i = 0, b = 0; i < len1; i++, b++) // First increment
{
    newstr[b] = str1[i];
    b++; // Second increment
    for (j = 0; j < len2; j++, b++)
        newstr[b] = str2[j];
}

解决方案

只需删除第一个b增量，代码即可运行：

for (i = 0, b = 0; i < len1; i++) // No more b increment
{
    newstr[b] = str1[i];
    ++b; // You only need this increment
    for (j = 0; j < len2; j++, b++)
        newstr[b] = str2[j];
}

Answer 2

你每次都在增加b。（这也是两次）只在你需要的时候去做。否则字符串中有孔。

for (i = 0, b = 0; i<len1; i++)
{
    newstr[b++] = str1[i];
    for (j = 0; j<len2; j++)
        newstr[b++] = str2[j];    
}

然后一个小小的改变就是

newstr[b] = 0;

循环结束后。

也不要强制转换malloc的返回值。检查malloc的返回值是否为NULL检查并适当处理。

当乘以检查是否有溢出时。如果溢出处理正确。

Answer 3

好的，我发现了问题，我的for循环又做了一个b ++，它在我的字符串中创建了一个空单元格。

在我的字符串中获取垃圾

3 个答案:

您的问题

解决方案