JavaRush/Помощь по Java задачам/Не проходит два пункта валидатора. Почему? Логика, на пер...

41 уровень

19.12.2020
697просмотров
9комментариев

Не проходит два пункта валидатора. Почему? Логика, на первый взгляд, правильная.

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

Решен

Не проходит два пункта валидатора, а именно:

1. Метод fix должен удалять из списка строк все слова, содержащие букву "р". Исключение: слова содержащие и букву "р" и букву "л" - их нужно оставить.
Проверь, что метод fix удаляет слова содержащие букву "р".
2. Метод fix должен удваивать слова, содержащие букву "л" (добавлять еще один элемент с этим словом). Исключение: слова содержащие и букву "л" и букву "р" - их не нужно удваивать.

Код в методе следующий:

for (int i = 0; i < strings.size(); i++) {
          String s = strings.get(i);
            if (!(s.contains("р") && s.contains("л"))) {
              if (s.contains("л")) {
                  strings.add(strings.get(i));
                  i++;
              } else if (s.contains("р")) {
                  strings.remove(i);
                  i--;
              }
              else   if ((s.contains("р") && s.contains("л")) || (!s.contains("р") && !s.contains("л"))) {
                    continue;
                }
            }

       }

1. Создай список слов, заполни его самостоятельно.
2. Метод fix должен:
2.1. удалять из списка строк все слова, содержащие букву "р"
2.2. удваивать все слова содержащие букву "л".
2.3. если слово содержит и букву "р" и букву "л", то оставить это слово без изменений.
2.4. с другими словами ничего не делать.

Пример:
роза лира лоза

Выходные данные:
лира лоза лоза

Требования:

Программа не должна считывать данные с клавиатуры.
Метод fix должен удалять из списка строк все слова, содержащие букву "р". Исключение: слова содержащие и букву "р" и букву "л" - их нужно оставить.
Метод fix должен удваивать слова, содержащие букву "л" (добавлять еще один элемент с этим словом). Исключение: слова содержащие и букву "л" и букву "р" - их не нужно удваивать.
Метод fix не должен ничего делать со словами, содержащими и букву "л" и букву "р".
Метод fix не должен ничего делать со словами, которые не содержат ни букву "л", ни букву "р".

package com.javarush.task.task07.task0716; import java.util.ArrayList; /* Р или Л */ public class Solution { public static void main(String[] args) { ArrayList<String> strings = new ArrayList<String>(); strings.add("роза"); strings.add("лоза"); strings.add("лира"); strings = fix(strings); for (String string : strings) { System.out.println(string); } } public static ArrayList<String> fix(ArrayList<String> strings) { //напишите тут ваш код // 1. Программа не должна считывать данные с клавиатуры. // 2. Метод fix должен удалять из списка строк все слова, содержащие букву "р". Исключение: слова содержащие и букву "р" и букву "л" - их нужно оставить. // 3. Метод fix должен удваивать слова, содержащие букву "л" (добавлять еще один элемент с этим словом). Исключение: слова содержащие и букву "л" и букву "р" - их не нужно удваивать. // 4. Метод fix не должен ничего делать со словами, содержащими и букву "л" и букву "р". // 5. Метод fix не должен ничего делать со словами, которые не содержат ни букву "л", ни букву "р". for (int i = 0; i < strings.size(); i++) { String s = strings.get(i); if (!(s.contains("р") && s.contains("л"))) { if (s.contains("л")) { strings.add(strings.get(i)); i++; } else if (s.contains("р")) { strings.remove(i); i--; } else if ((s.contains("р") && s.contains("л")) || (!s.contains("р") && !s.contains("л"))) { continue; } } } /* for (int i = strings.size() - 1; i >= 0; i--) { if (strings.get(i).indexOf("р") >= 0 && strings.get(i).indexOf("л") < 0) { strings.remove(strings.get(i)); } } for (int i = strings.size() - 1; i >= 0; i++) { if (strings.get(i).indexOf("л") >= 0 && strings.get(i).indexOf(i) < 0) { strings.add(strings.get(i)); } }*/ return strings; } }

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

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

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

hidden #2322530

Уровень 41

19 декабря 2020, 08:17

как ты думаешь, слово "рыбалка" должно удалиться?
} else if (s.contains("р")) { strings.remove(i); i--; }

Евгений Хлебников

Уровень 41

19 декабря 2020, 08:29

По условию задачи слово "рыбалка" не должно быть удалено:

Метод fix не должен ничего делать со словами, содержащими и букву "л" и букву "р".

И по факту при тестировании оно тоже не удаляется. Output при тестировании у меня такой:

рыбалка
лоза
лира
лоза
лоза

Process finished with exit code 0

А не удаляется оно потому, что перед тем как зайти в эту ветку:
else if (s.contains("р")) {
                   strings.remove(i);
                   i--;
 }

мы делаем следующую проверку:
if (!(s.contains("р") && s.contains("л")))
и только потом идём проверять условия дальше:
if (!(s.contains("р") && s.contains("л"))) {
               if (s.contains("л")) {
                   strings.add(strings.get(i));
                   i++;
               } else if (s.contains("р")) {
                   strings.remove(i);
                   i--;
 }

Выглядит правильно.

hidden #2322530

Уровень 41

19 декабря 2020, 08:32

окей. а слово "рыба" или "рама" удаляется? у тебя это условие никогда не выполнится.
else if (s.contains("р")) else if ((s.contains("р") && s.contains("л")) || (!s.contains("р") && !s.contains("л"))) {

Евгений Хлебников

Уровень 41

19 декабря 2020, 08:40

Да, слово "рыба" удаляется. Вот ввод:

ArrayList<String> strings = new ArrayList<String>();
        strings.add("рыба");
        strings.add("лоза");
        strings.add("лира");
        strings = fix(strings);

А вот вывод:

лоза
лира
лоза
лоза

Process finished with exit code 0

А по условию задачи в примере должно быть только 3, а не 4 слова, а именно:
лира
лоза
лоза

hidden #2322530

Уровень 41

19 декабря 2020, 08:47

да, ты прав. просто условие так написано, пока прочитаешь, поймёшь неправильно. не пиши так больше. "Пишите код так, как будто поддерживать его будет склонный к насилию психопат, который знает, где вы живёте" (с) это всё равно ни на что не влияет, можешь удалить совсем. если у тебя дошло до этого условия, итерация цикла итак закончилась, не зачем тут continue;

else   if ((s.contains("р") && s.contains("л")) || (!s.contains("р") && !s.contains("л"))) {
      continue;
  }

ошибка твоя в том. что ты добавляешь слово в конец, а не сразу за проверяемым. например вот таким примером, у тебя и "роза" не удалится.

strings.add("лоза");
strings.add("роза");
strings.add("лира");

из-за этой ошибки, у тебя одно слово "лоза" превращается в 3 на примере из условия.

Евгений Хлебников

Уровень 41

19 декабря 2020, 08:48

Условие

else if (s.contains("р"))  // строка содержит только "р"

выполняется. То условие, которое ему предшествует:

if (!(s.contains("р") && s.contains("л")))  // строка содержит и "р" , и "л"

никак не мешает ему выполняться.

Евгений Хлебников

Уровень 41

19 декабря 2020, 09:19

Я переделал код вот так:

for (int i = 0; i < strings.size(); i++) {
           String s = strings.get(i);
            if (s.contains("р") && (!(s.contains("р") && s.contains("л"))))
            // условие: если строка содержит только "р" и не содержит комбинацию "р" и "л", то удалить строку
            {
                strings.remove(i);
                i--;
            }
            if (s.contains("л") && (!(s.contains("р") && s.contains("л"))))
               // условие: если строка содержит только "л" и не содержит комбинацию "р" и "л", то удвоить строку
               {
                   strings.add(strings.get(i));
                   i++;
               }
            /*else   if ((s.contains("р") && s.contains("л")) || (!s.contains("р") && !s.contains("л"))) {
                     continue;
            }*/
}

Но всё равно, слово "роза" не удаляется, если ввод такой:

ArrayList<String> strings = new ArrayList<>();
        strings.add("лоза");
        strings.add("роза");
        strings.add("лира");
        strings = fix(strings);

роза
лира
лоза
лоза

Process finished with exit code 0

и слово "роза" удаляется, если ввод такой:

strings.add("роза");
        strings.add("лоза");
        strings.add("лира");
        strings = fix(strings);
лоза
лира
лоза
лоза

Process finished with exit code 0

Почему здесь из-за начального порядка ввода слов меняется результат? Не совсем мне понятно.

hidden #2322530

Уровень 41

19 декабря 2020, 09:23

решение

1) я ведь написал в чем ошибка: "ошибка твоя в том. что ты добавляешь слово в конец, а не сразу за проверяемым" с самого начала достаточно было добавить индекс для вставки нового элемента:

strings.add(i, strings.get(i));
i++;

таким образом, у тебя при слове ЛОЗА, добавится сразу за ним ещё одно слово ЛОЗА, а так как i++ ты его перепрыгнешь, и будешь проверять уже РОЗА, так как РОЗА удаляем, то i--, объекты сдвигаются влево, и ты проверяешь уже объект который появился вместо РОЗА. всё прозрачно и красиво. 2) вот это правильно вынести самым первым условием:

&& (!(s.contains("р") && s.contains("л")

и переписать вот так:

if (s.contains("р") && s.contains("л")) {
    continue;
}

а в остальных if удалить из условия, оставив только проверку на Р или на Л.

Евгений Хлебников

Уровень 41

19 декабря 2020, 10:16

Спасибо за помощь. Я думал, что где-то из-за счетчика i++ проблемы возникают, но не мог понять где и какие именно. Написал вот так, и всё в итоге получилось:

for (int i = 0; i < strings.size(); i++) {
           String s = strings.get(i);

 // Только если НЕ выполняем 4 и 5 условия (совпадают с исключениями в условиях 2 и 3), а именно:
 // 4. Ничего не делать со словами, содержащими И букву "л" И букву "р",
 // 5. ИЛИ со словами, которые не содержат НИ букву "л", НИ букву "р",
 // то только тогда проверяем условия 2 и 3 и добавляем или удаляем нужную строку
           if (!((s.contains("р") && s.contains("л")) || (!s.contains("р") && !s.contains("л")))) {

               if (s.contains("р") && (!(s.contains("р") && s.contains("л"))))
                // условие #2: если строка содержит только "р" и не содержит комбинацию "р" и "л", то удалить строку
                {
                    strings.remove(i);
                    i--;
                }

                if (s.contains("л") && (!(s.contains("р") && s.contains("л"))))
                // условие #3: если строка содержит только "л" и не содержит комбинацию "р" и "л", то удвоить строку
                {
                     strings.add(i, strings.get(i));
                     i++;
                }
           }
 }