C#継承やオーバーライドについて

Question

前提・実現したいこと

c#勉強で詰まったので質問させてください.
いつもお世話になります。
以下のプログラムでクラスnyuryokuの中のstartのスコープの中についてです。
最終的にConsole.WriteLineで入力した物を表示したいのですが、引数の設定がうまくいきません。
よろしくお願いします

発生している問題・エラーメッセージ

「引数がありません」となり実行することができません。

該当のソースコード

c#
ソースコード
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;

namespace nameage
{
public class Information
{
// メンバ変数
public string UserName; // ユーザ名

	// コンストラクタ
	public Information()
	{
		UserName = string.Empty;
	}
}


public abstract class Base
{
	/// コンストラクタ
	protected Base()
	{
	}


	// 開始
	public abstract void Start();

	// 終了処理
	public abstract void End();

	// 名前入力
	protected abstract void GetUserName(ref string userName);

}


class nyuryoku : Base
{
            Information gi = new Information();
	public override void Start()
	{
		GetUserName(ref gi.userName); //←試した事をそのまま反映しました
	}

	public override void End()
	{ }

	//名前入力 
	protected override void GetUserName(ref string userName)
	{
		Console.Write("名前を入力してください:");
		userName = Console.ReadLine();
	}
    }
	
class Program
{
	static void Main(string[] args)
	{
		Information gi = new Information();
		nyuryoku cr = new nyuryoku();


		cr.Start();
		Console.WriteLine("ようこそ{0}さん", gi.UserName);

	}
}

}

試したこと

間違えてるとは思うのですが、「Information gi = new Information();」をクラスnyuryokuに作って、getUserNameの引数などに「ref gi.UserName」などとしてみましたが、表示される時に反映されておらず空欄となっていました・・・。

Comments

[YAmaGNZ]

「引数がありません」そのままだと思いますが・・・
GetUserNameは引数ありますよね？

[退会済みユーザー]

コードは ``` と ``` で囲ってください。インデントされて見やすくなるので。インデントされてないコードは質問者さん自身も読む気がしないのでは？

Answer 1

なんというか，それぞれのクラスが何をするための存在なのか，それらをどう組み合わせて出力まで持っていくつもりなのか？　が不明瞭です．

• nyuryokuは入力されたものをどうするのか？
• Infomationとnyuryokuをどういう関係にしたいのか？
• 最後に出力されるべき情報は誰が持っていて，出力するのは誰か？（今はMain()で出力しているが，そうするのならば，Main()内の出力処理箇所は誰からどうやって出力すべき情報を取り出せば良いのか？
• 継承の必要性が謎（←これは単に「勉強」だから，なのかな？）

とりあえず，

// 名前入力
protected abstract void GetUserName(ref string userName); 

注釈をこんな端的な言葉で終わらせずに，メソッドの仕様をきちんと決めて書いてみることから始めてみてはどうでしょう．
例えば，「引数の参照先を入力結果文字列に差し替える」という話に決めたならば，メソッドの呼び出し元

GetUserName(); //?

では，結果を受け取るための引数を与える必要があることが（より）明確になるでしょう．
→ということは引数として与えるためのstringがどこかに存在する必要があって…
→それはどこにあるべきなのか？…

Comments

[hiro0215]

そうですね。単に継承の勉強で流れをつかむために、あえてこういう形にさせていただきました。

[fana]

継承の存在理由はわかりました．

入力した内容が表示に反映されない件については
> Main()内の出力処理箇所は誰からどうやって出力すべき情報を取り出せば良いのか？
の部分を解決すると良いかと思います．
Console.WriteLine("ようこそ{0}さん", gi.UserName); だと，入力処理とは無関係なインスタンスgiのUserNameを出力しています．

[hiro0215]

調べていったらcrの中のgiで数値が変わっている事がわかりました。
という事は、なんらかの方法でcrの中のインスタンスgiを呼びだせばよいということでしょうか？

[fana]

そうですね，とりあえず現状のnyuryokuクラスに自身が保有するgiのUserNameを返すようなメソッドかプロパティを追加すれば動作は達成できるかもしれません．
メソッドなら例えば
public string GetXXX(){ return gi.UserName; }
とかでしょうか．

[fana]

それはそれとして，勉強とのことですから，
nyuryokuとInformationとの関係等を今と違う形にしたらどうだろうか？
→別の組み合わせ方が考えられるなら，今の形と比較してどっちがどうだろうか？
…とか，色々やってみるのも良いのではないかと思います．

Answer 2

「引数がありません」となり実行することができません。

GetUserNameにはstring型の変数を参照渡し(refキーワード付き)で渡さないとコンパイルエラーです。

- GetUserName(); //?
+ string userName = hoge;
+ GetUserName(ref userName);

ところで、ユーザ名と年齢を自分のメンバーに保持するだけのメソッドにGetから始まる命名は誤解を招きます。GetではなくSetではないでしょうか。

refキーワードも使い方が適切でない気がします。

Comments

[BluOxy]

細かいことはたくさんありますが、今回は特に気になる点2つに留めました。

[BluOxy]

もう1つ気になる点がありました。
Programクラスがnyuryokuクラスの内部クラスとして定義されていて違和感があります。分けて定義するのではいけないのでしょうか。

[hiro0215]

純粋に記入ミスをしていました。
ご指摘もあるのですが、今回は学習のためにrefや継承を無理やり使っており、大変おかしな物になっています。申し訳ございません・・・。

[BluOxy]

全ては学習のためということですね。
1つのプログラムで色々な機能を試し書きするよりかは、1つの学習につき1つのプログラムを書くほうがコード全体の目的がすっきりして学習しやすいと思います。

Answer 3

GetUserNameを引数なしで呼び出してますが、GetUserNameの定義では、引数が必要ですよね？

そのせいです。GetUserNameの定義で、userNameを必要としないように修正してみてください。

エラーが起きる場所

        public override void Start()
        {
            GetUserName(); //?
        }

GetUserNameの定義

// 名前入力
protected abstract void GetUserName(ref string userName);

}

//名前入力 
protected override void GetUserName(ref string userName)
{
Console.Write("名前を入力してください:");
userName = Console.ReadLine();
}