Добавление значений, возвращаемых SQL-запросом, чтобы получить итог?

Question

Добавление значений, возвращаемых SQL-запросом, чтобы получить итог?

1

У меня есть функция, которая получает количество и вес каждого элемента для пользовательского заказа из таблицы SQL на основе идентификатора заказа:

protected decimal getOrderWeight(int orderid)
{
    string sqlCmd = "SELECT Weight, Quantity FROM OrderItems WHERE OrderId = @OrderId";
    int qty = 0;
    decimal wgt = 0;
    decimal totalWeight = 0;
    using (SqlConnection cn = new SqlConnection(ConfigurationManager.ConnectionStrings["AbleCommerce"].ToString()))
    {
        cn.Open();
        SqlCommand cmd = new SqlCommand(sqlCmd, cn);
        cmd.Parameters.AddWithValue("@OrderId", orderid);
        using (IDataReader reader = cmd.ExecuteReader())
        {
            while (reader.Read())
            {
                qty = Convert.ToInt32(reader["Quantity"]);
                wgt = Convert.ToDecimal(reader["Weight"]);     
                totalWeight = qty * wgt;
            }
            cn.Close();
        }
    }
    return totalWeight;
}

В таблице (которая называется OrderItems) каждый элемент имеет количество и вес. Тем не менее, вес, показанный в таблице, НЕ является результатом количества веса * (то есть, пункт ABC имеет количество 6 для заказа № 12345, и каждый элемент ABC весит 1,00 фунтов). Созданная мной функция работает в том, что она умножает количество единиц заказа на вес (т.е. Общий вес для всех 6 предметов ABC составляет 6,00 фунтов)

Тем не менее, теперь мне нужно принять эти общие весовые значения и добавить их вместе, чтобы получить общий вес для всего заказа. Прямо сейчас, я возвращаю все веса в порядке: 6,00, 5,00, 2,80, 146,76, 6,02, 2,25

Но я не уверен, как добавить все это вместе? Я думал о создании Split(), но тогда я бы преобразовал десятичные дроби в строки, чтобы преобразовать их обратно в десятичные числа, чтобы добавить их вместе "split". Это кажется ужасно сложным... есть ли лучший способ?

Dejsa Cocan 27 май 2014, в 16:33

Источник

Теги:

sql-server

c#

sql

add

2 ответа

0

Если вы не суммируете общие весовые значения следующим образом: totalWeight + = qty * wgt;

Потому что теперь ваш totalWeight будет содержать общее значение веса последнего элемента только в порядке.

Igos 27 май 2014, в 13:02

Ещё вопросы

Gordon Linoff · Accepted Answer · 2014-05-27T13-06-00.000Z

3

Лучший ответ

Это то, что вы ищите?

SELECT sum(Weight * Quantity) as TotalWeight
FROM OrderItems
WHERE OrderId = @OrderId;

Если это так, это пример запроса агрегации, который является базовой концепцией SQL. Вы должны выполнять эту работу в базе данных, поскольку база данных оптимизирована для работы с большими объемами данных и извлекает такие результаты из таблиц.

Gordon Linoff 27 май 2014, в 13:06

0

@SaraDeJaneiro Это переместит логику приложения из кода в базу данных, что я считаю плохой практикой в наши дни. Наилучший подход IMHO - это выполнить этот расчет на бизнес-уровне с использованием LINQ.
Oscar 27 май 2014, в 14:21
0

Это именно то, что я искал. Это было намного проще, чем я думал, что это будет ... Спасибо!
Dejsa Cocan 27 май 2014, в 14:21
0

@ Оскар Я думаю, в некоторых случаях (например, это решение) лучше использовать DataBase чем циклы внутри кода или использовать linq. Потому что это экономит время выполнения кода.
Bharadwaj 27 май 2014, в 14:25
0

@ Оскар. , , Достаточно сказать, что я категорически не согласен с вами. «Приложения базы данных» обычно состоят как из интерфейса приложения, так и из хранилища данных, логика, подходящая для каждого компонента, должна идти туда, где он лучше всего подходит.
Gordon Linoff 27 май 2014, в 14:27
0

@Bharadwaj Почему в этом случае лучше? Если вы используете linq, вы делаете то же самое, так как вычисления будут выполняться в базе данных, но сохраняя код вашей компании в коде домена приложения.
Oscar 27 май 2014, в 14:27
1

@Oscar Принимая этот случай в качестве примера, конечная цель состоит в том, чтобы получить total а не детали, так почему бы получить все результаты в приложении, а затем выполнить расчет? вместо того, чтобы просто выполнить то же самое в запросе?
Bharadwaj 27 май 2014, в 14:29
0

@Bharadwaj Если вы знаете, как работает LINQ, вы должны знать, что вам не нужно получать данные из db, чтобы сделать их в LINQ, только одна строка кода может сделать то же самое, что и ваш транзакционный код, но сохраняя логику в модель предметной области, как я уже сказал. var sum = dataContext.Sum (x => x.Weight * x.Quantity);
Oscar 27 май 2014, в 14:36
1

@ Оскар Я понимаю, что вы говорите, но, поскольку ОП не указал, что использует какой-либо ORM, я не уверен, как ваш аргумент применим. Он может просто заменить свой запрос в переменной sqlCmd на запрос Гордона.
Grant 27 май 2014, в 14:40

Показать ещё 6 комментариев