for loop in window forms applications

Question

Hello;-)

I just designed a simple "For Loop" using window form application. I would like that this will be only clickable once & that it will not repeat the same information if I click the button. How could I do that? thanks Here is my code:

        int count;

        for (count = 0; count < 5; count = count + 1)
        {
            listBox1.Items.Add("This is count: " + count);
        }
        const string textEnd = "Done!!!";
        {
            listBox1.Items.Add(textEnd);
        }

==== Additional Info === I've made it this way now. This will only click once, but the button is still enable. I think this is ok:

        int count;

        for (count = 0; count < 5; count++)
        {
            string newItem = "This is count: " + count; 
            if (listBox1.Items.IndexOf(newItem) < 0) 
            { 
                listBox1.Items.Add(newItem); 
            }
         }

        const string textEnd = "Done!!!";
        if (listBox1.Items.IndexOf(textEnd) <0)
        {
            listBox1.Items.Add(textEnd);
        }

Answer 1

button1.Enabled = false;

Answer 2

On the click event, add button1.Enabled = false

And maybe, after the loop, you may want to add button1.Enabled = true to reenable the button. :)

Answer 3

I'm assuming you don't want the same items added to the list multiple times?

Instead of

{
    listBox1.Items.Add("This is count: " + count);
}

You need something like

{
    string newItem = "This is count: " + count;
    if(listBox1.Items.IndexOf(newItem) < 0)
    {
        listBox1.Items.Add(newItem);
    }
}

Answer 4

You can just use a simple 'flag' to determine whether the loop has already run.

Create a global variable eg bool HasRun = false;

Then check the state of the flag before you execute the loop eg if HasRun == true

Set HasRun to true when you first run the loop.

Finally you could also disble the button on first run eg button1.Enabled = false;

Answer 5

If it's not too many items, you could always clear the listbox before adding the stuff.

listbox1.Items.Clear();
...your adding code...

But the probably best solution is just to disable the button, like Ian wrote.

Answer 6

Best one is what Ian Said however you can also hide the button completely by running either of the folowing commands

button.Hide();

Or

button.Visable = false;