C#　オブジェクト参照のエラー

エラーの意味がよくわかりません。
どうしてエラーが出るのか詳しく教えてください。
あと改善方法なども教えて下さい。

###発生している問題・エラーメッセージ
オブジェクト参照がオブジェクトインスタンスに設定されていません。

###該当のソースコード
using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Windows.Forms;

namespace WindowsFormsApplication1
{
public partial class Form1 : Form
{
public int[][] board = new int[8][];

    public Form1()
    {
        InitializeComponent();
    }

    private void Form1_Load(object sender, EventArgs e)
    {
        for (int x = 0; x < 8; x++)
        {
            board[x] = new int[8];
        }

        for (int x = 0; x < 8; x++)
        {
            for (int y = 0; y < 8; y++)
            {
                board[x][y] = 0;
            }
        }
    }

    public void jag(int x,int y)
    {
        if (board[x][y] == 0)
        {
            MessageBox.Show("OK");
        }
        else
        {
            MessageBox.Show("NO");
        }
    }

    private void Form1_Click(object sender, EventArgs e)
    {
        Test sample = new Test();

        sample.test();
    }

}

class Test
{
    Form1 form = new Form1();

    public void test()
    {
        form.jag(3,2);
    }
}

}

行動規範の内容に同意します

回答3件

エラーの原因等はChironian様の仰る通りですので、少しばかり補足を。

Form.Loadイベントは、フォームが初めて表示される直前に発生します。(参考)
Show()していればLoadイベントが発生しますが、インスタンスの生成だけでは発生しないので注意です。

また、改善案としてはForm1のコンストラクタでboardを設定するというのも一つの手かと思います。

投稿2016/05/26 14:40

gusao

総合スコア185

Chironian

2016/05/26 14:44

> また、改善案としてはForm1のコンストラクタでboardを設定するというのも一つの手かと思います。可能な場合は、必要な領域の獲得はコンストラクタで行った方が不具合が発生しにくいので、gusaoさんの改善案が好ましいと思います。

otfria

2016/05/27 00:41

コンストラクタをやってみたら成功しました。 gusaoさんの改善案を参考に改良していきたいと思います。

行動規範の内容に同意します

ベストアンサー

こんにちは。

Testクラスの中でForm1()をnewしていますが、Form1_Load()を呼び出していません。
従って、Form1_Load()の中で実行しているboard配列への設定処理が呼ばれていませんから、jag()の中のif (board[x][y] == 0)にて参照エラーが発生してしまいます。

通常は、Main()関数の中でApplication.Run(new Form1());のようにしてForm1のインスタンスを用いてメッセージ・ループを回るようになっています。そのメッセージ・ループの中でForm1_Load()が呼ばれます。

【追記】
TestクラスのコンストラクタでForm1への参照をわたせば良いはずです。

下記イメージです。（テストしてないのでビルドエラーがでたら適宜修正下さい。）

C#
1（略）
2        private void Form1_Click(object sender, EventArgs e) 
3        { 
4            Test sample = new Test(this);
5
6            sample.test(); 
7        } 
8   }
9
10    class Test 
11    { 
12        Form1 form;
13        Test(Form1 form1)
14        {
15            form=form1;
16        }
17
18        public void test() 
19        { 
20            form.jag(3,2); 
21        } 
22    } 
23}