Необработанное исключение типа "System.InvalidOperationException"

@Виктор27 · Регистрация: 21.11.2013

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

Здравствуйте. Ребят подскажите пожалуйста, выдает вот такую ошибку:Необработанное исключение типа "System.InvalidOperationException" в System.Data.dll

Дополнительные сведения: Существует назначенный этой команде Command открытый DataReader, который требуется предварительно закрыть.

Мучаюсь, мучаюсь понять не могу. Жалуется на эту строчку:

C#
1
label7.Text = comd.ExecuteScalar().ToString();

Сам код:

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
private void button1_Click(object sender, EventArgs e)
        {
            con.Open();
 
            if (comboBox1.Text == "")
            {
                MessageBox.Show("Выберите пользователя");
                con.Close();
                return;
            }
            SqlCommand com = new SqlCommand("Select Логин, Пароль From Регистрация Where Логин='" + comboBox1.SelectedItem.ToString() + "'", con);
            SqlDataReader reader = com.ExecuteReader();
 
            if (reader.HasRows)
            {
                while (reader.Read())
                {
                    if (textPassw.Text == reader.GetString(1))
                    {                                                
                        SqlCommand comd = new SqlCommand("SELECT ID FROM Сотрудники WHERE ID=(SELECT ID_Сотрудника FROM Регистрация WHERE Логин='" + comboBox1.SelectedItem.ToString() + "')", con);                        
                        label7.Text = comd.ExecuteScalar().ToString();            //Вот тут останавливается и жалуется            
                    }
                }
                MessageBox.Show("Вы ввели неверный пароль");
                textPassw.Text = "";
                textPassw.Focus();
                reader.Close();
            }
            else
            {
                MessageBox.Show("Вы ввели неверный логин и пароль");
                textPassw.Text = "";
                comboBox1.SelectedItem.ToString();
                comboBox1.Focus();
            }   
            con.Close();
        }

@HF · 19.06.2017, 21:01

Вы уже сделали

Code
1
com.ExecuteReader()

. Нельзя сделать ещё один вызов через

Code
1
comd.ExecuteScalar().ToString()

на том же SqlConnection (con).
Вы просто совершенно не правильно работаете с выборкой. Нужно почти всё переписать. Я напишу только об основных ошибках.

Сначала (как я понял) вы проверяете введённый пароль для возможности доступа к выбоке из базы.
Ошибки:
1) вы делаете ExecuteReader (это выборка данных, "многострочная"), а вам как раз нужен только один результат. Как раз здесь нужно сделать ExecuteScalar.
2) и раз вам нужно только сравнить пароль, то зачем выбирать из базы другие поля? (Логин, Пароль). Достаточно только одного - Пароль
3) Если вы знаете Пароль и Логин, то нужно сразу через БД и проверять наличие пользователя, а не сравнивать пароль потом в коде программы. Т.е. нужно добавить в условие

C#
1
Where Логин='" + comboBox1.SelectedItem.ToString() + "'"

C#
1
Where Логин='" + comboBox1.SelectedItem.ToString() + "' AND Пароль='" + textPassw.Text + "'"

и получить в результат ID пользователя из базы Регистрация.
И тогда если ID не пустой, то значит логин и пароль верны.
4) Дальше вообще не понятно. У вас ID в Таблице Регистрация или Сотрудники? Или он и там и там? или в Сотрудниках? Если в Сотрудниках то это вообще странно. Мало того, получается что если даже пользователь есть в Регистрации и есть ID, то не факт что есть в Сотрудниках?

Добавлено через 4 минуты
Получается что можно вообще всё сократить в половину:

C#
1
2
3
4
5
6
7
8
9
10
            SqlCommand com = new SqlCommand("Select ID From Регистрация Where Логин='" + comboBox1.SelectedItem.ToString() + "' AND Пароль='"+ textPassw.Text +"'", con);
            string userID = com.ExecuteScalar().ToString();
 
            if (userID == "")
            {
                MessageBox.Show("Вы ввели неверный пароль");
                textPassw.Text = "";
                textPassw.Focus();
            }
            reader.Close();

@Виктор27 · 19.06.2017, 21:35 **[ТС]**

HF, Большущее спасибо. Разжевали. У меня оказывается вообще все не правильно, надо переписать. Что то и на самом деле много всего не нужного))
Сделал так)))

C#
1
2
3
4
5
6
7
8
9
10
11
12
SqlCommand comd = new SqlCommand("SELECT Пароль FROM Регистрация WHERE Логин='" + comboBox1.SelectedItem.ToString() + "'", con);
            string passw = comd.ExecuteScalar().ToString();
            if (textPassw.Text == passw)
            {
                 
            }
            else
            {
                MessageBox.Show("Вы ввели неверный пароль");
                textPassw.Text = "";
                textPassw.Focus();
            }

Вот и весь код. Еще раз спасибо, а то бы бился головой об стену))

Новые блоги и статьи Все статьи Все блоги /
Как я обхитрил таблицу Word Alexander-7 21.03.2026 Когда мигает курсор у внешнего края таблицы, и нам надо перейти на новую строку, а при нажатии Enter создается новый ряд таблицы с ячейками, то мы вместо нервных нажатий Энтеров мы пишем любые буквы. . .	Krabik - рыболовный бот для WoW 3.3.5a AmbA 21.03.2026 без регистрации и смс. Это не торговля, приложение не содержит рекламы. Выполняет свою непосредственную задачу - автоматизацию рыбалки в WoW - и ничего более. Однако если админы будут против -. . .	Программный отбор значений справочника Maks 21.03.2026 Установка программного отбора значений справочника "Сотрудники" из модуля формы документа. В качестве фильтра для отбора служит предопределенное значение перечислений. Процедура. . .	Переходник USB-CAN-GPIO Eddy_Em 20.03.2026 Достаточно давно на работе возникла необходимость в переходнике CAN-USB с гальваноразвязкой, оный и был разработан. Однако, все меня терзала совесть, что аж 48-ногий МК используется так тупо: просто. . .
Оттенки серого Argus19 18.03.2026 Оттенки серого Нашёл в интернете 3 прекрасных модуля: Модуль класса открытия диалога открытия/ сохранения файла на Win32 API; Модуль класса быстрого перекодирования цветного изображения в оттенки. . .	SDL3 для Desktop (MinGW): Рисуем цветные прямоугольники с помощью рисовальщика SDL3 на Си и C++ 8Observer8 17.03.2026 Содержание блога Финальные проекты на Си и на C++: finish-rectangles-sdl3-c. zip finish-rectangles-sdl3-cpp. zip	Символические и жёсткие ссылки в Linux. algri14 15.03.2026 Существует два типа ссылок — символические и жёсткие. Ссылка в Linux — это запись в каталоге, которая может указывать либо на inode «файла-ИСТОЧНИКА», тогда это будет «жёсткая ссылка» (hard link),. . .	[Owen Logic] Поддержание уровня воды в резервуаре количеством включённых насосов: моделирование и выбор регулятора ФедосеевПавел 14.03.2026 Поддержание уровня воды в резервуаре количеством включённых насосов: моделирование и выбор регулятора ВВЕДЕНИЕ Выполняя задание на управление насосной группой заполнения резервуара,. . .