Сравнение с значением базы данных

Question

То, что я пытаюсь сделать, это захватить текущее имя пользователя и сравнить его с базой данных, которая содержит пользователей, а также включает флаг Active и флаг администратора. Я хочу сравнить текущего зарегистрированного пользователя в таблице tbl_Person и их соответствующих пользователей в таблице, чтобы увидеть, отмечены ли они как Active и Admin. Если оба они верны, они получают доступ к странице администратора. У меня есть до сих пор, что не работает. Некоторые из них я знаю, почему, а некоторые нет. Я думаю, что нахожусь на правильном пути, что, как я сказал, я уверен, что я делаю это неправильно. Я знаю, что вы используете ExecuteScalar(), чтобы возвращать что-то вместе с OUTPUT в строке запроса, но не могли заставить это работать. Другая вопиющая проблема заключается в том, что я пытаюсь вернуть целые числа, когда имя пользователя является строкой, а активные и админ-флаги - это Bools. Я знаю, что у меня только Актив, есть момент. Я пытался заставить это работать, прежде чем добавлять что-то еще.

Я прочитал, что с помощью ExecuteScalar вы можете анализировать и преобразовывать ToString, но это не сработало, и я нашел доказательства того, что это может быть неправильной задачей, но я действительно не уверен.

У меня есть несколько разных ошибок. Введите ошибки, недопустимый столбец, когда я попытался выполнить OUTPUT. С OUTPUT я пробовал как только OUTPUT, и потому, что я знаю, когда возвращаюсь после вставки, вы вставляете insert.name. Я попробовал selected.name как догадка, но это не сработало.

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

string userName = System.Security.Principal.WindowsIdentity.GetCurrent().Name; 

SqlConnection conn = new SqlConnection(ConfigurationManager.ConnectionStrings["HSEProjRegConnectionString1"].ConnectionString); 
conn.Open(); 
SqlCommand sqlUserName = new SqlCommand("SELECT [username] FROM [tbl_Person]", conn); 
SqlCommand sqlActive = new SqlCommand("SELECT [active] FROM [tbl_Person]", conn); 
int result1 = ((int)sqlUserName.ExecuteScalar()); 
int result2 = ((int)sqlActive.ExecuteScalar()); 

string userInfo = result1 + "." +result2; 
string userName = userName + "." +result2; 

if (userInfo == userName) 
{ 
    Woo, you have access. 
} 
else 
{ 
    Sorry, but no. 
} 

Запрос также не является окончательным. Как только он будет работать, я изменю его на параметризованный запрос.

Answer 1

Хорошо, рассмотрим следующий код:

string userName = System.Security.Principal.WindowsIdentity.GetCurrent().Name; 

using (SqlConnection conn = new SqlConnection(ConfigurationManager.ConnectionStrings["HSEProjRegConnectionString1"].ConnectionString)) 
{ 
    conn.Open(); 
    using (SqlCommand cmd = new SqlCommand("SELECT [active] FROM [tbl_Person] WHERE username = @username", conn)) 
    { 
     // since we can literally filter the results, if something comes back 
     // we know they are registered 
     cmd.Parameters.AddWithValue("@username", userName); 

     var res = cmd.ExecuteScalar(); 
     bool registeredAndActive = (bool)res; 

     // unless of course `[active]` is an INT -then do this 
     bool registeredAndActive = (int)res == 1 ? true : false; 

     // but really -set [active] up as a BIT if it's not **and** 
     // please make it non-nullable :D 
    } 
} 

Я уверен, что он делает то, что вы хотите. Но это также показывает некоторые лучшие практики, как:

1. Кредитное плечо using заявление для всех IDisposable объектов.
2. Отфильтруйте запрос как можно больше и совершите только одно путешествие в оба конца.