1

Есть два EditText и кнопка отправки с методом onClick().
При нажатии на кнопку в методе срабатывает код отправки данных из полей на сервер.
Нужно проверить количество символов на минимально допустимое значение (6 символов). И проверить что оба поля EditText не пустые.

Вот мой код:

user = (EditText) findViewById(R.id.user);
password = (EditText) findViewById(R.id.password);

l_user = user.getText().toString();
p_user = password.getText().toString();

if (l_user == "") {
    setContentView(R.layout.act_login);
}
else {
    if (p_user == "") {
        setContentView(R.layout.act_login);
    }
    else {
        // код отправки данных из форм на сервер

Этот способ проверки на заполнение почему-то работает некорректно. Если пусты оба поля, то все работает, а если в одно из полей ввести букву, то скрипт начинает отправлять запрос. А ведь не должен, ибо второе поле-то пустое!
По коду видно, что если хотя бы одно из полей пустое, грузится экран с той же формой регистрации, но, как я сказал, проверка плохо работает.

Как правильно сделать проверку на количество символов, введенных в поле EditText?

9
  • 4
  • @eugeneek однако при этом автор утверждает, что если оба поля пустые, то запрос не уходит. И вот это кажется странным.
    – Regent
    12 июл 2017 в 13:59
  • зачем дважды setContentView(R.layout.act_login);?
    – iFr0z
    12 июл 2017 в 14:00
  • @iFr0z а зачем строки сравнивать через == без веских на то причин?
    – Regent
    12 июл 2017 в 14:02
  • а в вашем случае "".equals(l_user), что проверит на пустоту вернет false, даже когда l_user == null.
    – Lex Hobbit
    12 июл 2017 в 14:04

4 ответа 4

8

Вам не нужно сравнивать строки, проще проверять количество символов в строке. Как то так:

if (l_user.length() > 5 && p_user.length() > 5) {
   // код отправки данных из форм на сервер
}
1
  • Спасибо, роботает!
    – Вадим
    12 июл 2017 в 14:58
6

А зачем вообще преобразовывать в строки? Все и так отлично работает:

if (user.getText().length() == 0 || password.getText().length() == 0) {
  setContentView(R.layout.act_login);
} else {
  // код отправки данных из форм на сервер
}

Меньше кода, легче читать.

7
  • про ограничения длинны укажите еще
    – Lex Hobbit
    12 июл 2017 в 14:16
  • на минимально допустимое значение (6 символов)
    – Lex Hobbit
    12 июл 2017 в 14:18
  • @LexHobbit так вроде так сейчас в ответе и есть? Слово "длина", кстати, с одной "н".
    – Regent
    12 июл 2017 в 14:20
  • Из вопроса не совсем ясно что проверять на минимум 6 символов. 12 июл 2017 в 14:21
  • @Regent все верно =) все пора отдыхать...
    – Lex Hobbit
    12 июл 2017 в 14:22
1

Проверять строки на пустоту следует следующим образом:

if(l_user.equals(""))
{
    //строка пуста
}

Вот вам чтиво на эту тему: http://developer.alexanderklimov.ru/android/java/string.php#equals

1

Я зделал так чтобы не пропускало только пробелы

String text = edittext.getText().toString();
   String[] arr = text.split(""); 
   Boolean isEmpty = true;

   for (String arg: arr){ // проверяем посимвольно
       if(arg.equals(""))
           isEmpty = true; // в массиве первый елемент всегда "", я не нашел для себя способа его удалить.
       else if(!arg.equals(" ")){
           isEmpty = false;
       }
   }
   if (isEmpty || text.equals("")){
       Toast.makeText(this, "Поле пустое", Toast.LENGTH_SHORT).show(); 
   }
   else 
       // код дальше

Думаю поможет)

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими условиями использования и подтверждаете, что прочитали и поняли наши политику конфиденциальности и нормы поведения.

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