Копирование элементов массива в новый - C

Question

В настоящее время я выполняю школьное задание, и теперь я действительно застрял. У меня проблема в том, что когда я пытаюсь скопировать элементы кубика массива в массив diceCheck, программа входит в какой-то бесконечный цикл, и я не понимаю, почему. Помощь будет принята с благодарностью! Редактировать: это внизу в функции printScores.

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

void printMenu(void);
void throwDice(int dice[], int nrOfDice, int nrOfDieValues);
void readDieValues (int dice[], int nrOfDice);
void printDice(const int dice[], int nrOfDice);
void printScores(const int dice[], int nrOfdice, int nrOfDieValues);
int isThreeOfAKind(const int dieValues[], int nrOfDieValues);
int isSmallStraight(const int dieValues[], int nrOfDieValues);

int main(void)
{
    const int nrOfDice = 5;
    const int nrOfDieValues = 6;
    int dice[4], menuChoice = 0;

    printMenu();
    printf("\nMake your choice: ");
    while(scanf("%d", &menuChoice) != -1)
    {
        switch (menuChoice)
        {
            case 0:
                printMenu();
            break;
            case 1:
                throwDice(dice, nrOfDice, nrOfDieValues);
                printf("Make your choice: ");
            break;
            case 2:
                readDieValues(dice, nrOfDice);
                printf("Make your choice: ");
            break;
            case 3:
                printDice(dice, nrOfDice);
                printf("Make your choice: ");
            break;
            case 4:
                printScores(dice, nrOfDice, nrOfDieValues);
            break;
            case -1:
                return 0;
            break;
            default:
                printf("Invalid choice!\n");
            break;
        }
    }
    return 0;
}

void printMenu()
{
    printf("MENU:\n");
    printf("0.  Display the menu\n");
    printf("1.  Make a random throw\n");
    printf("2.  Enter die values for a throw\n");
    printf("3.  Display the die values for the throw\n");
    printf("4.  Display the score for the throw\n");
    printf("-1. End program\n");
}

void throwDice(int dice[], int nrOfDice, int nrOfDieValues)
{
    int choice, i;
    printf("Enter seed (1 gives a random seed): ");
    scanf("%d", &choice);
    if(choice == 1)
    {
        srand(time(NULL));
        for (i = 0; i < nrOfDice; i++)
        {
            dice[i] = ( rand() % nrOfDieValues) + 1;
        }
        printf("\n");
    }
    else
    {
        srand(choice);
        for(i = 0; i < nrOfDice; i++)
        {
            dice[i] = ( rand() % nrOfDieValues) + 1;
        }
        printf("\n");
    }
}

void readDieValues(int dice[], int nrOfDice)
{
    int i;
    for(i = 0; i < nrOfDice; i++)
    {
        printf("Die %d: ", (i+1));
        scanf("%d", &dice[i]);
    }
}

void printDice(const int dice[], int nrOfDice)
{
    int i;
    printf("Your dice: ");
    for(i = 0; i < nrOfDice; i++)
    {
        printf("%d ", dice[i]);
    }
    printf("\n\n");
}

int isThreeOfAKind(const int dieValues[], int nrOfDieValues)
{
}

int isSmallStraight(const int dieValues[], int nrOfDieValues)
{
}

void printScores(const int dice[], int nrOfdice, int nrOfDieValues)
{
    int diceCheck[4], i;

    for(i = 0; i < nrOfdice; i++)
    {
        diceCheck[i] = dice[i];

        printf("%d ", dice[i]); //these are just for myself to check if it worked
        printf("%d ", diceCheck[i]); //these are just for myself to check if it worked
    }
}

Answer 1

У тебя есть:

const int nrOfDice = 5;

но

int dice[4];
int diceCheck[4];

Ваша идея копирования верна, но вы идете один за концом массива.

Чтобы избежать такого рода ошибок, инициализируйте оба из одного и того же выражения, например:

int dice[4];
const int nrOfDice = sizeof dice / sizeof dice[0];

или же

const int nrOfDice = 4;
int dice[nrOfDice];

и внутри функции PrintScores, а не int diceCheck[4];, делать:

int diceCheck[nrOfDice];