Как найти где NullReferenceException?

Question

[Сергей]

Добрый день.
Подскажите, как поймать неуловимый null?

Есть программа с таким методом:

private async Task checkFile(string path)
        {
            var fileType = await getFileAsync(path);
            
                if (fileType.FileExtension != "None")
                {
                    try
                    {
                        if (this.filesMap == null) Console.WriteLine("Map is null");
                        if (path == null) Console.WriteLine("path is null");
                        if (fileType == null) Console.WriteLine("type is null");

                        this.filesMap.Add(path, fileType);
                    }
                    catch(Exception e)
                    {
                        Console.WriteLine(e.Message);
                    }
                    
                }
            
        }

На выборке из 20000 файлов один раз из пяти запусков программы выпадает NullReferenceException при добавлении элемента в дикшинари filesMap.Add, в самом начале работы программы, когда размер мапы два - три элемента. Спотыкается каждый раз на разных файлах.

Все три переменные, которые содержатся в этом коде (path, fileType, filesMap) в момент ошибки содержат какое-то значение, я даже вот эти врайтлайны поставил, чтобы убедиться что не сошёл с ума. Они ничего в консоль не выводят.

Ещё, трай кетч не обрабатывает ошибку автоматически, и студия останавливает работу программы. Если продолжить, то exception из кетча выведется в консоль, и следом будет вторая такая же ошибка. Дальше до конца программа работает нормально. Ошибки случаются только по две, всегда по две. Или не случается ни одной.
Ощущение, будто в dictionary.Add() что-то происходит.

Весь код лежит тут: https://github.com/osija/ImgExtractor/blob/master/...

Картинку с ошибкой прилагаю:

Спасибо!

Answer 1

[twobomb]

Хех, так Dictinary же не потокобезопасный. Вам нужна или потокобезопасная коллекция или лок ставьте.
В FileMapper добавьте где-нибудь
private static object locker = new object();
И попробуй переписать на такое

private async Task checkFile(string path)
        {
            var fileType = await getFileAsync(path);
            
                if (fileType.FileExtension != "None")
                {
                      lock(locker){
                        this.filesMap.Add(path, fileType);
}                    
                }
            
        }

P.S. Thread-Safe Collections

Comments

[Сергей]

Вот же, попробуй угадай. С другой стороны, мог бы и догадаться. А Майкрософту стоило бы другой эксепшн для этого дела выдавать. Ну а я сегодня открыл новое пространство имен dotnet.

ConcurrentDictionary наверное в этом случае очень подойдет.

Спасибо!

[twobomb]

Сергей, На самом деле это относится ко всем спискам в вашем проекте, к которым обращение идет из разных потоков, особенно на запись.
А вообще работа с несколькими потоками, опасная штука, порой возникают такие ошибки, которые сложно найти, и дедлоки появляются только когда все звёзды сложатся в ряд. Поэтому нужно очень аккуратно всё делать, тут вся ответсвенность синхронизации ложится на ваши плечи, это уже другой уровень программирования.

[Сергей]

twobomb, Для того и нужны петпроджекты, чтобы ходить туда где живут драконы, и я бы поклялся, что в этом проекте не может быть дедлоков, ели бы был более уверен в себе, но я тут не вижу где они могут быть. И пишу свои таски в разные файлы, и читаю из каждого файла в одном потоке. Я с такими проблемами уже столкнулся когда пытался читать байты конкурентно, и получалось, точнее, не получалось читать из одного файла одновременно. В общем, на вот этом простом проекте я просёк как работает конкурентность в C#, хоть и читал об этом раньше очень много. Может не полностью просек, но концептуально.

В предыдущем варианте этой проги я писал всё по MSDN и там везде были await, в том числе в foreach-ах, и тот вариант не отличался по скорости от предыдущего, где не было асинхронности вообще.
А предыдущий работал чертовски долго, текущий асинхронный код работает раз в 10 быстрее.

А начиналось все с того, что мне нужно было просто вытянуть картинки из бэкапа айфона. И затянуло.

Спасибо!