Что не учла?

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

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

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; /* Проверка номера телефона */ public class Solution { public static boolean checkTelNumber(String telNumber) { if (telNumber.matches("\\d$") && telNumber.matches("\\d | \\+ |\$\$")) { if (telNumber.matches("^\\+") && telNumber.matches("[0-9]{12}") || (telNumber.matches("^\\d | \\^\$")) && telNumber.matches("[0-9]{10}")) { if (!(telNumber.matches("\\(\$"))) { return true; } else { return telNumber.matches("\\b\$\\d{3}\$"); } } else { return false; } } return false; } public static void main(String[] args) { } }

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

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

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

Стас Пасинков Software Developer в Zipy

Master

13 июля 2022, 07:53

попробуйте для начала убрать пробелы в регулярках

Ada

Уровень 46

13 июля 2022, 10:13

спасибо, уже разобралась получше с вопросом. одно осталось непонятным: скобки. возможно, вы знаете, как реализовать логику того, что скобки могут быть в любом месте выражения?

Стас Пасинков Software Developer в Zipy

Master

13 июля 2022, 13:16

полезный

нуу, судя по примерам - как-раз таки не в любом месте. а строго перед 7 и 10 цифрами с конца :) а вот валидацию того, что скобки корректно закрыты - джавовскими регулярками вроде невозможно сделать, сам движок не позволяет. в крайнем случае, я такое читал когда сам решал эту задачу. но судя по примерам кода ниже - тесты возможно не проверяют кейз когда одна скобка есть 0501)234567

Ada

Уровень 46

13 июля 2022, 19:46

мне кажется, это не все примеры) протестила на правильном решении: 0501)234567 возвращает false, а +38777(050)4567 возвращает true. в само решение не всматривалась, хочется как-то самой понять, как они этого добились) мне в связанном вопросе посоветовали цикл лекций, пойду еще там искать ответы, вдруг найдутся)

Стас Пасинков Software Developer в Zipy

Master

13 июля 2022, 22:05

ну я помню тоже что-то помаялся с регулярками, а потом добавил пару ифов и решение стало мега-простым. а написать одну огромную регулярку, которую никто не поймет и вообще будут бояться трогать ее в коде (это я про то, когда другие люди работают с вашим кодом) - не самое лучшее решение, как по мне) обычно, более простое решение - это более лучшее решение)

Виктор

Уровень 45

13 июля 2022, 06:42

полезный

Вот возможные regex для примера, у Вас лишние экранирования и неправильное форматирование

public static boolean checkTelNumber(String telNumber) {
    Pattern patternStartsWithPlusOrBracket = Pattern.compile(
            "(^[+](\\d{2}[(]\\d{3}[)]\\d{7}|\\d{12})|[(]\\d{3}[)]\\d{7})");
    Pattern patternStartsWithNumber = Pattern.compile("\\d{10}");
    Pattern patternStartsWithNumberIncludedBrackets = Pattern.compile(
             "[\\d|(|)]{11}[\\d]${1}");
    Pattern verifyNumberOfDigitsInsideBrackets = Pattern.compile(
             "\\d*[(][\\d]{3}[)]\\d+");
    Matcher matcher = patternStartsWithPlusOrBracket.matcher(telNumber);
    if(matcher.matches()) return true;
    matcher = patternStartsWithNumber.matcher(telNumber);
    if(matcher.matches()) return true;
    matcher = patternStartsWithNumberIncludedBrackets.matcher(telNumber);
    if(matcher.matches()){
        matcher = verifyNumberOfDigitsInsideBrackets.matcher(telNumber);
        if(matcher.matches()) return true;
    }
    return false;
}

Посмотрите логику проверки и regex, надеюсь пригодится )

+380501234567 true
+38(050)1234567 true
(050)1234567 true
0(501)234567 true
+38)050(1234567 false
+38(050)123-45-67 false
050xxx4567 false
050123456 false
(0)501234567 false

Ada

Уровень 46

13 июля 2022, 19:46

да, тут у меня совсем все сырое, спасибо, что направили)