Код работает но не проходит проверку

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

Обсуждается

Мой код работает, но почему-то не проходит проверку. Подскажите, пожалуйста, в чем причина.

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

import java.util.Scanner;
import java.util.Arrays;

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

public class Solution {
    public static String[] strings;

    public static void main(String[] args) {
        //напишите тут ваш код
        strings = new String[6];
        Scanner console = new Scanner(System.in);
        for (int i = 0; i < 6; i++) {
            strings[i] = console.nextLine();
        }

        for (int i = 0; i < 5; i++) {
            if (strings[i] != null) {
                for (int j = i + 1; j < 6; j++) {
                    if (strings[j] != null) {
                        if (strings[i].equals(strings[j])) {
                            strings[i] = "";
                            strings[j] = "";
                        }
                    }

                }
            }

        }

        for (int i = 0; i < 6; i++) {
            if (strings[i] ==  "") {
                strings[i] = null;
            }
        }

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

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

Считать 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; import java.util.Arrays; /* Удаляем одинаковые строки */ public class Solution { public static String[] strings; public static void main(String[] args) { //напишите тут ваш код strings = new String[6]; Scanner console = new Scanner(System.in); for (int i = 0; i < 6; i++) { strings[i] = console.nextLine(); } for (int i = 0; i < 5; i++) { if (strings[i] != null) { for (int j = i + 1; j < 6; j++) { if (strings[j] != null) { if (strings[i].equals(strings[j])) { strings[i] = ""; strings[j] = ""; } } } } } for (int i = 0; i < 6; i++) { if (strings[i] == "") { strings[i] = null; } } // for (int i = 0; i < strings.length; i++) { System.out.print(strings[i] + ", "); } } }

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

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

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

milyasow

Уровень 2

2 июня 2022, 19:16

Во внешнем цикле тоже 6-й элемент должен быть границей.

Dmitry Bolkhovskikh

Уровень 15

2 июня 2022, 20:27

А почему? 6 итерация работать не будет все равно.

milyasow

Уровень 2

2 июня 2022, 20:45

Ну да. Тогда возможно ему не нравится то, что где-то на середине цикла и "Hello", и "World" превратятся в "" и станут equals() друг другу. А это, по-хорошему, не соответствует условию задачи.

Dmitry Bolkhovskikh

Уровень 15

2 июня 2022, 22:01

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

Dmitry Bolkhovskikh

Уровень 15

2 июня 2022, 18:29

прошло проверку вот такое решение

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

import java.util.Scanner;
import java.util.Arrays;

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

public class Solution {
    public static String[] strings;

    public static void main(String[] args) {
        //напишите тут ваш код
        strings = new String[6];
        Scanner console = new Scanner(System.in);
        for (int i = 0; i < 6; i++) {
            strings[i] = console.nextLine();
        }

        for (int i = 0; i < 5; i++) {
            if (strings[i] != "") {
                String tmp = strings[i];
                for (int j = i + 1; j < 6; j++) {
                    if (strings[j] != "") {
                        if (strings[j].equals(tmp)) {
                            strings[i] = "";
                            strings[j] = "";
                        }
                    }

                }
            }

        }

        for (int i = 0; i < 6; i++) {
            if (strings[i] ==  "") {
                strings[i] = null;
            }
        }

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

Дмитрий Шпарута

Уровень 16

5 июня 2022, 08:22

А если подряд будут стоять 6 одинаковых слов, этот код работать не будет....

Dmitry Bolkhovskikh

Уровень 15

6 июня 2022, 10:23

Почему?

for (int i = 0; i < 5; i++) {
    if (strings[i] != null) {
        for (int j = i + 1; j < 6; j++) {
            if (strings[j] != null) {
                if (strings[i].equals(strings[j])) {
                    strings[i] = "";
                    strings[j] = "";

Этот фрагмент кода все 6 значений перебирает и преобразует.