Динамическое выделение памяти для массива, в чем ошибка?

Question

[Vadim kyklaed]

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

#include <stdio.h>
#include <malloc.h>
#include <stdlib.h>
int getline(char *p1,char *p2);
void copy (char *p2,char *p1);

int main(){
	char *p1,*p2;
	int len;
	int max;
	int n =100;
	p1 = (char*)malloc(n * sizeof(char));
	p2 = (char*)malloc(n * sizeof(char));
	
	while((len=getline(p1,p2)) > 0){
		
		if (len>max){
			max=len;
			copy(p2,p1);
		}
	}
	if (max>0){
		int i;
		printf("%s",p2);

	}
	free(p1);
	free(p2);
	return 0;
}

int getline(char *p1,char *p2){
	int c, i;
	for (i=0;  (c=getchar()) != EOF && c != '\n'; ++i){
		if (i==10){
			printf("%s\n","UP1");
			p1 = (char*)realloc(p1, 2 * sizeof(char));
			p2 = (char*)realloc(p2, 2 * sizeof(char));
			printf("%s\n","UP2");	
		}
		printf("c[%d] = %c\n",i,c);
		*(p1+i)=c;
	}
	if (c == '\n'){
		*(p1+i) = c;
		++i;
	}
	*(p1+i)='\0';
	return i;
}

void copy(char *p2, char *p1){
	int i;
	i=0;
	
	while ((*(p2+i) = *(p1+i)) != '\0'){
		++i;
	}
	free(p1);
}

Answer 1

[Денис Загаевский]

1) Realloc принимает размер ожидаемой памяти, ты суёшь ему просто 2.
2) После этого ты пишешь в какую-то рандомную память.
3) Ты в функцию передаешь одинарный указатель. Чтобы это работало и указатель менялся, надо передавать двойной. То есть, после выхода из функции, если у тебя был реаллок, в твоём буфере всё равно мусор.
4) Перевыделять каждый раз второй буфер не нужно.
5) копировать из одного буфера в другой не нужно. Достаточно буферы свопнуть(поменять местами через третью переменную).

Comments

[Vadim kyklaed]

1) если я укажу просто число ? в чем оно будет ? думал там нужно например умножить размер типа который я там буду хранить на какое то число
3) то есть инициализация масива вот так char **p1, **p2;
p1 = (char**)malloc(20 * sizeof(char*));
p2 = (char**)malloc(20 * sizeof(char*));
а когда нужно сделать реалок то вот так
p1 = (char**)realloc(p1, 100);
p2 = (char**)realloc(p2, 100);

а как правильно передать p1 p2 в функции ? если двойной указатель. то есть указатель на указатель.

[Vadim kyklaed]

в функцию я передаю массивы с одинарным указателем

и получаю вот такие предупреждения

[Note] expected 'char *' but argument is of type 'char **'

[Warning] passing argument 2 of 'getlen' from incompatible pointer type

[Денис Загаевский]

Vadim kyklaed, нет, не так. Объявления переменных не надо менять. Надо поменять сигнатуру функции и передавать &р.
Насчёт реаллока, да, ты должен передать размер, умноженный на число. В данном случае это число - 2, и чего ты хочешь этим добиться? Ты выделить два байта и всё.

[Vadim kyklaed]

Денис Загаевский, то есть надо передать адрес указатель с помощью ссылки ?

с реолаком я понял, спасибо

[Денис Загаевский]

void f(char** c) {
   с = whatever; //меняет локальное значение c. Значение за пределами функции не аффектится
   *c = realloc(100500*sizeof(char)); //перевыделяем память. Меняем значение вне функции
  **c = 'a';//меняем значение первого символа
}

void g() {
   char* c = malloc(1050*sizeof(char));
   f(&c);
}

[Vadim kyklaed]

Денис Загаевский, спасибо я понял наконец то синтаксис. изначально указатель указывает на адрес в памяти где лежит начало массива ? после мы в функцию передаем ссылку на указатель который указывает на начало массива ?
в заголовке функции передаем ** а в вызове отдаем вот так &
*с - это вроде называется разыменование ?
а почему **с получаем адрес первого символа ? в массиве

[Денис Загаевский]

Vadim kyklaed, Нет, не ссылку, а указатель на указатель. Ссылок в Си нет, но они есть в С++ и это абсолютно другая сущность.
Да, разыменование.
Потому же, почему *(p + i). Разыменовывая двойной указатель ты получаешь одинарный. **с <-> *(*с).