Add strings to the list of string

Question

I am trying to fill a list of strings but it throws null reference exception.

Code:

public class Validation
{
    public List<string> Errors { get; set; }
}

Class where all validation errors are to be stored:

public object Post(Currency currency)
{
    ClientData clientData = new ClientData();
    if (ModelState.IsValid)
    {
        new CurrencyProvider().Insert(currency);
        clientData.IsValid = true;
        clientData.Data = new CurrencyProvider().GetAll();
    }
    else
    {
        Validation validation = new Validation();
        foreach (var modelState in ModelState)
            foreach (var error in modelState.Value.Errors)
                validation.Errors.Add(error.ErrorMessage);

        clientData.IsValid = false;
        clientData.Data = validation;
    }   

    return clientData;
}

The problem occurs when I fill validation.Errors.Add(error.ErrorMessage). It throws me the null reference exception even though I have done exception handeling globally as follows in my global.asax.cs

public class MvcApplication : System.Web.HttpApplication
{
    protected void Application_Start()
    {
        GlobalConfiguration.Configure(WebApiConfig.Register);
        AreaRegistration.RegisterAllAreas();
        FilterConfig.RegisterGlobalFilters(GlobalFilters.Filters);
        RouteConfig.RegisterRoutes(RouteTable.Routes);
        BundleConfig.RegisterBundles(BundleTable.Bundles);
        GlobalFilters.Filters.Add(new ExceptionFilter());
    }
}

My Exception handler class:

public class ExceptionFilter : HandleErrorAttribute
{
    public override void OnException(ExceptionContext filterContext)
    {
        ViewResult view = new ViewResult();
        view.ViewName = "Error";
        view.ViewBag.Exception = filterContext.Exception.Message;
        filterContext.ExceptionHandled = true;
        filterContext.Result = view;
    }
}

I have my custom Error Handling page but its also not showing, when I debug then I came to know the there is the null reference exception at the time of filling the list of string in:

validation.Errors.Add(error.ErrorMessage)

What am I doing wrong when I fill the List<string> and why is that throwing null reference exception? And why that null reference exception is not coming on to my custom error page?

Try replacing your Foreach condtion of Errors with this one:- foreach(ModelError error in modelState.Errors){ ///Your Code } — yash
– yash, Commented Jan 2, 2016 at 11:29
Have you remembered to instantiate a new List<string> before doing validation.Errors.Add()? I think the null ref is validation.Errors — geedubb
– geedubb, Commented Jan 2, 2016 at 11:34
geedubb i think u got it!, plz tell me how do i instantiate the list??? — Uzair Ashraf
– Uzair Ashraf, Commented Jan 2, 2016 at 11:38

NightOwl888 · Accepted Answer · 2016-01-02 11:37:54Z

2

The issue is that you have not created an instance of List<string> in your Validation class. You can do that by initializing the instance in the class constructor.

public class Validation
{
    public Validation()
    {
         this.Errors = new List<string>();
    }

    public List<string> Errors { get; set; }
}

answered Jan 2, 2016 at 11:37

NightOwl888

57.1k28 gold badges148 silver badges222 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

Steve Harris Over a year ago

this keyword is not required

geedubb · Accepted Answer · 2016-01-02 11:36:48Z

You need to instantiate a new List<string> before adding to list with validation.Errors.Add().

You could try:

public object Post(Currency currency)
{
    ClientData clientData = new ClientData();
    validation.Errors = new List<string>(); // instantiate
    if (ModelState.IsValid)
    {
        new CurrencyProvider().Insert(currency);
        clientData.IsValid = true;
        clientData.Data = new CurrencyProvider().GetAll();
    }
    else
    {
        Validation validation = new Validation();
        foreach (var modelState in ModelState)
        {
            foreach (var error in modelState.Value.Errors)
            {
                validation.Errors.Add(error.ErrorMessage);
            }
        }
        clientData.IsValid = false;
        clientData.Data = validation;
    }
    return clientData;
}

Bimal Das · Accepted Answer · 2016-01-02 11:38:22Z

0

Try

Validation validation = new Validation();
validation.Errors validationError = new List<string>();

Then write into foreach loop :

validationError.Add(error.ErrorMessage);

answered Jan 2, 2016 at 11:38

Bimal Das

2,0125 gold badges33 silver badges55 bronze badges

Comments

Hardik · Accepted Answer · 2016-01-02 19:52:52Z

0

What you basically doing is called 'Composition' or 'has a' Relation between classes in object oriented programming ,Composition relation between classes, means that class 'has a' another class internally. for ex : Consider a Customer class below

  public class Customer 
    {
       public Address HomeAddress { get; set; } //'Composition' or 'has a' Relation
        public string FirstName { get; set; }
        public string LastName { get; set; }
        public string EMail { get; set; }
        ...........

    }

Note : (Assume that Address class is another class which contain properties like AddLine1,AddLine2,City,State etc.)

Now Once you initialize the object of Customer class all properties of that class will initialize with it's default values, below will be the default values of object of the above class

    HomeAddress = null
    FirstName = null
    LastName = null
    EMail = null

so this can lead to run time exception as in your case where you have just declared 'Errors' property but didn't initialized it , And in order to use it you should initialize it,using constructor like below :

public Customer 
{
HomeAddress  = new Address();
FirstName  = "";
LastName  = "";
EMail = "";
}

so similarly you can initialize 'Errors' property in class constructor

edited Jan 2, 2016 at 19:52

answered Jan 2, 2016 at 19:44

Hardik

2281 silver badge6 bronze badges

1 Comment

Uzair Ashraf Over a year ago

Thanks a lot, your explanation is awesome!

Collectives™ on Stack Overflow

Add strings to the list of string

4 Answers 4

1 Comment

Comments

Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

1 Comment

Comments

Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related