Выводит null

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

Решен

Подскажите пожалуйста, почему выводит все null, как это исправить? Голова кипит от непонимания Как сравнить так, чтобы на nul изменились только все дубли?

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

Считать 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; public static void main(String[] args) { //напишите тут ваш код Scanner sc = new Scanner(System.in); strings = new String[6]; String[] tmp = new String[strings.length]; for (int i = 0; i < strings.length; i++) { strings[i] = sc.nextLine(); tmp[i] = strings[i]; if(tmp[i].equals(strings[i])) strings[i]=null; System.out.print(strings[i] + ", "); } } }

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

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

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

solid

Уровень 33

21 октября 2022, 13:31

Для начала нужно циклом просто занести 6 элементов в созданный массив. Чтобы сразу находить дубликаты при вводе придётся заморачиваться, чего делать не стоит. Вторым шагом циклом проходишь по каждому элементу и во внутреннем цикле сравниваешь с последующими элементами. Если есть дубликаты, то удаляешь их и, соответственно, удаляешь данный элемент, для этого нужно завести булеву переменную "есть ли дубликаты". Также в циклы нужно ввести проверку на null, их просто пропускаем.

Владимир

Уровень 30

21 октября 2022, 12:55

tmp[i] = strings[i];            // дублируешь элемент массива
if(tmp[i].equals(strings[i]))  // всегда будет true, т.к. вверху tmp[i] = strings[i]
strings[i]=null;              // меняешь все значения на null

Один из вариантов решения: создать второй подцикл для прохода по тем же элементам массива, но + 1 позиция, и сравнить попарно элементы. Если совпадают, то заменить на null. NB: т.к. метод equals() сравнивает объекты, то будь внимательным, чтобы не вызвать его у null и не получить NullPointerException

Уровень 19

21 октября 2022, 13:22

Владимир, спасибо Подскажите пожалуйста, в правильном ответе посмотрел такое: public static void main(String[] args) { Scanner scanner = new Scanner(System.in); strings = new String[6]; for (int i = 0; i < 6; i++) { strings[i] = scanner.nextLine(); } for (int i = 0; i < strings.length; i++) { String currentString = strings[i]; for (int j = i + 1; j < strings.length; j++) { if (currentString == null) { break; } if (currentString.equals(strings[j])) { strings[j] = null; strings[i] = null; } } } for (int i = 0; i < strings.length; i++) { System.out.print(strings[i] + ", "); } } } Подскажите почему в строке for (int j = i + 1; j < strings.length; j++) { // такое значение у переменной j?

Уровень 19

21 октября 2022, 13:23

Зачем нужно добавлять эту 1 позицию?

Владимир

Уровень 30

21 октября 2022, 13:29

В 1ом цикле for i проходит по всем элементам, а во втором j по каждому последующему, по отношению к 1ому. Таким образом элемент strings[i] сравнивается попарно с остальными элементами массива.

Владимир

Уровень 30

21 октября 2022, 13:58

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

Олег

Уровень 7

23 октября 2022, 15:54

но ведь таким способом сравниваем и сработает только если одинаковые друг за другом идут. в данном случае 0 и 1 ячейки. а как быть со 2ой и 5ой ячейкой ?

Владимир

Уровень 30

23 октября 2022, 16:27

Проверь! Запусти в режиме дебагинга Alt+Shift+F9 и проследи каждый шаг, нажимая F8