[ASP.NET] Kiedy tworzyć asynchroniczne metody kontrolera?

Question

Cześć wszystkim! Ostatnio rozmyślam nad jednym dylematem - kiedy tworzyć asynchroniczne metody kontrolera a kiedy synchroniczne? Kiedyś spotkałem się z taką opinią, że metody kontrolera powinny być prawie zawsze asynchroniczne ze względu na responsywność serwera, chodzi mi o to, że jeśli będzie dużo żądań to kod synchroniczny może mieć problem, aby to obsłużyć i wątek główny może być zawalony przez te żądania. Oprócz tego spotkałem się jeszcze z taką opinią, że metody te powinny być asynchroniczne wtedy kiedy trzeba użyć słówka await, bądź wielokrotnie czytałem, że powinniśmy używać takich metody gdy mamy do czynienia z operacjami I/O oraz odpytywaniem bazy danych.

Odpowiedź na to pytanie jest mi potrzebna bo ostatnio napisałem taki kod:

public ViewResult Menu(string selectedCategory)
        {
            ViewBag.SelectedCategory = selectedCategory;
            var products = _context.Products;
            var allCategoryAndCount = (ConfigurationManager.AppSettings["all_category_option_name"], products.Count());
            
            IQueryable<(string option, int productsCount)> menuOptionsWithCounts = products
                .AsEnumerable()
                .Select(prod => prod.Category)
                .Distinct()
                .OrderBy(category => category)
                .Select(category => (category, products.Count(prod => prod.Category == category)))
                .Prepend(allCategoryAndCount);
            
            return View(menuOptionsWithCounts);
        }

I teraz nie wiem czy nie przekształcić tego w coś takiego:

public async Task<ViewResult> Menu(string selectedCategory)
        {
            ViewBag.SelectedCategory = selectedCategory;
            var products = this.repository.Products;
            var allCategoryAndCount = (ConfigurationManager.AppSettings["all_category_option_name"], products.Count());
            
            var menuOptionsWithCounts = await products
                .AsEnumerable()
                .Select(prod => prod.Category)
                .Distinct()
                .OrderBy(category => category)
                .Select(category => (category, products.Count(prod => prod.Category == category)))
                .Prepend(allCategoryAndCount)
                .ToListAsync();
            
            return View(menuOptionsWithCounts);
        }

Z góry dzięki za pomoc!

Comments

Metod asynchronicznych używasz, gdy wykonujesz operacje I/O lub długotrwałe prace. Metoda z async, ale bez await jest zwykłą metodą i z metodami asynchronicznymi nie ma nic wspólnego. Musisz użyć słowa await.

Answer 1

Warto używać await podczas korzystania z bazy danych, dlaczego ?

Ponieważ gdy mamy jakąś sekwencje zapytań do bazy (np. 2 - 3 zapytania) to w zależności od obciążenia serwera DB wartość 3 może być zwrócona szybciej niż 1 zapytania pomimo, że ono jest nam potrzebne na start