Не корректная работа с Map

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

Обсуждается

Ради интереса решил решить задачу при помощи Map. Не до конца понимаю в чем ошибка, прогонял в айдешке все работает, но тут не проходит чек домена.

В этой задаче тебе нужно выполнить проверку 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; import java.util.HashMap; import java.util.Map; /* Проверка 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) { //напишите тут ваш код Map<Integer, String> protocols = new HashMap<>(); protocols.put(url.indexOf("http"), "http"); protocols.put(url.indexOf("https"), "https"); for (Map.Entry<Integer, String> protocol: protocols.entrySet()) { if (protocol.getKey() > -1) { return protocol.getValue(); } } return "неизвестный"; } public static String checkDomain(String url) { //напишите тут ваш код Map<Integer, String> domains = new HashMap<>(); domains.put(url.indexOf("com"), "com"); domains.put(url.indexOf("net"), "net"); domains.put(url.indexOf("org"), "org"); domains.put(url.indexOf("ru"), "ru"); for (Map.Entry<Integer, String> domain : domains.entrySet()) { if (domain.getKey() > -1) { return domain.getValue(); } } return "неизвестный"; } }

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

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

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

hidden #3303140

Уровень 2

21 марта 2025, 09:07

"Некорректная работа с map" - довольно точное описание того, что ты делаешь. Мапа предназначена совсем не для этого. Ты полагаешься на то, что, что у тебя мапа будет заполнена набором значений, где будет один ключ больше минус одного, а остальные домены будут с индексом -1. Во-первых, понимаешь ли ты что эти пары (-1, домен) будут заменять одна другую при добавлении в мапу, если ни один из доменов не найден? Во-вторых, понимаешь ли ты, что в случае с протоколом при наличии https протокол http также будет найден в строке с тем же индексом? Получается, что по твоей задумке в мапе всегда будет не более 2 строк? Зачем вообще мапа, если наличие/отсутствие индекса по твоей задумке уже отвечает на вопрос, что за домен/протокол используется. Лучше сосредоточься на правильной работе со строками. Одного наличия подстроки в строке недостаточно для того, чтобы эта подстрока была доменом/протоколом данной ссылки. Ты проверяешь на "конвенциональных" примерах, где это определяется однозначно. А метод должен быть способен работать на любых примерах. Я тебе напишу вот такой url: http://my-https-website-com-ru-org-net.info и твой метод сойдёт с ума: он найдёт все вхождения, а поскольку мапа не гарантирует порядка, то он выдаст рандомный результат, вместо того, чтобы сказать, что протокол = http, а домен = неизвестный.

Денис Enterprise Java Developer

21 марта 2025, 07:15

Какой пи*ец у тебя в коде происходит... запомни, наркотики это зло, завязывай. Как думаешь, у строки https://google.com какой индекс будет у http и https? Вообще у строк есть методы startsWith и endsWith, они не просто так существуют. Если подумать то можно догадаться как решить задачу не создавая атомную сноповязалку. Если хочется чего-то необычного, можно сделать вот так:

public static String checkProtocol(String url) {
    return Optional.ofNullable(url)
            .filter(link -> link.startsWith("http") && link.contains("://"))
            .map(link -> link.substring(0, link.indexOf("://")))
            .orElse("неизвестный");
}

public static String checkDomain(String url) {
    return Optional.ofNullable(url)
            .filter(link -> link.contains("."))
            .map(link -> link.substring(link.lastIndexOf(".") + 1))
            .filter(domainWhitelist::contains)
            .orElse("неизвестный");
}