Error: Index was out of range. Must be non-negative and less than the size of the collection. Parameter name: index

Question

2.00/5 (1 vote)

See more:

hello everyone can you help me please when I Press button to publish the file this error was show

SQL

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

Note:I do GridView2 inside GridView1 and GridView1 have a CommandName and GridView2 also have CommandName, the error happen in GridView2.

the code for define index for GridView1:

C#

protected void GridView1_RowCommand(object sender, GridViewCommandEventArgs e)
    {
        if (e.CommandName == "publish")
        {
            int index = Convert.ToInt32(e.CommandArgument);

            GridViewRow row = GridView1.Rows[index];

            int artid = Convert.ToInt32(((Label)GridView1.Rows[index].FindControl("lblartid")).Text);
            bool pub = ((CheckBox)GridView1.Rows[index].FindControl("chkpub")).Checked;
.
.
.
.

the code for define the index for GridView2 and where the error located I mark it by do under line, but I thing the error in the Definition of index :

C#

  protected void GridView2_RowCommand(object sender, GridViewCommandEventArgs e)
    {
        if (e.CommandName == "publish1")
        {
            int index = Convert.ToInt32(e.CommandArgument);
           
            GridViewRow row =GridView1.Rows[index];

            GridView grd = new GridView();

            grd = ((GridView)GridView1.Rows[index].FindControl("GridView2"));
     

          int flsid = Convert.ToInt32(((Label)grd.Rows[index].FindControl("lblflsid")).Text);
           
           bool pub = ((CheckBox)grd.Rows[index].FindControl("chkpub1")).Checked;
.
.
.
.

Please help me Quickly

Posted 6-Nov-12 9:53am

starab20

Updated 6-Nov-12 10:17am

Matt T Heffron

v2

Add a Solution

4 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)

starab20 · Answer 1 · 2012-11-06T10:49:00

what you main when you say are you sure that this second GridView has at least index+1 rows?
if you main if the GridView2 have some rows inside it the answer yes but I thing my problem is how I can define new index for GridView2 where it is inside GridView1 when I do this the below new code the Problem is finish but the code not work the web page reload but No difference happen:

C#

 protected void GridView2_RowCommand(object sender, GridViewCommandEventArgs e)
    {
        if (e.CommandName == "publish1")
        {
          int index1 = Convert.ToInt32(e.CommandArgument);

          if (e.CommandName == "publish")
           {
              int index = Convert.ToInt32(e.CommandArgument);

                GridViewRow row = GridView1.Rows[index];

                GridView grd = new GridView();

                grd = ((GridView)GridView1.Rows[index].FindControl("GridView2"));

                int flsid = Convert.ToInt32(((Label)grd.Rows[index1].FindControl("lblflsid")).Text);

                bool pub = ((CheckBox)grd.Rows[index1].FindControl("chkpub1")).Checked;

can you help me

starab20 · Answer 2 · 2012-11-06T11:04:00

Solution 4

mr. csharpbd2 how I can try use 0 replace index variable in this code

C#

int flsid = Convert.ToInt32(((Label)grd.Rows[index].FindControl("lblflsid")).Text);

I'm sorry but I'm Beginner in programming

Posted 6-Nov-12 11:04am

starab20

Comments

csharpbd 6-Nov-12 22:45pm

in those line of code you need to following change. change is below:
your code:
int flsid = Convert.ToInt32(((Label)grd.Rows[index].FindControl("lblflsid")).Text);

bool pub = ((CheckBox)grd.Rows[index].FindControl("chkpub1")).Checked;
.
.
.
.

and changed code:
int flsid = Convert.ToInt32(((Label)grd.Rows[0].FindControl("lblflsid")).Text);

bool pub = ((CheckBox)grd.Rows[0].FindControl("chkpub1")).Checked;
.
.
.
.

Matt T Heffron · Answer 3 · 2012-11-06T10:16:00

It looks like you are using index as the index into two different collections: GridView1.Rows and the Rows of the GridView2 that is in the row GridView1.Rows[index] (i.e., grd). Are you sure that this second GridView has at least index+1 rows?
Also, you're creating and throwing away an instance of GridView in the declaration of grd. Simplify this line and the next into one line, like this:

C#

GridView grd = ((GridView)GridView1.Rows[index].FindControl("GridView2"));

csharpbd · Answer 4 · 2012-11-06T10:29:00

when you initialize gridview grd using this code

C#

grd = ((GridView)GridView1.Rows[index].FindControl("GridView2"));

gridview grd got just 1 row with index=0, then you try to get value of gridview grd using variable index. at that time you get this error.

now you can try use 0 replace index variable in this code

C#

int flsid = Convert.ToInt32(((Label)grd.Rows[index].FindControl("lblflsid")).Text);

i think it works fine...

Error: Index was out of range. Must be non-negative and less than the size of the collection. Parameter name: index

4 solutions

Solution 3

Solution 4

Solution 1

Solution 2

Add your solution here

Preview 0