Считать только 3 элемента строки, при этом не создавая отдельный конструктор

@ss1zaa · Регистрация: 14.12.2022

Author24 — интернет-сервис помощи студентам

Необходимо считать только 3 элемента строки, при этом не создавая отдельный конструктор. Соответственно, запрос SQL должен выглядеть следующим образом:

SQL

1	SELECT Name, Genre, Poster FROM 'Event'

Исходный код:

C#

            Event[] events = new Event[20];
            string query = "SELECT * FROM 'Event'";
            SQLiteConnection connection = new SQLiteConnection("Integrated Security = SSPI; Data Source = Films.db");
            connection.Open();
            SQLiteCommand command = new SQLiteCommand(query, connection);
            using (var dr = command.ExecuteReader())
            {
                int k = 0;
                while (dr.Read())
                {
                    events[k] = new Event(dr.GetString(0), dr.GetString(1), dr.GetString(2), dr.GetString(3), (byte[])dr.GetValue(4), dr.GetString(5), dr.GetString(6));
                    k++;
                }
            }
            connection.Close();
            EventdataGridView.DataSource = events;

@OwenGlendower · 07.06.2023, 23:42

ss1zaa, так исправьте запрос на нужный, в чем проблема? Или если ваш вопрос о том как получить значение колонки по имени, то это делается так

C#

1	dr.GetString(dr.GetOrdinal("Name"))

@ss1zaa · 08.06.2023, 11:45 **[ТС]**

OwenGlendower, вопрос заключается в том, как необходимо изменить код программы, ведь если я вбиваю необходимый запрос вместо исходного, при этом ничего другого в коде не трогая, при запуске программы я получаю ошибку System.IndexOutOfRangeException: "Индекс находился вне границ массива." на следующей строчке:

C#

1	events[k] = new Event(dr.GetString(0), dr.GetString(1), dr.GetString(2), dr.GetString(3), (byte[])dr.GetValue(4), dr.GetString(5), dr.GetString(6));

, так как конструктор принимает больше значений, нежели берёт команда, я это так понимаю.
Но и конструктор я заменить не могу на другой, который будет принимать лишь три значения, ведь остальные данные мне тоже нужны, хоть их я и не собираюсь выводить в dataGridView. Надеюсь, понятно объяснил

@OwenGlendower · 08.06.2023, 12:34

Нерабочий вариант

ss1zaa, ага, понял. Я бы сначала изменил класс Event:

C#

class Event
{
    public string? Name { get; init; }
    public string? Genre { get; init; }
    public string? Poster { get; init; }
    // Остальные свойства ...
    
    public Event(IDataRecord data)
    {
        Name = GetString(data, "Name");
        Genre = GetString(data, "Genre");
        Poster = GetString(data, "Poster");
        // Инициализация остальных свойств ...
    }
    
    private static string? GetString(IDataRecord data, string columnName)
    {
        int index = data.GetOrdinal(columnName);
        return index == -1 ? null : data.GetString(index);
    }
}

Тогда создание экземпляра Event будет всегда выглядеть так:

C#

1	events[k] = new Event(dr);

Так мы сможем прочитать любые колонки, в любом порядке. Для колонок которые могут быть NULL в БД можно добавить еще метод GetDbNullString. По хорошему метод GetString и GetDbNullString лучше сделать методами расширения для IDataRecord чтобы в классе Event (и других аналогичных) не было лишнего кода.

Добавлено через 5 минут

Сообщение от OwenGlendower

index == -1

СТОП! Тут я ошибку допустил. Метод GetOrdinal выбрасывает IndexOutOfRangeException если колонка не найдена.

Добавлено через 14 минут
ss1zaa, начал писать новый код, но понял что проще на Dapper переключиться чем снова изобретать велосипед.

@Andrey-MSK · 13.06.2023, 10:37

ss1zaa, А вот так не проще? Пример для FW 4.8, для Core использовать string?
Класс данных

C#

namespace DALModels.Models
{
    public class CPSHeader
    {
        public int HeaderID { get; set; }
        public int? PositionNumber { get; set; }
        public string PositionName { get; set; }
        public string Measure { get; set; }
        public int ContractID { get; set; }
    }
}

Метод чтения, в котором можно прочитать любое количество полей запроса, остальные в классе будут просто null

C#

public async Task<List<CPSHeader>> GetCPSHadersAsync(CPSContract contract)
{
    List<CPSHeader> headers = new List<CPSHeader>();
    string sqlText = "SELECT ID_CPSHeader, ID_CPSContract, PosNumber, PosName, Measure " +
        "FROM dbo.tblCPSHeaders WHERE ID_CPSContract = @idCPSContract";
 
    try
    {
        using (SqlConnection sqlConnection = new SqlConnection(_connectionString))
        {
            await sqlConnection.OpenAsync();
 
            using (SqlCommand sqlCommand = new SqlCommand(sqlText, sqlConnection))
            {
                SqlParameter pEObjectID = new SqlParameter
                {
                    ParameterName = "@idCPSContract",
                    Value = contract.ContractID,
                    SqlDbType = SqlDbType.Int,
                    Direction = ParameterDirection.Input
                };
                sqlCommand.Parameters.Add(pEObjectID);
 
                using (SqlDataReader sqlDataReader = await sqlCommand.ExecuteReaderAsync())
                {
                    if (sqlDataReader.HasRows)
                    {
                        while (await sqlDataReader.ReadAsync())
                        {
                            headers.Add(new CPSHeader
                            {
                                HeaderID = (int)sqlDataReader["ID_CPSHeader"],
                                ContractID = (int)sqlDataReader["ID_CPSContract"],
                                PositionNumber = sqlDataReader["PosNumber"]
                                    == DBNull.Value ? null : (int?)sqlDataReader["PosNumber"],
                                PositionName = (string)sqlDataReader["PosName"],
                                Measure = (string)sqlDataReader["Measure"]
                            });
                        }
                    }
                }
            }
        }
 
        return headers;
    }
    catch (Exception ex)
    {
        throw new ApplicationException("Ошибка загрузки списка заголовков ВДЦ.", ex);
    }
}

	13.06.2023, 10:37

Опции темы