Как удалить повторяющихся элементов из массива?

Question

[Elxan Mecidli]

package com.company;
import java.util.Arrays;
import java.util.LinkedHashSet;
public class Praktika15 {
    public static void main(String[] args) {
        int[] arr={10, 22, 10, 20, 11, 22};

        System.out.println("Исходный массив:"+Arrays.toString(arr));
        LinkedHashSet<Integer> set=new LinkedHashSet<Integer>();
        for(int i =0;i< arr.length-1;i++){
            for(int j=i+1;j< arr.length;j++){
                if(arr[i]==arr[j]){
                    set.remove();

                }
            }

        }



    }
}

Написал такой код.Как используя remove() удалить повторяющихся элементов из массива?
Пишу set.remove(arr,j); дает ошибку почему?

Comments

[Developer]

телепатов здесь нет

[Шамиль Хайрулаев]

Код ошибки какой?

[Шамиль Хайрулаев]

Зачем ты преобразуешь массив в строку?

[Шамиль Хайрулаев]

У тебя же set вообще пустой.

Answer 1

[Орхан Гасанлы]

Добрый день!
Рекомендую изучить JCF. А также обратить внимание на Map тоже.
В Set элементы являются уникальными.
Соответственно, необязательно все проверять на равенство.
Достаточно итерировать по массиву int[] arr={10, 22, 10, 20, 11, 22}; и добавить каждый элемент в Set. Повторяющиеся значения не будут добавлены

import java.util.Arrays;
import java.util.HashSet;
import java.util.List;
import java.util.Set;

public class Main {

    public static void main(String[] args) {
        int[] arr={10, 22, 10, 20, 11, 22};
        Set set = new HashSet();
        for (int i = 0; i < arr.length; i++) {
            set.add(arr[i]);
        }
        set.forEach(System.out::println);
    }

}

Comments

[Elxan Mecidli]

А можно в мой код добавить что-то чтобы сработал?Я не до конца сделал

[Орхан Гасанлы]

package com.company;
import java.util.Arrays;
import java.util.LinkedHashSet;
public class Praktika15 {
    public static void main(String[] args) {
        int[] arr={10, 22, 10, 20, 11, 22};

        System.out.println("Исходный массив:"+Arrays.toString(arr));
        LinkedHashSet<Integer> set=new LinkedHashSet<Integer>();
        for(int i =0;i< arr.length-1;i++){
          set.add(arr[i]);
        }

    }
}

Смысла реализовывать ваш код нет, так как он комплицирован.
1) вы итерируете двумя циклами по одному массиву, чтобы получить сам (элемент) и (элемент + 1).

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

2) Затем сравниваете (элемент) и (элемент+1)
if(arr[i]==arr[j]){

3) А затем при выолнении условия удаляете элемент из LinkedHashSet.
set.remove();
Притом, что сам элемент никогда и не был добавлен в LinkedHashSet. А если добавить элемент в LinkedHashSet, то в силу того, что Set гарантирует уникальность элементов, то и смысла в ваших проверках не будет.

Отсюда простое решение - просто добавьте элементы в Set и они будут уникальными

[Шамиль Хайрулаев]

E Java,

package com.company;
import java.util.Arrays;
import java.util.LinkedHashSet;
public class Praktika15 {
    public static void main(String[] args) {
        int[] arr={10, 22, 10, 20, 11, 22};

        System.out.println("Исходный массив:"+Arrays.toString(arr));
        LinkedHashSet<Integer> set=new LinkedHashSet<Integer>();
        
        for(int i = 0; i < arr.length-1; i++) {
            set.add(arr[i]);
        }
        
        System.out.println("Новый массив: " + set);
    }
}

[Шамиль Хайрулаев]

E Java, Проблема в том, что ты создал LinkedHashSet, но ничего в него не положил.
А это вообще не нужно :

for(int i =0;i< arr.length-1;i++){
            for(int j=i+1;j< arr.length;j++){
                if(arr[i]==arr[j]){
                    set.remove();

                }
            }

        }

так как, linkedHashSet сам удаляет одинаковые элементы внутри него

Answer 2

[BorLaze]

А что, на этом сайте уже можно просить решения ДЗ?

Answer 3

[Dmitry Roo]

1. В set все элементы уникальны, повторяющихся там нет.
2. Чтобы удалить элемент из set нужно чтобы он там был, а у вас set - пустой.