Не все ветви кода возвращают значение при работе с БД

@StereoRu · Регистрация: 13.12.2011

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

Помогите пожалуйста новичку!!!!
Никак не пойму в чем проблема.
Ошибка: Не все ветви кода возвращают значение.

ASP .NET C#

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
 protected bool Exist_User(string _UserName, string _UserPassword, out int _UserID)
    {
        string connectionString = WebConfigurationManager.ConnectionStrings["mySite"].ConnectionString;
        SqlConnection con = new SqlConnection(connectionString);
        con.Open();
 
        string sql = "SELECT [UserID],[Login],[Password]" +
         "FROM [mySite].[dbo].[Users]";
        SqlCommand command = new SqlCommand(sql, con);
        SqlDataReader reader = command.ExecuteReader();
 
        if (reader.Read())
        {
            while (reader.Read())
            {
                if (_UserName == reader.GetString(1) && _UserPassword == reader.GetString(2))
                {
                    _UserID = reader.GetInt32(0);
                    reader.Close();
                    con.Close();
                    return true;
                }
                else
                {
                    _UserID = 0;    //UserId не может =0, т.к. это ограничение забито в MS SQL Managem Studio
                    reader.Close();
                    con.Close();
                    return false;
                }
            }
        }
        else //если таблица Users пустая
        {
            _UserID = 0;    //UserId не может =0, т.к. это ограничение забито в MS SQL Managem Studio
            reader.Close();
            con.Close();
            return false; 
        }
    }

Psilon · 05.11.2012, 00:39

StereoRu, в конец запишите

C#
1
return true; //или false, на ваш выбор

Добавлено через 3 минуты
StereoRu, а вообще бредометод

Добавлено через 5 минут
Правильный вариант:

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
protected bool Exist_User(string _UserName, string _UserPassword, out int _UserID)
    {
        _UserID = 0;
        string connectionString = WebConfigurationManager.ConnectionStrings["mySite"].ConnectionString;
        using (SqlConnection con = new SqlConnection(connectionString))
        {      
        string sql = "SELECT [UserID],[Login],[Password]" +
         "FROM [mySite].[dbo].[Users]";
        SqlCommand command = new SqlCommand(sql, con);
        SqlDataReader reader = command.ExecuteReader(); 
            while (reader.Read())
            {
                if (_UserName == reader.GetString(1) && _UserPassword == reader.GetString(2))
                {
                    _UserID = reader.GetInt32(0);
                    return true;
                }
            }
        }
            return false; 
        }
        }
    }

@StereoRu · 05.11.2012, 00:59 **[ТС]**

Не увидел что можно ТАК упростить!
Спасибо большое!!!

Psilon · 05.11.2012, 18:13

StereoRu, только последние 2 скобки лишние)

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
        protected bool Exist_User(string _UserName, string _UserPassword, out int _UserID)
        {
            _UserID = 0;
            string connectionString = WebConfigurationManager.ConnectionStrings["mySite"].ConnectionString;
            using (SqlConnection con = new SqlConnection(connectionString))
            {
                string sql = "SELECT [UserID],[Login],[Password]" +
                             "FROM [mySite].[dbo].[Users]";
                SqlCommand command = new SqlCommand(sql, con);
                SqlDataReader reader = command.ExecuteReader();
                while (reader.Read())
                    if (_UserName == reader.GetString(1) && _UserPassword == reader.GetString(2))
                    {
                        _UserID = reader.GetInt32(0);
                        return true;
                    }
            }
            return false;
        }

А вообще имхо SELECT * не лучший запрос. Мб стоило спросить

C#
1
2
3
4
                string sql = "SELECT [UserID],[Login],[Password]" +
                             "FROM [mySite].[dbo].[Users]" +
                             string.Fromat("WHERE UsedID = {0}", _UserID) +
                             string.Fromat("WHERE Password = {0}", _UserPassword);

что-нибудь в таком роде(к сожалению с sql не очень дружу, в основном с linq работал)

Добавлено через 17 часов 9 минут

C#
1
2
3
string s = "SELECT [UserID],[Login],[Password]" +
           "FROM [mySite].[dbo].[Users]" +
           string.Fromat("WHERE UsedID = {0} AND Password = {1}", _UserID,_UserPassword);

Так не получится?

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Web (WebAssembly): Обработчик клика мыши в браузере ПК и касания экрана в браузере на мобильном устройстве 8Observer8 02.02.2026 Содержание блога Для начала пошагово создадим рабочий пример для подготовки к экспериментам в браузере ПК и в браузере мобильного устройства. Потом напишем обработчик клика мыши и обработчик. . .	Философия технологии iceja 01.02.2026 На мой взгляд у человека в технических проектах остается роль генерального директора. Все остальное нейронки делают уже лучше человека. Они не могут нести предпринимательские риски, не могут. . .	SDL3 для Web (WebAssembly): Вывод текста со шрифтом TTF с помощью SDL3_ttf 8Observer8 01.02.2026 Содержание блога В этой пошаговой инструкции создадим с нуля веб-приложение, которое выводит текст в окне браузера. Запустим на Android на локальном сервере. Загрузим Release на бесплатный. . .	SDL3 для Web (WebAssembly): Сборка C/C++ проекта из консоли 8Observer8 30.01.2026 Содержание блога Если вы откроете примеры для начинающих на официальном репозитории SDL3 в папке: examples, то вы увидите, что все примеры используют следующие четыре обязательные функции, а. . .
SDL3 для Web (WebAssembly): Установка Emscripten SDK (emsdk) и CMake для сборки C и C++ приложений в Wasm 8Observer8 30.01.2026 Содержание блога Для того чтобы скачать Emscripten SDK (emsdk) необходимо сначало скачать и уставить Git: Install for Windows. Следуйте стандартной процедуре установки Git через установщик. . . .	SDL3 для Android: Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 29.01.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами. Версия v3 была полностью переписана на Си, в. . .	Инструменты COM: Сохранение данный из VARIANT в файл и загрузка из файла в VARIANT bedvit 28.01.2026 Сохранение базовых типов COM и массивов (одномерных или двухмерных) любой вложенности (деревья) в файл, с возможностью выбора алгоритмов сжатия и шифрования. Часть библиотеки BedvitCOM Использованы. . .	SDL3 для Android: Загрузка PNG с альфа-каналом с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 28.01.2026 Содержание блога SDL3 имеет собственные средства для загрузки и отображения PNG-файлов с альфа-каналом и базовой работы с ними. В этой инструкции используется функция SDL_LoadPNG(), которая. . .

@StereoRu 0 / 0 / 0 Регистрация: 13.12.2011 Сообщений: 27
	05.11.2012, 00:59 [ТС]
	Не увидел что можно ТАК упростить! Спасибо большое!!! 0