JavaRush/Помощь по Java задачам/Не понимаю, в чём ошибка. Не проходят 2 и 3 пункты

14 уровень

12.02.2021
496просмотров
24комментариев

Не понимаю, в чём ошибка. Не проходят 2 и 3 пункты

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

Решен

break указан, так как получился бесконечный цикл. Но как?

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) { for (int i = 0; i < strings.size(); i++){ if (strings.get(i).contains("p") && strings.get(i).contains("л")) continue; if (strings.get(i).contains("p")) strings.remove(i); if (!strings.get(i).contains("p") && strings.get(i).contains("л")) strings.add(strings.get(i)); if (strings.get(i).contains("л")) strings.add(strings.get(i)); break; } return strings; } }

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

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

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

Михаил

Уровень 33

13 февраля 2021, 19:20

создай копию .и по копии проходи а удоляй в исходном по индексу .

Гофф

Уровень 20

Master

14 февраля 2021, 00:29

не об этом речь. Разве тут где-то создаётся итератор?

Михаил

Уровень 33

14 февраля 2021, 05:10

исправь у тебя должны выпооняться оба условия создаешь копию прохрдишь по копии первый if false р & false л else if true р & false л {remove i исходный } else if false p & true л {add в конец исходного} в твоем случае первое неправильные условия ,долдны выполняться оба . второе при добавление слова с л у тебя будет бесконечный цикл одного слова

Дмитрий

Уровень 36

12 февраля 2021, 19:03

Пробеги по списку дважды в первый раз определи index'ы на удаление. И удали. Индексы обновятся. Второй на добавление при том обратным от условия неизменной строки. И добавь.

Михаил

Уровень 33

12 февраля 2021, 18:38

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

Гофф

Уровень 20

Master

12 февраля 2021, 18:31

1. У тебя цикл проходит один раз и прерывается брейком. 2. Немного диковатый набор ифов. 3. Наверное имеет смысл при удалении или добавлении как-то манипулировать счётчик, нет? 4. Не всегда, но часто оператор continue - признак неудачно выстроенной логики. При правильно выстроенных условиях в ифах никаких continue не нужно. Намёк - задача решается просто в два ифа.

Ольга Бахута

Уровень 14

12 февраля 2021, 19:50

1. Если задача решается в два ифа, то вот такая идея. Но всё равно второй и третий пункты не работают.

for (int i = 1; i < strings.size()-1; i++)
{
            if (strings.get(i).contains("p") && !strings.get(i).contains("л"))
            strings.remove(i);
            i++;
            if (!strings.get(i).contains("p") && strings.get(i).contains("л"))
            strings.add(strings.get(i));
            i++;
  }
        return strings;

2. Если с нуля цикл, то четвёртый пункт не работает. Я на верном пути хоть ?

Гофф

Уровень 20

Master

13 февраля 2021, 16:03

Да, отличная идея, на верном пути. Надо только теперь подумать насчёт индекса, в каком случае как его изменять. Цикл надо, безусловно, начинать с нуля. Подумай, почему это он вдруг не работает. И обрати внимание, сколько раз за один проход цикла у тебя выполнится оператор i++; Подсказка - три раза.

Ольга Бахута

Уровень 14

13 февраля 2021, 16:39

Прогресс! теперь только второе условие не принимает. Ради эксперимента пробовала выносить первый иф с итератором в отдельный цикл - не помогло. Вроде правильно написано, но не работает.

for (int i = 0; i < strings.size()-1; i++)
        {
            if (strings.get(i).contains("p") && !strings.get(i).contains("л"))
               strings.remove(i);
            if (!strings.get(i).contains("p") && strings.get(i).contains("л"))
                strings.add(strings.get(i));
            i++;
        }

Гофф

Уровень 20

Master

13 февраля 2021, 17:03

1. Опять счётчик. Подумай, как должен изменяться счётчик, если ты удаляешь. 2. Снова счётчик. Подумай, в каких случаях у тебя выполняется i++ в строке 7 последней версии кода? Совет: если тело ифа или цикла из одного оператора, то либо пиши его всегда в той же строке:

if(условие) оператор;

Или всегда используй скобки:

if(условие) {
     оператор;
}

Ольга Бахута

Уровень 14

13 февраля 2021, 17:25

1. Если правильно поняла, то при удалении счётчик уменьшается. 2. Расставила скобки :) Результат: 2, 3, 4 шаги неправильные.

for (int i = 0; i < strings.size()-1; i++)
        {
            if (strings.get(i).contains("p") && !strings.get(i).contains("л"))
            {
               strings.remove(i);
               i--;
            }
            if (!strings.get(i).contains("p") && strings.get(i).contains("л"))
            {
                strings.add(strings.get(i));
                i++;
            }

Гофф

Уровень 20

Master

13 февраля 2021, 17:41

Но мы всё равно уже ближе)) Код уже красивый и почти рабочий )) Давай теперь подумаем, что произойдёт, если первое же слово содержит р и не содержит л? Мне кажется, что программа выбросит ошибку-исключение, что то вроде Out Of Bounds... АПДТ. Увидел ещё одну ошибку. Скажи, при добавлении строки, её надо добавлять в конец или в определённое место?

Ольга Бахута

Уровень 14

13 февраля 2021, 19:17

По Out Of Bounds: в принципе, указанное тобой условие: "если первое же слово содержит р и не содержит л" можно не учитывать, потому что Out Of Bounds исчезает при указании цикла до strings.size()-1, как писала в прошлый раз (я с ней уже сталкивалась, когда начинала решать задачу :)). Можно указать в лоб строку условия, тогда надо убрать этот -1. Но наверное лучше оставить -1, компактнее получается, меньше кода.

for (int i = 0; i < strings.size(); i++)
        {
            if (strings.get(i).contains("p") && !strings.get(i).contains("л"))
            {
               strings.remove(i);
               i--;
            }
            if (strings.get(0).contains("p") && !strings.get(0).contains("л")) {
               strings.remove(i);
             i--;
            }
            if (!strings.get(i).contains("p") && strings.get(i).contains("л"))
            {
                strings.add(i+1,strings.get(i));
                i++;
            }

Если оставить -1, но добавить вне условий ещё один итератор, то опять будет неправильно только второй пункт )) То есть получаем, что ошибка во втором пункте с удалением, но я в упор её не вижу.

for (int i = 0; i < strings.size()-1; i++)
       {

           if (strings.get(i).contains("p") && !strings.get(i).contains("л"))
           {
              strings.remove(i);
              i--;
           }
           if (!strings.get(i).contains("p") && strings.get(i).contains("л"))
           {
               strings.add(i+1,strings.get(i));
               i++;
           }
           i++;
       }

Гофф

Уровень 20

Master

14 февраля 2021, 00:08

решение

нет-нет-нет, это, прости, не то. Остаёмся с двумя ифами. Вернёмся к предыдущему коду:

for (int i = 0; i < strings.size()-1; i++)
{
     if (strings.get(i).contains("p") && !strings.get(i).contains("л"))
     {
          strings.remove(i);
          i--;
     }
     if (!strings.get(i).contains("p") && strings.get(i).contains("л"))
     {
            strings.add(strings.get(i));
            i++;
     }
}

Ошибку с добавлением ты исправила:

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

Отлично. Насчёт первой ошибки - выхода за пределы (out of bounds). Почему оно происходит? Если первое же слово (при первом проходе цикла, когда i=0) выполняет условие первого ифа (содержит р и не содержит л), то выполняется i--, i у нас ставится -1. Программа выполняется дальше и при следующей проверке if (!strings.get(i) программа вывалится в ошибку, потому что i = -1 и это out of bounds. Поэтому нам надо просто добавить else перед вторым ифом.Ведь если первое условие выполнилось, второе нам проверять вообще не надо.

Гофф

Уровень 20

Master

14 февраля 2021, 02:10

Если возникнут вопросы по поводу ифов-елсов, то спроси.

Anonymous #2484617 Backend Developer

14 февраля 2021, 02:24

полезный

Привет, в последнем варианте кода в ифах английсая "p", может запутать почему в секцию if не заходит. И такой вариант не отработает: strings.add("лира"); strings.add("лира"); strings.add("липа"); "липа" не удвоится, нужно условие цикла поправить

Гофф

Уровень 20

Master

14 февраля 2021, 02:28

сорян, не понял совсем. При чём тут английская пе? Она нас ваще не колышит АПДТ: в коде Ольги где-то английская пе вместо русской эр?

Anonymous #2484617 Backend Developer

14 февраля 2021, 02:38

if (strings.get(i).contains("р") && !strings.get(i).contains("л")) { Да, вот тут contains с английской "p", соответственно "роза" не удалится. В следующем ифе также.

Гофф

Уровень 20

Master

14 февраля 2021, 02:43

Ага, спасибо! Это из категории ошибок, которые легче не допускать, чем найти)) Надеюсь, Ольга исправит. Интересно, как сам решал эту задачу?

Anonymous #2484617 Backend Developer

14 февраля 2021, 02:53

Заглянул в старые записи, как попало решал) Через сравнение символов, т.е цикл в цикле. Но это было уже давно, почти 5 лет назад

Гофф

Уровень 20

Master

14 февраля 2021, 03:00

Всегда забавно смотреть на свой код пятилетней давности)) Да какой пятилетней! Двухнедельной давности и то бывает забавно... Полюбак про англйскую р спасибо. Вспомнил забавный лайфхак одного новостного онлайн ресурса. Они все русские буквы в текстах новостей заменяли на латинские, которые так же выглядят. о, е, р, с и т.д. Зачем? Для читателя разницы не видно, а для тех, кто чужие новости тырит... Konuпacmy лeгкo omслeжuвamь))

Ольга Бахута

Уровень 14

14 февраля 2021, 09:37

Ха! помог тупо else и убрала -1. Ну и последний i++ убрала заодно. Всё! и валидатор пропустил. 😅 Самая долгая задача за всё моё время тут.

Ольга Бахута

Уровень 14

14 февраля 2021, 09:39

Про букву р первым делом исправляла во время моих первых N попыток решения задачи этой. Но видимо потом обратно вернула. Вы там ASCII код посмотрели буквы? :)

Anonymous #2484617 Backend Developer

14 февраля 2021, 11:32

Код буквы не смотрел, просто догадался. В режиме дебага запустил, if должен обязательно отработать при определенном слове, а он не отрабатывает. Первая мысль - не та буква, они же похожи. Ну и встречался раньше с такими недочетами.