Выделение памяти в функции

[Hemul]

int func(char* str)
{
    str = new char[10];
    strcpy(str,"hello");
}

....
char *str;
func(str);
cout<<str[1]<<endl;
delete[]str;
....

Почему данный код генерирует Segmentation fault? Почему выделяемая память в func не сохраняется в указателе при выходе из функции?

[marenko.lilia]

Почему выделяемая память в func не сохраняется в указателе при выходе из функции?

Потому что при выходе из функции все данные уничтожаются. И копии переданных переменных, и все созданные в функции переменные, и адрес памяти которая выделена в функции. Это СТРАШНОЕ ДЕЛО - утечка памяти

Чтобы не потерять адрес памяти, выделенной в функции, надо вернуть из функции указатель на этот участок памяти и записать этот адрес в указатель, определённый в main()

#include <iostream>
using namespace std;

char* func(char* str)
{
	str = new char[10];
	strcpy_s(str, 10, "hello");

	return str;
}

int main()
{
	setlocale(LC_ALL, "rus");

	char* myStr = 0;
	myStr = func(myStr);
	cout << myStr[1] << endl;
	delete[] myStr;

	return 0;
}

[Stilet]

#include <iostream>
#include <string.h>

using namespace std;
int func(char **str)
{
    *str = new char[6];
    strcpy(*str,"hello\0");
    return 6;
}
int main()
{
char *str;
int i=func(&str);
cout<<i<<str<<endl;
delete[]str;
    return 0;
}

Не?

Кстатти: http://www.programmersforum.ru/showthread.php?t=92433

[8Observer8]

Hemul, а вообще, используйте C++ строки вместо сишных:

#include <iostream>
#include <string>

void func( std::string &str )
{
    str = "Hello";
}

int main()
{
    std::string str;
    func( str );
    std::cout << str << std::endl;
    return 0;
}

[_Bers]

Сообщение от Hemul

Почему данный код генерирует Segmentation fault? Почему выделяемая память в func не сохраняется в указателе при выходе из функции?

Aргументы функции всегда копируются.
Внутри функции фигурирует копия указателя, а вовсе не оригинал:

char* func(char* str)
{.
	str = new char[10];   //<--- str - копия указателя, который был передан в функцию
	strcpy_s(str, 10, "hello");
	return str;
}

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

А оригинальный указатель остается без изменений.


Что бы внутри функции влиять на оригинальные данные, а не на копии, можно пойти двумя путями:

1. Путь сишника. Передавайте в функцию указатель на указатель.
Господин Stilet показал вам пример.....

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


2. Путь сишника-с-крестами (в народе просто: плюснутого, приплюснутого, крестанутого, крестухана, крестобл... ну вы понэли)

Если функция ожидает реально существующий объект, и внутри функции нужно влиять на оригинал, а не на копию - передают ссылку на этот объект:

void func(char*& str)  //<--- str, это ссылка на оригинальный указатель
{
    str = new char[10]; //<--- изменяем содержимое оригинального указателя
    strcpy(str,"hello");
}

....
char *str;
func(str);
cout<<str[1]<<endl;
delete[]str;
....