JavaRush/Помощь по Java задачам/Не проходит валидацию

29 уровень

13.07.2018
2019просмотров
10комментариев

Не проходит валидацию

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

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

Архивный

Добрый день. Проверяю код - работает, но валидатор плюётся. Данные для проверки использую следующие:

HashMap<String, Boolean> phones = new HashMap<>();
       phones.put("+380501234567",true);
       phones.put("+38(050)1234567",true);
       phones.put("+38050123-45-67",true);
       phones.put("+38(050)1-23-45-6-7",false);
       phones.put("050ххх4567",false);
       phones.put("050123456",false);
       phones.put("(0)501234567",false);
       phones.put("050123-4567", true);
       phones.put("(380)501234567",false);
       phones.put("+38050123--4567",false);
       phones.put("050123--4567",false);
       phones.put("0-50123-4567",true);
       phones.put("+38)050(1234567",false);
       phones.put("050���4567",false);
       phones.put("050123456=false",false);
       phones.put("+38-(050)1234567",false);
       phones.put("+38((050)1234567",false);
       phones.put("+5(0--5)1234567",false);
       phones.put("1-23456789-0",true);
       phones.put("+38051202(345)-7",true);
       phones.put("+38051202(345)7",true);
       phones.put("(345)0512027",true);
       phones.put("+-313450531202",true);
       phones.put("+313450--531202",false);
       phones.put("38xx501A34567",false);
       phones.put("3805012345",true);
       phones.put("+38(0501234567",false);
       phones.put("+38(05)1234567",false);
       phones.put("(3)80501234567",false);
       phones.put("380-50123-45",true);
       phones.put("(380)501-234567",false);
       phones.put("(38-0)501234567",false);
       phones.put("380-(501)234567",false);
       phones.put("380(50-1)234567",false);
       phones.put("380(501)(23)4567",false);
       phones.put("+38050123(456)7",true);
       phones.put("+38050123(456)76",false);
       phones.put("+380501234(567)",false);
       phones.put("3-805-0123-45",false);
       phones.put("-3805-012345",false);
       phones.put("3805-012345-",false);
       phones.put("+38(05)01234567",false);
       phones.put("+38(0501)234567",false);
       phones.put("7-4-4123689-5",false);
       phones.put("+313450--531202Э",false);
       phones.put("++380501234567", false);
       phones.put("(111)111111111",false);
       phones.put("AB",false);
       phones.put("1AB1",false);
       phones.put("aB",false);
       phones.put("12345678910",false);
       phones.put("+38",false);
       phones.put("222222-2222",true);
       phones.put("(050)34(125)6-7",false);


       for (Map.Entry<String, Boolean> pair : phones.entrySet()) {
           if (checkTelNumber(pair.getKey()) != pair.getValue())
               System.out.println("ERROR:Should be:" + pair.getValue() + " checkTelNumber:" + checkTelNumber(pair.getKey()) + " " + pair.getKey());
       }

Метод 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.HashMap; import java.util.Map; /* Проверка номера телефона */ public class Solution { private static boolean checkTelNumber(String telNumber) { String temp = telNumber; int length = temp.replaceAll("\\D", "").length(); if (telNumber.contains("[a-aA-Z]")) {return false;} if (length==12) { return telNumber.matches("(^\\+{1})\\d*(\$\\d{3}\$)?\\d*(\\-?\\d+)?\\-?\\d*\\d$"); } else if (length==10) { return telNumber.matches("^(\\d|\$\\d{3}\$)\\d*(\\-?\\d+)?\\-?\\d*\\d$"); } return false; } public static void main(String[] args) { } }

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

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

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

Дмитрий

Уровень 32

25 января 2020, 16:07

Разобрался! Отличное решение!

Дмитрий

Уровень 32

25 января 2020, 16:01

Две строчки с регулярными выражениями и все! Супер! Я пол дня голову ломаю... Осталось разобраться с логикой проверки.

Алексей Клоков

Уровень 35

2 октября 2019, 07:02

Как тебя пропустил валидатор если оно не видит буквы в слове

if (telNumber.contains("[a-aA-Z]")) {return false;}

Александр Бойко

Уровень 29

22 октября 2019, 17:21

Сори за поздний ответ, давно не заходил на JavaRush. В задании мы отметали номера в которых присутствовали буквы. Сейчас что-то изменилось?

Alexander Avdoshin

Уровень 34

13 июля 2018, 22:05

Проверьте на null и пустую строку. В самое начало checkTelNumber:

if(telNumber == null || telNumber.isEmpty()){
            return false;
        }

Александр Бойко

Уровень 29

15 июля 2018, 09:39

добавил, ошибка та же. чего-то еще не хватает

Roman Java Developer

Expert

16 июля 2018, 10:55

решение

Сделай метод публичным.

Александр Бойко

Уровень 29

16 июля 2018, 11:13

а ларчик-то просто открывался) Спасибо большое!)

Артур Харатян

Уровень 41

5 февраля 2020, 12:09

Как так получается у многих , что изменяется модификатор доступа от изначального? Если меняют самостоятельно, то зачем?

Леонид Васильевич Ефимовfreelance в freelance

8 января 2021, 17:42

Очень просто: IDEA помечает модификатор серым цветом, и предлагает его поменять на public, так как по его мнению, это более корректно. Я и сам так же повёлся в одной задаче.