Ошибка на форме авторизации Visual Studio 2010

@Krish · Регистрация: 17.04.2015

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

При вводе логина и пароля и после нажатия кнопки вход выдает ошибку:

Неправильный синтаксис около ключевого слова "User".
И кидает на строку :dr = cmd.ExecuteReader();

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
private void button1_Click(object sender, EventArgs e)
{
SqlConnection cn = new SqlConnection("Data Source=ServerName;Initial Catalog=DB;Integrated Security=True");
cn.Open();
SqlCommand cmd = new SqlCommand("SELECT * FROM User WHERE FIO = '"+textBox3.Text+"' and Parol = '"+textBox2.Text+"'", cn);
SqlDataReader dr;
dr = cmd.ExecuteReader();
int count = 0;
while (dr.Read())
{
count += 1;
}
if (count == 1)
{
Form2 f2 = new Form2();
f2.Show();
}
else if (count > 0)
{
MessageBox.Show("Вы ввели неверный логин или пароль");
}
else
{
MessageBox.Show("Вы ввели неверный логин или пароль");
}
textBox2.Clear();
textBox3.Clear();
}

@Esper · 12.05.2015, 11:35

Не по теме:

Зачем городить while (dr.Read()){count += 1;}
Может сразу в запросе количество вычислять?

А что в таблице User?

@Krish · 13.05.2015, 10:19 **[ТС]**

В таблице User данные пользователей: FIO, Parol и Dostup(1 или 2).
В идеале если у пользователя доступ 1, то при авторизации переход на форму 7, если 2 - то на форму 2.

Добавлено через 19 часов 16 минут
Сколько видео примеров не смотрел везде такой код работает, возможно ли это из-за того что при выборе источника данных я выбрал не "Файл базы данных MSSQL Server", а "MSSQL Server" и программа не может получить доступ к таблице?

Добавлено через 57 минут
А может быть дело в том что в таблице User не только FIO, Parol, но еще 2 поля.
Неужели никто не знает...

@FroztBal · 13.05.2015, 10:46

Поставьте точку останова на строке 6 SqlDataReader dr; и посмотрите в переменных какая строка передается в качестве запроса, возможно ошибка там.
а вообще рекомендую строки запроса формировать либо через string.Format или через параметры (тут вообще трудно накосячить)
Ну и авторизацию тогда уж проводить логичнее так: если правильно ввел связку логин/пароль то пускает дальше если нет, остается на той же форме

@Krish · 13.05.2015, 13:41 **[ТС]**

Ошибки исправил, авторизация работает. Теперь хотелось бы узнать как в зависимости от поля в таблице User - Dostup пересылать на разные формы. Если в поле цифра "1", то на одну, а если "2", то на другую?

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
SqlConnection con = new SqlConnection("Data Source=КАРПЕНКО_Е;Initial Catalog=DB;Integrated Security=True");
            con.Open();
            SqlCommand cmd = new SqlCommand("SELECT [FIO], [Parol] FROM [User] WHERE [FIO] ='" + textBox3.Text + "' and [Parol] ='" + textBox2.Text + "'", con);
            SqlDataReader dr;
            dr = cmd.ExecuteReader();
            int count = 0;
            while (dr.Read())
            {
                count += 1;
            }
            if (count == 1)
            {
                OperatorsForm f2 = new OperatorsForm();
                f2.Show();
                f2.label100.Text = "Вы вошли как " + textBox3.Text;
            }
            else if (count > 0)
            {
                MessageBox.Show("Ошибка!");
            }
            else
            {
                MessageBox.Show("Ошибка!");
            }
            textBox2.Clear();
            textBox3.Clear();

@FroztBal · 13.05.2015, 14:04

оператор switch - case вам в помощь

@Krish · 13.05.2015, 14:45 **[ТС]**

FroztBal, а можно по-подробнее. Просто я не слишком силен в этом, набросал блоки оператора, а что в начале дописать не пойму

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
switch()
                {
                    case "1":
                        UsersForm secondForm = new UsersForm();
                        secondForm.ShowDialog();
                        textBox2.Clear();
                        textBox3.Clear();
                        break;
 
                    case "2":
                       OperatorsForm f2 = new OperatorsForm();
                       f2.Show();
                       f2.label100.Text = "Вы вошли как " + textBox3.Text;
                       textBox2.Clear();
                       textBox3.Clear();
                       break;
 
                    default:
                       MessageBox.Show("Ошибка!");
                       break;
                }

@wm_leviathan · 13.05.2015, 15:09

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

switch()

в сам свитч должен попадать результат доступа 1 или 2 в противном случае всегда будет default. а вначале авторизация должна быть

@Krish · 13.05.2015, 15:14 **[ТС]**

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

в сам свитч должен попадать результат доступа 1 или 2 в противном случае всегда будет default. а вначале авторизация должна быть

Я это понимаю, но не знаю как сделать, а авторизация и так есть:

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
38
39
40
41
42
43
44
45
private void button1_Click(object sender, EventArgs e)
        {
          
            SqlConnection con = new SqlConnection("Data Source=КАРПЕНКО_Е;Initial Catalog=DB;Integrated Security=True");
            con.Open();
            SqlCommand cmd = new SqlCommand("SELECT [FIO], [Parol] FROM [User] WHERE [FIO] ='" + textBox3.Text + "' and [Parol] ='" + textBox2.Text + "'", con);
            SqlDataReader dr;
            dr = cmd.ExecuteReader();
            MessageBox.Show("Соединение установлено!");
            int count = 0;
            while (dr.Read())
            {
                count += 1;
            }
            if (count == 1)
            {
                
                switch()
                {
                    case "1":
                        UsersForm secondForm = new UsersForm();
                        secondForm.ShowDialog();
                        break;
 
                    case "2":
                       OperatorsForm f2 = new OperatorsForm();
                       f2.Show();
                       f2.label100.Text = "Вы вошли как " + textBox3.Text;
                       break;
 
                    default:
                       MessageBox.Show("Ошибка!");
                       break;
                }  
            }
            else if (count > 0)
            {
                MessageBox.Show("Проверьте логин или пароль!");
            }
            else
            {
                MessageBox.Show("Проверьте логин или пароль!");
            }
            textBox2.Clear();
            textBox3.Clear();

@wm_leviathan · 13.05.2015, 15:35

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
38
39
40
41
42
43
44
        public int getData()
        {
            string connStr = "Data Source=КАРПЕНКО_Е;Initial Catalog=DB;Integrated Security=True";
            string sqlQueryString = "SELECT [FIO], [Parol] FROM [User] WHERE [FIO] ='" + textBox3.Text.TrimEnd() + "' and [Parol] ='" + textBox2.Text.TrimEnd() + "'";
            try
            {
                using (SqlDataAdapter dataAdapter = new SqlDataAdapter(sqlQueryString, connStr))
                {
                    DataTable table = new DataTable();
                    table.Locale = System.Globalization.CultureInfo.InvariantCulture;
                    dataAdapter.Fill(table);
                    if (table.Rows.Count != 0)
                    {
                        return Convert.ToInt32(table.Rows[0]["Dostup"].ToString());
                    }
                    else
                        return 0;
 
                }
            }
            catch
            {
                return 0;
            }
        }
 
 
                switch(getData())
                {
                    case 1:
                        UsersForm secondForm = new UsersForm();
                        secondForm.ShowDialog();
                        break;
 
                    case 2:
                       OperatorsForm f2 = new OperatorsForm();
                       f2.Show();
                       f2.label100.Text = "Вы вошли как " + textBox3.Text;
                       break;
 
                    default:
                       MessageBox.Show("Ошибка!");
                       break;
                }

@Krish · 13.05.2015, 15:42 **[ТС]**

wm_leviathan, к коду кнопки данные записи не подходят, что-то не совсем понятно...

@wm_leviathan · 13.05.2015, 15:56

getData э то метод он не пишется в коде кнопки а пишется в коде класса.

@Krish · 13.05.2015, 17:08 **[ТС]**

wm_leviathan, а что тогда в коде кнопки писать

@wm_leviathan · 13.05.2015, 17:10

switch(getData())
{

@Krish · 14.05.2015, 09:09 **[ТС]**

wm_leviathan, как буду за рабочим ПК попробую, спасибо за помощь.

Добавлено через 15 часов 57 минут
wm_leviathan, при любом исходе на логин форме нажатие кнопки выдает default

@wm_leviathan · 14.05.2015, 09:12

напишите поля базы, я мог "Dostup" неправильно написать. Я то писал так для примера, а не для копипаста. покажите полный код формы.

@Krish · 14.05.2015, 09:18 **[ТС]**

wm_leviathan,

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
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Windows.Forms;
using System.Data.SqlClient;
using System.Data.SqlTypes;
 
namespace Riaraelib
{
    public partial class LoginForm : Form
    {
        public LoginForm()
        {
            InitializeComponent();
 
        }
        public int getData()
        {
            string connStr = "Data Source=КАРПЕНКО_Е;Initial Catalog=DB;Integrated Security=True";
            string sqlQueryString = "SELECT [FIO], [Parol] FROM [User] WHERE [FIO] ='" + textBox3.Text.TrimEnd() + "' and [Parol] ='" + textBox2.Text.TrimEnd() + "'";
            try
            {
                using (SqlDataAdapter dataAdapter = new SqlDataAdapter(sqlQueryString, connStr))
                {
                    DataTable table = new DataTable();
                    table.Locale = System.Globalization.CultureInfo.InvariantCulture;
                    dataAdapter.Fill(table);
                    if (table.Rows.Count != 0)
                    {
                        return Convert.ToInt32(table.Rows[0]["[Dostup]"].ToString());
                    }
                    else
                        return 0;
 
                }
            }
            catch
            {
                return 0;
            }
        }
        private void button1_Click(object sender, EventArgs e)
        {
            switch (getData())
            {
                case 1:
                    UsersForm secondForm = new UsersForm();
                    secondForm.ShowDialog();
                    break;
 
                case 2:
                    OperatorsForm f2 = new OperatorsForm();
                    f2.Show();
                    f2.label100.Text = "Вы вошли как " + textBox3.Text;
                    break;
 
                default:
                    MessageBox.Show("Ошибка!");
                    break;
            }
}

Добавлено через 1 минуту
Поля таблицы User: ID_user, FIO, Parol, Dostup, Lab.

@wm_leviathan · 14.05.2015, 09:28

[Dostup] без скобок пишется

@Krish · 14.05.2015, 09:29 **[ТС]**

Я и так и так писал, просто сюда этот пример копернул

@wm_leviathan · 14.05.2015, 09:30

и в поле Dostup точно есть 1 или 2 ?

Добавлено через 47 секунд
и проверьте отладчиком table.Rows.Count возвращает ли строку ? может запрос ничего не возвращает?

Новые блоги и статьи Все статьи Все блоги /
PhpStorm 2025.3: WSL Terminal всегда стартует в ~ and_y87 14.12.2025 PhpStorm 2025. 3: WSL Terminal всегда стартует в ~ (home), игнорируя директорию проекта Симптом: После обновления до PhpStorm 2025. 3 встроенный терминал WSL открывается в домашней директории. . .	Как объединить две одинаковые БД Access с разными данными VikBal 11.12.2025 Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.	Новый ноутбук volvo 07.12.2025 Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro	Музыка, написанная Искусственным Интеллектом volvo 04.12.2025 Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .	От async/await к виртуальным потокам в Python IndentationError 23.11.2025 Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .
Поиск "дружественных имён" СОМ портов Argus19 22.11.2025 Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .	Сколько Государство потратило денег на меня, обеспечивая инсулином. Programma_Boinc 20.11.2025 Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .	Ломающие изменения в C#.NStar Alpha Etyuhibosecyu 20.11.2025 Уже можно не только тестировать, но и пользоваться C#. NStar - писать оконные приложения, содержащие надписи, кнопки, текстовые поля и даже изображения, например, моя игра "Три в ряд" написана на этом. . .	Мысли в слух kumehtar 18.11.2025 Кстати, совсем недавно имел разговор на тему медитаций с людьми. И обнаружил, что они вообще не понимают что такое медитация и зачем она нужна. Самые базовые вещи. Для них это - когда просто люди. . .	Создание Single Page Application на фреймах krapotkin 16.11.2025 Статья исключительно для начинающих. Подходы оригинальностью не блещут. В век Веб все очень привыкли к дизайну Single-Page-Application . Быстренько разберем подход "на фреймах". Мы делаем одну. . .

@Esper 10 / 11 / 3 Регистрация: 20.03.2015 Сообщений: 104
	12.05.2015, 11:35
	Не по теме: Зачем городить while (dr.Read()){count += 1;} Может сразу в запросе количество вычислять? А что в таблице User? 0

@Krish 1 / 1 / 1 Регистрация: 17.04.2015 Сообщений: 114
	13.05.2015, 10:19 [ТС]
	В таблице User данные пользователей: FIO, Parol и Dostup(1 или 2). В идеале если у пользователя доступ 1, то при авторизации переход на форму 7, если 2 - то на форму 2. Добавлено через 19 часов 16 минут Сколько видео примеров не смотрел везде такой код работает, возможно ли это из-за того что при выборе источника данных я выбрал не "Файл базы данных MSSQL Server", а "MSSQL Server" и программа не может получить доступ к таблице? Добавлено через 57 минут А может быть дело в том что в таблице User не только FIO, Parol, но еще 2 поля. Неужели никто не знает... 0

@FroztBal 27 / 27 / 10 Регистрация: 18.08.2014 Сообщений: 212
	13.05.2015, 10:46
	Поставьте точку останова на строке 6 SqlDataReader dr; и посмотрите в переменных какая строка передается в качестве запроса, возможно ошибка там. а вообще рекомендую строки запроса формировать либо через string.Format или через параметры (тут вообще трудно накосячить) Ну и авторизацию тогда уж проводить логичнее так: если правильно ввел связку логин/пароль то пускает дальше если нет, остается на той же форме 0

@FroztBal 27 / 27 / 10 Регистрация: 18.08.2014 Сообщений: 212
	13.05.2015, 14:04
	оператор switch - case вам в помощь 0

@Krish 1 / 1 / 1 Регистрация: 17.04.2015 Сообщений: 114
	13.05.2015, 15:42 [ТС]
	wm_leviathan, к коду кнопки данные записи не подходят, что-то не совсем понятно... 0

@wm_leviathan 214 / 214 / 73 Регистрация: 12.01.2011 Сообщений: 767
	13.05.2015, 15:56
	getData э то метод он не пишется в коде кнопки а пишется в коде класса. 0

@Krish 1 / 1 / 1 Регистрация: 17.04.2015 Сообщений: 114
	13.05.2015, 17:08 [ТС]
	wm_leviathan, а что тогда в коде кнопки писать 0

@wm_leviathan 214 / 214 / 73 Регистрация: 12.01.2011 Сообщений: 767
	13.05.2015, 17:10
	switch(getData()) { 0

@Krish 1 / 1 / 1 Регистрация: 17.04.2015 Сообщений: 114
	14.05.2015, 09:09 [ТС]
	wm_leviathan, как буду за рабочим ПК попробую, спасибо за помощь. Добавлено через 15 часов 57 минут wm_leviathan, при любом исходе на логин форме нажатие кнопки выдает default 0

@wm_leviathan 214 / 214 / 73 Регистрация: 12.01.2011 Сообщений: 767
	14.05.2015, 09:12
	напишите поля базы, я мог "Dostup" неправильно написать. Я то писал так для примера, а не для копипаста. покажите полный код формы. 0

@wm_leviathan 214 / 214 / 73 Регистрация: 12.01.2011 Сообщений: 767
	14.05.2015, 09:28
	[Dostup] без скобок пишется 0

@Krish 1 / 1 / 1 Регистрация: 17.04.2015 Сообщений: 114
	14.05.2015, 09:29 [ТС]
	Я и так и так писал, просто сюда этот пример копернул 0

@wm_leviathan 214 / 214 / 73 Регистрация: 12.01.2011 Сообщений: 767
	14.05.2015, 09:30
	и в поле Dostup точно есть 1 или 2 ? Добавлено через 47 секунд и проверьте отладчиком table.Rows.Count возвращает ли строку ? может запрос ничего не возвращает? 0

Ошибка на форме авторизации Visual Studio 2010

Решение