Лучший способ передать указатели между функциями

Question

Я играю с C. Просто задайте несколько вопросов, по которым лучше всего практиковать, и если я что-то делаю неправильно.

Я сделал программу, которая, кажется, работает:

void main() 
{ 
    char *text; 
    text = malloc(10 * sizeof(char)); 
    dosomethingwithtext(text); 
    printf("%s\n", text); 
} 

Однако я также видел версию, как это: (я знаю, что это не будет ничего значимого печати, потому что я не добавил символов , но представьте себе, что у меня было!)

void main() 
{ 
    char *text; 
    text = dosomething(); 
    printf("%s\n", text); 
} 
char * dosomething() 
{ 
    char *text; 
    text = malloc(10 * sizeof(char)); 
    return text; 
} 

Или, если любой из них не оптимален, что является лучшим способом. В основном я задаю самый общий/лучший способ сделать это.

Answer 1

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

б) SizeOf (Char) определяется как 1 в C, так что нет никакого смысла вызова SizeOf (Char)

с) Это не void main(), это int main(void) или int main(int argc, char** args) - вам необходимо предоставить возвращаемое значение ,

Чтобы ответить на фактический вопрос, все в порядке. Возможно, во втором случае вам становится сложнее управлять памятью, потому что вы не можете поместить функцию free() в ту же функцию, что и malloc() - ваш dosomething() возвращает выделенный блок. Тем не менее, очень часто заканчиваются программы, которые выделяют в одном месте и бесплатно в другом.

Answer 2

Вариант 1 имеет то преимущество, что распределение и снятие с производства осуществляются в том же объеме, что (в полностью основанном на мнениях) легче отслеживать и отлаживать. Вы можете использовать возвращаемое значение, чтобы сообщить о каком-то статусе.

Вариант 2 следует за Creational pattern, который является причиной утечки в неуправляемой памяти. Единственный возможный отчет о состоянии - пустой указатель. (что может быть или не быть достаточным).