BinaryReader.Dispose(bool disposing) создает локальную ссылку на поток. Зачем?

Question

[Illivion]

Вот сам метод:

protected virtual void Dispose(bool disposing) { 
            if (disposing) { 
                Stream copyOfStream = m_stream;
                m_stream = null; 
                if (copyOfStream != null && !m_leaveOpen)
                    copyOfStream.Close();
            }
            m_stream = null; 
            m_buffer = null;
            m_decoder = null; 
            m_charBytes = null; 
            m_singleChar = null;
            m_charBuffer = null; 
        }

m_stream — базовый поток

m_leaveOpen — задается в конструкторе, если желаем не закрывать поток при Dispose ридера.


Какой смысл скрывается за созданием ссылки на поток (copyOfStream)? Почему нельзя было вызвать

m_stream.Close();

m_stream = null;


Связано ли это с особенностями сборки мусора?

Answer 1

[AM5800]

Предположу, что это сделано, чтобы избежать NPE при вызове stream.Close(), если данный код вызывается одновременно из нескольких потоков. Ведь если написать:
if (stream != null) stream.Close();
То сразу после успешной проверки stream теоретически может стать равным null. При предварительном копировании в локальную переменную это исключено.

Comments

[Illivion]

Хорошая мысль. Осталось только придумать пример в котором Dispose() или Close() BinaryReader'а из разных потоков не будут абсурдными :) А может в данном случае это просто такой обычный паттерн, который разработчик применил для перестраховки.

[AM5800]

Если вы не можете придумать не абсурдный пример, ломающий BinaryReader, то это не значит, что абсурдному примеру позволительно это делать =)
Ведь какой-бы дурацкий код не писал пользователь — подобным библиотекам падать не позволительно.

Answer 2

[mayorovp]

Это сделано, чтобы избежать NullPointerException при одновременных вызовах Close из разных потоков.

При нормальном использовании такая ситуация невозможна, но смысл исключений как раз в отлове ненормального использования. Данный трюк с копированием, фактически, заменяет NullPointerException на более соответствующее причине ошибки ObjectDisposedException