C - 传递和操作字符指针和指针到指针

Question

作为C语言的新手，我正在使用指针，特别是双指针。

我的意图是

1. 主中的 malloc 字符指针
2. 将错误定位的指针传递给不同的函数
3. 获取同一指针内每个函数的结果
4. 释放主中的指针

这是我的代码：

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <ctype.h>

void process1(char **pointer) {
    //this should take and change the pointer value from a to x
    (*pointer)[0] = 'x';
    (*pointer)[1] = '\0';
}

void process2(char **pointer) {
    //this should take the pointer, free and NULL it, do a new malloc and set the pointer value to y
    char* p = *pointer;
    free(p);
    p = NULL;

    p = malloc(sizeof(char)*2);
    p[0] = 'y';
    p[1] = '\0';

    *pointer = p;
}

void main() {
    char* p = malloc(sizeof(char)*2);
    p[0] = 'a';
    p[1] = '\0';
    //should print a
    printf("%s\n",p);
    process1(&p);
    //should print x
    printf("%s\n",p);
    process2(&p);
    //should print y
    printf("%s\n",p);

    free(p);
    p=NULL;

}

//this is the Output as expectd
sh-4.2$ main
a
x
y

我现在的问题是：

1. 这是好的做法吗？
错误
2. 定位 p 指针时，我是否在函数进程 2 中泄漏了内存？我需要在某个地方释放这个 p 指针吗？

Answer 1

>

释放函数中的内存不是一个好主意，除非您 100% 确定传递的指针已错误定位。否则，如果您忘记它并传递任何其他指针，则非常容易出错。

您的职能

void process1(char **pointer) {
    //this should take and change the pointer value from a to x
    (*pointer)[0] = 'x';
    (*pointer)[1] = '\0';
}

不需要字符**参数

void process1(char *pointer) {
    //this should take and change the pointer value from a to x
    pointer[0] = 'x';   // or *pointer = 'x';          or *pointer++ = 'x';
    pointer[1] = '\0';  // or *(pointer + 1) = '\0';   or *pointer = '\0';
}

Answer 2

1. 这是好的做法吗？

>

当您不知道在编译时将收到多少输入数据时，动态内存非常有用。您可以轻松地重新分配动态数组的大小，而不能修改堆栈上数组的大小。

最大的缺点是内存泄漏和可能的分段错误。

您必须释放分配的内存。

在自动存储上声明的阵列更易于使用且速度更快

在您的情况下，无需传递指针到指针，只需使用

void process1(char *pointer) {
    pointer[0] = 'x';

process1(p);

在 process2 中，您可以使用 realloc（） 而不是释放和分配新内存。以免建议指针是字符*。

pointer = realloc (pointer, 4 * sizeof(int));

• 您不会丢失已存储在指针数组中的数据

不，没有任何内存泄漏或超出界限。

如果您正在使用指针，最好使用名为 valgrind 的工具调试程序。

如果不必使用动态分配，请不要使用。它容易出错，速度较慢，您必须释放数据。

Answer 3

这个程序表现良好。它正确地释放所有分配的内存，并且不会超出分配的内存范围进行写入。

从重新分配以前分配的内存的角度来看，process2 正在做什么是可以的。在这种特殊情况下，您分配的内存量与以前相同，但一般来说，如果这样的函数可能正在扩展分配的内存，那么传递双指针来修改调用函数中的指针变量是有意义的。

至于 process1，通过地址传入的指针没有被修改，只是它指向什么，所以这里不需要双指针。您可以将其定义为：

void process1(char *pointer) {
    pointer[0] = 'x';
    pointer[1] = '\0';
}

并像这样称呼它：

process1(p);