Funkcje agregujące na wielu tabelach nie dają poprawnych wyników

Gdy dodasz kolejną tabelę, możesz wpłynąć na liczbę wierszy, a gdy to nastąpi, wpłynie to również na agregacje. Aby uniknąć tego zagregowania tabeli szczegółów, tak aby w jednym zamówieniu mógł znajdować się tylko jeden wiersz, pozostałe agregacje pozostaną spójne.

SELECT
      Customers.EmailAddress
    , COUNT(Orders.OrderID)                                                                                            AS 'overall NumOrders'
    , SUM(Orders.PaymentAmount)                                                                                        AS 'overall TotalOrdered'
    , SUM(od.totalcost) AS totalcost
    , COUNT(CASE WHEN Orders.OrderDate >= '20170101' THEN Orders.OrderID END)                                          AS '2017 NumOrders'
    , SUM(CASE WHEN Orders.OrderDate >= '20170101' THEN Orders.PaymentAmount END)                                      AS '2017 TotalOrdered'
    , COUNT(CASE WHEN Orders.OrderDate BETWEEN ' 01/01/2015 00:00' AND '12/31/2015 23:59' THEN Orders.OrderID END)     AS '2015 NumOrders'
    , SUM(CASE WHEN Orders.OrderDate BETWEEN ' 01/01/2015 00:00' AND '12/31/2015 23:59' THEN Orders.PaymentAmount END) AS '2015 TotalOrdered'
FROM Customers
JOIN Orders ON Customers.Customerid = Orders.Customerid
JOIN (
      SELECT
            Orderid
          , SUM((Vendor_Price) * (Quantity)) AS totalcost
      FROM OrderDetails
      GROUP BY
            Orderid
) od ON Orders.Orderid = od.Orderid
WHERE Orders.OrderStatus NOT IN ('Cancelled', 'Payment Declined')
AND Orders.OrderDate BETWEEN ' 01/01/2015 00:00' AND GETDATE()
GROUP BY
      Customers.EmailAddress

EDYTUJ

Nie używaj „23:59” jako punktu końcowego dla zakresu dat, który jest niedokładny i może prowadzić do nieprawidłowych wyników. Istnieje bardzo prosta i dokładniejsza alternatywa, która wymaga po prostu zaprzestania używania „pomiędzy”. Dodatkowo „31.12.2015 23:59” to NIE bezpieczny sposób na określenie wartości daty/godziny. Użyj „20160101”, który JEST najbezpieczniejszy format literalny w SQL Server YYYYMMDD .

    , COUNT(CASE WHEN Orders.OrderDate >= '20150101' AND Orders.OrderDate < '20160101' THEN Orders.OrderID END)     AS '2015 NumOrders'
    , SUM(CASE WHEN Orders.OrderDate >='20150101' AND Orders.OrderDate < '20160101' THEN Orders.PaymentAmount END) AS '2015 TotalOrdered'