Откуда взялся null

[WorldMaster]

Здравствуйте.
Есть вот такой вот код:

List<Command> CommandsQueue = new List<Command>();

...

List<Command> exists = CommandsQueue.FindAll(item => item.CmdType == CommandTypes.GetState);

Переодически поиск вызывает Exception с указанием что item не определен. В отладчике он и правда нулл. Хотя список содержит нормальные здоровые объекты.
С чем это может быть связано?

[Stilet]

Может чушь сморожу, но среди свойств Command нет свойства с именем item?
И кстати как периодически? Не постоянно? уловить ситуацию с эксцепшном получается?

[WorldMaster]

internal class Command
    {
        /// <summary>
        /// Получает понятное имя команды
        /// </summary>
        public CommandTypes CmdType
        {
            get
            {
                if (CmdData == null) return CommandTypes.None;
                return (CommandTypes)CmdData[2];
            }
        }

        /// <summary>
        /// Команда
        /// </summary>
        public byte[] CmdData
        {
            get;
            protected set;
        }

        /// <summary>
        /// Данные
        /// </summary>
        public byte[] Data
        {
            get;
            protected set;
        }

        /// <summary>
        /// Получает готовый пакет для отправки
        /// </summary>
        public byte[] PackedData
        {
            get
            {
                List<byte> _comp = new List<byte>();
                _comp.AddRange(CmdData);
                _comp.AddRange(Data);
                return _comp.ToArray();
            }
        }

        public Command(CommandTypes cmdType, byte[] SendData)
        {
            CmdData = new byte[] { 0x00, 0x00, (byte)cmdType };
            Data = SendData == null ? new byte[] { } : SendData;
        }
    }

Вот класс.
Вылетает эксепшн достаточно редко. Технически можно и отладчиком поймать. Но когда ловил то список содержал в себе несколько объектов нормальных. Если после отлова перевести указатель на туже строку то уже выполняется нормально ...

[Alex11223]

Так в отладчике точно нет null в списке?

А если Where вместо FindAll? Тем более, что он должен быть лучше по производительности если не требуется именно List получить (если не вызывать ToList).

Со списком только 1 поток работает? Если нет, то может что-то не так с синхронизацией?

[Stilet]

Скорее всего действительно что-то вмешивается в список и чистит его элементы.

[WorldMaster]

Сообщение от Alex11223

Так в отладчике точно нет null в списке?

Точно нету.

Сообщение от Alex11223

А если Where вместо FindAll?

Вообще мне нужно просто узнать содержит ли список команду с указанным типом или нет. Производительность особо не имеет значения. Но если есть более изящное решение чем у меня то буду рад увидеть его.

Сообщение от Alex11223

Со списком только 1 поток работает? Если нет, то может что-то не так с синхронизацией?

Постановка команды в список может быть вызвана и в других потоках.

Сообщение от Stilet

Скорее всего действительно что-то вмешивается в список и чистит его элементы.

А удаление происходит в той же самой функции.


Попробую залочить объект. Только вот как лучше? перед добавлением или перед удалением блокировать?

[Alex11223]

Вообще мне нужно просто узнать содержит ли список команду с указанным типом или нет. Производительность особо не имеет значения. Но если есть более изящное решение чем у меня то буду рад увидеть его.

LINQ

bool exists = CommandsQueue.Any(item => item.CmdType == CommandTypes.GetState);

Еще есть FirstOrDefault если нужно и получить его (один).

Попробую залочить объект. Только вот как лучше? перед добавлением или перед удалением блокировать?

Разве не в обоих случаях надо? Ну точнее при любом доступе, иначе толку от лока на добавление/удаление если в том коде, о котором тут речь, не будет лока?

Еще есть стандартные потокобезопасные коллекци
https://msdn.microsoft.com/en-us/lib...vs.110%29.aspx
ConcurrentQueue может быть подойдет, раз уж в имени переменной есть слово Queue.

[WorldMaster]

Сообщение от Alex11223

ConcurrentQueue может быть подойдет, раз уж в имени переменной есть слово Queue.

Не работал с такими еще .. щас посмотрим ...
Спасибо.