NullReferenceException 在 C# 中未处理

Question

我在我评论的那一行中发现了这个错误，有什么问题？

private void pictureBox34_Click(object sender, EventArgs e)
{
    if (pictureBox34.Image == chess9.Properties.Resources.siyahsah2)
    {
        f();
    }
}

public void picarray()
{
    pic[0, 0] = pictureBox54;
    pic[0, 1] = pictureBox64;
    pic[0, 2] = pictureBox48;
    pic[0, 3] = pictureBox42;
    pic[0, 4] = pictureBox34;
    pic[0, 5] = pictureBox26;
    pic[0, 6] = pictureBox18;
    pic[0, 7] = pictureBox8;
    pic[1, 0] = pictureBox1;
    pic[1, 1] = pictureBox2;
    pic[1, 2] = pictureBox3;
    pic[1, 3] = pictureBox4;
    ...
}

public void f()
{
    // int i = 0, j = 0;
    int x = 3;
    int y = 3;

    for (int i = 1; i < x; i++)
    {
        for (int j = 1; j < y; j++)
        {
            pic[i, j] = new PictureBox();
            // pic[i, j] = pic[i + 1, j + 1];
            pic[i, j].Image = chess9.Properties.Resources.siyahsah2;
        }
    }
}

Answer 1

我碰巧看了你之前的问题，所以如果我理解正确的话，这是一个国际象棋游戏的实现。

建议您对设计进行一些更改，而不是尝试修复此代码中的异常。

主要变化：将游戏板状态的表示与 UI 层分开。 如果我理解正确，您使用的是一组 PictureBox 对象来表示板上的棋子。如果您编写代表棋盘、棋子及其所有动作的类，然后编写可以在窗口中显示这些类并通过 UI 接收用户命令的单独代码，您可能会更好。

当您通过操作 UI 对象来实现棋子移动和棋盘更新时，您必然会犯更多错误并花费更多时间进行调试。

希望对你有帮助……

Answer 2

• 确保指示的文件确实存在。此外，如果您在 Windows 系统上运行它，最好将文件路径中的 / 更改为 \\（或 \ 中的 @"..." 字符串）。 （虽然这不应该是您的问题的原因。）

  ... = Image.FromFile( @"pic\siyahsah2.jpg" );
  

• 确保索引 i 和 j 正确。您的for 循环表明它们将是1 和2 的任意组合。

此外，在您的 for 循环中：

pic[i, j] = new PictureBox();            //  <-- will get overwritten by (*)
pic[i, j] = pic[i + 1, j + 1];           //  <-- will get overwritten by (*)
pic[i, j] = new PictureBox();            //  <-- (*)
pic[i, j].Image = Image.FromFile(...);

即可以删除前两行。

Answer 3

我不知道你为什么要把它弄得这么复杂，但是假设骑士在picturebox34 或者换句话说在pic[0, 4] 并且你想将它向右移动并因此结束在pic[2, 3] 上，那无非就是这样做

// Move knight image from 0,4 to 2, 3
pic[2, 3].Image = pic[0, 4].Image;

// Make old knight position empty
pic[0, 4].Image = null;

我不知道你为什么要做for循环之类的......