更好的方式来分配内存以获得灵活性和处理

Question

不同的程序员有不同的编码风格。 我和同事正在研究图像数据处理，我们有 3 种不同的方法。

同事 1：

int IMAGE_WIDTH=1280;
int IMAGE_HEIGHT=800;

char* colleague1_way() //just allocate when he wants
{
char* mem = malloc(IMAGE_WIDTH*IMAGE_HEIGHT*2);
return mem;
}

int main(void)
{ 
    char* data;

    data=colleague1_way();
    function1(data); //pass by pointer
    function2(data); //pass by pointer
    function3(data); //pass by pointer

    free(data);
}

同事2：

int IMAGE_WIDTH=1280;
int IMAGE_HEIGHT=800;
char* data;    //set it as global memory

void colleague2_way()
{
data = malloc(IMAGE_WIDTH*IMAGE_HEIGHT*2);
}

int main(void)
{
    colleague2_way();

    function1(); //void input, proceed data inside function
    function2(); //void input, proceed data inside function
    function3(); //void input, proceed data inside function

    free(data);        
}

我：

int IMAGE_WIDTH=1280;
int IMAGE_HEIGHT=800;

int main(void)
{
    char* data = malloc(IMAGE_WIDTH*IMAGE_HEIGHT*2);

    function1(data); //pass by reference 
    function2(data); //pass by reference
    function3(data); //pass by reference

    free(data);        
}

我的想法是

1. 我可以清楚地看到分配的内存，并在 main() 结束时释放它。
2. function1~3可能在另一个cpp文件中，所以很容易处理。

任何人都可以给我评论，有没有更好的方法？ 另外，如果是C++，有什么好的方法吗？

Answer 1

我不知道为什么你想要堆内存但堆栈更快的细节。您的示例代码使您的程序看起来很小，所以我会这样做：

#define IMAGE_WIDTH 1280
#define IMAGE_HEIGHT 800
#define PIXEL_DEPTH 2

int main(void)
{
    char data[IMAGE_WIDTH][IMAGE_HEIGHT][PIXEL_DEPTH] = {0};

    function1(data); //pass by reference 
    function2(data); //pass by reference
    function3(data); //pass by reference

}

这就像您的解决方案，但您不必担心释放内存并且您可以获得更快的写入速度。就像你说的不同的程序员有不同的解决方案，这只是我的两分钱。

Answer 2

首先说一下“Colleague2”方式：char指针data全局声明

在真正需要之前，您应该避免全局声明变量。在这里，我看不出有任何理由在全局范围内声明 data 指针。由于它对其他函数可见，因此它的范围增加了，并且可能会被其他不希望的代码部分意外修改。

“同事1”和“我”方式：
在这两种方式中，一件好事是data 指针的范围是有限的。但是在“Colleague1”方式中，内存分配操作是另一个函数的一部分，这实际上很好，因为您可以在一个地方进行内存分配错误处理（在所有3 方式中都缺少）时间>。考虑一种情况，您需要在代码的其他部分分配相同大小的内存，然后您只需要调用此函数而不是再次执行malloc。如果您确定不需要再次分配相同大小的内存，那么在这种情况下“我”方式也可以。因此，在这两种方式中，请选择合适的方式，但请确保执行所需的错误处理。