Не проходит валидацию.

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

Обсуждается

Почему не проходит валидацию? не понимаю((( пробовал через

strings.get(i).contains

пробовал через

strings.get(i).indexOf() != -1

пишет

java.lang.NullPointerException
	at com.javarush.task.task07.task0716.Solution.main(Solution.java:17)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:498)

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("р")) && (!strings.get(i).contains("л"))){ /*если в слове есть Р и нет Л*/ strings.remove(i); } if ((!strings.get(i).contains("р")) && (strings.get(i).contains("л"))){ /*Если в слове есть Л и нет Р*/ strings.add(i, strings.get(i)); /*увеличиваем счетчик, чтобы он не добавлял слово бесконечно*/ i++; } } return null; } }

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

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

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

Vladimir Tanski

Уровень 22

22 июля 2019, 10:53

Ты в методе fix возращаешь null; strings = fix(strings) ты получаешь strings = null; Либо метод сделай void либо верни пофиксаный лист ;

МаксимJava Developer в Freightender

21 июля 2019, 17:14

Ну вот вы добавили строку и вручную переместили индекс списка. А если удалили строку, что происходит с ArrayList'ом? )

hidden #2111695

Уровень 25

21 июля 2019, 18:15

for (int i = 0; i < strings.size(); ){
            if ((strings.get(i).contains("р")) && (!strings.get(i).contains("л"))){
                /*если в слове есть Р и нет Л, счетчик не увеличиваем,
на этом месте будет новый элемент*/
                strings.remove(i);
            }
            if ((!strings.get(i).contains("р")) && (strings.get(i).contains("л"))){
                /*Если в слове есть Л и нет Р*/
                strings.add(i, strings.get(i));
                /*увеличиваем счетчик на 2 так как следующее слово -  это дубль*/
                i++;
                i++;
            }
        }

теперь компилится, но зацикливается

МаксимJava Developer в Freightender

21 июля 2019, 18:52

Я хотел намекнуть, что нужно что-то сделать в блоке, где строка удаляется )

if ((strings.get(i).contains("р")) && (!strings.get(i).contains("л"))){
                /*если в слове есть Р и нет Л, счетчик не увеличиваем,
на этом месте будет новый элемент*/
                strings.remove(i);
            }

Все верно вы рассуждаете. Список сдвигается, но при следующей итерации вы проскакиваете одну строчку

hidden #2111695

Уровень 25

21 июля 2019, 19:00

Не получается понять. Почему проскакиваю? Ведь при выполнении условия удаления я не увеличиваю i и из условия цикла я убрал i++. Т.е следующая итерация цикла должна происходить с тем же значением i но с новым элементом под этим индексом.

МаксимJava Developer в Freightender

21 июля 2019, 19:10

Ок. Тогда что происходит со строками которые не попадают ни под одно условие? Здесь то вы и получаете зацикливание на одном и том же i.

hidden #2111695

Уровень 25

24 июля 2019, 12:38

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

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 < 3;){
            if ((strings.get(i).contains("р")) && (!strings.get(i).contains("л"))){
                /*если в слове есть Р и нет Л*/
                strings.remove(i);
            }else if ((!strings.get(i).contains("р")) && (strings.get(i).contains("л"))){
                /*Если в слове есть Л и нет Р*/
                strings.add(i, strings.get(i));
                /*увеличиваем счетчик, чтобы он не добавлял слово бесконечно*/
                i++;
                i++;
            }else{ i++; }
        }
        return null;
    }
}

Теперь не компилируется.

java.lang.NullPointerException
	at com.javarush.task.task07.task0716.Solution.main(Solution.java:17)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:498)