Проверка на заполнение полей

@Huge · Регистрация: 26.01.2016

Author24 — интернет-сервис помощи студентам

Создаю регистрационную форму для ввода логина и пароля не могу понять как сделать проверку полей, что бы если они пустые(не заполнены), то писало, мол вы не заполнили поля.

Java

 public void LogPass(){
        try {
            Class.forName("com.mysql.jdbc.Driver");
            String str="insert into emplyeeinfo(name, password)"+ " values(?,?)";
            con = javaconectDB.ConectDB(); 
            prep = con.prepareStatement(str);
            prep.setString(1, name.getText());
 
            prep.setString(2, password.getText());
 
         //   prep.execute();
 
                }catch (Exception e){
            System.out.println("ERRORRRR!!!");
        }
//        if(name==null || password==null){
//            aError.setText("Use all fields!!!");
//
//        }
    }
   
    public void aNext(ActionEvent actionEvent) {
        if (name.getText()==null | password.getText()==null){
            aError.setText("Use all fields!!!");
            }
    else {
 
        LogPass();
 
        }}

aNext это действие при нажатии на кнопку далее
name, password - TextField

@Doctor_ · 14.03.2016, 21:38

Java

String username = name.getText();
String password = password.getText();
 
if (isNullOrEmpty(username) || isNullOrEmpty(password)){
JOptionPane.showMessageDialog(null, "Please provide username and password.");
}
....
private boolean isNullOrEmmpty(String str){
   return str == null || str.trim().lenght() == 0;
}

@Huge · 15.03.2016, 09:31 **[ТС]**

Огромное спасибо работает, я бы не додумался

Добавлено через 35 минут
Вопрос в "догонку", а как сделать проверку на телефон? т.е. если вводят цифры или - +, то все ок, а если символы, то ошибка?

Добавлено через 28 минут
почитал инет на ск-ко я понял это относится к регулярным выражениям, завтра буду лопатить

Добавлено через 10 часов 4 минуты

Java

 if (isAllEmpty(name) || isAllEmpty(Lname) || isAllEmpty(Pn)) {
            aMessDialog.setText("WOW problem. Not all fields are filled");
        }
        Pattern phone=Pattern.compile("[a-zA-Zа-яА-Я]+");
        Matcher matcher=phone.matcher(Pn);
 
            while (matcher.find()) {
                    aMessDialog.setText("Number contain words!");
            }
        Pattern mail=Pattern.compile("@+");
        Matcher mE=mail.matcher(Ename);
 
        while (!mE.find()) {
            aMessDialog.setText("Enter Email realy !!!");
        }

Как эти условия заставить работать?
Если я все положу в один if все работает, но если я хочу сказать, что в номере телефона есть буквы, а в емайле ошибка, нет @, все висит

@Doctor_ · 15.03.2016, 09:38

Это как вам ума хватило в while вставить. Бесконечный цикл.

@Huge · 15.03.2016, 09:50 **[ТС]**

я за утро и while и if и все в один лил все равно ((, да условие для емайла не рабочее

Добавлено через 7 минут
если я все лью в один if с разными условиями, то месседж пишет только последний минуя все предыдущие

@Doctor_ · 15.03.2016, 10:02

Зачем всё в один IF?

Добавлено через 3 минуты
Почему нельзя сделать что-то типа:

Java

public static void main(String[] args) {
        String name = "name";
        String password = "password";
        String phone = "37120000000";
        String email = "zhiznj@tochka.net";
 
        if (isNullOrEmpty(name) || isNullOrEmpty(password)
                || isNullOrEmpty(phone) || isNullOrEmpty(email)) {
            System.out.println("Please fill all fields.");
            return;
        }
        if (validatePattern(phone, Pattern.compile("[0-9]+"))) {
            System.out.println("Phone is not valid.");
            return;
        }
 
        if (validatePattern(email, Pattern.compile("^[_A-Za-z0-9-\\+]+(\\.[_A-Za-z0-9-]+)*@[A-Za-z0-9-]+(\\.[A-Za-z0-9]+)*(\\.[A-Za-z]{2,})$"))) {
            System.out.println("Email is not valid.");
            return;
        }
    }
 
    public static boolean isNullOrEmpty(String str) {
        return str == null || str.trim().length() == 0;
    }
 
    public static boolean validatePattern(String str, Pattern pattern) {
        return !pattern.matcher(str).matches();
    }

@Huge · 15.03.2016, 10:05 **[ТС]**

Java

   if (isAllEmpty(name) || isAllEmpty(Lname) ) {
            aMessDialog.setText("WOW problem. Not all fields are filled");
                   
        if (isAllEmpty(Pn)||matcher.find()) {
            aMessDialog.setText("Number contain words!");
 
        if (isAllEmpty(Ename)||!mE.find()){
            aMessDialog.setText("Enter Email realy !!!");}}

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

Добавлено через 1 минуту
у меня в else запуск добавления в БД, т.е. если все поля заполнены верно оно добавляет результат в БД

@Doctor_ · 15.03.2016, 10:07

Я повторяю вопрос. Зачем вы всё в этот иф вставляепте? Он туда входит елси поля пустые. Делайте как я написал выше. Пошаговая валидация. Что-то не проваледировалось, сразу выходить, зачем проверять что-то дальше, если в начале что-то уже не так?

Добавлено через 1 минуту
Зачем в else? Боги мой. Читайте что я написал. В ДБ добавлять только в том случае, если все валидации прошли и всё. Без каких либо else.

@Huge · 15.03.2016, 11:45 **[ТС]**

Спасибо оказывается мой код тоже работал, но нужно было soutы прописать, тогда бы я понял где ошибаюсь.
Еще раз спс огромное

Добавлено через 2 минуты
В Вашем коде не понял

{2,}

Добавлено через 3 минуты
разобрался не менее 2-х раз

Добавлено через 1 час 20 минут
ну и, надеюсь, последний вопрос почему ELSE в этом коде ловит только последний IF?

Java

    public void aNext(ActionEvent actionEvent) throws SQLException {
        String sql = "select*from emplyeeinfo where name=?";
 
        prep = con.prepareStatement(sql);
        prep.setString(1, name.getText());
        ResultSet rs = prep.executeQuery();
 
        String username = name.getText();
        String pass = password.getText();
        if (isNullOrEmpty(username) || isNullOrEmpty(pass)) {
            aError.setText("Use all fields!!!");
        }
        if (rs.next()) {
            aError.setText("Login exist, pls change Name");
        }
        else {
            LogPass();
        CancelBtn(actionEvent);}
    }

@Doctor_ · 15.03.2016, 12:02

Потому что else есть только у посленего IF'a.
Третий раз спрашиваю. Зачем вы используете ELSE? Почему не сделать как я уже два раза писал? Если валидация не прошла, просто выходить из метода....

@Huge · 15.03.2016, 22:59 **[ТС]**

да согласен, утро было, тупил трошки, сделал "комби", часть Вашего, часть своего оставил
все работает.

Проверка на заполнение полей

Решение

Решение

@Doctor_ 238 / 237 / 142 Регистрация: 03.02.2011 Сообщений: 1,437
	15.03.2016, 09:38	4
	Это как вам ума хватило в while вставить. Бесконечный цикл. 0

@Huge 3 / 3 / 14 Регистрация: 26.01.2016 Сообщений: 481
	15.03.2016, 09:50 [ТС]	5
	я за утро и while и if и все в один лил все равно ((, да условие для емайла не рабочее Добавлено через 7 минут если я все лью в один if с разными условиями, то месседж пишет только последний минуя все предыдущие 0

@Doctor_ 238 / 237 / 142 Регистрация: 03.02.2011 Сообщений: 1,437
	15.03.2016, 10:07	8
	Сообщение было отмечено Huge как решение Решение Я повторяю вопрос. Зачем вы всё в этот иф вставляепте? Он туда входит елси поля пустые. Делайте как я написал выше. Пошаговая валидация. Что-то не проваледировалось, сразу выходить, зачем проверять что-то дальше, если в начале что-то уже не так? Добавлено через 1 минуту Зачем в else? Боги мой. Читайте что я написал. В ДБ добавлять только в том случае, если все валидации прошли и всё. Без каких либо else. 1

@Doctor_ 238 / 237 / 142 Регистрация: 03.02.2011 Сообщений: 1,437
	15.03.2016, 12:02	10
	Потому что else есть только у посленего IF'a. Третий раз спрашиваю. Зачем вы используете ELSE? Почему не сделать как я уже два раза писал? Если валидация не прошла, просто выходить из метода.... 0

@Huge 3 / 3 / 14 Регистрация: 26.01.2016 Сообщений: 481
	15.03.2016, 22:59 [ТС]	11
	да согласен, утро было, тупил трошки, сделал "комби", часть Вашего, часть своего оставил все работает. 0