Валидатор не принимает, что думаете?

Вопрос по задаче

Проверка номера телефона

Java Multithreading, 2 уровень, 13 лекция

Решен

Проверил по условиям, применил разные варианты, вроде должно всё подходить

Метод checkTelNumber должен проверять, является ли аргумент telNumber валидным номером телефона.

Критерии валидности:
1) если номер начинается с '+', то он содержит 12 цифр
2) если номер начинается с цифры или открывающей скобки, то он содержит 10 цифр
3) может содержать 1 пару скобок '(' и ')'
4) скобки (если они есть) внутри содержат четко 3 цифры
5) номер может содержать только цифры, '+', '(' и ')'
6) номер заканчивается на цифру

Примеры:
+380501234567 - true +38(050)1234567 - true (050)1234567 - true 0(501)234567 - true +38)050(1234567 - false +38(050)123-45-67 - false 050ххх4567 - false 050123456 - false (0)501234567 - false

Требования:

Метод checkTelNumber должен возвращать значение типа boolean.
Метод checkTelNumber должен быть публичным.
Метод checkTelNumber должен принимать один параметр типа String.
Метод checkTelNumber должен корректно проверять валидность номера телефона переданного ему в качестве параметра.

package com.javarush.task.task22.task2212; /* Проверка номера телефона */ import java.util.regex.Matcher; import java.util.regex.Pattern; public class Solution { public static boolean checkTelNumber(String telNumber) { if (telNumber.charAt(0) == '+' && telNumber.contains("(") && telNumber.contains(")") && telNumber.length() - 3 != 12) { return false; } else if (telNumber.charAt(0) == '+' && !(telNumber.contains("(") || telNumber.contains(")")) && telNumber.length() - 1 != 12) { return false; } else if (telNumber.contains("(") && telNumber.contains(")") && !(telNumber.contains("+")) && telNumber.length() - 2 != 10) { return false; } else if (!(telNumber.contains("+")) && !(telNumber.contains("(") && telNumber.contains(")")) && telNumber.length() != 10) { return false; } if (telNumber.indexOf('(') > telNumber.indexOf(')')) { return false; } Pattern pattern = Pattern.compile("[^0-9+()]+"); Matcher matcher = pattern.matcher(telNumber); if (matcher.find()) { return false; } for (int i = 0; i < telNumber.length(); i++) { if (telNumber.charAt(i) == '(' && telNumber.charAt(i + 4) == ')') { break; } else if (telNumber.charAt(i) == '(') { return false; } } return true; } public static void main(String[] args) { /* +380501234567 - true +38(050)1234567 - true (050)1234567 - true 0(501)234567 - true +38)050(1234567 - false + +38(050)123-45-67 - false + 050ххх4567 - false 050123456 - false (0)501234567 - false */ String test = "+380501234567"; System.out.println(checkTelNumber(test)); } }

Комментарии (5)

популярные
новые
старые

Для того, чтобы оставить комментарий Вы должны авторизоваться

zxc17

Уровень 41

15 марта, 13:46

полезный

"(123)4(567)8" "(123)4567)89" и вот такое тоже не должен пропускать.

zxc17

Уровень 41

15 марта, 13:05

полезный

"123456789+" вот такой номер проверится некорректно.

Islam Yunusov

Уровень 31

16 марта, 08:57

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

Islam Yunusov

Уровень 31

16 марта, 08:59

решение

Итоговое решение:

public static boolean checkTelNumber(String telNumber) {
        if (telNumber == null || telNumber.equals("")) {
            return false;
        }
        if (telNumber.charAt(0) == '+' && telNumber.contains("(") && telNumber.contains(")") && telNumber.length() - 3 != 12) {
            return false;
        } else if (telNumber.charAt(0) == '+' && !(telNumber.contains("(") || telNumber.contains(")")) && telNumber.length() - 1 != 12) {
            return false;
        } else if (telNumber.contains("(") && telNumber.contains(")") && !(telNumber.charAt(0) == '+') && telNumber.length() - 2 != 10) {
            return false;
        } else if (!(telNumber.charAt(0) == '+') && !(telNumber.contains("(") && telNumber.contains(")")) && telNumber.length() != 10) {
            return false;
        } else if (telNumber.contains("+") && telNumber.charAt(0) != '+') {
            return false;
        } else if (telNumber.charAt(telNumber.length() - 1) == ')') {
            return false;
        }

Islam Yunusov

Уровень 31

16 марта, 08:59

int counter1 = 0;
        int counter2 = 0;
        for (int i = 0; i < telNumber.length(); i++) {
            if (telNumber.charAt(i) == '(') {
                counter1++;
            } else if (telNumber.charAt(i) == ')') {
                counter2++;
            }
        }
        if (counter1 != counter2) {
            return false;
        } else if (counter1 > 1) {
            return false;
        }

        if (telNumber.indexOf('(') > telNumber.indexOf(')')) {
            return false;
        }

        Pattern pattern = Pattern.compile("[^0-9+()]+");
        Matcher matcher = pattern.matcher(telNumber);
        if (matcher.find()) {
            return false;
        }
        for (int i = 0; i < telNumber.length(); i++) {
            if (telNumber.charAt(i) == '(' && telNumber.charAt(i + 4) == ')') {
                break;
            } else if (telNumber.charAt(i) == '(') {
                return false;
            }

        }
        return true;
    }