Подскажите пожалуйста правильно ли я, динамически распределяю память?

[Nata Golden rose]

Добрый день,

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

#include <stdio.h>
#include <stdlib.h>
 
int main(int argc, char *argv[])
{ 
    char *S;
    int *p, i;
    int t = 20;
   
    
    
    
   printf("Введите предложение");
  
   gets(S); // считываю предложение
  
   p = (char*)malloc(strlen(S)+1);// выделяю память
   
   if(!p) //проверяю есть ли свободная память
   
   printf("Невозможно выделить память");
   
    return 1;
  
   char tab[*p]; //создаю массив
   
   strcpy(tab[i],S); //копирую строку в массив
    
   system("PAUSE"); 
   return 0;
}

[kineziz]

Вы не освобождаете выделенную память.

[waleri]

Сообщение от Nata Golden rose

но я не знаю правильно ли он работает?

Например, куда прочитает данные fgets(S)?
А для чего нужен tab вообще непонятно.
Копировать программа ничего не будет, независимо от того, будет выделена память или нет, потому что return выполняется всегда.

Данную программу никто даже и не компилировал.

Ответ - программа вообще не работает, ни правильно, ни неправильно.

[Bugrimov]

Nata Golden rose вы на правильном пути... Развивайте мысль. Не ошибается только тот кто ничего не делает

if(p == null) { 
      printf("Невозможно выделить память");
      return 1;
}

Обработку например можно так описать.

[Nata Golden rose]

А если вот так сделать:

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

int main(int argc, char *argv[])
{
    char *S,*p;

    printf("Введите предложение");
    gets(S);                       // считываю предложение
    p = (char*)malloc(strlen(S)+1);// выделение памяти 
    if(!p) {                       //проверяю есть ли свободная память
        printf("Невозможно выделить память");
        return 1;
    }
    strcpy(p,S);              //копирую строку в массив
    printf(p);                //вывод содержимого динамического массива
    free(p);                  //освобождение памяти
    return 0;
}

Почему он все равно не работает?

[alekopoko]

gets не понимает сколько выделено памяти для указателя S
т.е. вместо

char *S;

надо так -

 char S[30];

или после char *S; -

S=(char*)malloc(sizeof(char[10]);

[_Bers]

Nata Golden rose,
Потому что вы не правильно пользуетесь функцией gets(S);

Эта функция на входе получает указатель на буффер, в котором она размещает строку.

Но у вас нет хранилища данных под строку.

А ваш указатель S в самом начале программы явно не проинициализирован.
А значит - содержит мусор. Некий случайный адрес.

gets записывает по этому адресу данные строки, перетирая при этом чужую информацию.
Это может приводить к самым непредсказуемым последствиям.

Иногда программа сможет нормально отработать. А иногда будет падать. Как повезет.

Правильное использование этой функции описано в документации.
Внимательно рассмотрите пример:
http://www.cplusplus.com/reference/cstdio/gets/

[_Bers]

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

Ваше решение не верное.
Использование gets предполагает заранее известный размер хранилища данных.
Но поскольку по условию длина строки может быть какой угодно, то вы на это закладываться не можете.

А значит эта функция вам не подходит.

[vandit86]

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
main()
{
    char ch;
    int num=0;
    char *str=NULL,*aux=NULL;
    while((ch = getchar())!='\n')
    {
        if (str) free(str);

        str = (char *) malloc((num+2)*sizeof(char));
        if (!str) return 1;
        if (aux) strcpy(str,aux);
        free(aux);
        str[num]=ch;
        str[num+1]='\0';
        aux = (char *) malloc((num+2)*sizeof(char));
        if (!aux) return 1;
        strcpy(aux,str);
        puts(aux);

        num++;
    }
free (aux);
return 0;
}

[_Bers]

vandit86, ваш код не компилируется.