EADever Ответов: 3

Как исправить ссылку на объект, не установленную на экземпляр объекта?


Im пытается прочитать данные из xml-файла в таком формате:

<npc_templates>	
	<npc_template npc_id="280001">
		<equipment>
			<item>100000005
			<item>100000006


Что я уже пробовал:

И я пытаюсь с помощью этого кода читать элементы оборудования/предметов:

public class NpcTemplate
    {
        [XmlAttribute("npc_id")]
        public int npc_id;
        
        [XmlElement("equipment")]
        public NpcEquippedGear equipment;
    }

и класс NpcEquippedGear:
public class NpcEquippedGear : IEnumerable<KeyValuePair<EquipmentSlot, ItemTemplate>>
    {
        [XmlElement("item")]
        public ItemTemplate[] items2;

        public Dictionary<EquipmentSlot, ItemTemplate> items;
        public int mask;
        public NpcEquippedGear()
        {
        }

        public void Add(object o)
        {

        }

        public int getItemsMask()
        {
            if (items == null)
                init();
            return mask;
        }

        public void init()
        {
            lock(this)
            {
                if (items == null)
                {
                    items = new Dictionary<EquipmentSlot, ItemTemplate>();
                    foreach (ItemTemplate item in items2)// Error Object reference not set to an instance of an object.
                    {
                        if (items[item.EquipmentSlot] == null)
                        {
                            items.Add(item.EquipmentSlot, item);
                            mask |= (int)item.EquipmentSlot;
                        }
                    }
                }
            }
        }

        public ItemTemplate GetItem(EquipmentSlot slot)
        {
            return items != null ? items[slot] : null;
        }

        public IEnumerator GetEnumerator()
        {
            return GetEnumerator();
        }

        IEnumerator<KeyValuePair<EquipmentSlot, ItemTemplate>> IEnumerable<KeyValuePair<EquipmentSlot, ItemTemplate>>.GetEnumerator()
        {
            return items.GetEnumerator();
        }
    }


Это все, что я пытаюсь, но во время запуска программы его сказать mw ошибка
Object reference not set to an instance of an object.

в NpcEquippedGear цикл по каждому элементу класса

Richard MacCutchan

Потому что вы никогда не инициализировались items2 ссылаться на что угодно.

3 Ответов

Рейтинг:
7

Mohtshm Zubair

В настоящее время у вас

foreach (ItemTemplate item in items2)// Error Object reference not set to an instance of an object.



просто поставьте эту строку

if (items2 !=null)

подобный этому


if(items2 !=null)
foreach (ItemTemplate item in items2)// Error Object reference not set to an instance of an object.



Это решит ваше нулевое исключение, но всегда хорошо инициализировать массивы или список внутри contructor или использовать соглашение о методе Initialize

Плз отметьте, что проблема решена!!!!


Richard MacCutchan

Нет, это просто замаскирует его.

EADever

Но проблема в том, что я получаю только данные оборудования npc_id, которые я даю на вопрос. его должен вернуть не null, но когда я проверить код, если (items2 != значение null) {} иначе {items2 возвращать null}

Рейтинг:
22

OriginalGriff

Это одна из самых распространенных проблем, которые нам задают, и это также та, на которую мы меньше всего готовы ответить, но вы больше всего готовы ответить сами.

Позвольте мне просто объяснить, что означает ошибка: вы пытались использовать переменную, свойство или возвращаемое значение метода, но оно содержит null - что означает, что в переменной нет экземпляра класса.
Это немного похоже на карман: у вас есть карман в рубашке, в котором вы держите ручку. Если вы сунете руку в карман и обнаружите, что там нет ручки, вы не сможете подписать свое имя на листе бумаги - и вы получите очень забавный вид, если попытаетесь! Пустой карман дает вам нулевое значение (здесь нет ручки!), поэтому вы не можете сделать ничего, что обычно делали бы, когда извлекли свою ручку. Почему он пуст? Вот в чем вопрос - может быть, вы забыли взять ручку, когда уходили из дома сегодня утром, или, возможно, вы оставили ручку в кармане вчерашней рубашки, когда снимали ее вчера вечером.

Мы не можем сказать, потому что нас там не было, и, что еще важнее, мы даже не можем видеть вашу рубашку, не говоря уже о том, что находится в кармане!

Вернемся к компьютерам, и вы каким - то образом сделали то же самое-и мы не можем увидеть ваш код, а тем более запустить его и узнать, что содержит null, когда это не должно быть.
Но вы можете - и Visual Studio поможет вам здесь. Запустите свою программу в отладчике, и когда она выйдет из строя, VS покажет вам строку, на которой она обнаружила проблему. Затем вы можете начать смотреть на различные его части, чтобы увидеть, какое значение равно null, и начать просматривать свой код, чтобы выяснить, почему. Поэтому поставьте точку останова в начале метода, содержащего строку ошибки, и снова запустите программу с самого начала. На этот раз VS остановится перед ошибкой и позволит вам изучить, что происходит, пройдя через код, глядя на ваши значения.

Но мы не можем этого сделать - у нас нет вашего кода, мы не знаем, как его использовать, если бы он у нас был, у нас нет ваших данных. Так что попробуйте - и посмотрите, сколько информации вы сможете узнать!


Рейтинг:
16

EADever

Спасибо за вашу помощь, ребята.
Я решил свою проблему, перекодировав класс NpcEquippedGear

public class NpcEquippedGear
    {
        [XmlElement("item")]
        public int[] items;
        private Dictionary<EquipmentSlot, ItemTemplate> dicItems;
        private int mask;
        public int getItemsMask()
        {
            if (dicItems == null)
                init();
            return mask;
        }
        private void init()
        {
            lock (this)
            {
                if (dicItems == null)
                {
                    dicItems = new Dictionary<EquipmentSlot, ItemTemplate>();
                    foreach (int itemId in items)
                    {
                        ItemTemplate item = Datastore.ItemsById[itemId];
                        if (!dicItems.ContainsKey(item.EquipmentSlot))
                        {
                            dicItems.Add(item.EquipmentSlot, item);
                            mask |= (int)item.EquipmentSlot;
                        }
                    }
                }
            }
        }
        public ItemTemplate GetItem(EquipmentSlot slot)
        {
            return dicItems != null ? dicItems[slot] : null;
        }
    }