メソッドをメインで動かしたい

Question

お世話になります。下記のメソッド(Make_nextgen)をメインで動かしたいです。メソッドの呼び出し方が分かりません。以下、コードになります。

以下メインメソッドです。

class Program
    {
        static void Main(string[] args)
        {
            
            //初期世代の生成
            List<int[]> temp_gen = new List<int[]>();
            Random rnd = new System.Random();

            for (int i = 0; i < 10; ++i)
            {
                var individual = new int[10];
                for (int j = 0; j < 10; ++j)
                {
                    individual[j] = rnd.Next(0, 2);
                }
                temp_gen.Add(individual);
            }


            foreach (int[] item in temp_gen)
                {
                    System.Console.Write("[");
                    bool first = true;
                    foreach (int value in item)
                    {
                        if (!first) System.Console.Write(","); else first = false;
                        System.Console.Write(value);
                    }
                    System.Console.WriteLine("]");
                }

                
            
        }
        
    }

以下呼び出したいメソッドがあるクラスです。

public class Onemax
    {
        //インスタンス変数
        public int[] Individual { get; }
        public List<int[]> Temp_gen { get; }

        //コンストラクタ(インスタンス変数の初期化)
        public Onemax(int[] individual, List<int[]> temp_gen)
        {
            Individual = individual;
            Temp_gen = temp_gen;
        }
        public List<int[]> Make_nextgen(List<int[]> temp_gen, int[] individual)
        {
            int individual_sum = individual.Sum();

            return temp_gen;
        }
    }

以上になります。解答のほうよろしくお願いします。

Comments

[退会済みユーザー]

> 下記のメソッドをメインで動かしたいです。

「下記のメソッド」というのは質問のコードで言うとどれでしょう？　「メインで動かしたい」というのはどういう意味ですか？（非同期にしたいということ？　違うかな？）　その理由は何ですか？　

[退会済みユーザー]

> 下記のメソッドをメインで動かしたいです。

「下記のメソッド」は質問の追記で分かりましたが、「メインで動かしたい」というのはどういう意味ですか？　その理由は何ですか？

[old.exe]

Make_nextgenをメインメソッドに呼び出したいです。理由は何回も繰り返す処理なので、メソッドとして書いておいた方が良いかと考えたからです(メソッド自体はまだ未完成です)。すみません、理解がないので曖昧な質問、および解答になってしまいました。

[退会済みユーザー]

Onemax クラスの Make_nextgen メソッドを Main メソッドの中で呼び出したいと理解しましたが、Make_nextgen メソッド自体がメチャクチャという感じで（未完成とのことですがあんまりです）、呼び出し例を書くにも困ってしまうのですが。

[old.exe]

申し訳ないです。現状ではコードに落とし込むスキルがないのでMake_nextgenの動きを言葉で説明させていただきますと、Mainクラスのリストのリストtemp_genを引数に取り、temp_genを並び替えなどの操作を加え、temp_genをMainクラスに返すという仕事をさせたいとイメージしています。

[退会済みユーザー]

> 理由は何回も繰り返す処理なので、メソッドとして書いておいた方が良い

ということだそうですが、であれば、Main の中のコードをメソッドとして切り出すという方向で考えるべきだと思いますよ。

Make_nextgen メソッドはテキトーすぎてその話につながりません。

Answer 1

Onemax onemax = new Onemax(individual[j],temp_gen);
onemax.Make_nextgen(temp_gen,individual[j]);

こんな感じでしょうか、引数の部分などは間違ってるかもです。
クラスをnewしてインスタンス化して、メソッドを呼び出せばよいと思います。

Comments

[mamemaru]

ごめんなさい、引数やreturnしたものをどう使うかは不明でしたので現状書いてある内容で単に呼び出すためだけの記載に修正しました。

[退会済みユーザー]

> Onemax onemax = new Onemax(individual[j],temp_gen);

Onemax クラスのコンストラクタの引数は Main のどれをどのように与えるのですか？

Onemax クラスの Make_nextgen を Main メソッドで呼び出したいのだそうですが、

> temp_gen.add(onemax.Make_nextgen(temp_gen,individual[j]));

で質問者さんがやりたいことができているのでしょうか？

[mamemaru]

あくまで質問は他クラスのメソッドの呼び出し方についてですので、文法的な指摘等はあるとは思いますがそれは質問者様自身が考えるべきであって、今回私たちが回答する範囲を超えていると思います。
ですのでクラスをインスタンス化して、その際の引数はコンストラクタと合わせる。
呼び出すときはインスタンス引数.メソッド名で呼び出せますよという回答をしたまでです。

>> temp_gen.add(onemax.Make_nextgen(temp_gen,individual[j]));

>で質問者さんがやりたいことができているのでしょうか？
すいませんこの部分は間違っていたのでSurferOnWww様がコメントをする前に修正を行っておりました、申し訳ないです。

[退会済みユーザー]

> あくまで質問は他クラスのメソッドの呼び出し方についてですので、文法的な指摘等はあるとは思いますがそれは質問者様自身が考えるべきであって、今回私たちが回答する範囲を超えていると思います。

その姿勢は自分は賛同できないですね。そもそもの質問内容にいろいろ問題があってまともに呼び出し例を書くのは難しいのは分かりますけど、それでもある程度例として成り立たないと・・・

例えば、あなたが書いた引数 individual[j] は質問がさんのコードを見ると int 型になるようですけど、それを引数に与えるコードを回答の例として書くのはいかがなものかと思います。

Answer 2

そのクラスのインスタンスを作成して、そこから実行するようにします

Comments

[退会済みユーザー]

点数乞食は止めましょう。最近マイナス評価がスコアにカウントされないのが残念。

[Zuishin]

この回答は合っていると思いましたが、どこか間違っていますか？

[退会済みユーザー]

間違ってはいないかもしれませんが質問の回答にはなっていないと理解。これだけの情報で質問者さんが理解して求める答えだと評価してベストアンサーを付けたら取り消します。