Только realloc

@Rondy · Регистрация: 23.02.2016

Не хочет работать программа(возвращает неправильные числа+проблема с выводом)

Во вложении скрин с условием!!!

Комментарий модератора

Перепишите задание на форум

C

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
int main()
{
    char *ram, *in;
    int p = 50;
 
    printf("Podaj tekst: ");
 
    ram =(char *)realloc(NULL, p*sizeof(char));
 
    if (ram == NULL)
    {
        printf("\nFailed to allocate memory");
 
        return 1;
    }
 
    in = (char *)realloc(NULL, 1010*sizeof(char));
    if (in == NULL)
    {
        printf("\nFailed to allocate memory");
 
        free(ram);
        return 1;
    }
 
 
    scanf("%[^\n]s", in);
 
    while (strlen(in) > (unsigned int)p)
    {
        p += 50;
        ram = (char *)realloc(ram, p*sizeof(char));
 
        if (ram == NULL)
     {
        printf("Failed to allocate memory");
 
        free(ram);
        free(in);
        return 1;
     }
    }
    ram = in;
 
    printf("%s", ram);
 
    free(ram);
    free(in);
 
    return 0;
}

@stake-k26 · 20.08.2018, 00:36

вне зависимости от задания, код составлен крайне неоптимально.

Зачем сначала объявлять массив ram[50], а затем делать
while (strlen(in) > (unsigned int)p)
?

Когда мы можем сразу сделать

C

int p = strlen(in);
if ((ram = (char *)malloc(p*sizeof(int))) == NULL)
{
    printf("\nFailed to allocate memory\n");
    free(in);
    return -1;
}

??

Ну, в вашем случае тут realloc, но не суть, смысл тот же.

Далее, когда вы делаете ram = in; вы просто записываете в ram адрес памяти, по которому хранится in, т.е. в этом случае во-первых, теряется вся память, которая была запрошена под ram, т.е. больше нет указателя, который бы на нее указывал, а во-вторых (что следует из первых), делать realloc для ram вообще нет никакого смысла, достаточно сделать:

C

#include <stdio.h>
#include <stdlib.h>
 
int main (void)
{
    char *ram, *in = NULL;
    if ((in = (char *)malloc(1010*sizeof(int))) == NULL)
    {
        printf("\nFailed to allocate memory\n");
        return -1;
    }
 
    printf("Podaj tekst: ");
    scanf("%1010[^\n]s", in);
 
    ram = in;
 
    printf("symbols in in[]:\n\t%s\n", in);
    printf("symbols in ram[]:\n\t%s\n", ram);
 
    ram = NULL;
    free(in);
    return 0;
}

вы бы тогда уж memcpy использовали что ли.

Просто для примера, посмотрите на код:

C

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
int main (void)
{
    char *ram, *in;
    if ((in = (char *)malloc(1010*sizeof(int))) == NULL)
    {
        printf("\nFailed to allocate memory\n");
        return -1;
    }
 
    printf("Podaj tekst: ");
    scanf("%1010[^\n]s", in);
 
    int p = strlen(in);
    if ((ram = (char *)malloc(p*sizeof(int))) == NULL)
    {
        printf("\nFailed to allocate memory\n");
        free(in);
        return -1;
    }
 
    int i;
    for (i=0; i<1010 && in[i] != '\0'; i++)
        ram[i] = in[i];
    ram[i] = '\0';
 
    printf("symbols in in[]:\n\t%s\n", in);
    printf("symbols in ram[]:\n\t%s\n", ram);
 
    free(in);
    free(ram);
    return 0;
}

Опции темы