1
string Spl_Chars = "$%^&*";
if (pas.IndexOf(Spl_Chars) > 0)
{
    ErrorFlag_5 = true;
    MessageBox.Show("Вы ввели недопустимые символы.", "Ошибка");
}
else
    MessageBox.Show("Вы ввели непопустимые символы.", "Ошибка");

pas - то TextBox, куда пользователь вбивает пароль.

У меня должен стоять запрет на ввод символов, указанных в Spl_Chars, но ничего не работает. Если ставить проблем между символами - это помогает, но в таком случае он считает пробел, как недопустимый символ, а этого быть не должно

2
  • Пожалуйста, приведите код текстом, а не картинкой 15 мар 2019 в 9:00
  • @АндрейNOP правил)
    – aquaprogit
    15 мар 2019 в 9:04

3 ответа 3

2

Ваш код работает не так как вы ожидаете, потому что метод string.IndexOf возвращает индекс вхождения подстроки в строку, т. е. он вернет не -1 только в том случае, если в проверяемой строке есть целиком подстрока $%^&*, а не какой-то символ из нее.

Исправить можно, например, так:

if (pas.Intersect(Spl_Char).Any())

Linq-операция Intersect возвращает пересечение множеств. В данном случае строки считаем множествами из символов. Если пересечение не пустое — значит введены какие-то запретные символы.

Если это место когда-то станет узким (в чем я сомневаюсь), можно будет сохранить набор запрещенных символов в HashSet (один раз) и переписать проверку как-то так:

var splCharSet = new HashSet<char>(Spl_Char);
...
if (pas.Any(c => splCharSet.Contains(c)))
    ...
0

Если не использовать Linq, то ваш пример можно модифицировать, например, так:

ErrorFlag_5 = false;
string Spl_Chars = "$%^&*";

foreach (var c in Spl_Chars)
{
    if (pas.IndexOf(c) > 0)
    {
        ErrorFlag_5 = true;
        MessageBox.Show("Вы ввели недопустимые символы.", "Ошибка");
        break;
    }
}

if (!ErrorFlag_5)
    MessageBox.Show("В пароле нет недопустимых символов", "ОК");
0
private static bool ValidPass(string password, string notAllowedChar)
{
   return !string.IsNullOrEmpty(password) && !Regex.IsMatch(password, notAllowedChar);
}

Я использовал это.

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.