JavaRush/Помощь по Java задачам/префикс в теле цикла for не работает как хотелось бы ожид...

18 уровень

03.05.2019
843просмотров
9комментариев

префикс в теле цикла for не работает как хотелось бы ожидать, почему? list.add(++i, str); //+0 ?????

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

Решен

Одно из условий по дублированию слова у меня было такое, это правильный вариант:

if (str.contains("л") && !str.contains("р")) {
                    i++;
                    list.add(i, str); //+1
                    }

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

list.add(++i, str); //+0 ?????

Это не работало, когда прогнал в отладчике, то увидел, что именно в таком варианте нет разницы, что i++, что ++i отладчик показывает старое значение до присвоения. Как итог, не в тот индекс дублировалось слово. Притом эксперимент с выводом в консоль, дает ожидаемый результат.

int i=0;
        System.out.println(++i); //1

Кто объяснит?

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

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

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

Sergey Rudeshko

Уровень 12

17 мая 2019, 10:31

if (str.contains("л") && !str.contains("р")) - подскажите, для чего ! знак перед str.contains("р")

hidden #598481

Уровень 27

17 мая 2019, 11:55

это инверсия (НЕ) стр содержит "л" и стр НЕ содержит "р"

ProgX

Уровень 18

4 мая 2019, 08:01

Спасибо что откликнулись на мой вопрос. Выкладываю полностью мой код. Результат как Вы сказали, на первый взгляд такой же, за исключением, что дубляж слова вставляется в индекс того слова, который надо продублировать, валидатор по фашистки не пропускал такой вариант как правильный, вот я и озадачился с этим казусом.

public class Solution {
    public static void main(String[] args) throws Exception {
        ArrayList<String> list = new ArrayList<String>();
        list.add("роза"); // 0
        list.add("лоза"); // 1
        list.add("лира"); // 2
        list = fix(list);

        for (String s : list) {
            System.out.println(s);
        }
    }

    public static ArrayList<String> fix(ArrayList<String> list) {
        String str;
        for (int i = 0; i < list.size(); i++) {
            str = list.get(i);
                if (str.contains("р") && !str.contains("л")) {
                    list.remove(i);
                    i--;
                }else if (str.contains("л") && !str.contains("р")) {
                    //i++;
                    list.add(++i, str);
                    //++i;
                }
        }
        return list;
    }
}

Прикладываю скриншот дебагера, который показыват что i=0 ???? загадка мать её!

Justinian Judge в Mega City One

Master

4 мая 2019, 18:28

решение

отладчик это мощный инструмент к которому нужно приспособиться. Я вот ленюсь пока еще, поэтому для меня фраза "дебагер показывает что i=0" вообще ни о чем не говорит. Если курсор дебагера стоит на какой-то строке, то это значения на момент исполнения строки - перед ней, или после ее исполнения, или во время. Я не знаю. А код запустил, хм, вроде работает все как нужно, дублирует, удаляет, оставляет. Есть такая категория ошибок, когда мы сфокусированы на одном моменте, и не замечаем других. Возможно была ошибка в другом месте незаметная, валидатор брыкнулся, ты алгоритм переписал думая в голове про индексы и инкремент и попутно изменил что-то еще. И вот валидатор принял. А ты думаешь что это связано с одним моментом, а валидатора устроил другой.. А какую ошибку он писал?

ProgX

Уровень 18

4 мая 2019, 19:03

Именно проблема была только в этом префиксе. Я 20 раз прогонял его на проверку через сайт. Ошибка была вообще лживая, ругался на второе условие проверки "Метод fix должен удалять из списка строк все слова, содержащие букву "р". Исключение: слова содержащие и букву "р" и букву "л" - их нужно оставить." Потом я начал просто по одному условию ему скармливать и наткнулся на этот косяк, что это условие он вообще не воспринимал. Отнесем это к магии, я уверен что когда вернусь к этому вопросу позже, то всё будет работать и так и так). Спасибо что принял участие, мне были интересны твои ответы!)

Justinian Judge в Mega City One

Master

3 мая 2019, 20:20

А по сути твоего вопроса (потому что все что я ниже написал это лирика на тему, у тебя видно просто ошибка в алгоритме причем в другом месте, поэтому в вопросе всегда лучше весь код прилагать). Как я понимаю, в данном алгоритме нету никакой разницы что list.add(i++, element); что list.add(++i, element); ведь речь о дублировании, если мы обнаружили что допустим, элемент под индексом 0 слово "лира". То метод list.add(i++, element); -> вставит новое слово в индекс 0 что list.add(++i, element); -> вставит новое слово в индекс 1 Но на выходе это все-равно будет один и тот же результат. Два подряд идущих слова "лира", под индексками 0 и 1

Justinian Judge в Mega City One

Master

3 мая 2019, 19:56

ArrayList<String> list = new ArrayList<>(Arrays.asList("0","1","2","3"));
        int i = 0;
        System.out.println(list.get(i++));  // 0
        System.out.println(list.get(++i));  // 2
        System.out.println(list.get(i+1));  // 3

ArrayList<String> list = new ArrayList<>(Arrays.asList("0","1","2","3"));
        int i = 0;
        list.add(i++, "100");
        list.add(++i, "200");
        list.forEach(System.out::println);

Вывод: 100 0 200 1 2 3 То есть все работает как надо. п.с. Код вида someCount++ + --otherCount / count-- нужно избегать всяческими силами. Я например реально полчаса не мог вбить в толк, вроде все сложилось, а почему n = 1; m = ++n + n++; будет 4 Хотя а как иначе? 2+2 = 4. Неочевидный код, а этого нужно избегать(многоэтажные конструкции постфикса/префикса, одиночные вполне читаемы и хороши).Это разве что на собеседовании спросить могут... Приоритет операций хорошее объяснение ассоциативности Еще есть направление , как int x = y = 2+5; присвоение по табличке справа налево, что логично, и тд..

Justinian Judge в Mega City One

Master

3 мая 2019, 19:08

Таблица приоритета операций Для себя прояснил момент в какой момент происходит окончание вычисления выражения. Ожидаемо, как только начинает выполнятся метод, все уже посчитано, переменная test уже с инкрементом.

static int test = 0;
public static void main(String[] args) {
   testPrefix(test++);
   testPostfix(++test);
}
private static void testPostfix(int i) {
        System.out.println(test);  //  1
        System.out.println(i);       //  0
    }

    private static void testPrefix(int i) {
        System.out.println(test);    //  2
        System.out.println(i);         //  2
    }

Justinian Judge в Mega City One

Master

3 мая 2019, 18:54

Хороший вопрос! Важная тема, для того чтобы разобраться. Это связано с ассоциативностью операций. В комментариях - вывод в консоль

int[] arr = {0, 1, 2};
       int index = 0;
       System.out.println(arr[++index]);   // 1
       System.out.println(arr[index++]);   // 1
       System.out.println(index);          // 2

       int n = 0;
       System.out.println(++n);        // 1
       System.out.println(++n + n++);   // 4