Index was out of range. Must be non-negative and less than the size of the collection.

Question

1.00/5 (1 vote)

See more:

Please find my code below:

If dtApp.Rows.Count <> 0 Then

If dtApp.Rows(0).Item(0).ToString = Nothing Then
ddlChangeStatus.SelectedValue = 0
Else
ddlChangeStatus.SelectedValue = dtApp.Rows(0).Item(0).ToString
Session("ChangeStatus") = ddlChangeStatus.SelectedItem.Text
End If

If ddlChangeStatus.SelectedItem.Text = "Change In Progress" Then

ddlChangeStatus.Items.RemoveAt(ddlChangeStatus.Items.IndexOf(ddlChangeStatus.Items.FindByText("Awaiting Change Management Approval")))
ddlChangeStatus.Items.RemoveAt(ddlChangeStatus.Items.IndexOf(ddlChangeStatus.Items.FindByText("Awaiting M&S CAB Approval")))
ddlChangeStatus.Items.RemoveAt(ddlChangeStatus.Items.IndexOf(ddlChangeStatus.Items.FindByText("Awaiting Ceridian CAB Approval")))
ddlChangeStatus.Items.RemoveAt(ddlChangeStatus.Items.IndexOf(ddlChangeStatus.Items.FindByText("Awaiting Ceridian Email CAB Approval")))
ddlChangeStatus.Items.RemoveAt(ddlChangeStatus.Items.IndexOf(ddlChangeStatus.Items.FindByText("Authorised & Awaiting Implementation")))
ddlChangeStatus.Items.RemoveAt(ddlChangeStatus.Items.IndexOf(ddlChangeStatus.Items.FindByText("Cancelled or Rejected")))
ddlChangeStatus.Items.RemoveAt(ddlChangeStatus.Items.IndexOf(ddlChangeStatus.Items.FindByText("Completed")))
ddlChangeStatus.Items.RemoveAt(ddlChangeStatus.Items.IndexOf(ddlChangeStatus.Items.FindByText("Awaiting Environment Approval")))
ddlChangeStatus.Items.RemoveAt(ddlChangeStatus.Items.IndexOf(ddlChangeStatus.Items.FindByText("On Hold/Pending")))
ddlChangeStatus.Items.RemoveAt(ddlChangeStatus.Items.IndexOf(ddlChangeStatus.Items.FindByText("Change Expired–On Hold")))
ddlChangeStatus.Items.RemoveAt(ddlChangeStatus.Items.IndexOf(ddlChangeStatus.Items.FindByText("Change Expired-Cancelled")))

End If

What I have tried:

Can someone please advise what is wrong with the index?

Posted 19-May-16 0:33am

Member 12533124

Updated 19-May-16 1:00am

Add a Solution

Comments

jimmson 19-May-16 6:50am

Hi, what is wrong with the index - explanation is in the title of your question. Did you tried to debug your code? Without debug, you hardly solve this problem..

F-ES Sitecore 19-May-16 7:07am

When you write code like

If dtApp.Rows(0).Item(0).ToString = Nothing Then

if dtApp.Rows has no elements, ie it is empty, of if .Item is empty then you'll get that error as you are trying to get the first element of an empty array. Debug your code to find out exactly what array is empty and either work out why it is empty when it shouldn't be (if that's the case) or else check that the array has at least one item in it before accessing it.

1 solution

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Member 12533124 · Answer 1 · 2016-05-19T01:00:00

Solution 1

Hello Jimmson,

When I debug, the code crashes after this line:

ddlChangeStatus.Items.RemoveAt(ddlChangeStatus.Items.IndexOf(ddlChangeStatus.Items.FindByText("On Hold/Pending")))

Posted 19-May-16 1:00am

Member 12533124

Comments

jimmson 19-May-16 7:08am

Hello, please don't post a your reply as a solution, use comments or "Improve your question" button. Anyway, your code is not very clear, you doing a lot of stuff in one line of code.. I assume exception is thrown from RemoveAt method - put everything inside into separate variable and check that index is in the range. Something like this:
var index = ddlChangeStatus.Items.IndexOf(ddlChangeStatus.Items.FindByText("On Hold/Pending"));
if(index >= 0 && index < ddlChangeStatus.Items.Count)
{
ddlChangeStatus.Items.RemoveAt(index);
}

jimmson 19-May-16 7:15am

I'm sorry, I didn't notice you use VB, not C#. But the idea is still same. Put index into separate variable and then check that this index is in the range of the collection you working with -> equal/bigger than zero AND smaller than the count of items in the collection.