C Программирование - Return Pointer to Freed

Question

У меня есть функция foo(), которая выделяет память и возвращает ее. Является ли стандартной практикой для меня освобождать ее в конце моей основной функции?

char* foo(){ 
char * p; 

p = malloc(sizeof(char) * 4); /* edit - thanks to msg board */ 
p[0] = 'a'; 
p[1] = 'b'; 
p[2] = 'c'; 
p[3] = '/0'; /* edit: thanks to the msg board. */ 

return p; 
} 

int main(int argc, char *argv[]) 
{ 
char * p2; 

p2 = foo(); 

printf("%s", p2);  

free(p2); 

return 0; 
} 

Answer 1

Освобождение в конце main() было бы правильной вещью, да. Тем не менее, вы можете подумать о нулевом завершении этой строки. Более идиоматический дизайн, так сказать, относится ко всему управлению памятью «на одном уровне». Что-то вроде:

void foo(char *p) 
{ 
    p[0] = 'a'; 
    p[1] = 'b'; 
    p[2] = 'c'; 
    p[3] = '\0'; 
} 

int main(int argc, char **argv) 
{ 
    char *p2 = malloc(4); 
    foo(p2); 
    printf("%s", p2); 
    free(p2); 
    return 0; 
} 

Answer 2

Я не знаю, если это «стандартная практика», но free() необходимо или будет протекать Memeory.

Я хотел бы сделать две функции, new_foo() и delete_foo(), как это:

char* new_foo() 
{ 
    char * p; 
    p = malloc(sizeof(int) * 4); 
    /* ... */ 
    return p; 
} 

void delete_foo(char* p) 
{ 
    free(p); 
} 

int main() 
{ 
    char * p2; 
    p2 = new_foo(); 
    /* ... */ 
    delete_foo(p2); 
    return 0; 
} 

Это "имеет смысл", на мой взгляд.

Answer 3

Вам необходимо освободить, когда вам больше не нужна ссылка на эту переменную. Однако, когда ваша программа выходит из всех выделенных ей ресурсов, они освобождаются.

И да, это хорошая практика, чтобы освободить его, когда вы закончите использовать его, даже если программа выходит на следующую строку.

Answer 4

и зачем вам malloc (sizeof (int) * 3), когда вы выделяете для 3-х символов?

Answer 5

«Стандартная практика», если есть такая вещь, заключается в том, чтобы освободить память, как только вы закончите с ней.

Мое личное убеждение состоит в том, что вы должны освободить память, даже если знаете, что программа готова закончить только потому, что это хорошая практика, и потому что когда-нибудь вы каким-то образом столкнетесь с окружающей средой, которая автоматически не выпускает malloced памяти при выходе программы.

Answer 6

Я бы подумал, что было бы лучше распределить и освободить память в той же функции (в данном случае main()).

Например:

void foo(char *p) 
{  
p[0] = 'a'; 
p[1] = 'b'; 
p[2] = 'c'; 

return ; 
} 

int main(int argc, char *argv[]) 
{ 
char * p2; 

p = malloc(sizeof(int) * 3); 
foo(p2); 

printf("%s", p2);  

free(p2); 

return 0; 
} 

Answer 7

Одна хорошая причина, чтобы сделать это так, что у вас есть/Фри соответствие таНос в. Если код будет перемещен, программист, естественно, будет искать подходящее для игры с malloc. Если его там нет, это может быть источником путаницы (надеюсь, надолго).

Вкратце, это делает использование ресурсов более очевидным.

Answer 8

Общее правило заключается в том, что память должна быть освобождена, как только вы заметите, что она вам больше не нужна. В конкретном случае памяти, которая необходима до завершения программы, есть две действительные точки зрения.

• Держите это просто: для каждого malloc должен быть соответствующий free. Главное преимущество в том, что если вы когда-либо захотите преобразовать свою автономную программу в компонент более крупной программы, ваш main превратится в вызов библиотеки, и потребуются вызовы free.

• Проще: освобождение ресурсов программы - это работа ОС.Это действительная стратегия управления - память программы - это region памяти, которая освобождается одним махом при выходе программы. Главное преимущество заключается в том, что вам не нужно отслеживать каждый выделенный блок, который иногда может быть затруднен (но тогда вы, возможно, захотите рассмотреть сборщик мусора).