JavaRush/Помощь по Java задачам/Логика вроде верная, но валидатор не принимает...

11 уровень

05.09.2021
431просмотров
20комментариев

Логика вроде верная, но валидатор не принимает...

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

Решен

Большую часть обсуждаемых вариантов решениz почитал, в логике решения задачи разобрался, код вроде как набрал, но не могу разобраться до конца, почему не срабатывает.

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

Считать 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 ru.javarush.task.pro.task05.task0508; import java.util.Scanner; /* Удаляем одинаковые строки */ public class Solution { public static String[] strings; 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 - 1; i++) { boolean d = true; for (int j = i + 1; j < strings.length; j++) { if (strings[i].equals(strings[j])) ; strings[i] = null; d = true; } if (d == false) strings[i] = null; } for (int i = 0; i < strings.length; i++) { System.out.print(strings[i] + ", "); } } }

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

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

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

Konstantin

Уровень 51

5 сентября 2021, 17:10

полезный

Чтобы разобраться самому сначала исправь опечатку:

for (int i = 0; i < strings.length - 1; i++) {
            boolean d = true;
            for (int j = i + 1; j < strings.length; j++) {
                if (strings[i].equals(strings[j])) ; // вот здесь нет кода который выполняется если условие верно, а стоит точка с запятой
                strings[i] = null;
                d = true;
            }
            if (d == false)
                strings[i] = null;
        }

Konstantin

Уровень 51

5 сентября 2021, 17:12

полезный

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

artem_ak_izh

Уровень 11

6 сентября 2021, 18:46

С опечаткой прям раскрыл глаза - спасибо ) Поставил на автомате и все криво работает... важная деталь! Теперь не совсем понятна вторая часть подсказки... под фразой "сделай чтобы проходился по одному массиву, а менял другой" и подразумевается выполнение условия с bolean?

Konstantin

Уровень 51

6 сентября 2021, 18:59

Можно и без этого решить, но я делал так, создал копию массива, искал в ней слова повторяющиеся а удалял в оригинале. Но можно и без копии, но чтобы что-то подсказать нужен актуальный код, что есть на данный момент. И не совсем понятно зачем boolean d = true;

artem_ak_izh

Уровень 11

6 сентября 2021, 19:05

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 - 1; i++) {
            boolean d = true;
            for (int j = i + 1; j < strings.length; j++) {
                if (strings[i] != null && strings[i].equals(strings[j]))
                strings[i] = null;
                d = true;
            }
            if (!d)
                strings[i] = null;
        }

        for (int i = 0; i < strings.length; i++) {
            System.out.print(strings[i] + ", ");
        }
    }
    }

artem_ak_izh

Уровень 11

6 сентября 2021, 19:07

Вот к чему в итоге пришел на текущий момент... в такой редакции последнее слово из условия не об'null'яется

artem_ak_izh

Уровень 11

6 сентября 2021, 19:09

Не хочу сдаваться ))) Третий день по часу курю тему - принципиально не смотрю готовое решение, только в подсказках роюсь

Konstantin

Уровень 51

6 сентября 2021, 19:34

strings[j] = null;

не вижу где второе слово удаляется Опечатка на 15 строке? boolean d = true; - зачем эта переменная, она всегда true

artem_ak_izh

Уровень 11

6 сентября 2021, 20:17

Да, насчет обнуления второй строки согласен - отдельного метода нет, но если в строке 15 прописать

strings[j] = null;

не изменяя остальной части кода, то код не компилируется, т.к. эта строка за пределами цикла, в котором объявлена переменная j. Объявив же ее до цикла, я получаю об'null'енными все введенные слова, кроме последнего...

Konstantin

Уровень 51

6 сентября 2021, 20:34

полезный

Я конечно могу написать решение, но подскажу еще раз: strings[j] = null; нужно добавить туда же где и strings[i] = null; Совсем убрать эту переменную boolean d = true; везде где она написана И основная подсказка: Хорошим тоном считается выделять код который выполняется в цикле или который выполняется если соблюдено условие, - в фигурные скобки, даже если там одна строчка.

if (strings[i] != null && strings[i].equals(strings[j]))
               strings[i] = null;

artem_ak_izh

Уровень 11

6 сентября 2021, 20:35

Внимательно вопросы и комментарии почитал, сопоставил с Вашим замечанием... вроде пазл начинает складываться... сейчас вернусь с новым кодом )

artem_ak_izh

Уровень 11

6 сентября 2021, 21:25

решение

Аллелуя! ))) Да уж, вот насколько важно корректное определение границ цикла... пойду еще раз про циклы почитаю. Спасибо, все сработало!

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 - 1; i++) {
            for (int j = i + 1; j < strings.length; j++) {
                if (strings[i] != null && strings[i].equals(strings[j])) {
                    strings[i] = null;
                    strings[j] = null;
                }
            }

        }

        for (int i = 0; i < strings.length; i++) {
            System.out.print(strings[i] + ", ");
        }
    }
}

artem_ak_izh

Уровень 11

6 сентября 2021, 21:31

Код при разных вводных компилируется, но валидатор не принимает... Это особенность javarush или действительно в коде что-то не то?

artem_ak_izh

Уровень 11

6 сентября 2021, 21:41

Задача не прошла проверку потому, что в верхней строке по какой-то причине было

package ru.javarush.task.pro.task05.task0508;

а должно было быть

package com.javarush.task.pro.task05.task0508;

Странно, откуда оно у меня вообще такое взялось 🤪

Никита

Уровень 31

Expert

5 сентября 2021, 16:24

ты вызываешь метод equals у null поэтому выскакивает ошибка

artem_ak_izh

Уровень 11

5 сентября 2021, 16:31

Так. Правильно понимаю, что в строку 21 надо добавить условие на проверку значения переменной, чтобы оно не равнялось null?

if (strings[i] != null && strings[i].equals(strings[j]))

Никита

Уровень 31

Expert

5 сентября 2021, 17:22

да сделай так, должно помочь

artem_ak_izh

Уровень 11

6 сентября 2021, 18:36

Добавил. Теперь все об'null'яется кроме последнего слова...

AlexVolt Backend Developer в MaDela

7 сентября 2021, 08:21

ты решил?

artem_ak_izh

Уровень 11

21 сентября 2021, 19:07

Да. Посмотри чуть выше по тексту специально отдельным комментарием выложил верное решение.