brown-aleks / CleannetCode_bot

7 stars 4 forks source link

Использовать Async/Await правильно #5

Open Alex2021cyberhuman opened 1 year ago

Alex2021cyberhuman commented 1 year ago

Использовать await при выполнении асинхронных методов Использовать постфикс Async для асинхронных методов

brown-aleks commented 1 year ago

Соглашусь. Именно с расстановкой async/await в этой теме я слабоват. Буду благодарен, если в ваших комментариях к коду будут чуть более конкретные замечания по этому поводу. Где и как делать нужно, а как не нужно.

Alex2021cyberhuman commented 1 year ago

https://github.com/brown-aleks/CleannetCode_bot/blob/bbbb0b23caa439ad83b58ea521b14d9ef6d6e88e/CleannetCode_bot/IStorageService.cs#L5 - должно быть AddObjectAsync() https://github.com/brown-aleks/CleannetCode_bot/blob/bbbb0b23caa439ad83b58ea521b14d9ef6d6e88e/CleannetCode_bot/Handlers.cs#L19 - завершение AddObject не ожидается с помощью await, а должен быть await, а соответствующие методы async Task. https://github.com/brown-aleks/CleannetCode_bot/blob/bbbb0b23caa439ad83b58ea521b14d9ef6d6e88e/CleannetCode_bot/Handlers.cs#L23

это для примера, есть аналогичные примеры далее по коду

Alex2021cyberhuman commented 1 year ago

https://github.com/brown-aleks/CleannetCode_bot/blob/bbbb0b23caa439ad83b58ea521b14d9ef6d6e88e/CleannetCode_bot/StorageFileService.cs#L41 тут тоже нужно await

brown-aleks commented 1 year ago

Огромное спасибо! Всё поправил. 😎👍