Размер массива не меняется (malloc, C++ )

[RIO]

Здравствуйте, не могу разобраться в чем недочет, размер массива постоянно равняется 4, вот код:

	char *String = "RIO ! ! !";
	char *S = NULL;
	S = (char *)malloc(sizeof(char) * strlen(String));
	for (I = 0; I < sizeof(S) - 1; I++)
	{
		 S[I] = String[I];
		 printf("Element - %c\n", S[I]);
	}
	delete[] S;

В чем собственно проблема?

[Sam Gold]

Дело в том, что

sizeof(S)=sizeof(char *)=4 байта

Память выделяется под всю строку S, а считываются только первые 4 символа. А вам в цикле видимо надо до strlen(S)
P.S И если уж выделили память malloc'ом то освобождать ее лучше

free(S)

[Blade]

Сообщение от Sam Gold

P.S И если уж выделили память malloc'ом то освобождать ее лучше

free(S)

Если это C++, то лучше не выделять память malloc`ом()

[Stilet]

for (I = 0; I < sizeof(S) - 1; I++)

У тебя S - указатель. А его размер 4 байта. Я бы не делал такую проверку. Лучше завести еще одну переменку - размера

	char *String = "RIO ! ! !";
	char *S = NULL;
        int n=strlen(String);
	S = (char *)malloc(sizeof(char) * n);
	for (I = 0; I < n; I++)
	{
		 S[I] = String[I];
		 printf("Element - %c\n", S[I]);
	}
	delete[] S;

И кстати Blade'а послушай - верно глаголит.

[RIO]

Спасибо всем Я как раз и думал использовать еще одну переменную. Но никак не мог понять, почему программа на sizeof() не реагирует, теперь разобрался. Спасибо вам.

PS: может кому-то пригодиться - для статических массивов можно использовать sizeof() что бы узнать его длину, а для динамических нет, в случае использования будет возвращаться размер указателя

Кстати, а почему не рекомендуется использовать malloc ?

[ISergeyN]

Кстати, а почему не рекомендуется использовать malloc ?

В С нормально.. в С++ будут проблемы с конструкторами классов..