如果元素的数量是变量,如何显示数组元素

时间:2013-05-01 01:40:09

标签: c#

我在GUI中制作计算器,我需要一些帮助。 当我在文本框中输入一些数据时,我需要将它存储在一个数组中。这就是我想到的。

    int numOfPackages;//used to get user input

    private void button3_Click(object sender, EventArgs e)
    {
        int[] weight = new int[numOfPackages];

        for(int i = 0; i < numOfPackages; i++)
        {
            weight[i] = Convert.ToInt32(weightBox.Text);
        }

        foreach (int i in weight)
            totalCostLabel.Text = "" + weight[i];

    }

当我尝试显示元素时,它会给我indexOutOfRange异常。 那么,我如何显示该数组的元素?

提前致谢。

3 个答案:

答案 0 :(得分:5)

这一行

foreach (int i in weight)
    totalCostLabel.Text = "" + weight[i];

应该是

foreach (int w in weight)
    totalCostLabel.Text = "" + w;

您当前的代码会迭代权重数组,并尝试使用权重作为权重数组的索引,从而导致索引超出范围异常。

另一个问题是第一个循环:您将weight的所有值设置为相同的数字:

weight[i] = Convert.ToInt32(weightBox.Text); // That's the same for all i-s

如果权重不同,它们应该来自不同的权重框,或者单个weightBox中的字符串应该以产生多个数字的方式处理(例如,使用{{1 }})。

答案 1 :(得分:0)

这里有多个问题。首先是:

    foreach (int i in weight)
        totalCostLabel.Text = "" + weight[i];

这是迭代权重数组并使用该数组中的每个值。然后,您将该值用作索引。请看以下示例:

weight[0] = 0
weight[1] = 1
weight[2] = 15

在您的代码中,前两个条目将起作用,因为索引为0且索引为1.但是当它到达最后一个条目时,它会查找15的索引。您可以通过以下两种方式修复此问题,第一种是使用常规for循环:

for(int i=0; i < weight.Length; i++) 
{
    totalCostLabel.Text += weight[i];
}

这带来了第二个错误。您没有在代码中向totalCostLabel添加任何内容,只是替换了值。这会将所有重量值附加在一起。

另一种方法是使用foreach循环:

foreach(int i in weight) 
{
    totalCostLabel.Text += i;
}

这与上面相同,但您不必担心索引。

最重要的是,即使你修复了你的循环,你可能需要修改标签获取文本的方式,否则你将无法得到你想要的结果。

答案 2 :(得分:0)

也许你想要更喜欢的东西?

public partial class Form1 : Form
{
    public Form1()
    {
        InitializeComponent();
        btnAdd.Enabled = false;
    }

    int[] weight;
    int entriesMade;
    int numOfPackages;

    private void btnReset_Click(object sender, EventArgs e)
    {
        if (int.TryParse(numEntriesBox.Text, out numOfPackages))
        {
            weight = new int[numOfPackages];
            entriesMade = 0;
            btnReset.Enabled = false;
            btnAdd.Enabled = true;
            totalCostLabel.Text = "";
        }
        else
        {
            MessageBox.Show("Invalid Number of Entries");
        }
    }

    private void btnAdd_Click(object sender, EventArgs e)
    {
        int value;
        if (int.TryParse(weightBox.Text, out value))
        {
            weight[entriesMade] = value;
            weightBox.Clear();

            totalCostLabel.Text = "";
            int total = 0;
            for (int i = 0; i <= entriesMade; i++)
            {
                total = total + weight[i];
                if (i == 0)
                {
                    totalCostLabel.Text = weight[i].ToString();
                }
                else
                {
                    totalCostLabel.Text += " + " + weight[i].ToString();
                }
            }
            totalCostLabel.Text += " = " + total.ToString();

            entriesMade++;
            if (entriesMade == numOfPackages)
            {
                btnAdd.Enabled = false;
                btnReset.Enabled = true;
                MessageBox.Show("Done!");
            }
        }
        else
        {
            MessageBox.Show("Invalid Weight");
        }
    }

}