0

Выдает ошибку в следующей функции:

public void Tipmat(string tipmat)
{
    connection.Open();
    command = new OleDbCommand($"SELECT * FROM Материал WHERE Тип_мат = [{tipmat}]", connection);
    command.ExecuteNonQuery();
    connection.Close();
}

При вызове функции, tipmat принимает одно из четырех значений (Детали, Раст_жив_сырье, Горючее_сырье, Инструменты) в зависимости от того, что выбрал пользователь в комбо боксе. Функция вызывается когда пользователь меняет значение в комбо боксе.

Комбо

введите сюда описание изображения

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

Таблица

2
  • Не используйте конкатенацию/интерполяцию строк для составления sql-запроса. Используйте параметризованный запрос. Commented 18 мая 2022 в 10:20
  • Я не силён в синтаксисе Access. Разве можно строковый литерал заключать в квадратные скобки? Должны быть одинарные кавычки, не? Commented 18 мая 2022 в 10:21

2 ответа 2

0

Вероятно, у вас ошибка в строке запроса. Попробуйте изменить свой код так:

    command = new OleDbCommand($"SELECT * FROM Материал WHERE Тип_мат = '{tipmat}'", connection);

Здесь мы просто берем в кавычки значение, передаваемое в запрос.

12
  • Не используйте конкатенацию/интерполяцию строк для составления sql-запроса. Используйте параметризованный запрос. Commented 18 мая 2022 в 10:22
  • Ошибка теперь не выдается, но проблема в том что теперь запрос просто не срабатывает(в том плане что если я выбираю например "Деталь" и нажимаю обновить то ничего не меняется). Commented 18 мая 2022 в 10:30
  • @КириллРахвалов - так ваш код ничего и не делает. Вы же вызываете ExecuteNonQuery. Commented 18 мая 2022 в 10:32
  • То есть стоит убрать эту строчку? Я просто немного профан в этом всем Commented 18 мая 2022 в 10:33
  • Я убрал строчку с ExecuteNonQuery, но ничего не поменялось. Commented 18 мая 2022 в 10:34
0

Используйте параметризированные запросы - https://metanit.com/sharp/adonet/2.9.php

После создания такого запроса (с нужными параметрами и тд), открывайте соединение, передавайте command в метод и закрывайте его.

Для конвертации полученных данных в List используйте следующий код:

public List<T> SqlQuery<T>(MySqlCommand command)
{
    var result = new List<T>();
    using var reader = command.ExecuteReader();
    var columns = Enumerable.Range(0, reader.FieldCount).Select(reader.GetName).ToArray();
    var properties = typeof(T).GetProperties();

    while (reader.Read())
    {
        var data = new object[reader.FieldCount];
        reader.GetValues(data);

        var instance = (T)Activator.CreateInstance(typeof(T));

        for (var i = 0; i < data.Length; ++i)
        {
            if (data[i] == DBNull.Value)
                data[i] = null;

            var property = properties.SingleOrDefault(x => x.Name.Equals(columns[i], StringComparison.InvariantCultureIgnoreCase));

            if (property is not null)
            {
                var dataValue = Guid.TryParse(data[i].ToString(), out Guid res) is true ? res : data[i];
                property.SetValue(instance, Convert.ChangeType(dataValue, property.PropertyType));
            }
        }
        result.Add(instance);
    }
    return result;
}

Не забывайте открывать и закрывать соединение. Проще всего это делать, заворачивая connection в using.

Пример использования:

List<T> GetMaterialsByType(MySqlConnection connection, MaterialTypeEnum materialTypeEnum)
{
    using (connection)
    {
        connection.Open();

        using var command = connection.CreateCommand();
        command.CommandText = "SELECT * FROM Материал WHERE Тип_мат = @MaterType";
        command.Parameters.AddWithValue("@MaterType", materialTypeEnum);
        return SqlQuery(command);
    }
}

enum MaterialTypeEnum //Лучше храните одинаковые типы в enum. Гораздо проще жизнь станет
{
    Iron = 0,
    Wood = 1
}

UPD:

Заметил, как вы именуете методы. Слов, кроме как, "Wtf?!" не могу подобрать. Неужто нельзя назвать метод по получению списка материалов как-нибудь типо GetMaterialsByType, как у меня выше в примере.

Прошу, прочтите данную статью для приведения вашего стиля написания кода в порядок https://docs.microsoft.com/en-us/dotnet/csharp/fundamentals/coding-style/coding-conventions

6
  • Писал без VisualStudio - просто через редактор на сайте. Поэтому могут быть опечатки в коде и тд. Если они есть, дайте знать
    – Frehzy
    Commented 18 мая 2022 в 11:13
  • 1
    Ух... Имхо, опять оверинжиниринг. Автор начинающий, не поймёт-с! ЗЫ: лучше Dapper заюзать. Commented 18 мая 2022 в 11:34
  • 1
    Не стоит использовать AddWithValue. ru.stackoverflow.com/a/1236621/184217 - внизу ответа три ссылки на статьи Commented 18 мая 2022 в 11:35
  • @AlexanderPetrov Никто до этого не использовал Dapper. Почитаю информацию об этом обязательно. Но касаемо того, что AddWithValue - не лучший выбор - впервые слышу. Обязательно прочту статьи, которые вы приложили к ответу по ссылке. Спасибо
    – Frehzy
    Commented 18 мая 2022 в 11:45
  • Спасибо конечно за проделанный код, но мне не обязательно требуется максимально эффективный код, я лишь хочу что бы у меня получилось то что делал я. Я попробовал вставить код выше, он выдает некоторые ошибки, например то что он не видит SqlQuery(command). А также ему не нравится using var, он говорит эта команда поддерживается только с версии 8.0, а у меня 7.3. Commented 18 мая 2022 в 12:10

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.