4

ここで完全な初心者。これは私の最初の C# の試みであり、'Left Right Center' と呼ばれる飲酒ゲームをシミュレートするコンソール アプリケーションです。コンソールでは、次のメッセージが表示されます。

コンソール

Unhandled Exception: System.NullReferenceException: Object reference not set to an instance of an object
at LeftRightCenter.MainClass.Main (System.String[] args) [0x00038] in     /Users/apple/Projects/LearningC/LearningC/Main.cs:80 
[ERROR] FATAL UNHANDLED EXCEPTION: System.NullReferenceException: Object reference not set to an instance of an object
 at LeftRightCenter.MainClass.Main (System.String[] args) [0x00038] in /Users/apple/Projects/LearningC/LearningC/Main.cs:80

C#

    using System;

    namespace LeftRightCenter
    {
        class Player
        {
            //fields        
            private int _quarters = 4;

            public int Quarters {
                get{ return _quarters; }
                set{ _quarters += value; }
            }

            public Player (string name)
            {

            }   

        }
        class Dice
        {
            Random random = new Random();
            public int Roll ()
            {
                random = new Random ();
                int diceSide;
                diceSide = random.Next (0, 6);
                diceSide = (diceSide > 2) ? 3 : diceSide;
                return diceSide;            
            }
        }
        class MainClass
        {
            static int activePlayer = 0;
            static int theCup       = 0;

            static Player[] thePlayers = { 
                new Player ("Jessica"), 
                new Player ("Isaac"), 
                new Player ("Ed"), 
                new Player ("Bella"),
                new Player ("Elisa"),
                new Player ("Fake RedHead"),
                new Player ("Linda"),
                new Player ("MJ"),
                new Player ("Irene"),
                new Player("Devin")
            };

            static Dice[] theDice = new Dice[2];

            private static void MoveQuarter (int direction)
            {
                int numberOfPlayers = thePlayers.Length - 1;
                switch (direction) {
                case 0: 
                    thePlayers [activePlayer].Quarters = -1;
                    theCup++;
                    break;
                case 1:
                    thePlayers [activePlayer].Quarters = -1;
                    int leftPlayer = (activePlayer == 0) ? numberOfPlayers : activePlayer - 1;
                    thePlayers [leftPlayer].Quarters = +1;
                    break;
                case 2:
                    thePlayers [activePlayer].Quarters = -1;
                    int rightPlayer = (activePlayer == numberOfPlayers) ? 0 : activePlayer + 1;
                    thePlayers [rightPlayer].Quarters = +1;
                    break;                          
                }           
            }

            public static void Main (string[] args)
            {
                int cupEndPoint = thePlayers.Length * 4 - 1;
                while (theCup < cupEndPoint) {
                    foreach (Dice rattle in theDice) {
                        if (thePlayers [activePlayer].Quarters > 0) {
                            MoveQuarter (rattle.Roll ()); // this line seems to be the problem  
                        }                   
                    }
                    Console.WriteLine ("{0} Quarters In the Cup", theCup);
                }

            }
        }
    }

問題が何であるか、またはその理由がわかりません。私のグーグル検索では、有用というよりは混乱を招きやすいことが証明されています。

興味のある方のために、私は現在ほとんど実験を行っていません

http://pastebin.com/jxCCW2cd

4

1 に答える 1

8

この行

static Dice[] theDice = new Dice[2];

Dice クラスの 2 つのオブジェクトを格納できる配列を宣言していますが、この配列の各値はまだ null です。

Main メソッド内の foreach ループで使用する前に、配列の各スロットに Dice を作成する必要があります。

theDice[0] = new Dice();
theDice[1] = new Dice();

行でデバッガを停止した場合

 MoveQuarter (rattle.Roll ());

rattleDice が null であることがわかります。

編集: あなたのコードを見ると、問題のある状況が見つかりました。Roll メソッドでは、ランダム ジェネレーターを再作成しますが、これはランダム性には適していません。(この質問で受け入れられた回答を参照してください
)

static Dice[] theDice = new Dice[2] {new Dice(), new Dice()};

これはあなたの Dice クラスの完全なリビジョンです

class Dice 
{
    private static Random random;
    public  Dice()
    {
           // create the static random generator only on the first instance
        if(random == null) random = new Random();
    }

    public int Roll () 
    { 
        int diceSide; 
        diceSide = random.Next (1, 7); 
        diceSide = (diceSide > 2) ? 3 : diceSide; 
        return diceSide;             
    } 
}
于 2012-06-15T21:07:44.613 に答える