if文の中で作成したインスタンスをif文の外で使いたい

Question

前提・実現したいこと

C#でif文を用いてデータをロードするか新規でデータを作る簡単なコードを書きたい

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

エラー	CS0165	未割り当てのローカル変数 'player' が使用されました。

該当のソースコード

using System;

namespace ConsoleQuest
{
	class Program
	{
		static void Main(string[] args)
		{
			Logger.Inject(new ConsoleLogger(), new ConsoleInput());

			string currentDirectory = System.IO.Directory.GetCurrentDirectory();

			string jsonPath = currentDirectory + "\playerData.json";

			string Playname;

			int num;

			Logger.Log("Start Game!");

			Player player;

			do
			{
				Logger.Log("新しく始めますか:0\n続きから始めますか:1");

				string start = Logger.ReadInput();

				int.TryParse(start, out num);

				if (num == 0)
				{

					Logger.Log("プレイヤーの名前を入力してください");

					Playname = Logger.ReadInput();

					//create player
					player = new Player(Playname, 100f, 10f, 5f, 1, 0);

					Logger.Log("プレイヤーデータを生成します");
					Logger.Log("出力先:" + jsonPath);

					SaveUserJson(player, jsonPath);
				}
				else if (num == 1)
				{
					Player loadedData;
					if (!LoadUserJson(jsonPath, out loadedData))
					{
						return;
					}
					Console.WriteLine(loadedData.MaxHP.ToString(), loadedData.AttackPoint.ToString(),
						loadedData.Level.ToString());

					player = loadedData;
                }
                else
                {
					Logger.Log("0か1以外が入力されました");
					Logger.Log("もう一度入力してください");
                }
			} while (num != 0 || num != 1);

			//create world
			World world = new World(player);

			//worldが終了判定(false)を返すまでループ
			while (world.Loop())
			{
				//Enter入力を待つ
				Logger.ReadInput();
			}

			//THE END
			Logger.Log("game over.");
		}

Answer 1

Player player = null;としておくのが手間がないですが、コンパイラに分かりやすいフロー？にすればエラーを消せます。

あとint.TryParseは失敗時に0になるので戻り値をチェックしてください（もしくは選択肢を1/2にする等）

using System;
using System.IO;

namespace Questions300161
{
    class Program
    {
        static void Main()
        {
            //Logger.Inject(new ConsoleLogger(), new ConsoleInput());
            var currentDirectory = Directory.GetCurrentDirectory();
            var jsonPath = currentDirectory + "\playerData.json";
            Player player;

            Logger.Log("Start Game!");
            while(true)
            {
                Logger.Log("新しく始めますか:0\n続きから始めますか:1");
                var start = Logger.ReadInput();
                if(int.TryParse(start, out var num))
                {
                    if(num == 0)
                    {
                        Logger.Log("プレイヤーの名前を入力してください");
                        var Playname = Logger.ReadInput();
                        player = new Player(Playname, 100f, 10f, 5f, 1, 0);

                        Logger.Log("プレイヤーデータを生成します");
                        Logger.Log("出力先:" + jsonPath);
                        SaveUserJson(player, jsonPath);

                        break;
                    }
                    else if(num == 1)
                    {
                        if(!LoadUserJson(jsonPath, out player))
                        {
                            return;
                        }
                        break;
                    }
                }

                Logger.Log("0か1以外が入力されました");
                Logger.Log("もう一度入力してください");
            }

            var world = new World(player);
            while(world.Loop())
            {
                Logger.ReadInput();
            }
            Logger.Log("game over.");
        }

        private static bool LoadUserJson(string jsonPath, out Player loadedData)
        {
            loadedData = new Player("Player", 100f, 10f, 5f, 1, 0);
            return true;
        }
        private static void SaveUserJson(Player player, string jsonPath) { }
    }

    internal class World
    {
        public World(Player player) { }
        internal bool Loop() => false;
    }
    internal class Player
    {
        public Player(string playname, float v1, float v2, float v3, int v4, int v5) { }
    }
    internal class Logger
    {
        internal static void Log(string v) => Console.WriteLine(v);
        internal static string ReadInput() => Console.ReadLine();
    }
}

提示されていない部分は雑にでっち上げました^^;

Comments

[k.ryou]

ありがとうございます！
おかげでエラーが消えて実行できました！

Answer 2

下記と同じです。

        string n;
        for(int i=0;i<100;i++){
            n += i+"";
        }
        
        System.Console.WriteLine("Hello "+n);

//error CS0165: Use of unassigned local variable `n'

例え、実装上はループやifを必ず通るものだったとしても、プログラムはそのように解釈してくれません。
「分岐はあくまで分岐で通らないケースもある」として解釈しますので、何かしらで初期化は必要。

        string n=null;
        for(int i=0;i<100;i++){
            n += i+"";
        }
        
        System.Console.WriteLine("Hello "+n);

※各クラスの定義が提示されてないのでそれだけで想定通りで動くかは知らないです

Comments

[Zuishin]

分岐は else があればそれで網羅性のチェックをしてくれます。この回答と質問では、一回もループしない可能性のある for と必ず一回ループすることが保証されている do というところが違いますし、ループ内でも初期化していないところが違います。

[m.ts10806]

なるほど。勉強になります。
エラーから推測できるのはそこかなというのでコード組んでみましたが、
do whileで確認したほうが良かったですね。

Answer 3

まず、numの初期化がされてません。

            if (num == 0)

numにはなにがはいってるでしょうか

---

ああ、

int.TryParse(start, out num);

ここでいれてるのね。見落としてました。
さて、ここで、numに、２とか３が入ったときはどうなるか考えてみよう

って、

} while (num != 0 || num != 1);

条件がfalseになる場面が存在しませんね