object reference not set to an instance of an object(i got an error through the below code)

Question

0.00/5 (No votes)

See more:

C#

DataTable dt = ds.Tables[0];


                HolidayReg objHoliday = new HolidayReg();

                for (int row = 0; row < dt.Rows.Count; row++)
                {
                    objHoliday = new HolidayReg();
                    foreach (DataColumn dc in dt.Columns)
                    {
                        foreach (PropertyInfo prop in objHoliday.GetType().GetProperties())
                        {

                            if (prop.Name == dc.ColumnName && prop.CanWrite)
                            {
                                if (dt.Rows[row][dc.ColumnName] != DBNull.Value)
                                {

                                    prop.SetValue(objHoliday, dt.Rows[row][dc.ColumnName], null);
                                }
                            }
                        }
                    }


                    ListHoliday.Add(objHoliday);
                }


            }
            catch (Exception ex)
            {
                throw ex;
            }

            return ListHoliday;
        }

i got an error at catch(Expression ex)

Posted 10-Jan-13 0:42am

renjimaramanan mr

Updated 10-Jan-13 1:11am

Manas Bhardwaj

v3

Add a Solution

Comments

OriginalGriff 10-Jan-13 6:45am

Where did you get the error? Which line?

[no name] 10-Jan-13 7:01am

it means there is some null value getting to your variable.

4 solutions

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

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

syamrulezzz · Answer 1 · 2013-01-10T01:22:00

Solution 4

Hi,

HolidayReg objHoliday = null;

then it will work fine!!!!!!!!!

Posted 10-Jan-13 1:22am

syamrulezzz

Comments

Manas Bhardwaj 10-Jan-13 7:43am

Please don't poullute the forum with irrelevent answers.

Manas Bhardwaj · Answer 2 · 2013-01-10T01:16:00

1.) You didn't get error at catch. It happened much before that and was handled by catch block.
2.) It could be any where in the code. It's difficult for us to pin point where it could be. It could as early as in first statement:

ds.Tables[0];

where the ds could be null. To prevent that, you should also check if the object is not null like this:

C#

DataTable dt;
if(ds != null)
{
    if(ds.Tables.count > 0)
    {
        dt = ds.Tables[0];
    }
}

3.) Use Visual Studio to debug the code, and you would know where the error occured. Look at the great article from Abhijit Jana here:
Mastering Debugging in Visual Studio 2010 - A Beginner's Guide[^]

Maruthiram_99 · Answer 3 · 2013-01-10T01:07:00

Solution 2

Some null value is being passeed in your data table..check it ones by using quick watch

Posted 10-Jan-13 1:07am

Maruthiram_99

Zafar Sultan · Answer 4 · 2013-01-10T00:48:00

Solution 1

Actually it is not possible to find out the error with this little code you have posted but my assumption is you should change this line

C#

prop.SetValue(objHoliday, dt.Rows[row][dc.ColumnName], null);

to

C#

prop.SetValue(objHoliday, dt.Rows[row][dc.ColumnName].ToString(), null);

Posted 10-Jan-13 0:48am

Zafar Sultan

Comments

Manas Bhardwaj 10-Jan-13 7:18am

Your first statement is correct, but I don't think the assuption is correct. Your propsoed solution will still give an error if there is no row at specified index.

P.S. : I did not vote.

Zafar Sultan 10-Jan-13 7:31am

If there is no row at specified position is also an assumption and a wrong one because the OP is using for loop, for (int row = 0; row < dt.Rows.Count; row++). In case of 0 rows the control will never enter into the block. Anyways, I will vote for your solution

object reference not set to an instance of an object(i got an error through the below code)

4 solutions

Solution 4

Solution 3

Solution 2

Solution 1

Add your solution here

Preview 0

Existing Members

...or Join us