cotdarz Ответов: 2

Не удается получить sum excel-столбец с помощью csharp


Я хочу получить сумму одного столбца, используя фильтр другого столбца, но получить пустую строку в datatable. Новичок здесь.. Пожалуйста помочь.

Что я уже пробовал:

да = новый объект oledbdataadapter("выбрать сумма("+ sum_src.Текст +") как сумма из [Лист1$], где [" + criteria_src.Текст + "] = [" + col_row.Text + "]", conn);

2 Ответов

Рейтинг:
2

OriginalGriff

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

Когда вы объединяете строки, вы вызываете проблемы, потому что SQL получает такие команды, как:

SELECT * FROM MyTable WHERE StreetAddress = 'Baker's Wood'
Цитата, добавленная пользователем, завершает строку в том, что касается SQL, и вы получаете проблемы. Но могло быть и хуже. Если я приду и наберу вместо этого: "x';DROP TABLE MyTable;--", то SQL получит совсем другую команду:
SELECT * FROM MyTable WHERE StreetAddress = 'x';DROP TABLE MyTable;--'
Которые SQL видит как три отдельные команды:
SELECT * FROM MyTable WHERE StreetAddress = 'x';
Совершенно правильный выбор
DROP TABLE MyTable;
Вполне допустимая команда "удалить таблицу"
--'
А все остальное-это комментарии.
Так оно и происходит: выбирает любые совпадающие строки, удаляет таблицу из базы данных и игнорирует все остальное.

Поэтому всегда используйте параметризованные запросы! Или будьте готовы часто восстанавливать свою БД из резервной копии. Вы ведь регулярно делаете резервные копии, не так ли?

Во-вторых, вам нужно начать смотреть на фактическое содержание ваших переменных, чтобы понять, что здесь происходит, особенно содержание ваших переменных. criteria_src.Text и данные столбца, на которые он ссылается, и то же самое для col_row.Text и это данные. Мы не можем сделать это за вас - у нас просто нет доступа к переменным или данным, на которые они ссылаются.

Так что все будет зависеть от тебя.
К счастью, у вас есть инструмент, который поможет вам выяснить, что происходит: отладчик. Если вы не знаете, как его использовать, то быстрый Google для "Visual Studio debugger" должен дать вам необходимую информацию.

Поместите точку останова в первую строку функции и запустите код через отладчик. Затем посмотрите на свой код и на свои данные и определите, что должно произойти вручную. Затем по одному шагу в каждой строке проверяйте, что то, что вы ожидали, произойдет именно так, как и произошло. Когда это не так, тогда у вас есть проблема, и вы можете вернуться назад (или запустить ее снова и посмотреть более внимательно), чтобы выяснить, почему.

Извините, но мы не можем сделать это за вас - пришло время вам освоить новый (и очень, очень полезный) навык: отладку!

Я бы начал с того, что вернул бы фактические строки вместо суммы: это должно дать вам ключ к пониманию того, что на самом деле происходит.


Wendelius

Комплименты OP: #OriginalGriff Спасибо за все превосходные советы, я научусь отладке!

Рейтинг:
19

Wendelius

Запрос кажется немного странным. Вы имеете в виду сравнение значений в файле excel со значением, предоставленным пользователем? Если это так, то сравнение, вероятно, должно быть чем-то другим, чем-то вроде

da = new OleDbDataAdapter("select sum("+ sum_src.Text +") as summ from [Sheet1$] where [" + criteria_src.Text + "] = [Conditioncolumn]", conn); 

Кроме того, столбец, который будет суммирован, вероятно, должен быть назван, что-то вроде
da = new OleDbDataAdapter("select sum([ColumnToBeSummed]) as summ from [Sheet1$] where [" + criteria_src.Text + "] = [Conditioncolumn]", conn);


cotdarz

Все три combobox должны быть вызваны пользователем из выбранного excel_file . Первый и второй combobox загружают все имена столбцов, а третий combobox загружает column_data из второго выбранного имени столбца. Я дошел до этого, и имена столбцов на листе не содержат пробелов.

Все, чего я хочу добиться, - это "выбрать сумму(сумму) из [Sheet1$], где customer=John"
{с тремя выбранными пользователем combobox}

#OriginalGriff Спасибо за все превосходные советы, я научусь отладке! #Венделиус замечательный комментарий!

Wendelius

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

select sum([amount]) from [Sheet1$] where [customer]='John'"

Также убедитесь, что эти столбцы существуют на листе

cotdarz

Спасибо. Я сделал это. Мне не хватало одной кавычки в части "Джон", и вместо этого я добавил скобку к [Джон]: D

Wendelius

Рад, что все разрешилось!