配列のなかで最大値と最小値を表示させたい。

Question

乱数で1～10の数字を発生させて最大値と最小値を表示させたいのですが、以下のように書いてもうまくできません。

            Random rnd = new Random();
            string num = " ";


            for(int i = 0; i < 11; i++)
            {
                int dise = rnd.Next(1, 11);
                num += dise + " ";
            }

            Console.WriteLine(num);
            Console.WriteLine(num.Max());
            Console.WriteLine(num.Min());

結果が以下のようになってしまいます。

9が最大値なのもおかしいし最小値が表示されません。

｛追記｝
自分なりに調べて、最大値と最小値はたぶん以下のコードで出てくるようになりました。たぶんというのは、乱数で発生した10個の数字が表示できないから確認できないのです。以下の通り、listを使って書いてみたのですが、listの中身を表示させるために一回stringに変換してからconsole.writelineで表示させようとしたところ失敗してしまいました。

            var samplelist = new List<int>();
            Random rnd = new Random();
            

            for(int i = 0; i < 11; i++)
            {
                int a = rnd.Next(1, 11);
                samplelist.Add(a);
            }

            string n = samplelist.ToString();
            Console.WriteLine(" " + n);
            Console.WriteLine(samplelist.Max());
            Console.WriteLine(samplelist.Min());

出てくる結果

Comments

[dodox86]

たぶん先の質問、回答の延長で考えているのかもしれませんが、
https://teratail.com/questions/327609

> string num = " ";
...
> int dise = rnd.Next(1, 11);
> num += dise + " ";

はstring(文字列)で扱っていて、それすなわち配列ではありません。

[退会済みユーザー]

文字列に対してMax、Minを行っても、文字列の中で辞書順で一番最大の文字'9'と最小の文字' 'が出力されてるだけです。

[BluOxy]

samplelist.ToString では List の各要素は表示されません。
List を何かの文字で区切って表示したい場合には string.Join メソッドが使えます。

Answer 1

C#は強力なプログラミング言語なので、やりたいことがあってもそれを実現するに様々便利な方法があり、「調べる」と称してWEBを検索すると色んな情報が得られてしまいます。それは良いことではあるのですが、初心の方が一人で学習するには情報に振り回されがちになり、消化不良になることもあります。まずは「基本を学んだ上で、今、自分が扱える手持ちのテクニックだけでどこまでできるか」を考えて挑まれた方が良いケースも多いと思います。

言葉の捉え方の問題ですが、「調べる」とは自分にとって未知の情報を探しにいくことであり、配列やリストの中から最大のもの、最小のものを探しにいくことは普段の日常生活でも頭の中で同じようにやっていることで、その行為の仕方を「調べる」と言うよりは、普段やっていることをコードに書き起こすだけです。

そんな訳で、比較的素直、それほど深く考えずにコードに落とし込むと、例えば以下のようになります。

using System;
using System.Collections.Generic;
using System.Linq;

namespace ConApp1
{
    class Program
    {
        static void Main(string[] args)
        {
            var list = new List<int>();
            Random rnd = new Random();

            // 10個作りたいのであれば、0から9まで
            for (int i = 0; i < 10; i++)
            {
                int a = rnd.Next(1, 11);
                list.Add(a);
            }

            // 乱数の範囲は1から10の値なので、最小値の初期値を11、最大値の初期値を0とでも
            // しておき、それぞれの要素と比較して自力で取得すれば良い。
            int minValue = 11;
            int maxValue = 0;
            for (int i = 0; i < list.Count; i++)
            {
                if (list[i] < minValue)
                {
                    minValue = list[i];
                }

                if (maxValue < list[i])
                {
                    maxValue = list[i];
                }
            }

            // 生成したList要素数分の乱数値を確認したいのであれば、
            // Listの各要素を順番にConsole.WriteLine()すれば良い。要素の間は" "で区切る。
            // ※ foreach文を使えば少しだけシンプルになるが、あえて普通のfor文で。
            for (int i = 0; i < list.Count; i++)
            {
                if (0 < i)
                {
                    Console.Write(" ");
                }

                // Console.WriteLine()の引数は、int型の変数を渡せる。
                Console.Write(list[i]);
            }

            Console.WriteLine();
            Console.WriteLine("MAX=" + maxValue);
            Console.WriteLine("MIN=" + minValue);

            // おまけ： Linqを使った例
            Console.WriteLine("---- Linq:");

            // List中のint型要素をstring型にして配列にし、
            // string.Join()メソッドで' ' 区切りの文字列にする。
            var s = string.Join(' ', list.Select(v => v.ToString()).ToList().ToArray());
            Console.WriteLine(s);

            // Linqを使えば、List中のMaxもMinも得られる。
            Console.WriteLine("MAX=" + list.Max());
            Console.WriteLine("MIN=" + list.Min());
        }
    }
}

これを実行すると、例えば以下のような結果が出力されます。

C> ConApp1.exe
7 5 3 9 6 5 4 10 7 4
MAX=10
MIN=3
---- Linq:
7 5 3 9 6 5 4 10 7 4
MAX=10
MIN=3

Linqを使ったやり方はおまけで入れたものですが、同じ結果が出力されているのが分かりますね。ですが、そういった最適、効率的、便利な方法を学ぶのは後でも良くて、まずは愚直でも基本的な方法を自分が困らない程度に扱えるようになるのが先だと、少なくとも個人的には思います。

Comments

[Seeeveeen]

回答ありがとうございます。
疑問なんですが
int minValue = 11;
int maxValue = 0;
for (int i = 0; i < list.Count; i++)
{
if (list[i] < minValue)
{
minValue = list[i];
}

if (maxValue < list[i])
{
maxValue = list[i];
}
}
のところで、minvalueを０に設定していて、乱数は1～10の間で発生しているのになんでlist[i]<minvalueのように0より小さいときに代入、としているのでしょうか。

[dodox86]

> のところで、minvalueを０に設定していて、

いえ、最初にminValueの初期値は11（10でもいいですけど）としています。"考えられる最大の値"を"最小値の初期値"とすることで、発生した乱数1～10の値で、順次更新できるようにします。

[dodox86]

また、minVallueやmaxValueについては必ずしも初期値をセットしておかなくても良くて、例えば最初の要素list[0]の値をそれぞれに必ずセットし、list[1]からのものを比較して更新していっても良いです。更に、乱数を生成させるときに同時に比較、更新してしまえば、乱数生成時のループだけで処理できます。それだと最大、最小値を取得する為のループを減らすことができます。ただの最小値、最大値の取得にしても、色々な方法（考え方）でできる、ということです。

[Seeeveeen]

見間違えてた。。。
わかりやすい回答ありがとうございます。
もっと柔軟に考えれるように頑張ります。

Answer 2

発生乱数の確認だけならこれでできます。

for(int i = 0; i < 11; i++)
{
    int a = rnd.Next(1, 11);
    samplelist.Add(a);
    Console.Write($" {a}");  // これ追加
}
Console.WriteLine(""); // これも追加。改行しときます。※個人的にWriteLine("")で改行してる派

Answer 3

既に指摘が出ておりますが
string型に対してMax()、Min()を行った場合
stringをchar配列として扱い、最大、最小を取得するので
文字コード上の最小としてスペース、文字コード上の最大として9が取得されます。

数値配列に入れるか、Split()で分割してからSelect()メソッドでint変換を挟んでからMax()、Min()を使うか
方法は色々ありますので、工夫してみてください。

Answer 4

もう指摘してくださってる人がいる人もいると思うのですが、
string num = " ";
これが非常に怪しく見えます。
数字の大きさを比較するのに、stringでやるのは違和感を感じますね。

Answer 5

多分、仕様通りの動作ではないでしょうか。 (MSDN string)

string num = " ";

文字列に数字を入れてますが、文字列に入れることで、数字でなく、文字となります。　string.Max()は、文字列の中の最大の文字を取得し、string.Min()は最小の文字を取得します。
上記の場合、最大は、9ですが、最小は、 (空白)となります。 (多分、空白が出力されている)

数字として扱いたいならば、List<int> とか、int[] で配列として扱う必要があります。