Жалуется на break в условном операторе swich

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

Решен

Выдаёт ошибку: com/javarush/task/pro/task09/task0912/Solution.java:22: error: unreachable statement break; Подскажите пожалуйста в чём беда

В этой задаче тебе нужно выполнить проверку URL-адреса.

Простая схема URL-адреса выглядит так:
<сетевой протокол>://<название ресурса>.<домен>

Метод checkProtocol(String) проверяет, какой сетевой протокол (http или https) у URL-адреса, полученного входящим параметром, и возвращает результат проверки — строку название сетевого протокола. А метод checkDomain(String) проверяет, какой домен (com, net, org или ru) у URL-адреса, полученного входящим параметром, и возвращает результат проверки — строку название домена.

Если URL-адрес начинается не с http или https, то результат будет — "неизвестный". Если URL-адрес заканчивается не на com, net, org или ru, то результат будет — "неизвестный".

main не принимает участие в тестировании.

Требования:

Нужно, чтобы метод checkProtocol(String) был реализован согласно условию.
Нужно, чтобы метод checkDomain(String) был реализован согласно условию.

package com.javarush.task.pro.task09.task0912; /* Проверка URL-адреса */ public class Solution { public static void main(String[] args) { String[] urls = {"https://javarush.ru", "https://google.com", "http://wikipedia.org", "facebook.com", "https://instagram", "codegym.cc"}; for (String url : urls) { String protocol = checkProtocol(url); String domain = checkDomain(url); System.out.println("У URL-адреса - " + url + ", сетевой протокол - " + protocol + ", домен - " + domain); } } public static String checkProtocol(String url) { //напишите тут ваш код switch(url) { case "https": return "https"; break; case "http": return "http"; break; default: return "неизвестный"; break; } } public static String checkDomain(String url) { //напишите тут ваш код switch(url) { case "com": return "com"; break; case "net": return "net"; break; case "org": return "org"; break; case "ru": return "ru"; break; default: return "неизвестный"; break; } } }

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

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

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

Дмитрий Соснин

Уровень 40

Expert

9 февраля 2021, 20:32

полезный

это значит что этот участок кода ни при каких условиях не может быть выполнен, что обычно свидетельствует об ошибках где-то выше. А выше у тебя return, после которого как известно ничего уже не будет выполнено обычно это означает, что нужно или убрать недостижимый участок, или сделать так, чтобы до него доходило выполнение.

Олег А

Уровень 29

9 февраля 2021, 20:52

Спасибо, переписал код, теперь возвращает только значение default: "неизвестное". Не пойму в чём проблема

package com.javarush.task.pro.task09.task0912;

/*
Проверка URL-адреса
*/

public class Solution {
    public static void main(String[] args) {
        String[] urls = {"https://javarush.com", "https://google.com", "http://wikipedia.org", "facebook.com", "https://instagram", "codegym.cc"};
        for (String url : urls) {
            String protocol = checkProtocol(url);
            String domain = checkDomain(url);

            System.out.println("У URL-адреса - " + url + ", сетевой протокол - " + protocol + ", домен - " + domain);
        }
    }

    public static String checkProtocol(String url) {
        //напишите тут ваш код
        String urlStr;
        switch(url) {
            case "https": urlStr = "https";
                break;
            case "http": urlStr = "http";
                break;
            default: urlStr = "неизвестный";
                break;
        }
        return urlStr;
    }

    public static String checkDomain(String url) {
        //напишите тут ваш код
        String strUrl;
        switch(url) {
            case "com": strUrl = "com";
                break;
            case "net": strUrl = "net";
                break;
            case "org": strUrl = "org";
                break;
            case "ru": strUrl = "ru";
                break;
            default: strUrl = "неизвестный";
                break;
        }
        return strUrl;
    }
}

Дмитрий Соснин

Уровень 40

Expert

9 февраля 2021, 21:04

решение

эт потому что switch сравнивает свой аргумент с предложеными вариантами. т.е. проверяет верно ли равенство "https://javarush.com" == "https" и так со всеми вариантами что ты предлагаешь. И как можно ожидать все эти сранения дают в результате false

Александр Работает в безработный

20 февраля 2021, 07:43

полезный

тоже для потомков - используйте swich правильно

String str = url.substring((url.length()) -3, (url.length()));
	   switch(str)
	   {
		   case  ".ru" : return "ru";
		   case  "com" : return "com";
		   case  "net" : return "net";
		   case  "org" : return "org";
		   default : return "неизвестный";
	   }

Вероника Григорович

Уровень 35

28 апреля 2021, 07:06

Подскажите пожалуйста, а для чего тут -3, не понимаю

Александр Работает в безработный

28 апреля 2021, 08:11

потому ,что url.length()) -3 это начальный индекс адреса , а url.length() последний индекс адреса в итоге здесь

String str = url.substring((url.length()) -3, (url.length()));

нам вернётся имя домена , в даной задаче домены имеют не больше троёх символов , поэтому и отнимаем 3

Вероника Григорович

Уровень 35

28 апреля 2021, 09:08

спасибо