Solved

problem on closing form in vb.net (Object reference not set to an instance of an object.)

Posted on 2009-04-02
6
309 Views
Last Modified: 2013-11-26
hi,
 
i have an error when closing a form, the error is

Object reference not set to an instance of an object.

i have included the code below and the line of code the error comes from is

 voc = curRow.Row.Item(1)

how do i overcome this, im stuck. have tried setting the selected index value of the combox on the exit button but this does not work

any ideas?

thanks in advance
Private Sub ComboBox1_SelectedIndexChanged(ByVal sender As System.Object, ByVal e As System.EventArgs) Handles ComboBox1.SelectedIndexChanged
 
        Dim curRow As DataRowView = ComboBox1.SelectedItem
        Dim voc
        Dim SQLString As String
        Dim dtevents As New DataTable()
        Dim dbDataAdapter As OleDbDataAdapter
        'Dim ConnectString As String = "Provider=Microsoft.Jet.OLEDB.4.0;Data Source=A470s media.mdb;Persist Security Info=True"
        ' ComboBox1Event.ValueMember 
        'Dim ConnectString As String = "Provider=Microsoft.Jet.OLEDB.4.0;Data Source=C:\Documents and Settings\Administrator\Desktop\Copy of vb 30th march\A470s Media Manager\A470sMedManager\A470s media.mdb;Persist Security Info=True"
        voc = curRow.Row.Item(1)
        'events = Me.ComboBox1Event.SelectedIndex + 1
        'events = Me.ComboBox1Event.SelectedItem
        SQLString = "SELECT * FROM [MediaAsset] WHERE [Vocalist Code] = " & voc & ""
        dbDataAdapter = New OleDbDataAdapter(SQLString, ConnectString)
        dbDataAdapter.Fill(dtevents)
        Dgvsearch.DataSource = dtevents
        ListBox3.DataSource = dtevents
        ListBox3.DisplayMember = "AssetFileName"
        ListBox3.ValueMember = "AssetSource"

Open in new window

0
Comment
Question by:kipl20
  • 4
  • 2
6 Comments
 
LVL 48

Expert Comment

by:jpaulino
ID: 24050754
Try include this
Private Sub ComboBox1_SelectedIndexChanged(ByVal sender As System.Object, ByVal e As System.EventArgs) Handles ComboBox1.SelectedIndexChanged
 
If Me.ComboBox1.SelectedIndex <> -1 Then
   'you code
End If
 
End Sub

Open in new window

0
 
LVL 48

Expert Comment

by:jpaulino
ID: 24050765
Or
Dim curRow As DataRowView = ComboBox1.SelectedItem
If curRow Isnot Nothing Then
   'You code
End if

Open in new window

0
 

Author Comment

by:kipl20
ID: 24050980
hi, thanks for the reply but his doesnt seem to change any thing. my comboboxes are filled wit data from my access database using the code supplied. so i have a search from and on this form is a combobox, if the combobox text is Vocalist then it will fill another combo with the data.

does this have anything to do with it?
 ElseIf Me.ComboBox3.Text = "Vocalist" Then
                     Dim dbConn As System.Data.OleDb.OleDbConnection = New System.Data.OleDb.OleDbConnection(strConn)
            dbConn.Open()
            Dim DSet As New DataSet, SQLStr As String
            Dim cmd As System.Data.OleDb.OleDbCommand
            Dim dbAdaptr As System.Data.OleDb.OleDbDataAdapter = New System.Data.OleDb.OleDbDataAdapter
            Dim tTbl As DataTable
            With dbAdaptr
                .TableMappings.Add("Table", "Vocalist ")
                SQLStr = "Select [Vocalist Name],[Vocalist Code] from Vocalist"
                cmd = New System.Data.OleDb.OleDbCommand(SQLStr, dbConn)
                cmd.CommandType = CommandType.Text
                .SelectCommand = cmd
                .Fill(DSet)
                .Dispose()
            End With
            ' DSet.AcceptChanges() ' --> This line is useless
            tTbl = DSet.Tables.Item(0)
            DSet.Dispose()
            dbConn.Close()
 
            'fill out array by Event Names for combobox
            ComboBox1.DataSource = tTbl
            ComboBox1.DisplayMember = "Vocalist Name"
            ComboBox1.ValueMember = "Vocalist Code"
            'ComboBox1Event.SelectedItem = Nothing
            Me.ComboBox1.Show()
            Me.Label1.Show()
            DataGridView1.Hide()
            ComboBox1.Text = "---Select---"
            txtSearchEvent.Hide()
            txtSearchAr.Hide()
            txtSearchVoc.Show()
            Label2.Show()
            Me.Label1.Text = "Search By Vocalist"
            ListBox3.DataSource.clear()

Open in new window

0
How our DevOps Teams Maximize Uptime

Our Dev teams are like yours. They’re continually cranking out code for new features/bugs fixes, testing, deploying, responding to production monitoring events and more. It’s complex. So, we thought you’d like to see what’s working for us. Read the use case whitepaper.

 
LVL 48

Accepted Solution

by:
jpaulino earned 500 total points
ID: 24051162
Have you tried ?
Private Sub ComboBox1_SelectedIndexChanged(ByVal sender As System.Object, ByVal e As System.EventArgs) Handles ComboBox1.SelectedIndexChanged
 
Dim curRow As DataRowView = ComboBox1.SelectedItem
If curRow Isnot Nothing Then
   ' You code
End if
 
End Sub

Open in new window

0
 

Author Comment

by:kipl20
ID: 24051178
hi sorry jpaulino i read your code wrong, i was putting it down as

 Dim curRow As DataRowView = ComboBox1.SelectedItem
If curRow Isnot Nothing Then
   'You code
else
voc = curRow.Row.Item(1)
End if

instead of the other way round which works, i think i was readin the isnot nothing as it is nothing

thanks alot
 
0
 
LVL 48

Expert Comment

by:jpaulino
ID: 24051238
Glad I could help!
0

Featured Post

MIM Survival Guide for Service Desk Managers

Major incidents can send mastered service desk processes into disorder. Systems and tools produce the data needed to resolve these incidents, but your challenge is getting that information to the right people fast. Check out the Survival Guide and begin bringing order to chaos.

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

Welcome my friends to the second instalment and follow-up to our Minify and Concatenate Your Scripts and Stylesheets (http://www.experts-exchange.com/Programming/Languages/.NET/ASP.NET/A_4334-Minify-and-Concatenate-Your-Scripts-and-Stylesheets.html)…
Many of us here at EE write code. Many of us write exceptional code; just as many of us write exception-prone code. As we all should know, exceptions are a mechanism for handling errors which are typically out of our control. From database errors, t…
Microsoft Active Directory, the widely used IT infrastructure, is known for its high risk of credential theft. The best way to test your Active Directory’s vulnerabilities to pass-the-ticket, pass-the-hash, privilege escalation, and malware attacks …

829 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question