I get error "Index was out of range. Must be non-negative and less than the size of the collection. Parameter name: index".How I'll solve it?

Question

0.00/5 (No votes)

See more:

C#

private void BindGrid()
    {
        int selectedRowIndex = GridView1.SelectedIndex;
        int pId = (int)GridView1.DataKeys[selectedRowIndex].Value;//I get error in this line.
        SqlConnection conn;
        SqlCommand comm;
        SqlDataReader reader;
        string connectionString =
        ConfigurationManager.ConnectionStrings[
        "ConnectionString"].ConnectionString;
        conn = new SqlConnection(connectionString);
        comm = new SqlCommand("SELECT ProductId, ProductName, Price FROM Product " +
        "WHERE ProductId=@ProductId", conn);
        comm.Parameters.Add("ProductId", SqlDbType.Int);
        comm.Parameters["ProductId"].Value = pId;
        try
        {
            conn.Open();
            reader = comm.ExecuteReader();
            employeeDetails.DataSource = reader;
            employeeDetails.DataKeyNames = new string[] { "ProductId" };
            employeeDetails.DataBind();
            reader.Close();
        }
        finally
        {
            conn.Close();
        }
    }

Posted 21-Jan-14 21:22pm

Dhaliwal1

Updated 21-Jan-14 21:23pm

v2

Add a Solution

2 solutions

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)

CPallini · Accepted Answer · 2014-01-21T21:29:00

Solution 1

From the documentation[^]:

The zero-based index of the selected row in a GridView control. The default is -1, which indicates that no row is currently selected.

You could modify your code this way:

C#

private void BindGrid()
    {
        int selectedRowIndex = GridView1.SelectedIndex;
        if ( selectedRowIndex < 0) return;
        //...

Posted 21-Jan-14 21:29pm

CPallini

Comments

Dhaliwal1 22-Jan-14 4:37am

thank you for giving me a solution.Please elaborate it because i'm beginner in c# programming and can't rectify my error.

CPallini 22-Jan-14 4:45am

Well, I gave you a detailed answer. If no index is currently selected in the GridView then the GridView1.SelectedIndex property is -1, this is an error condition for the remaining code. Adding the line
if (selectedRowIndex < 0) return;
prevents the execution of the remaining code.

Dhaliwal1 23-Jan-14 6:38am

Thank you so much..finally it works..

CPallini 23-Jan-14 6:44am

You are welcome.

praveenagarkar · Accepted Answer · 2014-01-22T01:27:00

GridView1.SelectedIndex = -1 because no index is selected in the gridview

try the below code...

private void BindGrid()
{
int selectedRowIndex = GridView1.SelectedIndex;
if(selectedRowIndex>0)
{
int pId = (int)GridView1.DataKeys[selectedRowIndex].Value;
SqlConnection conn;
SqlCommand comm;
SqlDataReader reader;
string connectionString =
ConfigurationManager.ConnectionStrings[
"ConnectionString"].ConnectionString;
conn = new SqlConnection(connectionString);
comm = new SqlCommand("SELECT ProductId, ProductName, Price FROM Product " +
"WHERE ProductId=@ProductId", conn);
comm.Parameters.Add("ProductId", SqlDbType.Int);
comm.Parameters["ProductId"].Value = pId;
try
{
conn.Open();
reader = comm.ExecuteReader();
employeeDetails.DataSource = reader;
employeeDetails.DataKeyNames = new string[] { "ProductId" };
employeeDetails.DataBind();
reader.Close();
}
finally
{
conn.Close();
}
}

C#

}