Почему при записи в массив выходит ошибка NullReferenceException?

Question

[Light 777]

Почему при записи в массив выходит ошибка NullReferenceException?

using System;
					
public class Program
{
	public static void Main()
	{
		XXXclass xxx = new XXXclass();
		string[] cars = {"Volvo", "BMW", "Ford", "Mazda"};
		
		int i = 0;
		foreach(String car in cars){
		
		xxx.zzz[i]=car;
		i++;	
		}
	}
}


    public class XXXclass
    {
        public string[] zzz { get; set; }
	}

Run-time exception (line 14): Object reference not set to an instance of an object.

Stack Trace:

[System.NullReferenceException: Object reference not set to an instance of an object.]
at Program.Main() :line 14

Answer 1

[Владимир Коротенко]

public class XXXclass
{
public XXXclass(){
string[] zzz = new string[4];
}
public string[] zzz { get; set; }
}

Comments

[Light 777]

не помогло, такая же ошибка :(

[Владимир Коротенко]

Light 777, Что то вы не то делаете.

public class Program
    {
        public static void Main()
        {
            
            string[] cars = { "Volvo", "BMW", "Ford", "Mazda" };
            var xxx = new XXXclass(cars.Length);  // передаем размерность массива, что бы не вылезти за границы
            var i = 0;
            foreach (var car in cars) xxx.zzz[i++] = car;
        }
    }
    public class XXXclass
    {
        public XXXclass(int size)
        {
            zzz = new string[size];
        }
        public string[] zzz { get; set; }
    }

[Light 777]

Владимир Коротенко, Теперь заработало , спасибо!

[edward_freedom]

Владимир Коротенко, просто у тебя в конструкторе обьявляется заново переменная

string[] zzz = new string[4];

[Владимир Коротенко]

edward_freedom, так и было задумано. Я вообще часто в конструкторе инициализирую переменные. Можно было через ioc сделать но тут это излишнее. Хотя логика кода от меня ускальзывает зачем этот фореач и переменная, когда можно просто присвоить массив?

[edward_freedom]

Владимир Коротенко, я про этот кусман кода
public XXXclass(){
string[] zzz = new string[4]; - создали новою переменную
}
public string[] zzz { get; set; } - эта переменная как была неицилизирована, так и осталась
}

[edward_freedom]

Light 777, а действительно, зачем тебе два массива и циклы, когда у тебя есть массив уже заполненный, просто передай его аргументом в класс и не надо циклов вообще

[Владимир Коротенко]

edward_freedom, ты старый кусок кода смотришь, глянь чуть ниже

[Light 777]

edward_freedom, в полном коде там есть ifы. Просто сюда я их обрезал

[edward_freedom]

Light 777, а цикл зачем?

[Light 777]

edward_freedom, для вопроса сюда я задачу сильно упростил. У меня в полном коде дёргается сервис, оттуда получаю хмлку , далее циклом записываю в массив.

[edward_freedom]

Light 777, убери вообще массив стрингов, сделай список и не парься

public List<string> zzz { get; set; } = new List<string>();

[Light 777]

edward_freedom, у меня большой легаси проект. Посмотрим, может получится что нибудь поменять, не задевая текущий функционал. Спасибо))

[Light 777]

Владимир Коротенко, edward_freedom, Помогите пожалуйста
Что я делаю не так??
Опять столкнулся с данной проблемой

[Serializable]
    public partial class ContractClassShort
    {
        public ContractClassShort(){

        }

        public ValueClass AgreementNumber
        {
            get;
            set;
        }

    }

    [Serializable]
    public partial class ContractListClassShort : ValueClass
    {
        public ContractListClassShort()
        {

        }

        public ContractListClassShort(int x)
        {
            Contract = new ContractClassShort[x];
        }

        [System.Xml.Serialization.XmlElementAttribute("Contract")]
        public ContractClassShort[] Contract
        {
            get;
            set;
        }
    }

    public class FCBRequestResultShort
    {

        public ContractListClassShort ExistingContracts { get; set; }

    }



    [WebMethod]
    public FCBRequestResultShort GetReportShort(int processType, int processID, string iinbin, int isJur)
    {

        FCBRequestResult result = new FCBRequestResult();        
        result = GetReport(processType, processID, iinbin, isJur);        
        FCBRequestResultShort res = new FCBRequestResultShort();
        res.ExistingContracts = new ContractListClassShort(result.otherReport.Result.Root.ExistingContracts.Contract.Length);
        DateTime dt1 = DateTime.Parse("01.01.2021");       
        int i = 0;

        foreach (ContractClass contract in result.otherReport.Result.Root.ExistingContracts.Contract)
        {
            
            if (Convert.ToDateTime(contract.DateOfCreditEnd.Value) >= dt1)
            {
                if (contract.AgreementNumber.Value == "111")
                {
                    res.ExistingContracts.Contract[i] = new ContractClassShort();
                    res.ExistingContracts.Contract[i].AgreementNumber.Value = contract.AgreementNumber.Value;////<<<<<ОШИБКА
                    i++;
                }
            }
        }
        return res;
    }

[Владимир Коротенко]

Light 777, Добавьте тест для этого метода и включите отладку. В месте где ваша ошибка появится подумайте что делаете не так и проинициализируйте переменную

Answer 2

[Сергей Чамкин]

Поле zzz в классе XXXclass не создается, то есть не выделяется память как раз под Ваши 4 машины.
Поэтому нельзя обратиться к i элементу xxx.zzz. Создайте конструктор, как в ответе выше