Домашнее задание на стек программирования C и куча со строкой - PullRequest
Новая
       22

Домашнее задание на стек программирования C и куча со строкой

0 голосов
/ 21 марта 2019

Это домашнее задание, с которым я мог бы помочь.Кажется, у меня все работает, но у меня проблемы с названием игр.Как вы можете видеть, мои имена не отображаются должным образом в выходных данных, и в то время как я пытаюсь отладить, это большая тарабарщина и тому подобное.Может кто-нибудь объяснить, что является причиной этого, или дать мне пример только одного из games_stack[i].name[i], чтобы заставить его работать должным образом?

Output

#define _CRT_SECURE_NO_WARNINGS
#include <stdio.h>
#include <stdlib.h>
#define MAX 1000
#define PAUSE system("pause")

//Games Struct
typedef struct
{
    char name[50]; // name of game
    int mScore; // metacritic score
    char rating; // ESRB rating
}GAMES;

int main(void) {

    GAMES games_stack[MAX];

    int numElements = 10;

    int i;

    //Populate array in stack
    //games_stack[0].name[0] = "Bioshock Infinite";
    games_stack[0].name[0] = "Bioshock Infinite";
    games_stack[0].mScore = 94;
    games_stack[0].rating = 'M';

    games_stack[1].name[1] = "Half-life 2";
    games_stack[1].mScore = 96;
    games_stack[1].rating = 'M';

    games_stack[2].name[2] = "Mario Kart Double Dash";
    games_stack[2].mScore = 87;
    games_stack[2].rating = 'E';

    games_stack[3].name[3] = "Legend of Zelda: Twilight Princess";
    games_stack[3].mScore = 96;
    games_stack[3].rating = 'T';

    games_stack[4].name[4] = "Rocket League";
    games_stack[4].mScore = 86;
    games_stack[4].rating = 'E';

    games_stack[5].name[5] = "Counter-Strike: Global Offensive";
    games_stack[5].mScore = 83;
    games_stack[5].rating = 'M';

    games_stack[6].name[6] = "Assassin's Creed II";
    games_stack[6].mScore = 80;
    games_stack[6].rating = 'M';

    games_stack[7].name[7] = "Batman: Arkham Asylum";
    games_stack[7].mScore = 91;
    games_stack[7].rating = 'T';

    games_stack[8].name[8] = "Middle-eart: Shadow of Mordor";
    games_stack[8].mScore = 84;
    games_stack[8].rating = 'M';

    games_stack[9].name[9] = "Portal";
    games_stack[9].mScore = 90;
    games_stack[9].rating = 'T';

    // Create an array on the heap to store the same number of populated elements from the stack array.

    GAMES *games_heap = (GAMES*)malloc(numElements * sizeof(GAMES));

    // if memory not allocated, exit

    if (games_heap == NULL)

    {

        printf("\n Memory not allocated");

        return EXIT_FAILURE;

    }

    //Copy the values from the stack array into the dynamic array.

    for (i = 0; i < numElements; i++)

    {

        games_heap[i].name[i] = games_stack[i].name[i];

        games_heap[i].mScore = games_stack[i].mScore;

        games_heap[i].rating = games_stack[i].rating;

    }

    // print the elements from both the array

    for (i = 0; i < numElements; i++)

    {

        printf("\n Stack : ");

        printf(" Name: %c\t MetaScore: %d\t ESRB Rating: %c", games_stack[i].name[i], 
            games_stack[i].mScore, games_stack[i].rating);

        printf("\n Heap : ");

        printf(" Name: %c\t MetaScore: %d\t ESRB Rating: %c", games_heap[i].name[i],
            games_heap[i].mScore, games_heap[i].rating);

    }
    PAUSE;
    // write contents of heap to binary file

    FILE *filePtr = fopen("games.bin", "w");

    if (filePtr == NULL)

    {

        printf("Unable to open file");

        return EXIT_FAILURE;

    }

    for (i = 0; i < numElements; i++)

    {

        if (i != numElements - 1)

            fprintf(filePtr, "%c %d %c\n", games_heap[i].name[i],
                games_heap[i].mScore, games_heap[i].rating);

        else

            fprintf(filePtr, "%c %d %c", games_heap[i].name[i],
                games_heap[i].mScore, games_heap[i].rating);

    }

    fclose(filePtr);

    return EXIT_SUCCESS;

}

//end of program

1 Ответ

1 голос
/ 21 марта 2019

Есть несколько проблем с вашим кодом:

  • Ваше имя игры является массивом символов, поэтому ваша игра [i] является только персонажем. И вы не можете назначить строку на символ. Вам лучше использовать strncpy.
  • Вы можете присвоить значение массиву только при его инициализации, поэтому games_heap [i] .name [i] = games_stack [i] .name [i] не работает. Более того, в этом случае вы копируете только 1 символ.
  • Используйте% s, чтобы напечатать массив символов.

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

#define _CRT_SECURE_NO_WARNINGS
#include <stdio.h>
#include <stdlib.h>

#define GAME_NAME_LENGTH 50
#define MAX 1000
#define PAUSE system("pause")

//Games Struct
typedef struct
{
    char name[GAME_NAME_LENGTH]; // name of game
    int mScore; // metacritic score
    char rating; // ESRB rating
}GAMES;

int main(void) {

    GAMES games_stack[MAX];

    int numElements = 10;

    int i;

    //Populate array in stack
    //games_stack[0].name[0] = "Bioshock Infinite";
    strncpy(games_stack[0].name, "Bioshock Infinite", GAME_NAME_LENGTH);
    games_stack[0].mScore = 94;
    games_stack[0].rating = 'M';

    strncpy(games_stack[1].name, "Half-life 2", GAME_NAME_LENGTH);
    games_stack[1].mScore = 96;
    games_stack[1].rating = 'M';

    strncpy(games_stack[2].name, "Mario Kart Double Dash", GAME_NAME_LENGTH);
    games_stack[2].mScore = 87;
    games_stack[2].rating = 'E';

    strncpy(games_stack[3].name, "Legend of Zelda: Twilight Princess", GAME_NAME_LENGTH);
    games_stack[3].mScore = 96;
    games_stack[3].rating = 'T';

    strncpy(games_stack[4].name, "Rocket League", GAME_NAME_LENGTH);
    games_stack[4].mScore = 86;
    games_stack[4].rating = 'E';

    strncpy(games_stack[5].name, "Counter-Strike: Global Offensive", GAME_NAME_LENGTH);
    games_stack[5].mScore = 83;
    games_stack[5].rating = 'M';

    strncpy(games_stack[6].name, "Assassin's Creed II", GAME_NAME_LENGTH);
    games_stack[6].mScore = 80;
    games_stack[6].rating = 'M';

    strncpy(games_stack[7].name, "Batman: Arkham Asylum", GAME_NAME_LENGTH);
    games_stack[7].mScore = 91;
    games_stack[7].rating = 'T';

    strncpy(games_stack[8].name, "Middle-eart: Shadow of Mordor", GAME_NAME_LENGTH);
    games_stack[8].mScore = 84;
    games_stack[8].rating = 'M';

    strncpy(games_stack[9].name, "Portal", GAME_NAME_LENGTH);
    games_stack[9].mScore = 90;
    games_stack[9].rating = 'T';

    // Create an array on the heap to store the same number of populated elements from the stack array.

    GAMES *games_heap = (GAMES*)malloc(numElements * sizeof(GAMES));

    // if memory not allocated, exit

    if (games_heap == NULL)

    {

        printf("\n Memory not allocated");

        return EXIT_FAILURE;

    }

    //Copy the values from the stack array into the dynamic array.

    for (i = 0; i < numElements; i++)

    {

        strncpy(games_heap[i].name, games_stack[i].name, GAME_NAME_LENGTH);

        games_heap[i].mScore = games_stack[i].mScore;

        games_heap[i].rating = games_stack[i].rating;

    }

    // print the elements from both the array

    for (i = 0; i < numElements; i++)

    {

        printf("\n Stack : ");

        printf(" Name: %s\t MetaScore: %d\t ESRB Rating: %c", games_stack[i].name, 
            games_stack[i].mScore, games_stack[i].rating);

        printf("\n Heap : ");

        printf(" Name: %s\t MetaScore: %d\t ESRB Rating: %c", games_heap[i].name,
            games_heap[i].mScore, games_heap[i].rating);

    }
    PAUSE;
    // write contents of heap to binary file

    FILE *filePtr = fopen("games.bin", "w");

    if (filePtr == NULL)

    {

        printf("Unable to open file");

        return EXIT_FAILURE;

    }

    for (i = 0; i < numElements; i++)

    {

        if (i != numElements - 1)

            fprintf(filePtr, "%s %d %c\n", games_heap[i].name,
                games_heap[i].mScore, games_heap[i].rating);

        else

            fprintf(filePtr, "%s %d %c", games_heap[i].name,
                games_heap[i].mScore, games_heap[i].rating);

    }

    fclose(filePtr);

    return EXIT_SUCCESS;

}

//end of program
...