问:请指出下列程序的错误并修改。 void GetMemeory(char* p) { p = (char*) malloc (100); } void test() { .......}

问:请指出下列程序的错误并修改。
void GetMemeory(char* p)
{
    p = (char*) malloc (100);

void test()
{
    char *str=NULL;
    GetMemory(str);
    strcpy(str,”Thunder”);
    strcat(str+2, “Downloader”);
    printf(str);
}

答:
分析:
(1)如果需要改变str,改变变量用一级指针,改变指针用二级指针,
但传入的是一级指针,所以没有对str中存放的地址NULL进行改变,
所以后面执行strcmp和strcat的内容会让程序崩溃。
(2)动态申请内存有可能失败,所以应该增加判断。
(3)动态创建的内存没释放。

改正:
一级指针传递的函数内部更改不影响实参一级指针的值,所以此处要么使用二级指针,要么使用引用。
二级指针:
void GetMemeory(char** p)
{
    *p = (char*) malloc (100);
    if(NULL == *p)
    {
        printf("malloc failed!\n");
        exit(0);
    }

void test()
{
    char *str=NULL;
    GetMemory(&str);
    strcpy(str,”Thunder”);
    strcat(str+2, “Downloader”);
    printf(str);
    free(str);
}

引用:
void GetMemeory(char* &p)
{
    p = (char*) malloc (100);
    if(NULL == p)
    {
        printf("malloc failed!\n");
        exit(0);
    }

void test()
{
    char *str=NULL;
    GetMemory(str);
    strcpy(str,”Thunder”);
    strcat(str+2, “Downloader”);
    printf(str);
    free(str);
}

还有一种方法是,将动态申请的内存以返回值的形式传出。
char* GetMemeory(void)
{
    char* p=NULL;
    p = (char*) malloc (100);
    if(NULL == p)
    {
        printf("malloc failed!\n");
        exit(0);
    }
    return p;

void test()
{
    char *str=NULL;
    str = GetMemory();
    strcpy(str,”Thunder”);
    strcat(str+2, “Downloader”);
    printf(str);
    free(str);
}

修改后再运行程序,
不管strcat(str+n,"Downloader")中的n是几,结果都是ThunderDownloader

猜你喜欢

转载自blog.csdn.net/u014689845/article/details/88532609