Hi All,

I am trying to insert selected contents of a listbox into a database by looping through the collection. The loop is working and the contents are going into the database. Starnge thing is i come up with this error as soon as the contents are inserted in the database!.

     Dim iCount As Integer
        Dim ddlcount As Integer
        Dim strItem As String

        For iCount = 0 To CInt(ddlSelectedLocations.Items.Count)

            If ddlSelectedLocations.Items.Item(iCount).Selected <= CInt(ddlSelectedLocations.Items.Count) Then


                If ddlSelectedLocations.Items.Item(iCount).Selected Then
                    'strItem = lst.Items(iCount).value

                    SqlCommand1.Parameters("@townid").Value = CInt(ddlSelectedLocations.Items(iCount).Value)
                    SqlCommand1.Parameters("@userid").Value = CInt(Request.QueryString("u"))
                    SqlConnection1.Open()
                    SqlCommand1.ExecuteNonQuery()
                    '
                    SqlConnection1.Close()

                End If
            End If

        Next









Index was out of range. Must be non-negative and less than the size of the collection. Parameter name: index
Description: An unhandled exception occurred during the execution of the current web request. Please review the stack trace for more information about the error and where it originated in the code.

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

Source Error:


Line 346:
Line 347:
Line 348: If ddlSelectedLocations.Items.Item(iCount).Selected Then
Line 349: 'strItem = lst.Items(iCount).value
Line 350:


Source File: C:\Inetpub\wwwroot\BJ\candidatereg_3.aspx.vb Line: 348

hi there..

the error is clear... you are looping for one item more than the collection count...

since it's a base 0 collection, you go from 0 to count -1 because if you have only one element the index of it is 0, if you have 2 elements the index are 0 and 1, etc....

so.. the only thing you have to change is:

HTH

Gonzalo

================================================== =========
Read this if you want to know how to get a correct reply for your question:
http://www.catb.org/~esr/faqs/smart-questions.html
^^Took that from dparsons signature and he Took that from planoie's profile
================================================== =========
My programs achieved a new certification (can you say the same?):
WORKS ON MY MACHINE
http://www.codinghorror.com/blog/archives/000818.html
================================================== =========

 God bless you gbianchi , you were right.... it worked. Thanks a million!!!