Checksum

[Andrw-404]

No description provided.

[yurii-litvinov]

Тестов не хватает

[yurii-litvinov]

Вроде как у него нет никаких причин быть не static

[yurii-litvinov]

Не, всё-таки возвращать хешер должен был хеш, а не его строковое представление. Мы же не знаем, как он может быть использован в дальнейшем.

[yurii-litvinov]

Потенциально длительные операции в современном мире должны быть асинхронными, а считать хеш точно может быть долго.

[yurii-litvinov]

Некий копипаст с предыдущим методом. Можно было бы сделать один метод и параметризовать его лямбдой. И сделать два метода-обёртки, чтобы клиенту не надо было лямбду передавать.

[yurii-litvinov]

Suggested change

	using (FileStream fileStream = new FileStream(filePath, FileMode.Open, FileAccess.Read))
	using (FileStream fileStream = new(filePath, FileMode.Open, FileAccess.Read))

Древний C#, значит генерено нейросетью :)

[yurii-litvinov]

! не используйте вовсе. А ну как там null?

[yurii-litvinov]

Есть статическая версия ComputeHash (правда, называется HashData), которая позволила бы это записать в одну строчку. Объект-то Вам тут не нужен.

[yurii-litvinov]

Уже давно в C# не надо писать один тип дважды. Тем более такой.