search

DWD73 / DDW.WorkWithDocument

Навыки работы с документами
0 stars 0 forks source link

CodeReview #1

Open ZyuDev opened 3 years ago

ZyuDev commented 3 years ago

Работа сделана хорошо, требуемая по заданию функционально реализована.

Понравилось, что автор создал собственные аргументы для событий и передает информацию, которая может оказаться полезной в подписчиках событий. https://github.com/DWD73/DDW.WorkWithDocument/blob/master/DDW.Document/DocumentsReceiver.cs#L100

Заметил, что автор планировал освобождать ресурсы и реализовать интерфейс IDisposable. Однако метод Dispose() пустой, видимо просто забылось. https://github.com/DWD73/DDW.WorkWithDocument/blob/master/DDW.Document/DocumentsReceiver.cs#L94

Еще в качестве косметического замечания можно указать, что согласно официального style guide от MS приватные поля следует объявлять с префиксом "_". https://docs.microsoft.com/en-us/dotnet/csharp/fundamentals/coding-style/coding-conventions#camel-case

DWD73 commented 3 years ago

Благодарю. И да с Dispose() продинамил. Обязательно исправлю.