Hi guys
I hope you guys can help me. This is my first post.
Iam having trouble with this error ExecuteReader: Connection property has not been initialized.

My code is as follows.

 Dim reader As SqlClient.SqlDataReader
        Dim SqlCommand1 As New SqlClient.SqlCommand
        Dim Con As New SqlClient.SqlConnection
        Dim strsql As String
        Dim server As String
        Dim database As String
        Dim user As String
        Dim pass As String



        server = "GREEDY"
        database = "Northwind"
        user = "sa"
        pass = ""
        If Con.State = 1 Then Con.Close()


   Con.ConnectionString() = "Data Source=" & server & ";Initial Catalog=" & database & ";User Id=" & user & ";Password=" & pass

        Con.Open()


        strsql = "Select customerid,orderid,employeeid From Orders"

        SqlCommand1.CommandText = strsql

        reader = SqlCommand1.ExecuteReader

        'This sets the columns of the ListView to the same column names of the table in the database
        With lstTest
            .Columns.Add(reader.GetName(0), 80, HorizontalAlignment.Center)
            .Columns.Add(reader.GetName(1), 80, HorizontalAlignment.Center)
            .Columns.Add(reader.GetName(2), 80, HorizontalAlignment.Center)
        End With

        'Now scrolling through the DataReader and populating the ListView with the data
        While reader.Read
            Dim ls As New ListViewItem(reader.Item("customerid").ToString()) ' you can also use reader.GetSqlValue(0)
            ls.SubItems.Add(reader.Item("orderID").ToString())
            ls.SubItems.Add(reader.Item("employeeid").ToString ())
            lstTest.Items.Add(ls)
        End While

        Con.Close()

O.o first to answer your question.

You have never supplied the Connection to the SqlCommand so it has no idea what it is supposed to connect to! You need to do something like: SqlCommand1.Connection = con

Also, am I right in assuming that you are fairly new to programming?? The reason I ask is because this code does not lend itself well to reuseability at all since everything is hardcoded.

I would suggest adding a class to your solution that handles all of your database functionality and you just return the necessary objects from that class and work with those objects in your calling class.

Also, you should never never never leave SQL Servers SA account setup with a blank password, that is asking for all sorts of trouble.

================================================== =========
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 planoie's profile^^
^^Modified text taken from gbianchi profile^^
================================================== =========
Technical Editor for: Professional Search Engine Optimization with ASP.NET
http://www.wiley.com/WileyCDA/WileyT...470131470.html
================================================== =========
Why can't Programmers, program??
http://www.codinghorror.com/blog/archives/000781.html
================================================== =========

Hi dparsons

I really do appreciate it alot. I sorted out the error. I am fairly new to programming but i really enjoy it so much. Again thank you for the very useful advice.

check this information about executereader()

http://vb.net-informations.com/ado.n...sqlcommand.htm

lee