Click here to Skip to main content
15,902,492 members
Please Sign up or sign in to vote.
0.00/5 (No votes)
In my WinForms app, i have this below code which fetches the lot number from MSSQL table when the button is clicked ,
the function was working fine till yesterday , but from today it is throwing below exception and stopping the application even though try block is present.

System.NullReferenceException: 'Object reference not set to an instance of an object.'


i don't know why is this happening , but i was adding another form called "formedit" to edit the existing record , which is very similar to "FornewEntry" , but with prefilled values of selected datagridview row , maybe i have messed up something while doing the same as this is first application as i am learning C# below is the link to my solution kindly help me .

What I have tried:

private void getlotnumbers()
        {
            try
            {
                SqlConnection con = new SqlConnection(cs);
                SqlCommand cmd;
                con.Open();
                string s = "select LotNumber from Lot_Numbers where CoilNumber = @p1";
                cmd = new SqlCommand(s, con);
                cmd.Parameters.AddWithValue("@p1", coilNoTextBox.Text);
                cmd.CommandType = CommandType.Text;
                int i = cmd.ExecuteNonQuery();
                lotNoTextBox.Text = cmd.ExecuteScalar().ToString();
                con.Close();
            }
            catch(Exception ex)
            {
                MessageBox.Show("Lot Number Not Found:" + ex.Message.ToString(), "Lot Number",
                    MessageBoxButtons.OK, MessageBoxIcon.Error);
            }
            
        }
Posted
Updated 20-Dec-20 20:59pm
v2

1 solution

This is one of the most common problems we get asked, and it's also the one we are least equipped to answer, but you are most equipped to answer yourself.

Let me just explain what the error means: You have tried to use a variable, property, or a method return value but it contains null - which means that there is no instance of a class in the variable.
It's a bit like a pocket: you have a pocket in your shirt, which you use to hold a pen. If you reach into the pocket and find there isn't a pen there, you can't sign your name on a piece of paper - and you will get very funny looks if you try! The empty pocket is giving you a null value (no pen here!) so you can't do anything that you would normally do once you retrieved your pen. Why is it empty? That's the question - it may be that you forgot to pick up your pen when you left the house this morning, or possibly you left the pen in the pocket of yesterday's shirt when you took it off last night.

We can't tell, because we weren't there, and even more importantly, we can't even see your shirt, much less what is in the pocket!

Back to computers, and you have done the same thing, somehow - and we can't see your code, much less run it and find out what contains null when it shouldn't.
But you can - and Visual Studio will help you here. Run your program in the debugger and when it fails, VS will show you the line it found the problem on. You can then start looking at the various parts of it to see what value is null and start looking back through your code to find out why. So put a breakpoint at the beginning of the method containing the error line, and run your program from the start again. This time, VS will stop before the error, and let you examine what is going on by stepping through the code looking at your values.

At a guess, there are no rows in your DB that match the text the user typed - so SQL returns no value. Either test for that instead of automatically calling ToString, or call
ExecuteReader and check the Read method return value.

But we can't do that - we don't have your code, we don't know how to use it if we did have it, we don't have your data. So try it - and see how much information you can find out!
 
Share this answer
 
Comments
Member 14898617 21-Dec-20 2:37am    
hi. Thank you for your suggestion i found out that the error is coming at the line: ```lotNoTextBox.Text = cmd.ExecuteScalar().ToString();``` as the "coilnotext" box and lot number was not matched , but again i have used try block right? ..it must show the exception in messagebox and let me use my application , but why it stepping out of the application itself?
OriginalGriff 21-Dec-20 2:57am    
Because that's what the debugger is there for: to help you fix problems so that try catch isn't used as part of "regular processing"

You shouldn't just litter your code with try...catch block to cope with user mistakes: instead you check for valid results and behave sensibly.
So retrieve the value from the SQL, check if it exists, and replace it with "No matching records found" instead of the value when they mistype.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)



CodeProject, 20 Bay Street, 11th Floor Toronto, Ontario, Canada M5J 2N8 +1 (416) 849-8900