code:

---

        Dim CheckBox As CheckBox
        Dim KeyId As Int32

        For Each GridViewRow As GridViewRow In GridView1.Rows
            '---Look at each checkbox here
            CheckBox = CType(GridViewRow.FindControl("chkSelector"), CheckBox)

            If CheckBox.Checked Then
                '---Get the database id
                KeyId = CType(GridView1.DataKeys.Item(GridViewRow.RowIndex).Value, Int32)

                '---Use the KeytId to delete from the database.

            End If
        Next
        GridView1.DataBind()

---

quote:

---

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

---

code:

---

        myDataColumn = New Data.DataColumn
        myDataColumn.DataType = GetType(String)
        myDataColumn.ColumnName = "Product ID"
        myDataColumn.AutoIncrement = False
        myDataColumn.Caption = "Product ID"
        myDataColumn.ReadOnly = False
        myDataColumn.Unique = False
        myDataTable.Columns.Add(myDataColumn)

---

code:

---

            gt = Session("sessionvar")

            myrow = gt.NewRow()
            myrow(0) = ddlPr_Code.SelectedValue
            myrow(1) = ddlPr_Name.SelectedValue
            myrow(2) = txtPrice.Text
            myrow(3) = txtQuantity.Text
            myrow(4) = txtShipping.Text
            myrow(5) = (txtQuantity.Text * txtPrice.Text)

            gt.Rows.Add(myrow)
            GridView1.DataSource = gt
            GridView1.DataBind()

---

quote:

---

type ctrl+alt+w 1 when you've hit a breakpoint

---

quote:

---

gt = Session("sessionvar")

---

Wha-huh? Am I reading this correctly: you're using a data table stored in the Session object?!? This is probably beyond the scope of what you're learning now, but are you aware that IIS serializes access to the Session object? Not to mention the amount of chaos that will ensue if you do things like double-post the page (hit "F5" repeatedly) by accident. This method sounds... well... painful and risky.

Secondly, when I do something related (I don't do it the way you suggest; I do it the "old fashioned" way), I add a unique identifier in to the data set that's a HIDDEN column in the grid. Therefore, when I'm doing things like this, I don't have to worry about the "DataKeys" collection, but I simply access "GridViewRow[UniqueKey]".

I don't even know that the "DataKeys" collection is, but I don't know why you're checking the size of the "Rows" collection. You should be comparing "GridView1.DataKeys.Count" (not "GridView1.Rows.Count") to "GridViewRow.RowIndex".

I'm not a fan of Microsoft's way of doing this; I usually don't even use the ViewState (which I've seen be monumental in size when I do something similar) in cases like this and just create non-ASP controls with the unique identifier as the value.
 

---

Posted by Blayne Bradley (Member # 8565) on :
 
how do I compare "GridView1.DataKeys.Count" to "GridViewRow.RowIndex"?
 

---

Posted by MrSquicky (Member # 1802) on :
 
If you weren't you, I wouldn't believe that that was a serious question. For pete's sake, think for yourself.
 

---

Posted by Blayne Bradley (Member # 8565) on :
 
I don't know what I am doing, how the heck would I know what to do? Thinking accomplishes nothing if no amount of thinking determins what the frak am I supposed to do and google fails on me.
 

---

Posted by MrSquicky (Member # 1802) on :
 
I'm going to give you the benefit of the doubt and figure you're just really stressed out.

Answer this question. Why would you want to compare GridView1.DataKeys.Count to GridViewRow.RowIndex?
 

---

Posted by Morbo (Member # 5309) on :
 
People got stuff done before google, Blayne. Debugging sucks but keep plugging.
Good luck.
 

---

Posted by King of Men (Member # 6684) on :
 
Um. You know, Blayne, if you wanted to inspire my confidence that I want to work with you on the converter project, this was kind of not the way to do it. But let me give you a hint anyway. When people say 'think', they don't mean that you should try figuring out the Grand Unified Theory from first principles, they mean that you should step through the code by hand. You do understant what I mean by that, yes?

*Walks off muttering about schools these days*
 

---

Posted by El JT de Spang (Member # 7742) on :
 

quote:

---

Originally posted by Blayne Bradley:
Thinking accomplishes nothing if no amount of thinking determins what the frak am I supposed to do and google fails on me.

---

quote:

---

Originally posted by Blayne Bradley:
I cant figure it out, it is not figurable out, I do not know anything about how these asp/vb classes/methods work so I do not know where to begin looking or what to look for. How am I supposed to know which .etc does what I want it to do?

---

code:

---

        For Each GridViewRow1 As GridViewRow In GridView1.Rows
            '---Look at each checkbox here
            CheckBox1 = CType(GridViewRow1.FindControl("chkSelector"), CheckBox)
            Dim instance As New GridViewRowEventHandler

            If CheckBox1.Checked Then
                chkBox = True

                'Dim ProductID As String = GridView1.DataKeys(GridViewRow1.RowIndex)("pr_code").ToString
                '---Get the database id
                KeyId = GridViewRow1.RowIndex
                'KeyId = CType(GridView1.DataKeys.Item(GridViewRow1.RowIndex).Value, Int32)
                '---Use the KeytId to delete from the database.
                'GridView1.DeleteRow(KeyId)

                gt.Rows(KeyId).Delete()
                GridView1.DeleteRow(KeyId)



            End If
        Next
        GridView1.DataSource = gt
        GridView1.DataBind()

---

quote:

---

Exams coming up no time to "go back to basics" and the exams require the work having actually being done since they open book open notes.

---

quote:

---

but you've made me regret trying to help you.

---

quote:

---

This is just 1 tiny problem and while learning to debug programs is a valuable skill and something I am decent at in any other language VB/ASP pisses the hell out of me because it is overly complicated

---

quote:

---

Originally posted by Blayne Bradley:
google or ask other people.

---

quote:

---

Originally posted by fugu13:
You are not learning how to make an online store. Any online store you try to make using these methods will be dysfunctional. Neither are you learning to program. You should complain about the quality of the course to your registrar. Heck, I'm near to complaining about the quality of the course to your registrar.

---

quote:

---

Originally posted by Blayne Bradley:
As for those hours we were playing CK at the same time, so no soup for you!

---

quote:

---

And why the heck does the Vista version for VS2005 not have a proper debug mode!?

---

In order to properly debug code in real-time in VS2005 when running on a Vista machine, it is necessary to run your instance of Visual Studio as an administrator. The easiest way to do this, assuming you have rights, is to right-click on the icon and choose "Run as Administrator" instead of double-clicking on the icon. You can also customize the icon, but any lab system worth its salt would remove that customization when you logged out.
 

---

Posted by King of Men (Member # 6684) on :
 
In my experience, students do not get admin rights to blow their own noses when it comes to university lab computers. In many cases there are good reasons for this.
 

---

Posted by steven (Member # 8099) on :
 
"In many cases there are good reasons for this."

Based on the guys I knew who were getting computer degrees, and/or worked in the computer labs in college, yes. More than half of them were worthless hardware-stealing trouble-making vandals. They gave intelligent geeky white guys a very bad name.
 

---

Posted by Blayne Bradley (Member # 8565) on :
 
riiiiiiiiight I remember now vaguely that my teacher had difficulties and fixed it that way after 3 hours of googling.


And yes I do have admin rights. Advantages of making it to the third year, I also know how to get around nearly ever single firewall and proxy restriction as well.
 

---

Posted by MrSquicky (Member # 1802) on :
 

quote:

---

All I wanted to know is what was wrong with that line of code because as far as I can tell with the debugger everything is fine, I dont know what parameter index relates to, there is no variable called index. All I wanted was what is wrong and what I could do to fix it not the friggin secret of life.

---

quote:

---

Originally posted by Blayne Bradley:
[QB]I know how to debug, I know how to use the intermediate window, but VB is the only language I work with that uses this many methods, all other languages we were taught we built our own from scratch to learn how to do make our own algorithms.

---

quote:

---

Lets look at it this way, I made a line of code comparing the value of a txtbox and a lbl, how was I supposed to know you are supposed to put val(etc.text) there? To my view a number is a number is a number, didn't know that therwise it simply compares them to see which string is bigger.

---

quote:

---

I made a line of code comparing the value of a txtbox and a lbl, how was I supposed to know you are supposed to put val(etc.text) there?

---

code:

---

Index     Name     Price     Quantity
0       Halo 3      79$         50
1       Resistence  79$         50
2       Smash Bros  79$         50

---

code:

---

Index     Name     Price     Quantity
0       Halo 3      79$         50
1       Smash Bros  79$         50

---

quote:

---

Originally posted by Blayne Bradley:
All I wanted to know is what was wrong with that line of code because as far as I can tell with the debugger everything is fine, I dont know what parameter index relates to, there is no variable called index. All I wanted was what is wrong and what I could do to fix it not the friggin secret of life.

---

quote:

---

Originally posted by King of Men:

quote:

---

Lets look at it this way, I made a line of code comparing the value of a txtbox and a lbl, how was I supposed to know you are supposed to put val(etc.text) there? To my view a number is a number is a number, didn't know that therwise it simply compares them to see which string is bigger.

---

You are supposed to know that by checking the documentation to see whether it's a string or an int, and not make excuses. Nobody is interested in your excuses. We're willing to help you if you show some slightest hint of a good attitude. You're not doing that.

---

code:

---

        For Each GridViewRow1 As GridViewRow In GridView1.Rows
            '---Look at each checkbox here
            CheckBox1 = CType(GridViewRow1.FindControl("chkSelector"), CheckBox)

            If CheckBox1.Checked Then
                'chkBox = True

                'Dim ProductID As String = GridView1.DataKeys(GridViewRow1.RowIndex)("pr_code").ToString
                '---Get the database id
                KeyId = GridViewRow1.RowIndex
                'KeyId = CType(GridView1.DataKeys.Item(GridViewRow1.RowIndex).Value, Int32)
                '---Use the KeytId to delete from the database.

                If KeyId = gt.Rows.Count Then
                    gt.Rows(KeyId - 1).Delete()
                    ' Larry said its okay to delete one row at a time for now. Iff there's time fix it.
                ElseIf KeyId >= gt.Rows.Count Then
                    gt.Rows(KeyId - KeyId).Delete()
                Else
                    gt.Rows(KeyId).Delete()
                    ' Larry said its okay to delete one row at a time for now. Iff there's time fix it.

                End If
            End If

        Next
        GridView1.DataSource = gt
        GridView1.DataBind()

---

code:

---

0 Halo3 79.99
1 COH 89.99
2 NWN2 79/999

---

quote:

---

Simple logic game, how do you take two objects - let's say the second and the fifth from the left - out of a line up such that at the time you take them out, they will still be the second and the fifth from the left, respectively?

---

code:

---

If KeyId = gt.Rows.Count Then
	gt.Rows(KeyId - 1).Delete()
ElseIf KeyId >= gt.Rows.Count Then
	gt.Rows(KeyId - KeyId).Delete()

---

code:

---

0 Halo3 79.99
1 COH 89.99
2 NWN2 79/999

---

code:

---

0 Halo3 79.99
2 NWN2 79/999

---

code:

---

0 Halo3 79.99
1 NWN2 79/999

---

code:

---

0 Halo3 79.99
2 NWN2 79/999

---

code:

---

0 Halo3 79.99
1 NWN2 79/999

---

quote:

---

Gridview is refreshed and fills its rows based on the new datatable until there's no datatable left thus no gridview.

---

code:

---

0 Halo3 79.99
1 COH 89.99
2 NWN2 79/999
3 Halo2 79.99
4 WOW 89.99

---

code:

---

0 Halo3 79.99
1 COH 89.99
2 NWN2 79/999
4 WOW 89.99

---

code:

---

0 Halo3 79.99
1 COH 89.99
2 NWN2 79/999
3 WOW 89.99

---

Thus there is no more index(4) thus when i tried to loop through and went to the last row (in this case 4) to delete it, there was no row and it blew up.

Say I havr 5 rows, and I want to dlete all of them.

It will successfully delete all of the rows until the last row which through merging has become row 0.

Thus if I have 10 rows, have KeyId - KeyId and delete that row.

Thus because there is only ONE row left I automatically delete the first row being the only one.

This was the case of having alot of rows to start off with and deleting all of them.

I go back by 1 only when I am deleting a comparitevly small number of rows that are the end rows.

My solution DOES in FACT work, I have thoroughly tested it, and it works, if you think it doesn't or shouldn't work then welcome to Ornery You are Wrong.
 

---

Posted by MrSquicky (Member # 1802) on :
 
No, Blayne, your solution doesn't work. The operations that Tom provided will show this to you.
 

---

Posted by TomDavidson (Member # 124) on :
 
Blayne, do exactly what I describe. Create ten items. Check #2, #7, #9, and #10. Click your "Delete" button.
 

---

Posted by Blayne Bradley (Member # 8565) on :
 
I did do what Tom showed, works just fine on my end.
 

---

Posted by MattP (Member # 10495) on :
 
I'd expect that when you went to delete #7, you'd actually delete #8 (because the index is off by one at that point because of the previous delete of #2).

Note that deleting a single row will work. Also, because of your if/else, deleting x number of rows at the end will also work. What I don't expect to work is deleting a row in the middle, then another row in the middle.

Edit: No, deleting at the end won't work either. It'll start deleting rows from the beginning once the index gets high enough. Deleting ALL rows will work though, since you'll always delete the quantity of rows that you are asking to delete, just not the rights ones, but if you delete all rows than it doesn't matter that you aren't deleting the correct ones since your if/else fall back will cause the first or last row to always be deleted once the index goes wonky.

Again, think about it. You already know the index is getting out of sync with the rows, but the only time you deal with that is when the index exceeds the total number of rows. How do you make sure that the index matches the middle rows once you've started deleting some? Why would it be off by one (or more) only when it's on the last row?

[ November 26, 2007, 04:15 PM: Message edited by: MattP ]
 

---

Posted by MrSquicky (Member # 1802) on :
 
THere's a difference between didn't throw an error and works fine. If you did what Tom described, you'd end up with
3
4
5
6
7
9
10
 

---

Posted by MrSquicky (Member # 1802) on :
 
Yeah, oops, I took that out, then I looked back and thought "Why did I take 3 out? 1 and 2 would be deleted by the 9 and 10 calls." Silly me.

edit: Which is to say, the list should look like
4
5
6
7
9
10
 

---

Posted by MattP (Member # 10495) on :
 
Oops. I was having doubts and deleted my post. Sorry about that.
 

---

Posted by Blayne Bradley (Member # 8565) on :
 
I just had to grandstand, I could have just left it alone but nooooooo my 6th sense just had to force me to double check and no of course I just had to stick to my guns I just couldn't stay quiet, fix my code, I just had to make an ass of myself before realizing I was slightly a smigden mistaken.

code:

---

        Dim count = 0
        For Each GridViewRow1 As GridViewRow In GridView1.Rows
            '---Look at each checkbox here
            CheckBox1 = CType(GridViewRow1.FindControl("chkSelector"), CheckBox)

            If CheckBox1.Checked Then
                'chkBox = True

                '---Get the database id
                KeyId = GridViewRow1.RowIndex
                'KeyId = CType(GridView1.DataKeys.Item(GridViewRow1.RowIndex).Value, Int32)
                '---Use the KeytId to delete from the database.

                If KeyId = gt.Rows.Count Then
                    gt.Rows(KeyId - 1).Delete()
                ElseIf KeyId > gt.Rows.Count Then
                    gt.Rows(KeyId - KeyId).Delete()
                Else
                    If count = 0 Then
                        gt.Rows(KeyId).Delete()
                    Else
                        gt.Rows(KeyId - count).Delete()
                    End If
                End If
                count = count + 1
            End If

        Next
        GridView1.DataSource = gt
        GridView1.DataBind()

---

quote:

---

*blink* okay I am 101% positive that this does the job, as the rows merge it now anticipates it so that it will delete the proper rows regardless of how far they've moved sicne each time I delete something I am 1 row off, thus if I dlete 3 things I am 3 rows off, hence, I use a counter to keep track of how many times I dleted something and adjust what row to dlete accordingly.

---

code:

---

If count = 0 Then
     gt.Rows(KeyId).Delete()
Else
     gt.Rows(KeyId - count).Delete()
End If

---

code:

---

gt.Rows(KeyId - count).Delete()

---

code:

---

int iPos = 0
while ( iPos < gt.Rows.Count )
{
     if ( delete row iPos? )
          delete row from DB using the unique key
          delete "gt.Rows[iPos]" from collection     
     else
          iPos++
}

---

quote:

---

Originally posted by TomDavidson:
Actually, it just occurred to me why Blayne might be resisting our suggestions to iterate backwards; he might not know how, since you can't use foreach backwards. He'll need a for loop for that, and his teacher may not have shown him that.

---

quote:

---

MattP, don't you need a "Step -1"? I admit I haven't done that myself in VB for quite some time...

---

quote:

---

Originally posted by Blayne Bradley:
I only know how to loop backwards in C.

For (e=0;e<10;e--)

---

quote:

---

Don't like one SQL statement myself either becuase I need to write more code to maintain the collection array and eventually convert that in to the actual SQL statement.

---

code:

---

        Dim count = 0
        For Each GridViewRow1 As GridViewRow In GridView1.Rows
            '---Look at each checkbox here
            CheckBox1 = CType(GridViewRow1.FindControl("chkSelector"), CheckBox)

            If CheckBox1.Checked Then
                'chkBox = True

                'Dim ProductID As String = GridView1.DataKeys(GridViewRow1.RowIndex)("pr_code").ToString
                '---Get the database id
                KeyId = GridViewRow1.RowIndex
                'KeyId = CType(GridView1.DataKeys.Item(GridViewRow1.RowIndex).Value, Int32)
                '---Use the KeytId to delete from the database.

                If KeyId = gt.Rows.Count Then
                    gt.Rows(KeyId - 1).Delete()
                ElseIf KeyId > gt.Rows.Count And (gt.Rows(KeyId - count)(1) = Nothing) Then
                    gt.Rows(KeyId - KeyId).Delete()
                Else
                    If count = 0 Then
                        gt.Rows(KeyId).Delete()
                    Else
                        gt.Rows(KeyId - count).Delete()
                    End If
                End If
                count = count + 1
            End If

        Next
        GridView1.DataSource = gt
        GridView1.DataBind()

---

quote:

---

gt.Rows(KeyId - KeyId).Delete()

---

code:

---

 
Dim count = 0        
For Each GridViewRow1 As GridViewRow In GridView1.Rows            
'---Look at each checkbox here    
CheckBox1 = CType(GridViewRow1.FindControl("chkSelector"), CheckBox)            
If CheckBox1.Checked Then                   
gt.Rows(GridViewRow1.RowIndex - count).Delete
count = count + 1            
End If        
Next        
GridView1.DataSource = gt   
GridView1.DataBind()

---

[ November 27, 2007, 12:45 PM: Message edited by: TomDavidson ]
 

---

Posted by MattP (Member # 10495) on :
 
Fish giver!
 

---

Posted by Blayne Bradley (Member # 8565) on :
 
Ah, now i see what you mean.
 

---

Posted by King of Men (Member # 6684) on :
 
You know, a few days ago I was idly paging through Joel's archives, and I came across this article,which contains among much wisdom the words

quote:

---

I know a few superstar programmers who could crank out a Fortran compiler by themselves in one week, and lots of programmers who couldn’t write the code to print the startup banner if they had six months.

---

code:

---

package test;

public class paulaBean {

  private String paula = "Brillant";

  public String getPaula() {
    return paula;
  }
}

---

quote:

---

Originally posted by King of Men:
You know, a few days ago I was idly paging through Joel's archives, and I came across this article,which contains among much wisdom the words

quote:

---

I know a few superstar programmers who could crank out a Fortran compiler by themselves in one week, and lots of programmers who couldn’t write the code to print the startup banner if they had six months.

---

For some reason I thought he was exaggerating. I can't imagine what I was thinking. Although cheer up, Blayne, google "Paula Bean" and realise that you haven't yet hit rock bottom.

---