0

1 から 100 までの乱数で配列を 11 個のスポットで埋めるプログラムを作成しようとしています。ランダムなものが機能し、最小値が機能しているように見えますが、最大値は、配列に投げられた11個の数字の一部でさえない非常に高い数字になっています。

問題が何であるかはよくわかりませんが、私が過去を見ているとてつもなく単純なものであると確信しています.

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

int main( void )
{
   int i  = 0;
   int a[11];

   int min = a[0];
   int max = a[0];

   srandom( (unsigned) time(NULL) );

   for (i=0;i<11;i++)
   {
       a[i]=random( ) % 100 ;

       printf("%d\n", a[i]);

       if (a[i] > max)
       {
           max = a[i];
       }
       else if (a[i] < min)
       {
           min = a[i];
       }
   }

   printf("Min: %d\n", min);
   printf("Max: %d\n", max);

   return ( 0 ) ;
}

出力：

16
28
27
58
8
53
76
35
27
19
41
Min: 8
Max: 152908968
4

7 に答える 7

2

あなたの問題はここにあります：

int a[11];

int min = a[0];
int max = a[0];

それ自体が初期化される前に、a[0]初期化minとを使用しています。maxa[0]

を初期化し、とをその値にa[0]設定してから、そこから続行します。minmax

于 2013-10-19T06:05:30.617 に答える
2

max は非常に高い値に初期化されます.. max を 0 に初期化します。

int max = 0;

固定コードは次のとおりです。

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

int main( void )
{
   int i  = 0;
   int a[11];

   int min = 0;
   int max = 0;

   srandom( (unsigned) time(NULL) );
   a[0]=random( ) % 100 ;
   min = a[0];

for (i=1;i<11;i++)
     {
       a[i]=random( ) % 100 ;


       printf("%d\n", a[i]);

       if (a[i] > max)
            {
          max = a[i];
            }
       if (a[i] < min)
            {
          min = a[i];
            }
    }
            printf("Min: %d\n", min);
            printf("Max: %d\n", max);

return 0;
}

出力：

Notra:Desktop Sukhvir$ ./try
82
91
33
8
60
48
60
6
59
62
60
Min: 6
Max: 91
于 2013-10-19T06:06:09.293 に答える
1

問題はあなたのコードスニペットです 

int min = a[0];
int max = a[0];

の要素を初期化する前に に割り当てていa[0]ます。最初に配列の要素を初期化してから、その要素を他の変数に割り当てることができます。 maxmina

 int min ;
 int max ;

 srandom( (unsigned) time(NULL) );   

 min = max = a[0] = random() % 100 + 1;  

 for (i = 1; i < 11; i++) 
 { 
        a[i]=random( ) % 100  + 1; // to generate numbers between 1 to 100  
        ...
于 2013-10-19T06:05:26.830 に答える
1

最初にmin&maxをガベージ値で初期化し、次にそれらを配列の他の要素と比較していますか?

  • まず、配列の各要素に値を割り当てます。

  • 最小値と最大値を a[0] に設定します(簡単にするため)。

  • 次に、max と配列の他の要素との比較を開始します。

次のコードを試してください。

int main( void )
{
   int i  = 0;
   int a[11];

   int min;
   int max;

   srandom( (unsigned) time(NULL) );

for (i=0;i<11;i++)
       a[i]=random( ) % 100 ;

min = a[0];
max = a[0];

for (i=1;i<11;i++)
     {
       printf("%d\n", a[i]);

       if (a[i] > max)
            {
          max = a[i];
            }
       else if (a[i] < min)
            {
          min = a[i];
            }
    }
            printf("Min: %d\n", min);
            printf("Max: %d\n", max);

return ( 0 ) ;
于 2013-10-19T06:06:28.017 に答える
1

すでに 4 つの正解を得ているので、代替手段として。

変数minとを初期化するmaxには、次のいずれかを実行できます。

  • それらを初期化してa [0]の値を取得します（a [0]がその値を受け取った後;））。
  • 値が 0 未満または 100 を超えることはできないことがわかっているため、min を 0 に、max を 100 に初期化します。
  • テストに追加の条件を追加しますif (i == 0 || a[i] > max)。ループを 0 から開始します。

私が提供している他の 2 つの代替手段を使用すると、データがまだ利用できない場合に対処できます。たとえば、値がユーザーまたはネットワークなどから取得された場合。

于 2013-10-19T07:42:00.477 に答える
0 
//Just another version...

void minArray(int v[])
{

    int m=100,len=0;   //for max, put m=0
    for(int i=0;v[i];i++)
        len++;

    for(int i=0;i<len;i++)
        if(v[i]<m)   //for max, change '<' to '>'
            m=v[i];

    printf("%d",m);
}

int main(void)
{
    int v[100]={5,2,3,4,22,5};
    minArray(v);

    return 0;
}
于 2014-01-29T16:24:26.583 に答える