Проверка или псевдоним уже существует в списке <User>

Question

Я создаю веб-сайт ASP с возможностью регистрации.

Прозвище, которое посетители выбирают для регистрации, должно быть уникальным.

Everytime когда пользователь регистрируется, я выбираю все пользователь из базы данных, а затем я использую цикл Еогеаспа для проверки или имя пользователя уже существует:

private List<User> listExistingUsers; 
listExistingUsers = Database.GetUsers(); 
foreach (User u in listExistingUsers) 
{ 
    if (u.Nickname == txtNickname.text) 
    { 
     Error = "Username already in use."; 
    } 
} 

Но приведенный выше код не работает должным образом. Он не проверяет все элементы в списке, которые считываются из базы данных. Таким образом, возможно иметь пользователей с теми же именами пользователей, которых я не хочу.

Что делать, чтобы решить эту проблему? Я читал о LINQ, но считаю, что это неверный способ проверить имена пользователей со списком <> по-моему. Я думаю, что это имя пользователя-проверки должно быть выполнено по-другому.

Вы можете мне помочь? Я также мог бы выполнить эту проверку с помощью SQL-запроса, но я хотел бы сделать это в C#.

Answer 1

Вместо возврата всех пользователей из БД, передать имя пользователя для запроса/хранимой процедуры, и пусть бэкенд сделать проверьте, а затем верните только флаг состояния 1/0 - существует/нет.

Answer 2

Вы можете использовать Contains operator в порядке tocheck

listExistingUsers.Select(x => x.Nickname).Contains(txtNickname.text); 

ссылка: http://msdn.microsoft.com/fr-fr/library/bhkz42b3%28v=vs.80%29.aspx

Примечание: Вы можете использовать Any или count (очень дорого последнее решение)

Answer 3

if (Database.GetUsers().Select(x => x.Nickname).Contains(txtNickname.Text)) должны делать то, что вы хотите.

Я сгустил все в одну строку, поэтому дам краткое объяснение; Сначала я использую ваш метод Database.GetUsers() для извлечения пользователей, затем я использую select для проецирования Nickname, поскольку это то, что мы сравниваем. Если бы это было выполнено самостоятельно, это привело бы к IEnumerable<string> со всеми прозвищами. Оттуда я использую contains, чтобы увидеть, содержит ли этот список псевдоним, который (я предполагаю) был введен в пользовательский интерфейс.

Answer 4

Использование Any Оператор. Он проверяет, удовлетворяет ли какой-либо элемент последовательности некоторому условию. В вашем случае условие является пользователь ник составляет текст в TextBox:

if (Database.GetUsers().Any(u => u.Nickname == txtNickname.Text)) 
    Error = "Username already in use."; 

BTW, если вы измените GetUsers вернуть IQueryable<User> затем проверить будет происходить на стороне сервера.

Answer 5

ли получить список прозвищ раз

var nickNames = new List<string>(); 

for(int i=0;i<listExistingUsers.Count;i++) 
{ 
    nickNames.Add(listExistingUsers.NickName); 
} 

Тогда и можно просто использовать

if(nickNames.Contains(txtNickname.text)) 
{ 
    Error = "Username already in use."; 
} 

Answer 6

1) Вы подтвердили, что Database.GetUsers() фактически возвращает полный список без проблем SQL?

2) Вам нужно, чтобы это было нечувствительным к регистру?

3) Вы можете использовать LINQ, чтобы сделать запрос, как это:

if (listExistingUsers.Any(u => string.Equals(u, txtNickname.Text, StringComparison.CurrentCultureIgnoreCase))) 
{ 
    // Process error 
} 

Answer 7

Если Database.GetUsers() возвращает все пользователи из базы данных, поэтому не использовать его! Представьте, что у вас уже 1000 пользователей, для каждого нового пользователя он загрузит всех пользователей, и у вас будут проблемы с производительностью.

Вместо этого создайте новый метод, который ищет вашу базу данных и возвращает только один результат, если он существует. Что-то вроде:

private bool Database.UserExists(txtNickname.text) { 
     //Your query to database with a where statment looking for the nickname. It could be a LINQ query, or any other way you use in your system. 
     //If it brings 1 result, it has to return true. 
} 

Answer 8

Я думаю, что самая сложная часть вашей задачи, чтобы заполнить базу данных правильно.

В частности:

• Отрезать отставая и заканчивая пространства
• Решите, если имена пользователей должны Becase чувствительны
• Убедитесь, что при создании нового имени пользователя не имеет ника уже

О Загрузка пользователей и проверки:

Как уже упоминалось выше LINQ является наиболее эффективным С # как проверка дубликатов (если (Database.GetUsers(). Выберите (х => x.Nickname) .Contains (txtNickname.Text)))

Я Я больше привык писать записи SQL, чем использовать LINQ. Если у вас много пользователей, SQL будет читать только выбранные, но я не знаю, будет ли приведенный выше оператор LINQ вытаскивать всех пользователей в пул памяти или только с одним и тем же псевдонимом.