Как посчитать строки в динамическом массиве?

Question

[BuckvicK]

#define STR 3

int len_text(char** text){
	int i = 0;
	while (*(text+i)){
		i++;
	}
	return i;
}

char* get_string(){
	int i = 0;
	char *str;
	str = (char*)malloc(sizeof(char));
	while ((*(str + i) = getchar()) != '\n'){
		i++;
		str = (char*)realloc(str, (i + 1) * sizeof(char));
	}
	*(str + i) = '\0';
	return str;
}

int main(){
int j = 0, i = 0;
text = (char**)malloc(sizeof(char*));
	do{
		j = 0;
		text[i] = (char*)malloc(sizeof(char));
		text[i][j] = (char)malloc(sizeof(char));
		*(text + i) = get_string();
		i++;
		text = (char**)realloc(text, (i + 1) * sizeof(char*));
	} while (i < STR);
printf("%d", len_text(text));
rerurn 0;
}

Текст должен вводится до определенного условия, i STR это частный случай.
Функция len_text работает не совсем правильно, она возвращает неверное количество строк, в чем может быть проблема?
Как сделать это более грамотно, с точки зрения этики программирования?

Answer 1

[MiiNiPaa]

printf("%d", len_text);
Радостно выведет значение указателя на функцию len_text. Функции вызывать надо.

Также огромное количество выделений памяти. Код можно назвать Изнасилование кучи

Comments

[BuckvicK]

Прошу прощения, опечатался. Без выделения памяти никуда, я только учусь, делаю как могу и умею)

[MiiNiPaa]

BuckvicK: У вас функция len_text ожидает что после всех осмысленных строк будут идти нулевой указатель. Но в main в text не ложится нулевой указатель после всего. Поэтому этот вызов len_text считает погоду на Марсе.

>Без выделения памяти никуда
Но не по аллокации на символ же. Выделите буфер длиной, скажем, 30 символов. Если он использован, перевыдели в 2 раза больше.
Чтобы понять масштаб проблемы: один вызов malloc/realloc отработает медленнее чем остальной код без malloc и ввода-вывода. А здесь их тьма.

Answer 2

[werktone]

Так вроде бы код работает.

p.s. Автор, зачем вы используете конструкции вида *(text + i)? Это абсолютно тоже самое, что и text[i].

Comments

[BuckvicK]

Раньше использовал text[i], потом сказали что по индексам это не самое лучшее, посоветовали таким образом, так и не понял почему)

[MiiNiPaa]

BuckvicK:
>сказали что по индексам это не самое лучшее
Фигню сказали. Стандарт требует чтобы text[i] был абсолютно идентичен *(text + i)