Чтение данных из базы данных SQL Server с C#

Question

У меня есть список BuilderString, который я хочу, чтобы содержать данные

public List<int> IDS = new List<int>(); 

public List<StringBuilder> Items = new List<StringBuilder>(); 

Что случилось с этим кодом?

SqlConnection con2 = new SqlConnection("Data Source=aya-PC\\SQLEXPRESS;Initial Catalog=ItemSet;Integrated Security=True"); 
SqlDataReader rdr2; 
SqlCommand cmd2; 

con2.Open(); 

for (int i = 0; i < IDS.Count; i++) 
{ 
    cmd2 = new SqlCommand("select item From TransactiontData where idT=@IDS[i]", con2); 
    cmd2.CommandType = CommandType.Text; 
    rdr2 = cmd2.ExecuteReader(); 
    SqlParameter param = new SqlParameter(); 
    param.ParameterName = "@IDS[i]" 

    while (rdr2.Read()) 
    { 
     Items[i].Append((StringBuilder)rdr2["item"]); 
    } 
} 

Answer 1

Вы должны изменить ваш код немного:

using (SqlConnection con2 = new SqlConnection("Data Source=aya-PC\\SQLEXPRESS;Initial Catalog=ItemSet;Integrated Security=True")) 
using (SqlCommand cmd2 = new SqlCommand("select item From TransactiontData where idT = @IDS", con2)) 
{ 
    // add the paramter to the command 
    cmd2.Parameter.Add("@IDS", SqlDbType.Int); 

    con2.Open(); 

    for (int i = 0; i < IDS.Count; i++) 
    { 
     // set the parameter value 
     cmd2.Parameter["@IDS"].Value = IDS[i]; 

     // only *THEN* call ExecuteReader() 
     using (SqlDataReader rdr2 = cmd2.ExecuteReader()) 
     { 
       while (rdr2.Read()) 
       { 
        // **NOT SURE** what you're trying to do here..... 
        // First of all, you need to just call Items.Add() 
        // to add new items to the list - and I'm TOTALLY 
        // UNCLEAR what you're trying to do casting the reader 
        // value to a StringBuilder....... 
        // 
        // Items[i].Append((StringBuilder)rdr2["item"]); 
        // 
        // replaced with what *might* make more sense..... 
        Items.Add(rdr2["item"].ToString()); 
       } 

       rdr.Close(); 
     } 
    }   

    con2.Close(); 
} 

Очки отметить:

• Я бы рекомендовал всегда поставить SqlConnection, SqlCommand и SqlDataReader в using() {...} блоки для обеспечения надлежащей утилизации

• Вам необходимо добавить свой параметр и установить его значениеПЕРЕД НАЙТИ Вы звоните .ExecuteReader()!

• Поскольку сам запрос никогда не изменяется - нет смысла создавать новый SqlCommand на каждой итерации. Создать команду раз - а затем просто установить значение параметра (что единственное изменение) один раз в итерации

Answer 2

Вам необходимо присвоить значение параметра в коде приложения, а не в запросе , Я не уверен, что вы пытаетесь выполнить, выставив значение столбца как StringBuilder. Предполагая, что каждый элемент StringBuilder должен содержать одну строку, полученную из столбца varchar/nvarchar, пример ниже сделает это.

for (int i = 0; i < IDS.Count; i++) 
{ 
     var cmd2 = new SqlCommand("select item From TransactiontData where idT=@IDS", con2); 

     SqlParameter param = new SqlParameter("@IDS", SqlDbType.Int) { Value = IDS[i] }; 

     var rdr2 = cmd2.ExecuteReader(); 

     while (rdr2.Read()) 
     { 
      Items.Add(new StringBuilder((string)rdr2["item"])); 
     } 
}