Как избежать этой ошибки обратной трассировки после того, как моя программа отобразит свой вывод? - PullRequest
0 голосов
/ 11 апреля 2020

Это подробная программа, которую я пытаюсь реализовать: натуралист отправляется исследовать джунгли амазонки и нуждается в компьютерной программе для записи информации обо всех обнаруженных новых видах. Для каждого нового вида необходимо хранить имя (максимум 128 символов), размер (действительное число) и тип животного. млекопитающее, насекомое, птица или фи sh). Вот как должен выглядеть пример выполнения (с курсором, выделенным курсивом) ...

> NewSpecies
Enter animal information ("exit" to exit)
What is the name : bloatfish
What is the size : 12.47
What is the type : fish
Enter animal information ("exit" to exit)
What is the name : stingybeasty
What is the size : 0.13
What is the type : insect
Enter animal information ("exit" to exit)
What is the name : toothfulsloth
What is the size : 33.33
What is the type : mammal
Enter animal information ("exit" to exit)
What is the name : exit

The following new species were found:
bloatfish            has size  12.47 and is a fish
stingybeasty         has size   0.13 and is a insect
toothfulsloth        has size  33.33 and is a mammal 
You must ...
Implement the program in C.

Необходимо использовать массив структур, чтобы каждый новый вид мог быть записан в элементе массив. Тип животного представлен как тип enum, обозначающий одно из млекопитающих, насекомых, птиц или Fi sh. Заранее неизвестно, сколько новых видов будет найдено, поэтому программа должна выполнить mallo c для начального массива размера 1 и использовать метод удвоения reallo c, чтобы получить больше памяти при необходимости. Вы всегда должны проверять возвращаемое значение из mallo c, как это делается в функции оболочки Mallo c (или просто использовать Mallo c: -).

Моя попытка:

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

    #define MAX_CHAR 128
    #define LENGTH(A) (sizeof(A)/sizeof(A[0]))

    typedef char String[MAX_CHAR];
    typedef enum {mammal, insect, bird, fish, error} AnimalType;

    typedef struct{
      String name;
      double size;
      AnimalType type;
    } Animal;

    void * Malloc(size_t Size) {

        void * Memory;

        if ((Memory = malloc(Size)) == NULL) {
            perror("Cannot malloc");
            exit(EXIT_FAILURE);
        } else {
            return(Memory);
        }
    }

    AnimalType CheckAnimalType(String type) {
        if (!strcmp(type,"mammal")) {
            return (mammal);
        }
        if (!strcmp(type,"insect")) {
            return (insect);
        }
        if (!strcmp(type,"bird")) {
            return (bird);
        }
        if (!strcmp(type,"fish")) {
            return (fish);
        }
        return (error);
    }

    char *PrintAnimalType(AnimalType type){
      switch(type){
        case mammal: return "mammal"; break;
        case insect: return "insect"; break;
        case bird: return "bird"; break;
        case fish: return "fish"; break;
        case error: return "error";
      }
      return "error";

    }
    void printData(Animal *animal, int size) {
        printf("The following species were found:\n");
        for(int i = 0; i < size-1; i++)
            printf("%s has size %.2lf and is a %s\n", animal[i].name, animal[i].size, PrintAnimalType(animal[i].type));
    }

    void MainMenu(Animal *animal, int *size){
      for(;;){
        Animal newAnimal;
        String animalName;
        double animalSize;
        String animalType;

        printf("Enter animal information (\"exit\" to exit)\n");

        printf("What is the name : ");
        scanf("%s", animalName);
        if(!strcmp(animalName, "exit")) break;
        strcpy(newAnimal.name, animalName);

        printf("What is the size : ");
        scanf("%lf", &animalSize);
        if(newAnimal.size == 0) break;
        newAnimal.size = animalSize;

        printf("What is the type : ");
        scanf("%s", animalType);
        newAnimal.type = CheckAnimalType(animalType);

        if((animal = realloc(animal, sizeof(newAnimal)*((*size)+1))) == NULL) {
                printf("MEMORY ERROR: problem reallocating array\n");
                return;
            }
         animal[(*size)-1] = newAnimal;
            (*size)++;
      }
      printData(animal, *size);
    }



    int main(void) {
      int size = 1;
      Animal *animal = Malloc(sizeof(Animal));
      MainMenu(animal, &size);
      free(animal);
      return 0;
    }

Я пытаюсь реализовать вышеуказанное в C, но я получаю эту ошибку сразу после выполнения:

Ссылка на ошибку: https://pastebin.com/Wcu0wtet

enter image description here

1 Ответ

3 голосов
/ 11 апреля 2020

В MainMenu вы звоните realloc на animal. Это может изменить размер существующего буфера, но обычно выделяет новый, изменяя значение указателя. Новый указатель сохраняется в локальной переменной animal, а не в переменной main.

. Когда вы вернетесь к main, вы вызовете free(animal), что попытается освободить исходное животное буфер, который уже был освобожден при вызове realloc.

Вы захотите передать измененный указатель буфера обратно вызывающей стороне, либо в качестве возвращаемого значения, либо путем передачи указателя на исходную переменную (Animal **).

...