8 уровень

Один случай не реализуется в задаче.

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

Решен

Если была одинаковая пара чисел (0 и 1), и затем еще одно (5)- то значения удаляются (null). Но как исправить код для случая, если была удалена пара чисел (null) (2 и 3), затем, в 12 строке для корректного сопоставления снова обновляется значение элемента на исходное (стр 12), но идентичных элементов больше нет и этот обновленный элемент так и не зануляется.. (то есть осталась t , ее пара теперь null, но она заново стала t)... как с этим быть?

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

Считать 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 console = new Scanner(System.in); strings = new String[6]; for (int i = 0; i < 6; i++) { strings[i] = console.nextLine();} for (int h = 0; h < 6; h++) { String temp = strings[h]; for (int u = 0; u < 6; u++) { strings[h] = temp; if (h != u && strings[h] != null){ if (temp.equals(strings[u])) { strings[u] = null; strings[h] = null; } } }} for (int i = 0; i < strings.length; i++) {//ПРАВИЛЬНЫЙ ВВЫВОД МАССИВА В КОНСОЛЬ!!! System.out.print(strings[i] + ", "); }}} // for (int j = 0; j < 6; j++) { //для сревнения первого элемента со всеми // for (int y = 0; y < 6; y++) { // if ((strings[j].equals(strings[y])) && (j != y) /*&& (strings[j] != null)*/) { // strings[y] = null;} // if (strings[y] == null) { // strings[j] = null;} // // } // } // for (int i = 0; i < strings.length; i++) {//ПРАВИЛЬНЫЙ ВВЫВОД МАССИВА В КОНСОЛЬ!!! // System.out.print(strings[i] + ", "); // } // // } //} // public static String[] strings; // public static void main(String[] args) { // Scanner console = new Scanner(System.in); // strings = new String[6]; // for (int i = 0; i < 6; i++) { // strings[i] = console.nextLine();} // // for (int j = 0; j < 6; j++) { // // String temp =strings[j]; // for (int j = 0; j < 6; j++) { //для сревнения первого элемента со всеми // //String temp =strings[j]; // for (int y = 0; y < 6; y++) { // if ((strings[j]==strings[y])&& (j!= y) /*&& (strings[j] != null)*/) { // strings[y] = null;} // if (strings[y] == null) { // strings[j] = null; // } continue; // //System.out.print(strings[j] + ", "); // } // } // for (int i = 0; i < strings.length; i++) {//ПРАВИЛЬНЫЙ ВВЫВОД МАССИВА В КОНСОЛЬ!!! // System.out.print(strings[i] + ", "); // } // //}}

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

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

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

Anonymous #3303140

Уровень 1

24 мая, 15:30

Ты его обнуляешь, а потом обратно записываешь:

strings[h] = temp;

Если ты хотела таким образом обойти возникающий NullPointerException, то это плохая попытка. У тебя же для этого и создана временная переменная temp, чтобы в ней всегда сохранялось неизменённое значение. Просто добавь проверку на то, что temp не равно null. PS Твои циклы делают немного лишней работы и лишних проверок. Не нужно внутренний цикл гонять от нуля, нужно перебирать только те строки, которые после h-того индекса. Поэтому внутренний цикл можно начинать с u = h + 1, тогда и проверка на то, что u != h не понадобится. А внешний цикл можно остановить на одну итерацию раньше. При этом в обоих случаях стоит вместо 6 использовать strings.length:

for (int h = 0; h < strings.length - 1; h++) {
    // ...
   for (int u = h + 1; u < strings.length; u++) {

Anonymous #3303140

Уровень 1

24 мая, 15:32

PPS. Экономия пустых строк и отступов программу не украшают, а ухудшают. Твой код в читаемом виде выглядит вот так:

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 console = new Scanner(System.in);
        strings = new String[6];
        for (int i = 0; i < 6; i++) {
            strings[i] = console.nextLine();
        }

        for (int h = 0; h < 6; h++) {
            String temp = strings[h];
            for (int u = 0; u < 6; u++) {
                strings[h] = temp;
                if (h != u && strings[h] != null) {
                    if (temp.equals(strings[u])) {
                        strings[u] = null;
                        strings[h] = null;
                    }
                }
            }
        }

        for (int i = 0; i < strings.length; i++) {//ПРАВИЛЬНЫЙ ВВЫВОД МАССИВА В КОНСОЛЬ!!!
            System.out.print(strings[i] + ", ");
        }
    }
}

ZLATA

Уровень 8

24 мая, 17:08

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

ZLATA

Уровень 8

24 мая, 17:20

1) спасибо огромное за подробный комментарий!! 2) а какая ,будет смысловая разница между 6 и strings.length в данном случае?

ZLATA

Уровень 8

24 мая, 17:23

а у меня есть проверка что temp!=null.строка 15 strings[h] != null

ZLATA

Уровень 8

24 мая, 17:28

проблема осталась (

ZLATA

Уровень 8

24 мая, 17:33

если уберу условие strings[h] = temp;, то подобная ситуация с повторяющейся стройкой на конце не прочитается ...

Anonymous #3303140

Уровень 1

25 мая, 05:53

А теперь давай вместе подумаем о том, зачем тебе вообще была нужна временная переменная, почему нельзя сразу сравнивать со strings[h]? Ответ: потому что после первого обнуления нам останется не с чем сравнивать, ведь первая пара станет null, и для третьей такой строки не найдется соответствия. Поэтому нам и требуется временная переменная, которая бы не изменялась в ходе внутреннего цикла. Так если у тебя есть такая переменная, то почему на null ты проверяешь не её, а strings[h], которую только что обнулила. Конечно у тебя это условие больше не выполнится после первого раза. А учитывая, что temp во внутреннем цикле не изменяется, проверку можно выполнить один раз до захода во внутренний цикл и всё.

ZLATA

Уровень 8

25 мая, 06:20

ааа получилось!!! <Поэтому нам и требуется временная переменная, которая бы не изменялась в ходе внутреннего цикла. >

Можно пожалуйста дополнительный вопрос, a почему после обнуления strings[h] = null; (1) (18 строка) не переинициализируется переменная String temp = strings[h]; на тоже на null (12 строка)? И разве не получается, что temp будет всегда не равна null (15 строка)? (спасибо огромное за помощь!)

Anonymous #3303140

Уровень 1

25 мая, 06:52

Вообще, это отличная задача для того, чтобы освоить отладку (дебаг). Ты можешь выполнять эту программу пошагово, наблюдая "онлайн" как при этом изменяются значения каждой из переменных. Это ответит на большинство твоих вопросов. strings[h] = null; - выполняется внутри внутреннего цикла. И внутренний цикл продолжает своё выполнение, пока не дойдёт до конца массива. И только после того, как он завершится, начнётся новая итерация внешнего цикла, в которой будет заново выполнена команда, создающая и инициализирующая переменную String temp = strings[h]. Только вот это самое strings[h] уже будет другое, ведь между итерациями цикла выполняется команда h++, то есть это strings[h] - это уже следующий элемент массива, не тот, который мы обнулили. И разве не получается, что temp будет всегда не равна null (15 строка)? Ну вот смотри, допустим у тебя первый и второй элементы массива одинаковые. Ты нашла эту пару, обнулила оба элемента, и при этом сравнила первый ещё и со всеми остальными, убедившись что больше итаких элементов нет. Что происходит дальше? Ты начинаешь сравнивать второй элемент аналогичным образом со всеми оставшимися. Но чему равен второй элемент? Он уже равен null, ведь он был одинаковым с первым элементом. Получается, что ты такой элемент уже искала по всему массиву, тебе можно этот шаг полностью пропустить. Более того, тебе уже и сравнивать-то не с чем, ведь элемент равен null, и у null нельзя вызывать метод equals (никакой метод вызывать нельзя :)). Поэтому тебе можно было бы сразу переходить к следующему h, а твой алгоритм делает следующее: берёт второй элемент, равный null, и начинает перебирать все оставшиеся, проверяя: если элемент, с которым сравниваем, не равен null... но он равен null, с каким бы элементом ты не собиралась его сравнить. Именно поэтому я писал, что проверку на null целесообразно вынести во внешний цикл, вообще не заходя во внутренний в этом случае.

Anonymous #3303140

Уровень 1

25 мая, 06:56

2) а какая ,будет смысловая разница между 6 и strings.length в данном случае? Вообще одно хорошее правило заключается в том, что у тебя в коде не должно использовать числовых литералов кроме 0 и 1, которые не хранились бы в переменных. Разница в том, что твоё число 6 в условии цикла взято просто с потолка, основываясь на том допущении, что в массиве всегда будет лежать 6 элементов. А еслси я вызову твой класс и выполню команду Solution.strings = new String[10];, но твоя шестёрка в цикле так и останется шестёркой, то вероятно код не станет работать так, как ты от него ожидаешь.

ZLATA

Уровень 8

31 мая, 10:16

Спасибо огромное! Разобралась!

Артём Злая Чебурашка

24 мая, 15:21

пересмотреть алгоритм. как вариант тебе никто не запрещает промежуточные коллекции использовать в работе.

ZLATA

Уровень 8

24 мая, 17:09

на 6 уровне еще не знаешь,кто такие коллекции ..

Артём Злая Чебурашка

24 мая, 18:54

1) массив это такая же "коллекция", если точнее структура данных для хранения. под тем же ArrayList под капотом точно такой же массив) 2) тебе никто не мешает изучить самостоятельно.

ZLATA

Уровень 8

25 мая, 06:22

Это правда) Спасибо за наводку ! :)

Anonymous #3303140

Уровень 1

25 мая, 06:38

Зачем в этой задаче коллекции?

Артём Злая Чебурашка

25 мая, 07:54

как пример можно в отдельный массив собрать строки, которые встречаются 2+ раз, а потом использовать это в работе)