0

配列にランダムな整数を入力し、ユーザーに配列内の数値を推測させようとする推測ゲームで問題が発生しました。私はそれのほとんどが正しいと信じていますが、プログラムは私の関数が機能しているかどうかを確認するための「ラッキーナンバーの初期化...」印刷コマンドを超えることはありません。問題は、配列を関数に渡す方法、または関数を呼び出す方法（おそらくプロトタイプで？）にあると思います。助けていただければ幸いです。これは私の最初の投稿なので、コードを正しく含めることができなかった場合はお詫び申し上げます。ありがとうございました！

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

/*
This program is a lucky number guessing game. A series of unique, random numbers 
will be generated and stored in an array. Users will have up to three chances to
guess a number stored in the array.*/

//Function Prototypes
int fillArray (int,int []);
int searchArray (int, int []);
int printArray (int, int []);

int main(){
 const int MAX = 25;
 int iArray[MAX]={0};
 int count;

 printf("Initializing lucky numbers...");
 //printf("%d", iArray[0]);
 //for(count=0;count<MAX;count++){
   //printf("%d ",iArray[count]);
 //}//end for loop
 fillArray(MAX,iArray);
 //ONLY TO CHECK ARRAY CONTENTS
 //printf("%d",iArray[0]);
 //for(count=0;count<MAX;count++){
 //  printf("%d ",iArray[count]);
 //}//end for loop
 //CARRY ON WITH PROGRAM
 searchArray(MAX,iArray);

 getch();
 return 0;   
}

//Function Definitions

//Fill Array Function
int fillArray(int max,int Array[]){
    int i;
    int j;
    int uniqueFlag;
    int random;
    srand(time(NULL));

    for(i = 0; i < max; i++) {
     /* Assume things are unique... we'll reset this flag if not. */
     uniqueFlag = 1;
     do {
        random = (rand() % 101);
        /* This loop checks for uniqueness */
        for (j = 0; j < i && uniqueFlag == 1; j++) {
           if (Array[j] == random) {
              uniqueFlag = 0;
           }//end if statement
        }//end for loop
     } while (uniqueFlag != 1);
     Array[i] = random;
}//end while
//ONLY TO CHECK ARRAY CONTENTS
printf("%d",Array[0]);
printf("\n%d\n",Array[1]);
for(i=0;i<max;i++){
  printf("%d ",Array[i]);
}//end for
}//end function

//Search Array Function
int searchArray(int max, int Array[]){
    int i;
    int guessCounter=0;
    int iGuess;
    do{
       printf("Please guess a number between 0 and 100: ");
       scanf("%d",&iGuess); 
       for(i=0;i<max&&guessCounter<3;i++){
          if(Array[i] == iGuess){
              printf("Correct");
              guessCounter = 3;
              }//end if
          else{
               printf("%d is not one of the lucky numbers.\n",iGuess);
               guessCounter++;
               }//end else
               }//end for
    }while(guessCounter < 3); //end do-while loop
    if(guessCounter==3){
        printf("Sorry! You lose!\n");
        printArray(max, Array);
        }//end if
}//end function   

//Print Array Function
int printArray(int max, int Array[]){
    int i;
    printf("Array contents: ");
    for(i=0;i<max;i++){
       printf("%d ",Array[i]);
    }//end for
}//end function
4

3 に答える 3

1

uniqueFlagの外側のループの開始時だけでなく、乱数を選択するたびにリセットする必要がありますfillArray。

于 2012-10-23T00:41:25.133 に答える
1

みんな、ありがとう！チェックする人、または同様の問題を抱えている人のために、私は私の訂正を含めました。checkArrayと呼ばれる別の関数で一意の数値をチェックすることができ、fillArray関数からポインターを返すことでfillArrayの問題を解決し、他のいくつかの（含まれている）微調整を行いました。

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


//Function Prototypes
int fillArray (int, int []);
int checkArray (int, int, int []);
int searchArray (int, int []);
int printArray (int, int []);

int main(){
 const int MAX = 25;
 int iArray[MAX]={0};
 int count;

 printf("Initializing lucky numbers...\n");
 fillArray(MAX,iArray);
 searchArray(MAX,iArray);

 getch();
 return 0;   
}

//Function Definitions

//Fill Array Function
int fillArray(int max, int Array[]){
int number;
int i=0;
int uniqueFlag;
srand(time(NULL));

while(i<max){
     number = (rand() % 101);
     if(checkArray(max, number, Array)==0){ //passes to another function to check for uniqueness
          Array[i]=number;
          ++i;
     }//end if
}//end while
return *Array;
}//end function

//Check Array Function
int checkArray(int max, int number, int Array[]){
int i;
int uniqueFlag;
for(i=0;i<=max;i++){
     if(Array[i]==number){  //determines if random number has already been used in array
          uniqueFlag = 1;
          return uniqueFlag;
     }//end if
     else{
          uniqueFlag = 0;
     }//end else
}//end for
     return uniqueFlag;
}//end function

//Search Array Function
int searchArray(int max, int Array[]){
int i;
int x=0;
int guessCounter=0;
int iGuess;

while(x !=1 && guessCounter<3){
       printf("Please guess a number between 0 and 100: ");
       scanf("%d",&iGuess);
       for(i=0;i<max;i++){
            if(iGuess==Array[i]){
                 x = 1;
                 }//end if
       }//end for
       switch(x){
          case 1:
               printf("%d is one of the lucky numbers. Congratulations! You win!", iGuess);
               printArray(max, Array);
               getch();
               return 0;
          default:
               printf("%d is not one of the lucky numbers.\n",iGuess);
               ++guessCounter;
          }//end switch statement
          }//end while condition
if(guessCounter==3){   //condition limits to 3 guesses per run
          printf("Sorry! You lose!");
          printArray(max, Array);
          }//end if   
}//end function

//Print Array Function
int printArray(int max, int Array[]){
int i;
printf("\nArray contents: ");
for(i=0;i<max;i++){
   printf("%d ",Array[i]);
}//end for
//getch();
//return 0;
}//end function
于 2012-10-29T03:30:12.803 に答える
0

searchArray一見問題があるかどうかわからなかったので、単純化してください。uniqueFlag全体を独自の機能に移すことをお勧めします、doesArrayContain多分？

于 2012-10-23T00:48:43.287 に答える