我在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异常。 那么,我如何显示该数组的元素?
提前致谢。
答案 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");
}
}
}