Проверка существующего пользователя при регистрации

@EvgenNews · Регистрация: 08.03.2017

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

Всем доброго времени. Пилю форму регистрации в БД а для этого проверяю по имени,фамилии и отчеству - есть ли пользователи с такими данными в базе. Но не смотря на то что таких юзеров уже >10 прога продолжает добавлять одни и те же данные. Вот код :

C#

  internal string createNewCustomer(string FN, string N, string ON, string Phone, string Mail)
        {
            string result;
            using (SqlConnection conn = new SqlConnection(connectionString))
            {
                SqlCommand select = new SqlCommand("select [customer_Name], [customer_FName], [customer_OName] from customers where [customer_Name] = @Name and [customer_FName] = @FName and [customer_OName] = @OName", conn);
                select.Parameters.AddWithValue("Name", N);
                select.Parameters.AddWithValue("FName", FN);
                select.Parameters.AddWithValue("OName", ON);
                conn.Open();
 
                using (SqlDataReader read = select.ExecuteReader())
                {
                    
                    if (read.HasRows)
                    {
                        result = "Человек с введенными данными уже есть в базе";
                    }
                    result = "Такого человека еще нет";
                }
            }

Наверняка какую то мелочь упустил. Помогите найти, спасибо!

@NewOrdered · 27.03.2017, 16:17

EvgenNews, строка 19 будет выполняться всегда

@EvgenNews · 27.03.2017, 16:23 **[ТС]**

Почему? ведь

C#

1	if (read.HasRows)

проверяет есть ли строки с значениями ФИО. а если строк нет то не должен выполняться. Или я что то не правильно понял?

@pincet · 27.03.2017, 16:25

открой для себя синтаксис if

@EvgenNews · 27.03.2017, 16:45 **[ТС]**

код

C#

1	read.HasRows

- если есть строки принимает значение true и делает то что в { } иначе не делает. При чем тут синтаксис if ?

Добавлено через 7 минут
при отладке : read свойство HasRows = false.

@NewOrdered · 27.03.2017, 16:49

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

При чем тут синтаксис if ?

есть ведь еще оператор else

@EvgenNews · 27.03.2017, 16:56 **[ТС]**

И? программа всегда дойдет до else если

C#

1	reader.HasRows = false.

Добавлено через 2 минуты
Что измениться если я так напишу ?

C#

if (read.HasRows)
 
                    {
                        result = "Человек с введенными данными уже есть в базе";
                    }
                    else
                    {
                        result = "Такого человека еще нет";
                    }

@NewOrdered · 27.03.2017, 17:03

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

при отладке : read свойство HasRows = false.

тем более, тогда и 17 строка не выполняется. Хотя это неважно, т.к. 19 строка выполняется безусловно.

Добавлено через 56 секунд

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

И? программа всегда дойдет до else если

где у вас else в исходной версии?

@EvgenNews · 27.03.2017, 17:06 **[ТС]**

C#

      using (SqlConnection conn = new SqlConnection(connectionString))
            {
                SqlCommand select = new SqlCommand( @"select * from customers where 
                                                [customer_Name] = @Name and [customer_FName] = @FName and [customer_OName] = @OName", conn);
 
                select.Parameters.AddWithValue("Name", N);
                select.Parameters.AddWithValue("FName", FN);
                select.Parameters.AddWithValue("OName", ON);
                conn.Open();
 
                using (SqlDataReader read = select.ExecuteReader())
                {
 
                    if (read.HasRows)
                    {
                        result = "Человек с введенными данными уже есть в базе";
                    }
                    else
                    {
                        result = "Такого человека еще нет";
                    }
                }
            }

Для наглядности

Добавлено через 52 секунды

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

тогда и 17 строка не выполняется

в том и проблема

@NewOrdered · 27.03.2017, 18:31

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

в том и проблема

Убедитесь, что
а) запись, которую вы ищете, существует
б) запрос, который вы выполняете, возвращает хотя бы одну строку. Это можно сделать извне вашей программы, например, в Management Studio

@EvgenNews · 27.03.2017, 18:36 **[ТС]**

SQL

1
2
3

SELECT * FROM customers WHERE
  [customer_Name] = 'Петров' AND [customer_FName] = 'Петр' AND [customer_OName] = 'Олегович'
  GO

такой запрос у меня в managment studio выдает 1 строку а в коде всегда 0. И при чем тут может быть синтаксис if ?

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

C#

 
select.Parameters.AddWithValue(@"Name", N);
select.Parameters.AddWithValue(@"FName", FN);
select.Parameters.AddWithValue(@"OName", ON);

Здесь N, FN, ON и есть петр,петров,олегович

@NewOrdered · 27.03.2017, 18:54

EvgenNews, посмотрите пример использования SqlDataReader
https://msdn.microsoft.com/ru-... .110).aspx
Мне кажется, вы забыли вызвать метод .Read()

@Igr_ok · 27.03.2017, 22:50

EvgenNews, самое простое объяснение - порядок фамилии, имени и отчества перепутан. Убедитесь, что в запросе нет чего-то типа [customer_Name] = 'Петр'.

@NewOrdered 471 / 234 / 73 Регистрация: 25.05.2012 Сообщений: 1,054 Записей в блоге: 1
	27.03.2017, 18:54	12
	EvgenNews, посмотрите пример использования SqlDataReader https://msdn.microsoft.com/ru-... .110).aspx Мне кажется, вы забыли вызвать метод `.Read()` 0

@Igr_ok 784 / 615 / 273 Регистрация: 04.08.2015 Сообщений: 1,707
	27.03.2017, 22:50	13
	EvgenNews, самое простое объяснение - порядок фамилии, имени и отчества перепутан. Убедитесь, что в запросе нет чего-то типа [customer_Name] = 'Петр'. 0

@NewOrdered 471 / 234 / 73 Регистрация: 25.05.2012 Сообщений: 1,054 Записей в блоге: 1
	27.03.2017, 16:17	2
	EvgenNews, строка 19 будет выполняться всегда 1

@pincet 1643 / 1144 / 171 Регистрация: 23.07.2010 Сообщений: 6,794
	27.03.2017, 16:25	4
	открой для себя синтаксис if 0

@NewOrdered 471 / 234 / 73 Регистрация: 25.05.2012 Сообщений: 1,054 Записей в блоге: 1
	27.03.2017, 16:49	6
	Сообщение от EvgenNews При чем тут синтаксис if ? есть ведь еще оператор else 0

@NewOrdered 471 / 234 / 73 Регистрация: 25.05.2012 Сообщений: 1,054 Записей в блоге: 1
	27.03.2017, 17:03	8
	Сообщение от EvgenNews при отладке : read свойство HasRows = false. тем более, тогда и 17 строка не выполняется. Хотя это неважно, т.к. 19 строка выполняется безусловно. Добавлено через 56 секунд Сообщение от EvgenNews И? программа всегда дойдет до else если где у вас `else` в исходной версии? 1

@NewOrdered 471 / 234 / 73 Регистрация: 25.05.2012 Сообщений: 1,054 Записей в блоге: 1
	27.03.2017, 18:31	10
	Сообщение от EvgenNews в том и проблема Убедитесь, что а) запись, которую вы ищете, существует б) запрос, который вы выполняете, возвращает хотя бы одну строку. Это можно сделать извне вашей программы, например, в Management Studio 1