Help with "Input string was not in a correct format" error

Riemann Harrison

Hi, first of all, I wanna thank this helpful forum which seeks to aid those of us who are just getting started with the world of programming.

My problem is as fallows, I want to add elements to an array, for that, I created a textbox wich would add the elements to the array,my code is as fallows:

int[] table = new int[24];

private void btnInput_Click(object sender, EventArgs e)

{

int elem = 0;

for (int i = 0; i < table.Length; i++)

{

if (i < table.Length)

{

tabla[elem] = Convert.ToInt32(txtEntrada.Text);//Input string was not in a correct format

txtEntrada.Clear();

}

else if (i==tabla.Length)

{

MessageBox.Show("Array full");

}

}

How ever, I get a code error while debugging, it says that the input string was not in correct format, which I don't understand why since I converted to int32, can someone please guide me as how can I fix this issue?

Hope someone can help me out with this, many thanks.

4 Replies

Know the answer? Post it — somebody with the same question will find it here.

Vulpes15y ago
Accepted answer
I think what you need to do here is get rid of the for loop and find a different way of counting the elements.

See if this is any better:

private int i = 0; // use private field

private void btnInput_Click(object sender, EventArgs e)
{
if (i < tabla.Length)
{
tabla[i] = Convert.ToInt32(txtEntrada.Text);
i++;
txtEntrada.Clear();
}
else if (i==tabla.Length)
{
MessageBox.Show("Array full");
txtEntrada.Enabled = false;
}
}
Riemann Harrison15y ago
wow thanks! this worked much better!! now I add the values with no debugging error! thank you so much Vulpes!
Riemann Harrison15y ago
One of the errors that I found is that once converted, I intended to clear the textbox so it would read the next value, but when the for loops again, it would read nothing, so that is why I'm getting that error, so what I did is move "txtEntrada.Clear()" at the bottom, as fallows:

private void btnInput_Click(object sender, EventArgs e)
{
for ( i = 0; i < 4; i++)
{
if (i < tabla.Length)
{
tabla[i] = Convert.ToInt32(txtEntrada.Text);
}

else if (i==tabla.Length)
{
MessageBox.Show("Array full");
txtEntrada.Enabled = false;
}
}

txtEntrada.Clear();
}

I still have some errors though, but I'm working on them, if you can provide me some hint, it would be welcome.
Vulpes15y ago
Well, the error message is telling you that the textbox doesn't contain a valid integer.

One possibility is that it includes a thousand separator or a decimal point.

Could that be the case here?