C# NullReferenceException was unhandled - Object reference not set to an instance of an object

Danny picture Danny · Feb 27, 2013 · Viewed 10.1k times · Source

I'm trying to create a application using Windows Forms in C#, however I keep getting an error stating:

NullReferenceException was unhandled - Object reference not set to an instance of an object

It's pointing to the line which has this code on:

carBootSaleList.AddCarBootSale(newCarBootSale);

The method I have on the forms interface:

  CarBootSaleList carBootSaleList;

  public void AddCarBootSale()
    {
        AddNewCarBootSale addForm = new AddNewCarBootSale();
        if (addForm.ShowDialog() == DialogResult.OK)
        {
            CarBootSale newCarBootSale = addForm.GetCarBootSaleData();

            carBootSaleList.AddCarBootSale(newCarBootSale);

            txtCarBootSale.Clear();
            txtCarBootSale.Text = newCarBootSale.Display();
        }
    }

The CarBootSaleList class:

public class CarBootSaleList : IDisplay
{

    private List<CarBootSale> carbootsales;

    public CarBootSaleList()
    {
        carbootsales = new List<CarBootSale>();
    }

    public bool AddCarBootSale(CarBootSale carbootsale)
    {
        bool success = true;
        foreach (CarBootSale cbs in carbootsales)
        {
            if (cbs.ID == carbootsale.ID)
            {
                success = false;
            }
        }
        if (success)
        {
            carbootsales.Add(carbootsale);
        }
        return success;
    }

    public int GetListSize()
    {
        return carbootsales.Count();
    }

    //public CarBootSale FindCarBootSale(string cbsID)
    //{
    //    CarBootSale carbootsale = null;
    //    foreach (CarBootSale cbs in carbootsale)
    //    {
    //        if (cbs.ID == cbsID)
    //        {
    //            carbootsale = cbs;
    //        }
    //    }
    //    return carbootsale;
    //}

    public List<CarBootSale> ReturnList()
    {
        return carbootsales;
    }

    public string Display()
    {
        string msg = "";

        foreach (CarBootSale cbs in carbootsales)
        {
            msg += String.Format("{0}  {1}", cbs.ID, cbs.Location, cbs.Date);
            msg += Environment.NewLine;
        }
        return msg;
    }
}

The CarBootSale class:

public class CarBootSale : IDisplay
{

    public string ID { get; set; }
    public string Date { get; set; }
    public string Location { get; set; }
    public double PitchCost { get; set; }
    public int Capacity { get; set; }
    public string Charity { get; set; }
    public string CharityName { get; set; }
    public string Catering { get; set; }

    public CarBootSale(string id, string date, string location, double pitchcost, int capacity, string charity, string charityname, string catering)
    {
        ID = id;
        Date = date;
        Location = location;
        PitchCost = pitchcost;
        Capacity = capacity;
        Charity = charity;
        CharityName = charityname;
        Catering = catering;
    }

    public CarBootSale() { }

    public override string ToString()
    {
        return String.Format("{0}  {1}", ID, Date, Location);
    }

    public string Display()
    {
        string msg;
        string CR = Environment.NewLine;

        msg = String.Format("ID: {0} {1}", ID, CR);
        msg += String.Format(" Date: {0} {1}", Date, CR);
        msg += String.Format(" Location: {0} {1}", Location, CR);
        msg += String.Format(" Pitch Cost: {0} {1}", PitchCost, CR);
        msg += String.Format(" Capacity: {0} {1}", Capacity, CR);
        msg += String.Format(" Charity: {0} {1}", Charity, CR);
        msg += String.Format(" CharityName: {0} {1}", CharityName, CR);
        msg += String.Format(" Catering: {0} {1}", Catering, CR);

        return msg;
    }
}

Answer

Habib picture Habib · Feb 27, 2013

You haven't initialized carBootSaleList in your code anywhere.

You can either instantiate it in the method or with the declaration.

CarBootSaleList carBootSaleList = new CarBootSaleList();