Ist es in Ordnung, viele Datenbank-Roundtrips in einer Controller-Aktion

Question

zu machen Ich schreibe eine Web-App, die ein Admin-Dashboard hat und eine Zusammenfassung aller Daten in der db hat.

MSSql wird hier verwendet.

Gibt es eine Möglichkeit, alle diese Daten aus verschiedenen Tabellen in einer Runde zu erhalten?

Ich bin auch besorgt über meine Repository-Design. Ich gebe ein IQueryable zurück, da es auf keinen Fall effizient sein wird, alle Daten als IEnumerable in den Speicher zu bekommen und mehr Filterung/Paginierung in der Mitte mit Erweiterungsmethoden durchzuführen.

Gibt es eine bessere Möglichkeit, mein Repository zu erstellen?

Hier ist meine ViewComponent Aktion (die auch eine Controller-Aktion sein kann):

public async Task<IViewComponentResult> InvokeAsync() 
    { 
     var agents = _repository.AgentData.GetAll(); 
     var applications = _repository.ApplicationData.GetAll(); 
     var paymentRequests = _repository.PaymentRequestData.GetAll(); 

     var universityCommissionCalcuator = new CommissionUniversityCalculator(0); 
     var commissionCalcuator = new CommissionReferralCalculator(universityCommissionCalcuator); 
     var commission = await _repository.AgentData.GetTotalCommissions(applications, commissionCalcuator); 

     var result = AdminSummaryMapper.ToViewModel(agents, applications, paymentRequests, commission); 

     return View(result); 
    } 

AdminSummaryMapper:

public static class AdminSummaryMapper 
{ 
    public static AdminSummaryViewModel ToViewModel(IQueryable<Agent> agents, 
                IQueryable<Application> applications, 
                IQueryable<PaymentRequest> paymentRequests, 
                Commission commission) 
    { 
     var result = new AdminSummaryViewModel() 
     { 
      TotalStudents = applications.Count(), 
      ConfirmedStudents = applications.Count(app => app.ApplicationStatus == ApplicationStatus.Confirmed), 
      UnderEvaluationStudents = applications.Count(app => app.ApplicationStatus == ApplicationStatus.UnderEvaluation), 
      TotalAgents = agents.Count(), 
      PaymentRequests = paymentRequests.Count(), 
      TotalCommission = commission.TotalComission, 
      NetCommission = commission.NetComission, 
     }; 

     return result; 
    } 
} 

Answer 1

Wenn Sie mehrere Anrufe tätigen müssen die Daten von verschiedenen Orten kommt und kann nicht in einer einzigen Datenbankabfrage angefordert werden, dann ist dies kein Problem. Optimieren und cachen Sie dort, wo es Sinn macht.

Answer 2

Traditionell, wenn Sie viele Daten von einer Rundreise wollen, ist die Antwort für SQL Server eine gespeicherte Prozedur. Das Entitätsframework hat wahrscheinlich eine Möglichkeit, direkt auf gespeicherte Prozeduren zuzugreifen, aber unter Beibehaltung des Formats kann aus einer solchen gespeicherten Prozedur, die Daten aus mehreren Abfragen abruft, eine Sicht auf die Datenbank aufgebaut werden, so dass jede Abfrage serverseitig und dann die Ergebnisse ausgeführt wird werden auf einmal zurückgezogen.

Direkt eine gespeicherte Prozedur zugreifen:

http://www.entityframeworktutorial.net/stored-procedure-in-entity-framework.aspx

How to call Stored Procedure in Entity Framework 6 (Code-First)?

Dies erfordert, dass die gespeicherte Prozedur, die Daten zu manipulieren, so dass es wieder in einer Tabelle kommt. Entity Framework ist wirklich nicht der beste Weg, dies zu tun. Der eigentliche Zweck ist, Rookie-Fehler wie UPDATEs und DELETES ohne WHERE-Klauseln beim Hinzufügen von Auto-Completion zu erschweren.

Answer 3

Sie haben nicht erwähnt, welche Art von Datenbank Sie mit EF verwenden (SQLServer, MySql, Oracle, ...).

Das Abrufen von aggregierten Daten aus einer relationalen Datenbank ist eine recht einfache Aufgabe mit SQL. Sie könnten eine Sicht definieren und Subselects wie diese hier verwenden: https://www.essentialsql.com/get-ready-to-learn-sql-server-19-introduction-to-sub-queries/