files-community / Files

A modern file manager that helps users organize their files and folders.
https://files.community
MIT License
34.19k stars 2.18k forks source link

Code Quality: Convert into `await using` statement or declaration #15092

Open gumbarros opened 6 months ago

gumbarros commented 6 months ago

Description

using (var stream = File.OpenRead(_item.ItemPath))

to

await using (var stream = File.OpenRead(_item.ItemPath))                    

In .NET, classes that own unmanaged resources usually implement the IDisposable interface to provide a mechanism for releasing unmanaged resources synchronously. However, in some cases they need to provide an asynchronous mechanism for releasing unmanaged resources in addition to (or instead of) the synchronous one. Providing such a mechanism enables the consumer to perform resource-intensive dispose operations without blocking the main thread of a GUI application for a long time.

The IAsyncDisposable.DisposeAsync method of this interface returns a ValueTask that represents the asynchronous dispose operation. Classes that own unmanaged resources implement this method, and the consumer of these classes calls this method on an object when it is no longer needed.

Source: https://stackoverflow.com/questions/58610350/what-is-the-difference-between-using-and-await-using-and-how-can-i-decide-which

Concerned code

Generated using Jetbrains Rider:

\\\Utils\Storage\StorageItems\VirtualStorageItem.cs (1 issue) \\\Utils\Storage\StorageBaseItems\BaseStorageFile.cs (1 issue) \\\Utils\Shell\LaunchHelper.cs (2 issues) \\\Utils\Storage\StorageItems\ZipStorageFile.cs (7 issues) \\\Utils\Storage\StorageItems\VirtualStorageFile.cs (4 issues) \\\Utils\Storage\Operations\ShellFilesystemOperations.cs (4 issues) \\\Utils\Storage\StorageItems\ZipStorageFolder.cs (9 issues) \\\Utils\Storage\StorageItems\NativeStorageFile.cs (4 issues) \\\Utils\Storage\StorageItems\SystemStorageFile.cs (9 issues) \\\ViewModels\Properties\HashesViewModel.cs (1 issue) \\\Utils\Storage\StorageItems\FtpStorageFile.cs (3 issues) \\\Utils\Cloud\Detector\SynologyDriveCloudDetector.cs (3 issues) \\\Utils\Cloud\Detector\GoogleDriveCloudDetector.cs (3 issues) \\\Services\SideloadUpdateService.cs (2 issues) \\\Services\UpdateService.cs (2 issues) \\\Extensions\ImageSourceExtensions.cs (1 issue) ### Gains - UI will not freeze while disposing a resource ### Requirements - Convert all asynchronous using to `async using` ### Comments ![image](https://github.com/files-community/Files/assets/52143624/4ac9f73c-f404-4a83-b402-bc655646e954) It's possible to make a single PR using this button :)
0x5bfa commented 6 months ago

Out of curious, what IDE are you using? I also wanna generate this list lol.

gumbarros commented 6 months ago

Out of curious, what IDE are you using? I also wanna generate this list lol.

Rider, but you can install ReSharper at VS to obtain the same result. You can get a free license if you are a FOSS contribuitor.

0x5bfa commented 6 months ago

Oh they are kind. Why do you think those warnings won't be shown in VS2022 by default and what should we do to should those issues that would improve performance.

0x5bfa commented 6 months ago

@yaira2 Can I use the name of Files Community for: image

gumbarros commented 6 months ago

Oh they are kind. Why do you think those warnings won't be shown in VS2022 by default and what should we do to should those issues that would improve performance.

I guess this is why JetBrains makes money lol, if VS showed these problems ReSharper wouldn't exist. Maybe because of some limitation?

Yair said about benchmarking, I think this is valid. There is also dozens of issues types when I run the analyzer, but I opened just the most importants, some of them I think they don't make sense, like var instead of the explicit type, not worth a PR.

image

marcofranzen99 commented 6 months ago

I'm not sure if Files counts as a project, since users have to pay for the version in the Microsoft Store

From here https://www.jetbrains.com/community/opensource/#support

image

0x5bfa commented 6 months ago

That's absolutely right lol I was wondering why VS doesn't show warnings that comes from the official analyzers(IDE????, C????) while Rider shows. But I seemed to have misunderstood. Thanks.

Really nice features.

gumbarros commented 6 months ago

I'm not sure if Files counts as a project, since users have to pay for the version in the Microsoft Store

From here https://www.jetbrains.com/community/opensource/#support

image

image I saw some projects with "Thanks for JetBrains" at the end of the README file, maybe if you contact JetBrains sales they can make a exception for Files as this is a big project .

0x5bfa commented 6 months ago

But that's for donation.

yaira2 commented 6 months ago

I'm not sure if Files counts as a project, since users have to pay for the version in the Microsoft Store

I wouldn't consider the store version as commercial, the sideload version has all the same features and users get the same level of support. In fact, the sideload version has many times the number of users as the store version.