Не удается прочитать из базы данных SQL Server в C#

Question

В настоящее время я работаю над заданием колледжа, в котором у меня возникают проблемы с чтением данных из базы данных SQL Server. Я пытаюсь прочитать столбец Dentist Name, а затем добавить эти имена в поле со списком.

Однако при вводе имени столбца отображается ошибка.

Моя таблица называется dentistInfo с колоннами Dentist ID, Dentist Name, Dentist Surname, DOB и Gender.

В конце концов, когда я вернусь к чтению, я буду надеяться, что он сможет заполнить их данные, когда имена будут выбраны из выпадающего списка.

public partial class Dentist_Info : Form 
{ 
    Surgery mySurgery = new Surgery(); 
    private SqlConnection conn; 
    private SqlCommand cmd; 
    private SqlDataAdapter da; 
    Surgery _formsSurgery; 

    public Dentist_Info(Surgery SurgeryToDisplay) 
    { 
     _formsSurgery = SurgeryToDisplay; 
    } 

    public void FillCombo() 
    { 
     SqlConnection conn = new SqlConnection(@"Data Source = GGJG; Initial Catalog = DentistDB; Integrated Security = True"); 
     SqlCommand SelectCommand = new SqlCommand("SELECT * FROM DentistInfo", conn); 

     SqlDataReader myreader; 
     conn.Open(); 

     try 
     { 
      myreader = SelectCommand.ExecuteReader(); 

      while (myreader.Read()) 
      { 
       string dname = myreader.GetString("Dentist Name"); 
       comboBox1.Items.Add(dname); 
      } 

      conn.Close(); 
     } 
     catch (Exception ex) 
     { 
      throw new Exception(ex.Message, ex); 
     } 
     finally 
     { 
      if (conn != null) 
      { 
       conn.Close(); 
      } 
     } 
    } 

Answer 1

Попробуйте это: Я рекомендую вам положить [] в Стоматолог Имени, так как оно имеет место между этими двумя словами, что может привести вас ошибку или изменить имя из базы данных в DentistName

public void FillCombo() 
    { 
     SqlConnection conn = new SqlConnection(@"Data Source = GGJG; Initial Catalog = DentistDB; Integrated Security = True"); 
     SqlCommand SelectCommand = new SqlCommand("SELECT * FROM DentistInfo", conn); 
     conn.Open(); 
     DataSet ds = new DataSet(); 
     SqlDataAdapter da = new SqlDataAdapter(SelectCommand); 
     da.fill(ds); 

     foreach(DataRow dr in ds.Tables[0].Rows) 
     { 
      comboBox1.Items.Add(dr["[Dentist Name]"].ToString()); 
     } 
     conn.Close(); 
    } 

в соответствии с добавлением вместо использования conn.Open() и conn.Close(), в качестве ответа первого пользователя вы можете окружить соединение внутри, используя так:

using(SqlConnection conn = new SqlConnection(@"Data Source = GGJG; Initial Catalog = DentistDB; Integrated Security = True")) 
{ 
    //your codes here no need for conn.Open() and conn.Close() 
} 

Answer 2

Pro-совет: Если вы хотите спросить об ошибке, вывесить ошибку.

В любом случае проблему легко обнаружить в этом случае. Нет перегрузки GetString, который принимает строку в качестве аргумента - вы можете использовать только индекс столбца.

Таким образом, либо вам необходимо передать индекс столбца (myreader.GetOrdinal("Dentist Name")), либо вам необходимо использовать индекс ((string)myreader["Dentist Name"]). В любом случае убедитесь, что вы правильно обрабатываете возможные значения NULL - считыватель данных просто выдает исключение, если вы пытаетесь прочитать значение NULL SQL.

Как и в сторону, ваш try ... catch можно упростить (и полезнее):

• Если вы хотите повторно выдать исключение, используйте throw; (нет «аргумент»). Обведите исключение только в том случае, если у вас есть какая-либо информация для добавления.
• Статья catch не требуется. Кажется, что вы используете его только для finally - это прекрасно, просто используйте try ... finally без catch.
• conn никогда не может быть null в предложении finally - ваш try не хватает.
• Для рисунка, подобного этому, вы хотите использовать using вместо try ... finally в любом случае. Вы также должны использовать using для устройства чтения данных.