JavaRush/Помощь по Java задачам/Мне кажется или условие задачи хромает?

16 уровень

20.12.2020
496просмотров
17комментариев

Мне кажется или условие задачи хромает?

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

Архивный

В условии задачи написано ввести 10 строк, однако входные данные содержат 11 строк из за чего возникает неверный резултат) Входные данные: Java Java Hello Increment Decrement Hello World Integer Double Integer Android Результат: null null null Increment Decrement null World null Double null

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

Считать 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.Arrays; import java.util.Scanner; /* Удаляем одинаковые строки */ public class Solution { public static String[] strings; public static void main(String[] args) { Scanner sc = new Scanner(System.in); String[] strings = new String[10]; for (int i = 0; i < strings.length; i++) { strings[i] = sc.nextLine(); } for (int i = 0; i < strings.length; i++) { for (int j = 0; j < strings.length; j++) { if ( j == i ) { continue; } else { if (strings[i] != null && strings[j] != null ){ if (strings[i].equals(strings[j])) { strings[i] = null; strings[j] = null; } } } } } Arrays.stream(strings).forEach(System.out::println); } }

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

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

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

hidden #2322530

Уровень 41

20 декабря 2020, 09:40

не надо тут создавать локальную переменную под массив. надо использовать статическую, что было дано изначально

public static String[] strings;

public static void main(String[] args) {
    Scanner sc = new Scanner(System.in);
    String[]  strings = new String[10];

если изменить условие на int j = i + 1, то от условия if можно избавиться.

for (int j = 0; j < strings.length; j++) {
    if ( j == i ) {
        continue;

Антон Антошин

Уровень 16

20 декабря 2020, 09:41

а вопрос, чем мое решение отлично от правильного?

hidden #2322530

Уровень 41

20 декабря 2020, 09:42

я тебе написал. ты инициализируешь, заполняешь и удаляешь не в той переменной!

String[]  strings = new String[10];

убери тут String[] , тогда будет не объявление локальной переменной, а инициализация статической. работать надо со статической переменной, а не создавать свои.

Антон Антошин

Уровень 16

20 декабря 2020, 09:48

Так? public class Solution { public static String[] strings = new String[10]; public static void main(String[] args) {

hidden #2322530

Уровень 41

20 декабря 2020, 09:51

я выше написал как. ты вообще читаешь комментарии?

Антон Антошин

Уровень 16

20 декабря 2020, 09:56

Да, еще раз прочел и понял, что вы имели ввиду. Однако подскажите в условии задачи сказано ввести 10 строк, а почему входных параметров 11?

hidden #2322530

Уровень 41

20 декабря 2020, 09:58

а где ты увидел 11 ? если при сдаче задачи, то не обращай внимания, значит тест коряво написан, считано будет всё равно 10.

Антон Антошин

Уровень 16

20 декабря 2020, 10:03

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

hidden #2322530

Уровень 41

20 декабря 2020, 10:05

покажи обновлённый код

Антон Антошин

Уровень 16

20 декабря 2020, 10:12

package com.javarush.task.pro.task05.task0508; import java.util.Arrays; 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[10]; for (int i = 0; i < strings.length; i++) { strings[i] = sc.nextLine(); } for (int i = 0; i < strings.length; i++) { for (int j = 0; j < strings.length; j++) { if ( j == i ) { continue; } else { if (strings[i] != null && strings[j] != null ){ if (strings[i].equals(strings[j])) { strings[i] = null; strings[j] = null; } } } } } Arrays.stream(strings).forEach(System.out::println); } }

hidden #2322530

Уровень 41

20 декабря 2020, 10:16

ты не учитываешь, что одинаковых строк может быть больше 2х.

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

у тебя если две одинаковых нашлись, ты обнуляешь

strings[i] = null;

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

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

обнулять strings[i] = null; надо только тогда, когда вложенный цикл закончит полностью свою работу, и будет хотя бы одна strings[j] = null;

Антон Антошин

Уровень 16

20 декабря 2020, 10:54

Спасибо, решил...

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

import java.util.Arrays;
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[10];
        int s;
        for (int i = 0; i < strings.length; i++) {
            strings[i] = sc.nextLine();
        }

        for (int i = 0; i < strings.length; i++) {
            s = 0;
            for (int j = 0; j < strings.length; j++) {
                if ( j == i ) {
                    continue;
                    } else {
                    if (strings[i] != null && strings[j] != null ){
                        if (strings[i].equals(strings[j])) {
                            strings[j] = null;
                            s++;
                        }
                        }
                      }
                }
            if ( s > 0 ) {
                strings[i] = null;
            }
        }
        Arrays.stream(strings).forEach(System.out::println);
    }
}

Антон Антошин

Уровень 16

20 декабря 2020, 10:55

Подскажите, как бы я мог упростить эту конструкцию, заранее спасибо

Pavlo PlynkoJava-разработчик в CodeGym

Expert

22 декабря 2020, 17:12

1. 17 строку можно убрать, а в 23 строке сделать int s = 0; Не критично, но минус одна ненужная строка. И s лучше переименовать в что-то типа count. 2. в 27 строке убрать else, т.е. получится так:

if ( j == i ) {
    continue;
}
if (strings[i] != null && strings[j] != null ) {
    ...

3. как вариант два if-а заменить одним:

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

но тут под вопросом, что монструозней...

Павел

Уровень 33

23 декабря 2020, 15:38

Примерно так можно упростить:

import java.util.Arrays;
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[10];
        for (int a = 0; a < 10; a++)
            strings[a] = scanner.nextLine();
        for (int i = 0; i < 10; i++) {
            int cout = 0;
            for (int j = i + 1; j < 10; j++) {
                if (strings[i] != null && strings[j] != null && strings[i].equals(strings[j])) {
                    strings[j] = null;
                    cout++;
                }
            }
            if (cout > 0)
                strings[i] = null;
        }
         Arrays.stream(strings).forEach(System.out::println);
    }
}

Павел

Уровень 33

23 декабря 2020, 15:39

если учесть все советы.

hidden #2322530

Уровень 41

23 декабря 2020, 18:11

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

вот так будет аккуратнее. если strings[i] != null, то без разницы с чем сравнивать. у null нельзя метод вызывать, но как аргумент его сравнивать можно. хардкодинг правильней исправить на переменные.

for (int i = 0; i < strings.length; i++) {
    for (int j = i + 1; j < strings.length; j++) {

ещё если бороться за строчками, то можно вот так cout перекинуть в условие цикла)

for (int i = 0, cout = 0; i < 10; i++, cout = 0) {

объявляем 0, и после каждой итерации, обнуляем снова.