How can I reduce this code

if(acet_name.getText().length()==0&&acet_surname.getText().length()==0){ til_name.setErrorEnabled(true); til_surname.setErrorEnabled(true); til_name.setError("Обязательно для заполнения"); til_surname.setError("Обязательно для заполнения"); } else if(acet_name.getText().length()==0){ til_name.setErrorEnabled(true); til_name.setError("Обязательно для заполнения"); } else if(acet_surname.getText().length()==0){ til_surname.setErrorEnabled(true); til_surname.setError("Обязательно для заполнения"); } 

First, there is a check of both AppCompatEditText for emptiness, and then each separately. If you make a logical OR || , in any case, you need to register the check of which one is empty.

    1 answer 1

    1. Delete the first if completely.
    2. Delete all else .
     if(acet_name.getText().length() == 0) { til_name.setErrorEnabled(true); til_name.setError("Обязательно для заполнения"); } if(acet_surname.getText().length() == 0) { til_surname.setErrorEnabled(true); til_surname.setError("Обязательно для заполнения"); } 
    • Tochnyak, stupid)) - Flippy