Ответ выводит правильный, а в условии неверно получается

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

Удаляем одинаковые строки

Java Syntax, 6 уровень, 4 лекция

Обсуждается

Где ошибка?

В этой задаче тебе нужно:

Считать 6 строк и заполнить ими массив strings.
Удалить повторяющиеся строки из массива strings, заменив их на null (null должны быть не строками "null").

Примеры.

Массив после чтения строк:
{"Hello", "Hello", "World", "Java", "Tasks", "World"}

Массив после удаления повторяющихся строк:
{null, null, null, "Java", "Tasks", null}

Требования:

В методе main(String[]) считай с клавиатуры 6 строк и заполнить ими массив strings.
В методе main(String[]) удали(заменить строку на null) элементы из массива strings с одинаковыми строками.

package com.javarush.task.pro.task05.task0508; import java.util.Scanner; /* Удаляем одинаковые строки */ public class Solution { public static String[] strings = new String[6]; public static void main(String[] args) { Scanner s = new Scanner(System.in); boolean is; int a; for(int i = 0; i < 6; i++) strings[i] = s.nextLine(); for(int i = 0; i < 6; i++){ for(int j = 0; j < 6; j++){ if(i != j && strings[i] != null && strings[j] != null) { if (strings[i].equals(strings[j])) { strings[i] = null; strings[j] = null; i++; j++; } } } } for (int i = 0; i < strings.length; i++) { System.out.print(strings[i] + ", "); } } }

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

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

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

Денис Enterprise Java Developer

6 июня 2022, 14:50

Проверьте на вот этом: {"Hello", "Hello", "Hello", "World", "Java", "Tasks", "World"} Ну и в циклах наведите порядок. Там явно можно сделать оптимизации навроде

for (int i = 0; i < strings.length; i++) {
            String current = strings[i];
            for (int j = i + 1; j < strings.length; j++) {...}
}

Anonymous #3086485Java Developer в BELKA_Cheats.com

6 июня 2022, 16:38

Это понятно, а какое условие тогда должно быть при Hello Hello Hello?

Денис Enterprise Java Developer

6 июня 2022, 16:55

Да в общем то ничего военного, просто сравнивай не со strings[i] (раз ты его зануляешь во вложенном цикле), а с добавочной переменной как в моем примере current. Как альтернатива - заведи булеву переменную и ставь ее в true если найден хоть один дубль во вложенном цикле, в нем же зануляй только найденный дубль, а как вложенный цикл доработает - проверяй твой флаг и если он true - зануляй strings[i]

Виктор

Уровень 45

6 июня 2022, 17:08

сначала удалить 2-й и последующие повторы, а уж потом первое включение повторяющегося значения

for (int i = 0; i < strings.length-1; i++) {
    if(strings[i]==null) continue;
    for (int j = i+1; j < strings.length; j++) {
        if(strings[i].equals(strings[j])){
            isRep = true;
            strings[j] = null;
        }
    }
    if (isRep){
        strings[i] = null;
        isRep = false;
    }
}

Денис Enterprise Java Developer

6 июня 2022, 17:13

Если перенести создание isRep в начало первого цикла, то не нужно будет ее сбрасывать руками ;)

Виктор

Уровень 45

6 июня 2022, 17:19

кстати - да ))) прав100% )))