C# An unhandled exception of type 'System.ArgumentOutOfRangeException' Error

Question

I am getting the error An unhandled exception of type 'System.ArgumentOutOfRangeException' occurred in RCPYC Membership Program.exe Additional information: Index was out of range. Must be non-negative and less than the size of the collection. in the following code.

private void ApplyBtn_Click(object sender, EventArgs e)
    {
        int index = 0;
        bool loop = true;
        string text;
        List<string> Data = new List<string>();
        while (loop == true)
        {
            if (index == 17)             //If statment takes a boolean value and sets it to the correct radio button
            {
                if (BtOwnerRdio.Checked == true)
                { Data[index] = "true";}
                else if(SocialRdio.Checked == true)
                {
                    Data[index] = "false";
                }
            }
            else                        //Else statement finds the correct textbox and sets the data to its text value
            {
                if (index < 10)
                { text = "tb0" + index.ToString(); }
                else
                { text = "tb" + index.ToString(); }

                Control tb = Controls.Find(text, true).FirstOrDefault() as Control; //Finds control with the name of the text string

                if (index != 0) //Does not encrypt the ID value
                {

                    try
                    {
                        MessageBox.Show("Try");
                        Data[index] = P.encrypt(tb.Text);  //Sets the list value to the textbox text value      
//This is the line that causes the error
                    }
                    catch (Exception)
                    {
                        MessageBox.Show("Fail");   
                        throw;         **Error is thrown here**
                    }
                }
            }

            index = index + 1;  //Adds 1 to the index to move to the next loop
            if (index == Data.Count - 1) //***
            { loop = false; }   //Ends the loop if the Data list is filled
                }

The code is getting the text value from a textbox (tb) and adding it to a list (Data) but returns the error when a blank value is returned. I've checked the P.encrypt method completes when a blank value is returned and the error occurs when the string is added to the list. The index when the list is called is 1 and I have tried manually setting the list capacity to 30, however I still get the error.

MY understanding of the error is that the index is either too high or negative, however if I have manually set the list capacity to 30 how can 1 be too high or negative?

which line gets the error, show the declaration and all accesses to the container (array, list,...) that gives the problem — pm100
– pm100, Commented Apr 25, 2017 at 20:22

Ecnerwal · Accepted Answer · 2017-04-25 20:24:25Z

2

There are two miss usage in this code.

You are setting the Capacity. Capacity does not mean current element quantity, it means the maximum quantity of elements.
List does not act like array do.

Change Data[index] = P.encrypt(tb.Text); for Data.Add(P.encrypt(tb.Text));

answered Apr 25, 2017 at 20:24

Ecnerwal

3701 gold badge2 silver badges17 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

Will Anderson Over a year ago

Thanks, this fixed the original error, however it unfortunately just revealed another one elsewhere :')

Ecnerwal Over a year ago

Glad this fixed the original error. Just a quick note, try to include variables declaration next time.

Chris · Accepted Answer · 2017-04-25 20:25:54Z

2

I think you have some confusion about what the capacity of a list is. For a List<T> the [Capacity][1] is just the number of elements the list can hold before it needs to resize its internal data structure. Importantly this does not mean that the list has 30 entries, it means that while the size of the list is less than 30 it will not need to do any resizing. It will still start with 0 entries when first created.

I assume that you haven't actually put anything into the list yet so Data[1] will indeed be out of range. If you want 30 initialised slots then you either need to add 30 items to initialise your list or use something like an array which would have 30 items if you set its size to 30.

answered Apr 25, 2017 at 20:25

Chris

27.7k7 gold badges77 silver badges94 bronze badges

Collectives™ on Stack Overflow

C# An unhandled exception of type 'System.ArgumentOutOfRangeException' Error

2 Answers 2

2 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

2 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related