валидатор не принимает

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

Решен

for (String[] strings : morals) {
            for (int i = 0; i < strings.length; i++) {
                if (strings[i].toLowerCase().contains("killer")) {
                    strings[i] = strings[i].toLowerCase().replace("killer", "compassion");
                } else if (strings[i].toLowerCase().contains("thief")) {
                    strings[i] = strings[i].toLowerCase().replaceAll("thief", "fairness");
                } else if (strings[i].toLowerCase().contains("bully")) {
                    strings[i] = strings[i].toLowerCase().replaceAll("bully", "respect");
                }
            }
        }
    }
}

в чем моя ошибка?

Моральный блок Стального Стива заполнен вредоносными элементами, которые делают пиратского капитана очень кровожадным и аморальным.

В этой задаче требуется в методе cleanMorals исправить статический массив morals класса Solution, заменив негативные моральные элементы на позитивные.

Для этого требуется:

Заменить все элементы, в которых присутствует текст "killer" (без учета регистра), на слово "compassion" (сострадание).
Заменить все элементы, в которых присутствует текст "thief" (без учета регистра), на слово "fairness" (справедливость).
Заменить все элементы, в которых присутствует текст "bully" (без учета регистра), на слово "respect" (уважение).

Метод main не участвует в проверке.

Требования:

Не изменяй массив morals в классе Solution.
Метод cleanMorals должен изменять массив morals согласно условиям.

package com.javarush.task.pro.task09.task0924; /* Взлом Стального Стива */ import java.util.Arrays; public class Solution { public static String[][] morals = {{"iAmAKiller", "iAmAThief", "honesty", "responsibility", "iAmABully"}, {"BuLLyIAmA", "honesty", "iamakiller", "iamathief", "responsibility"}, {"honesty", "iamabully", "responsibility", "AthiefIAm", "ikillerAmA"}, }; public static void main(String[] args) { cleanMorals(); for (String[] strings : morals) { System.out.println(Arrays.toString(strings)); } } // создаем пустой массив чтобы заполнить его массивом morals. а затем убрать в этом массиве killers. public static void cleanMorals() { for (String[] strings : morals) { for (int i = 0; i < strings.length; i++) { if (strings[i].toLowerCase().contains("killer")) { strings[i] = strings[i].toLowerCase().replace("killer", "compassion"); } else if (strings[i].toLowerCase().contains("thief")) { strings[i] = strings[i].toLowerCase().replaceAll("thief", "fairness"); } else if (strings[i].toLowerCase().contains("bully")) { strings[i] = strings[i].toLowerCase().replaceAll("bully", "respect"); } } } } }

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

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

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

hidden #598481

Уровень 27

18 января 2023, 18:09

Требуется заменить элементы целиком

ivan дворник

18 января 2023, 18:14

и как это сделать? чем мое решение отличается от правильного:

public class Solution {

    public static String[][] morals = {{"iAmAKiller", "iAmAThief", "honesty", "responsibility", "iAmABully"},
            {"BuLLyIAmA", "honesty", "iamakiller", "iamathief", "responsibility"},
            {"honesty", "iamabully", "responsibility", "AthiefIAm", "ikillerAmA"},
    };

    public static void main(String[] args) {
        cleanMorals();
        for (String[] strings : morals) {
            System.out.println(Arrays.toString(strings));
        }
    }

    public static void cleanMorals() {
        for (String[] strings : morals) {
            for (int i = 0; i < strings.length; i++) {
                String element = strings[i];
                boolean isKiller = element.toLowerCase().contains("killer");
                boolean isThief = element.toLowerCase().contains("thief");
                boolean isBully = element.toLowerCase().contains("bully");
                if (isKiller) {
                    strings[i] = element.replace(element, "compassion");
                } else if (isThief) {
                    strings[i] = element.replaceAll(element, "fairness");
                } else if (isBully) {
                    strings[i] = element.replaceAll(element, "respect");
                }
            }
        }
    }
}

hidden #598481

Уровень 27

18 января 2023, 19:23

Так сравни, и найдёшь отличие. Попробуй запустить по очереди каждое решение и посмотреть разницу в выводе, если не можешь найти отличие, читая код. Или мне всё-таки надо самому показать, в чём разница?

ivan дворник

19 января 2023, 13:56

в выводе разницы нет. можно даже вместо replace написать просто =. просто запомню что можно писать так как в правильном и возьму это на заметку.

hidden #598481

Уровень 27

19 января 2023, 14:19

А если запустить и проверить? Вывод твоей программы:

[iamacompassion, iamafairness, honesty, responsibility, iamarespect]
[respectiama, honesty, iamacompassion, iamafairness, responsibility]
[honesty, iamarespect, responsibility, afairnessiam, icompassionama]

Вывод правильного решения:

[compassion, fairness, honesty, responsibility, respect]
[respect, honesty, compassion, fairness, responsibility]
[honesty, respect, responsibility, fairness, compassion]

Всё ещё нет разницы? Правильное решение тоже не оптимальное, потому что replace там действительно ни к чему, но по крайней мере оно делает то, что требует задание: Заменить все элементы, в которых присутствует текст "killer" (без учета регистра), на слово "compassion" (сострадание). Я подчеркнул ключевые слова. Заменить элементы в массиве по сути означает присвоить элементам массива новые значения, те, которые требуются. А твоё решение заменяет одно слово внутри элемента на другое слово. В этом твоя ошибка. А сделать требовалось просто следующее:

if (...)
    strings[i] = "compassion";

и всё. Об этом я и написал в самом первом комментарии.

ivan дворник

19 января 2023, 15:00

спасибо тебе огромное. странно, вчера делал, используя обычный оператор присвоения, но валидатор не одобрил. ❤️