2

Что неверно в коде? Почему не работает? Нужно удалить дубликаты из массива.

public class Dublicate {

    public String[] delDublicate(String[] args) {
        StringBuffer result = new StringBuffer();
        StringBuffer esult = new StringBuffer();
        int i=1;

        for (int j=0; j <args.length; j++) {
            result.append(args[i]);
            String res = result.toString();
            esult.append(args[j]);
            String re = esult.toString();
            if (res.equals(re)){
                if (args[j]!= args[i]) {
                    args[i] = " ";
                }
            }
        }

        return args;
    }

    public static void main(String[] args){
        String [] res={"Привет", "Привет", "задание", "выполнено", "верно", "задание"};
        Dublicate dub=new Dublicate();
        dub.delDublicate(res);
        System.out.print(Arrays.deepToString(res));
    }
}
4
  • 5я строка int i = 1; что она делает? Вы используете ее, как ключ для доступа к элементам массива, но она у Вас все время равна 1.
    – XelaNimed
    5 сен 2016 в 18:55
  • Это индекс массива, для наглядности, если даже вместо него нарисовать цикл for c увеличением на 1, код не дает ожидаемого результата, пытаюсь понять где ошибка
    – Azamat
    5 сен 2016 в 18:59
  • Вы сравниваете res.equals(re), при этом res заполняется " Привет" на каждой итерации, внутрь if никогда не попадете. Вообще зачем использовать StringBuffer?
    – a.chugunov
    5 сен 2016 в 19:05
  • При args[0], тоже почему то не попадаю, хотя должен, StringBuffer для форматирования из args[i] в String
    – Azamat
    5 сен 2016 в 19:11

3 ответа 3

5

Я бы сделал примерно так

public String[]delDublicate(String[] args) {
    List<String> withoutDublicate = new ArrayList<>();
    for (String str : args) {
        if (!withoutDublicate.contains(str)) {
            withoutDublicate.add(str);
        }
    }

    String[] returnArr = new String[withoutDublicate.size()];
    return withoutDublicate.toArray(returnArr);
}

public static void main(String[] args){
    String [] res={" Привет ", " Привет", " задание", " выполнено", " верно", " задание"};
    Dublicate dub=new Dublicate();
    String[] arr = dub.delDublicate(res);
    System.out.print(Arrays.deepToString(arr));

}
3
import java.util.Arrays;

public class Dublicate {
    public void delDublicate(String[] args) {
        for (int i = args.length-1; i > 0; i--) {
            StringBuffer result = new StringBuffer();
            result.append(args[i]);
            String res = result.toString();
            for (int j = 0; j < args.length; j++) {
                StringBuffer esult = new StringBuffer();
                esult.append(args[j]);
                String re = esult.toString();
                if (i != j) {
                    if (res.equals(re)) {
                        args[i] = "";
                    }
                }
            }
        }
    }
    public static void main(String[] args) {
        String[] res = {"Привет", "Привет", "задание", "выполнено", "верно", "задание"};
        Dublicate dub = new Dublicate();
        dub.delDublicate(res);
        System.out.print(Arrays.deepToString(res));
    }
}
8
  • Убрал пробелы, не помогло
    – Azamat
    5 сен 2016 в 19:13
  • как предлагаешь написать НЕ РАВНО с применением equals? Мне нужно сравнить индексы а не значения, значения я сравниваю здесь: res.equals(re), т.е. одинаковые значения в разных индексах
    – Azamat
    5 сен 2016 в 19:58
  • 1
    !str1.equals(str2) вот твое не равно. И уточни, какие индексы сравниваешь здесь: args[j]!= args[i] ? Посмотри здесь про сравнение строк ru.stackoverflow.com/questions/417405/… 5 сен 2016 в 20:10
  • 1
    Между циклами бесполезный код инициализации объекта StringBuffer и перегон его обратно в строку
    – a.chugunov
    5 сен 2016 в 20:48
  • 1
    Спасибо, очень помог!
    – Azamat
    5 сен 2016 в 20:53
2

Можно попробовать вот так:

 String[] res = {"Привет", "Привет", "задание", "выполнено", "верно", "задание"};
 String[] array = Arrays.stream(res).distinct().toArray(String[]::new);
 System.out.println(Arrays.toString(array));

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.