74% of employees feel they are not achieving their full potential. With Linux Academy, not only will you strengthen your team's core competencies but also their knowledge of of the newest IT topics.
With new material every week, we'll make sure that you stay ahead of the game.
Course Of The Month
10 days, 7 hours left to enrollBecome a Premium Member and unlock a new, free course in leading technologies each month.
Solved
VS C# Connection Property Not Been Initialized
Posted on 2012-12-27
I have upgraded a program from VS C# 2008 to VS C# 2010. It is now set for .Net FW 3.5 where before it was .Net 2.0
When I run the code and this routine is called, I am now getting the message "Connection Property Has Not Been Initialized" on the cmd.ExecuteNonQuery() line. I tried inserting cmd.Connection.Open() before the execute but then I get "Object has not been set to an instance of an object" message. It appears something is different. How do I structure this code so the sp will fire?
private void InsertReasonForChange()
{
try
{
SqlCommand cmd = new SqlCommand();
/* Insert/Update the reason for the sell price change */
cmd.CommandType = CommandType.StoredProcedur
cmd.Parameters.Add(new SqlParameter("@SOPNUMBE", SOPNUMBE));
cmd.Parameters.Add(new SqlParameter("@SOPTYPE", SOPTYPE));
cmd.Parameters.Add(new SqlParameter("@LNITMSEQ", LNITMSEQ));
cmd.Parameters.Add(new SqlParameter("@CMPSEQNM", CMPSEQNM));
cmd.Parameters.Add(new SqlParameter("@USERID", GP_User_ID));
cmd.Parameters.Add(new SqlParameter("@REASON", txtReason.Text));
cmd.CommandText = "rbs_ChangeSellPriceReason
cmd.Connection = DataConnection;
cmd.ExecuteNonQuery();
}
catch (Exception ex)
{
string eMsg = "003: ERROR: " + ex.Message;
if (stackTraceWanted) eMsg += "\n" + ex.StackTrace;
if (stackTraceWanted) MessageBox.Show(eMsg);
}
}
When I run the code and this routine is called, I am now getting the message "Connection Property Has Not Been Initialized" on the cmd.ExecuteNonQuery() line. I tried inserting cmd.Connection.Open() before the execute but then I get "Object has not been set to an instance of an object" message. It appears something is different. How do I structure this code so the sp will fire?
private void InsertReasonForChange()
{
try
{
SqlCommand cmd = new SqlCommand();
/* Insert/Update the reason for the sell price change */
cmd.CommandType = CommandType.StoredProcedur
cmd.Parameters.Add(new SqlParameter("@SOPNUMBE", SOPNUMBE));
cmd.Parameters.Add(new SqlParameter("@SOPTYPE", SOPTYPE));
cmd.Parameters.Add(new SqlParameter("@LNITMSEQ", LNITMSEQ));
cmd.Parameters.Add(new SqlParameter("@CMPSEQNM", CMPSEQNM));
cmd.Parameters.Add(new SqlParameter("@USERID", GP_User_ID));
cmd.Parameters.Add(new SqlParameter("@REASON", txtReason.Text));
cmd.CommandText = "rbs_ChangeSellPriceReason
cmd.Connection = DataConnection;
cmd.ExecuteNonQuery();
}
catch (Exception ex)
{
string eMsg = "003: ERROR: " + ex.Message;
if (stackTraceWanted) eMsg += "\n" + ex.StackTrace;
if (stackTraceWanted) MessageBox.Show(eMsg);
}
}
[X]
Welcome to Experts Exchange
Add your voice to the tech community where 5M+ people just like you are talking about what matters.
- Help others & share knowledge
- Earn cash & points
- Learn & ask questions
4-
3
2
9 Comments
Active today
Where is DataConnection initialized? You should have a line similar to:
SqlConnection DataConnection = new SqlConnection("your connection string here");
You need an open SqlConnection in order to do what you're attempting... You appear to be assigning the connection:
cmd.Connection = DataConnection;
however I don't see where DataConnection is coming from in your code snippet above.
DataConnection would have to be a SqlConnection() created somewhere. It would also need to be opened before you execute:
cmd.ExecuteNonQuery();
Typically you will want to open the connection, execute, then close. Often this is done inside a using statement so that you don't leave resources hanging around.
Microsoft's example code looks like this:
cmd.Connection = DataConnection;
however I don't see where DataConnection is coming from in your code snippet above.
DataConnection would have to be a SqlConnection() created somewhere. It would also need to be opened before you execute:
cmd.ExecuteNonQuery();
Typically you will want to open the connection, execute, then close. Often this is done inside a using statement so that you don't leave resources hanging around.
Microsoft's example code looks like this:
private static void CreateCommand(string queryString,
string connectionString)
{
using (SqlConnection connection = new SqlConnection(
connectionString))
{
SqlCommand command = new SqlCommand();
command.Connection = connection;
command.CommandTimeout = 15;
command.CommandType = CommandType.Text;
command.CommandText = queryString;
connection.Open();
SqlDataReader reader = command.ExecuteReader();
while (reader.Read())
{
Console.WriteLine(String.Format("{0}, {1}",
reader[0], reader[1]));
}
}
}
If, by chance, DataConnection is a property set on your object class containing InsertReasonForChange, you may need to put some code in to verify that DataConnection was properly set up before you try and use it in your function. It may be that your code is failing due to your function assuming that DataConnection initialization was handled before InsertReasonForChange is called and that not being the case. Placing some error checking code in to validate that DataConnection is properly initialized may help you correct this problem and prevent others.
This could be in the form of a private internal flag set when DataConnection is initialized or some equivalent sort of mechanism.
This could be in the form of a private internal flag set when DataConnection is initialized or some equivalent sort of mechanism.
I have attached the entire section of this code. We have a connect reference call GPConnet that actually makes the connection. I have used this for years just as in this code but now that I have upgraded it no longer appears to work as originaly designed. There never was a line that actually opens the connection. I know it is open because I get an error message if I try to open it while it is open. I get the message "Object has not been set to an instance of an object" . If you could provide some guidance as to how to validate the connection. All I have ever done before is create the connection, do whatever I need to do and then close the connection upon termination.
All this code is doing is popping up a box teling the user the change something and asks for a reason for the change. I need this reason written back to a table in the database. InsertReasonForChange is called when the Save button is clicked.
CodeSnippet.txt
All this code is doing is popping up a box teling the user the change something and asks for a reason for the change. I need this reason written back to a table in the database. InsertReasonForChange is called when the Save button is clicked.
CodeSnippet.txt
P.S. I put in some breakpoints and I can see the GP _User_ID has the correct value however, the value of DataConnection is null. So I do not see the variables returne dfrom GPConnect have values but the connection is null.
Active today
Change line 28 of the file you posted from:
to...
Then uncomment line 104.
SqlConnection DataConnection = GetConnection();
to...
this.DataConnection = GetConnection();
Then uncomment line 104.
Looking at your code I see several items that don't make much sense to me...
You create a private variable DataConnection at the top of your class declaration:
private System.Data.SqlClient.SqlC
In most of your functions, you declare another DataConnection local to that function:
Your constructor has one:
try
{
SqlConnection DataConnection = GetConnection();
}
GetConnection has one:
SqlConnection DataConnection = new SqlConnection();
Both of those declarations will hide the one at the class level.
Your InsertReasonForChange() function references the private DataConnection at the class level which has never been set. If this code worked for you before, I am not sure how...
A one line fix would be to change your constructor from:
to:
That would set the class level DataConnection so it would not be null. Normally, I'd say you also need a DataConnection.Open(), but I've not worked with Dynamics before, so the
connection you create in GetConnection using the GPConnObj.Connect() may take care of that for you.
You create a private variable DataConnection at the top of your class declaration:
private System.Data.SqlClient.SqlC
In most of your functions, you declare another DataConnection local to that function:
Your constructor has one:
try
{
SqlConnection DataConnection = GetConnection();
}
GetConnection has one:
SqlConnection DataConnection = new SqlConnection();
Both of those declarations will hide the one at the class level.
Your InsertReasonForChange() function references the private DataConnection at the class level which has never been set. If this code worked for you before, I am not sure how...
A one line fix would be to change your constructor from:
public frmSOPCheckSellPrice(string ordnmbr, Int16 ordtype, int linenmbr, int compnmbr)
{
InitializeComponent();
try
{
SqlConnection DataConnection = GetConnection();
}
catch (Exception ex)
to:
public frmSOPCheckSellPrice(string ordnmbr, Int16 ordtype, int linenmbr, int compnmbr)
{
InitializeComponent();
try
{
// SqlConnection DataConnection = GetConnection();
DataConnection = GetConnection();
}
catch (Exception ex)
That would set the class level DataConnection so it would not be null. Normally, I'd say you also need a DataConnection.Open(), but I've not worked with Dynamics before, so the
connection you create in GetConnection using the GPConnObj.Connect() may take care of that for you.
Another thing to think about is variable naming. Current convention has private class variables named with a leading underscore to help you differentiate what your setting.
Private class variables
private object _myObject;
Private/Public/Protected Parameter:
public object MyObject { get;set; }
Variables local to function:
object myObject;
The capitalization and the leading underscore can make it a lot easier to figure out what it is you are setting and makes code easier to read.
Private class variables
private object _myObject;
Private/Public/Protected Parameter:
public object MyObject { get;set; }
Variables local to function:
object myObject;
The capitalization and the leading underscore can make it a lot easier to figure out what it is you are setting and makes code easier to read.
Featured Post
![[Live Webinar] The Cloud Skills Gap](https://filedb.experts-exchange.com/files/public/2017/5/29/3249c5dc-214e-4f8f-afab-ef354630e19f.png)
As Cloud technologies come of age, business leaders grapple with the impact it has on their team's skills and the gap associated with the use of a cloud platform.
Join experts from 451 Research and Concerto Cloud Services on July 27th where we will examine fact and fiction.
Question has a verified solution.
If you are experiencing a similar issue, please ask a related question
Introduction
Hi all and welcome to my first article on Experts Exchange.
A while ago, someone asked me if i could do some tutorials on object oriented programming.
I decided to do them on C#. Now you may ask me, why's that?
Well, one of the re…
Performance in games development is paramount: every microsecond counts to be able to do everything in less than 33ms (aiming at 16ms). C# foreach statement is one of the worst performance killers, and here I explain why.
In this video, viewers are given an introduction to using the Windows 10 Snipping Tool, how to quickly locate it when it's needed and also how make it always available with a single click of a mouse button, by pinning it to the Desktop Task Bar.
Int…
NetCrunch network monitor is a highly extensive platform for network monitoring and alert generation. In this video you'll see a live demo of NetCrunch with most notable features explained in a walk-through manner. You'll also get to know the philos…
Suggested Courses
Earn Certification
HTML5 Specialist - Certification
Free withPremium
Course of the Month10 days, 7 hours left to enroll
631 members asked questions and received personalized solutions in the past 7 days.
Join the community of 500,000 technology professionals and ask your questions.