C#のメンバ変数について

質問

下記のコードで enterName の変数を static void Resurut()
のメソッドへの受け渡しが上手くいかず、どなたか修正点を教えて頂けないでしょうか？

【ソースコード】
static class Program
{
static string enterName;
static int cntWin = 0; //勝利回数
static int cntLose = 0; //敗北回数
static int cntDraw = 0; //引分回数
//ユーザー名の入力
static void Main(string[] args)
{
string enterName = Console.ReadLine();
Console.WriteLine("ユーザ名を入力してください: {0}", enterName);
Start();
Resurut();
}

*一部省略

static void Resurut()
{
if (cntWin > cntLose)
{
Console.WriteLine("{0}さんの総合勝利です", enterName);
}
else if (cntLose > cntWin)
{
Console.WriteLine("CPUの総合勝利です");
}
else if (cntWin == cntLose)
{
Console.WriteLine("引き分けです");
}
}
}

回答No.1

AsarKingChang · ベストアンサー

static class Program
{
static string enterName;
ここで、せっかく同一クラス内メンバーを作ったのに、
static void Main(string[] args)
{
string enterName = Console.ReadLine();
ここで、同じ名前のローカル変数を作った為です。

Console.WriteLine("{0}さんの総合勝利です", enterName);
こいつが見てるのは、
static string enterName;
こっち側です！

対処法、
string enterName2 = Console.ReadLine();
Console.WriteLine("ユーザ名を入力してください: {0}", enterName2);
とやってから、enterName=enterName2;と代入してやるか、

Resurut();こいつにパラメーターを与えるようにする。
Resurut(enterName);などとして、
static void Resurut()ここを
static void Resurut(string enterName)と書き換えて新規のローカル変数を作るなど。
＃なお、スペルは、resultです。

あと、可能なら、関数の入り口は、voidとせず、
パラメーターをしっかり書いた方がいいです。
static void Resurut()
こうではなく、
static void result(string enterName,int cntWin,int cntLose,int cntDraw)
など！
こうすることで、その関数が、プログラムの中でどんな変数を
必要としているのかが、他の人が見た時はっきりわかるので、
メンテナンスしやすく、バグが減ります。

ひとまず、原因は、
名前空間内で、クラスメンバー名をローカル変数名で上書きしたことが、原因です。

お礼

ありがとうございます。

回答No.2

AsarKingChang

直接今回の件に関係ないけど、
else if (cntWin == cntLose)
これいらないかも、
直前で、大きいか？小さいか？の残りがここなので、
すでに、同じってのは比較しなくてもわかってるので、
if 不要ですね。

しっかし、今の人って、覚えるのが早くていいね～
質問者さんやたら、覚えるのが早いですよ。
私なんて結構苦労したのに～（独り言！）
んじゃ！がんばってね！