Чтение строк с неопределенной длиной в C

• вы всегда выделяете на один байт меньше, чем используете. Например, в начале вы выделяете пространство для нулевых символов, а затем пытаетесь установить (несуществующий) первый символ в ''.

• realloc не принимает указатель на указатель в качестве первого параметра. Он должен использоваться следующим образом:

  lst = realloc(lst, largo * sizeof (char));
  

• если вы хотите обрабатывать условия нехватки памяти, вам придется проверить, если malloc() или realloc() возвращать null.

• было бы более эффективно выделять больший буфер в начале и увеличивать его большими шагами вместо перераспределения каждого добавленного символа отдельно.

первый аргумент для вызова realloc на

realloc(&lst, largo * sizeof (char));

должно быть lst, а не &lst

также указатель, возвращенный realloc не всегда нужно быть таким же, как его первый аргумент. Если свободная память рядом с существующей памятью не найдена, выделяется совершенно другой фрагмент и возвращается его адрес.

char *new_lst = realloc(lst, largo * sizeof (char));
if(new_lst != NULL) {
  lst = new_lst;
}

помимо ошибок в вашем коде, я думаю, что лучше создать строку переменной длины в C. Как только у вас это будет, вы можете написать функцию getLine (). Эта строка переменной длины включает в себя понятие емкости, поэтому ее размер увеличивается в блоках степеней 2, а не один за другим.

#include <string.h>
#include <stdio.h>

typedef struct _mystring {
    char * native;
    size_t size;
    size_t capacity;
} String;

size_t String__len(String this)
{
    return this.size;
}

String String__create(char native[], size_t capacity) {
  String this;

  this.size = strlen( native );
  if ( capacity < ( this.size + 1 ) )
        this.capacity = this.size + 1;
  else  this.capacity = capacity;

  this.native = (char *) malloc( capacity * sizeof( char ) );
  strcpy( this.native, native );

  return this;
}

String * String__set(String *this, char native[]) {
    this->size = strlen( native );

    if ( this->size >= this->capacity ) {
        do {
            this->capacity <<= 1;
        } while( this->size > this->capacity );

        this->native = realloc( this->native, this->capacity );
    }

    strcpy( this->native, native );

    return this;
}

String * String__add(String *this, char ch) {
    ++( this->size );

    if ( this->size >= this->capacity ) {
        do {
            this->capacity <<= 1;
        } while( this->size > this->capacity );

        this->native = realloc( this->native, this->capacity );
    }

    char * zeroPos = this->native + ( this->size -1 );
    *( zeroPos++ ) = ch;
    *zeroPos = 0;

    return this;
}

void String__delete(String *this)
{
    free( this->native );
}

после того, как вы сделали эту реализацию, которая полезна для этой проблемы и многих других, вы можете создать функцию getLine:

String String__getLine()
{
    int ch;
    String this = String__create( "", 16 );

    do {
        ch = fgetc( stdin );
        String__add( &this, ch );
    } while( ch != EOF
          && ch != '\n' );

    size_t len = String__len( this );
    this.size = len -1;
    *( this.native + this.size ) = 0;

    return this;
}

Теперь вы можете просто использовать это:

int main()
{
    printf( "Enter string: " );
    String str = String__getLine();
    printf( "You entered: '%s'\n", str.native );
    String__delete( &str );

    return EXIT_SUCCESS;
}

вот рабочий пример для realloc и fgets. Его C89, не требуется POSIX. Параметр можно задать с помощью собственной предварительно выделенной памяти или NULL. Окончательная "свобода" всегда необходима.

#include <string.h>
#include <stdlib.h>
#include <stdio.h>

char *getstringStdin(char *s)
{
  char buffer[9];
  s=realloc(s,1);
  *s=0;
  while( fgets(buffer,9,stdin) )
  {
    s=realloc(s,strlen(s)+1+strlen(buffer));
    strcat(s,buffer);
    if( strchr(s,'\n') )
    {
      *strchr(s,'\n')=0;
      break;
    }
  }
  return s;
}

main()
{
  char *s;
  while( *(s=getstringStdin(0)) ) /* a single Enter breaks */
  {
    puts(s);
    free(s);
  }
  free(s);
  puts("end");
  return 0;
}