По всем условиям проходит

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

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

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) { boolean a=false; // с + и скобками if (telNumber.length()==15) { if (telNumber.matches("\\+\\d+\\(\\d{3}\\)\\d+")) a=true; } // без + со скобками if (telNumber.length()==12) { if (telNumber.matches("\\d+\\(\\d{3}\\)\\d+") || telNumber.matches("\\(\\d{3}\\)\\d+")) a=true; } // с + без скобок if (telNumber.length()==13){ if (telNumber.matches("\\+\\d{12}")) a=true; } // без + без скобок if (telNumber.length()==10){ if (telNumber.matches("\\d{10}")){ a=true; } } return a; } public static void main(String[] args) { System.out.println(checkTelNumber("(0)501234567")); } }

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

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

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

Alexander

Уровень 7

19 марта 2023, 17:53

Спасибо большое, помогло! Нашел ошибку. Еще проверку на null запросил. И все прошло

Lexman (Алексей)Full Stack Developer в TrendSoft

19 марта 2023, 09:31

Map<String, Boolean> tests = new HashMap<>();
       // empty string and null
       tests.put("", false);
     //  tests.put(null, false);
       // characters
       tests.put("characters", false);
       tests.put("+abc123456789012", false);
       tests.put("050xxx4567", false);
       // without '+', less than 10 digits
       tests.put("12345678", false);
       tests.put("123456789", false);
       // without '+', more than 10 digits
       tests.put("12345678901", false);
       tests.put("123456789012", false);
       tests.put("1234567890123", false);
       // with '+', less than 12 digits
       tests.put("+1234567890", false);
       tests.put("+12345678901", false);
       // with '+', more than 12 digits
       tests.put("+1234567890123", false);
       tests.put("+12345678901234", false);
       // with '+' and parentheses, less than 12 digits
       tests.put("+12(345)67890", false);
       tests.put("+12(345)678901", false);
       // with '+' and parentheses, more than 12 digits
       tests.put("+12(345)67890123", false);
       tests.put("+12(345)678901234", false);
       // spaces
       tests.put(" +(380)501234567", false);
       tests.put("+(380)501234567 ", false);
       tests.put("+(380) 501234567", false);
       // parentheses
       tests.put("+38)050(1234567", false);
       tests.put("+38050(1234567", false);
       tests.put("+38)0501234567", false);
       tests.put("(380501234567)", false);
       tests.put("(0)501234567", false);
       tests.put("(05)01234567", false);
       tests.put("12(3456789)0", false);
       tests.put("+38(0501234567)", false);
       tests.put("+380501234(567)", false);
       tests.put("0501234(567)", false);

Lexman (Алексей)Full Stack Developer в TrendSoft

19 марта 2023, 09:31

// dashes
       tests.put("+38(050)123-45-67", false);
       // correct numbers
       tests.put("+380501234567", true);
       tests.put("+(380)501234567", true);
       tests.put("+3(805)01234567", true);
       tests.put("+38(050)1234567", true);
       tests.put("+380(501)234567", true);
       tests.put("+3805(012)34567", true);
       tests.put("+38050(123)4567", true);
       tests.put("+380501(234)567", true);
       tests.put("+3805012(345)67", true);
       tests.put("+38050123(456)7", true);
       tests.put("0501234567", true);
       tests.put("(050)1234567", true);
       tests.put("0(501)234567", true);
       tests.put("05(012)34567", true);
       tests.put("050(123)4567", true);
       tests.put("0501(234)567", true);
       tests.put("05012(345)67", true);
       tests.put("050123(456)7", true);

Lexman (Алексей)Full Stack Developer в TrendSoft

19 марта 2023, 09:32

String pattern = "%-23s%-20s%-13sYour result: %-10bMust be: %b";
       List<String> failedTests = new ArrayList<>();

       System.out.println("\n========== ALL TESTS ==========\n");
       for (Map.Entry<String, Boolean> phoneNumber : tests.entrySet()) {
           String action = "Checking phone number:";
           if (phoneNumber.getKey() == null) action = "Checking \"null\":";
           else if (phoneNumber.getKey().equals("")) action = "Checking empty string:";
           boolean myResult = checkTelNumber(phoneNumber.getKey());
           boolean desiredResult = phoneNumber.getValue();
           String testResult = myResult == desiredResult ? "✓ passed" : "✗ failed";
           System.out.println(String.format(pattern, action, phoneNumber.getKey(), testResult, myResult, desiredResult));
           if (myResult != desiredResult)
               failedTests.add(String.format("%-20s Must be: %b", phoneNumber.getKey(), desiredResult));
       }

       System.out.println("\n========== FAILED TESTS ==========\n");
       for (String phoneNumber : failedTests) System.out.println(phoneNumber);

Проверьте по этим тестам

Lexman (Алексей)Full Stack Developer в TrendSoft

19 марта 2023, 09:33

в моем случае регулярка была такая:

public static boolean checkTelNumber(String telNumber) {
       if(telNumber ==null)
           return false;
        Pattern p1 =Pattern.compile("^\\+(\\d[()]?){12}$|^([()]?\\d){10}$");
        Pattern p2 =Pattern.compile("^(\\+)?(\\d+)?(\\(\\d{3}\\))?\\d+$");
        Matcher m1 =p1.matcher(telNumber);
        Matcher m2 = p2.matcher(telNumber);

        return m1.matches() && m2.matches();
    }