object reference not set to an instance of an object

Question

1.00/5 (1 vote)

See more:

C#

foreach (DataGridViewRow row in LanguageGridView.Rows)
              {                  
                  for (int i = 0; i < LanguageGridView.Rows.Count; i++)
                  {
                                        
                          string selectedLanguage;
                          selectedLanguage = LanguageGridView.Columns[2].HeaderText;
                          string id = row.DataGridView.Rows[i].Cells["GID"].Value.ToString().Trim();
                          string name = row.DataGridView.Rows[i].Cells[2].Value.ToString();
                        //  LanguageGridView.AutoGenerateColumns = false;    
                          if (id!=null && name.Trim() == null)
                          {
                              continue;
                          }
                     

                          _localizationBusinessService.UpdateColumn(selectedLanguage, name, id);
                     
                  }

Posted 3-Oct-13 19:26pm

Member 10314566

Updated 3-Oct-13 19:45pm

Ron Beyer

v2

Add a Solution

Comments

Member 10314566 4-Oct-13 1:27am

please tell me how to solve

Ron Beyer 4-Oct-13 1:46am

How about you tell us what line its on, simply pasting a code dump does not help us find the error, you need to tell us what line and exactly what variable is giving the error. We can't see your screen or access your computer, and we can't reproduce your program with the little bit you gave, so help us help you and tell us where you get the error.

1 solution

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)

OriginalGriff · Answer 1 · 2013-10-03T19:54:00

What are you trying to do?
You have an outer loop which runs through each row, then an inner loop which also runs through each row in the same grid. Inside the inner loop, you use the row from the outer loop to get the grid it is part of and then use the value from the inner loop to access every row in the grid. You then do a test which will always fail (because AA string.Trim call will never return null) before calling an update method.

So if you have 100 rows, you will call the update method 10,000 times...

Rather than suggesting a solution to the specific problem you have, I would recommend that you take a good, hard look at this code (and anything else that looks like it) and rethink your design...