-1

我正在编写这个函数,它将 n 个字符从 s2 复制到 s1 中。如果 s2 的长度小于 n,则其余 n 个字符将由空字符组成。

main()
{
    char sourceStr[10];
    char destStr[80];
    int myInt;

    printf("Enter a string: ");
    gets(sourceStr);

    printf("Enter the number of characters: ");
    scanf("%d", &myInt);

    printf("Returned string: %s ", copyastring(destStr, sourceStr, myInt));
    return 0;

}

char *copyastring(char * s1, char * s2, int n)
{

    int a = n;
    for( n ; n > 0 ; n--)
    {
            // if end of s2 is reached, the rest of s1 becomes null
        if(*s2 == '\0')
        {
            while(n > 0)
            {
                *s1 = '\0';
                s1++;
                n--;
            }
            break;
        }
            //if-not, copy current s2 value into s1
            //increment both pointers
        else
        {
            *s1 = *s2;
            s2++;
            s1++;
        }
    }
    // Just incase s2 is longer than n, append a null character
    s1++;
*s1 = '\0';
s1--;

    //Reset s1's pointer back to front of s1
    s1 = s1 - a;
    return s1;
}

运行此代码并打印出函数返回的字符串后,我意识到所有空字符都被打印为垃圾字符(不可读)。为什么呢?空字符不会终止字符串吗?

提前致谢

4

2 回答 2

1

如果s2n字符长,则不要将终止符添加到字符串s1,因此将返回的指针打印为字符串会产生垃圾。

于 2013-02-11T12:55:29.457 回答
0

调用copyastringwheremyInt < strlen(sourceStr)将无法添加空终止符 tp destStr。然后,您将打印出现在其余部分中的任何数据destStr以及其后的堆栈,仅在您的程序崩溃或遇到零字节时终止。

一个简单的解决方法是始终在末尾添加一个空终止符copyastring

s1 = s1 - a;
s1[n-1] = '\0';
return s1;
于 2013-02-11T12:51:49.347 回答