List配列で重複した値を持つオブジェクトの値を編集したい

実現したいこと

javaでとあるシステムを開発しているのですが、ロジックが思いつかないので、何かいい案を教えてください。

FruitのListでnameが重複してるFruitのnameを空文字にしたいです。

java
1public class Sample {
2
3    public static void main(String[] args) {
4
5        // リストに値を詰める
6        List<Fruit> fruit = new ArrayList<>();
7        fruit.add(new Fruit("りんご", "0001"));
8        fruit.add(new Fruit("ばなな", "0002"));
9        fruit.add(new Fruit("りんご", "0003"));
10        fruit.add(new Fruit("ぶどう", "0004"));
11        fruit.add(new Fruit("ぶどう", "0005"));
12        fruit.add(new Fruit("りんご", "0006"));
13        fruit.add(new Fruit("いちご", "0007"));
14        fruit.add(new Fruit("ばなな", "0008"));
15
16        // 重複したnameを空文字にしたい...
17        // 期待値　↴
18        // name = "りんご"、no = "0001"
19        // name = "ばなな"、no = "0002"
20        // name = ""      、no = "0003"
21        // name = "ぶどう"、no = "0004"
22        // name = ""      、no = "0005"
23        // name = ""      、no = "0006"
24        // name = "いちご"、no = "0007"
25        // name = ""      、no = "0008"
26
27    }
28}

java
1public class Fruit() {
2    // フィールド
3    private String name;
4    private String no;
5
6    // コンストラクタ
7    public Fruit(String name, String no) {
8        this.name = name;
9        this.no = no;
10    }
11    // getter,setterは割愛...
12    ...
13}

行動規範の内容に同意します

回答3件

ベストアンサー

方法1

あるFruitよりあとのFruitで、同じ名前のものがあればそれを空文字に変える

java
1for (int i = 0; i < list.size(); i++) {
2    String nameA = list.get(i).getName();
3    if (nameA.equals("")) continue;
4    for (int j = i + 1; j < list.size(); j++) {
5        if (nameA.equals(list.get(j).getName())){
6            list.get(j).setName("");
7        }
8    }
9}

方法2

名前の重複をSetで管理

java
1Set<String> names = new HashSet<>();
2for (Fruit fruit : list) {
3    if (!names.add(fruit.getName())){
4        fruit.setName("");
5    }
6}

投稿2018/06/18 13:35

swordone

総合スコア20675

Yux

2018/06/21 02:49 編集

方法2のHashSetを使ったやり方は大変勉強になりました。ありがとうございました。

行動規範の内容に同意します

addメソッドで追加するときに、要素を検索して存在してれば””にして登録する、ということをすればいいんじゃないかと

投稿2018/06/18 13:27

y_waiwai

総合スコア88163

Yux

2018/06/21 12:17

addメソッドを追加するタイミングで悩んでいました。

行動規範の内容に同意します

javascript
1(()=>{
2  class Fruit {
3    constructor(name, no){ this.setName(name); this.setNo(no); }
4    setName(name){ this.getName = ()=>{ return name;} }
5    setNo(no){ this.getNo = ()=>{ return no; } }
6  };
7
8  class ArrayList {
9    constructor(){ let array = []; this.toArray = ()=>{return array;}; }
10    add(item){ this.toArray().push(item); return this; }
11    get(index){ return this.toArray()[index]; }
12    forEach(callback){ return this.toArray().forEach(callback); }
13    indexOf(item){ return this.toArray().indexOf(item); }
14  };
15  
16  let fruits = new ArrayList();
17  fruits.add(new Fruit("りんご", "0001"))
18        .add(new Fruit("ばなな", "0002"))
19        .add(new Fruit("りんご", "0003"))
20        .add(new Fruit("ぶどう", "0004"))
21        .add(new Fruit("ぶどう", "0005"))
22        .add(new Fruit("りんご", "0006"))
23        .add(new Fruit("いちご", "0007"))
24        .add(new Fruit("ばなな", "0008"));
25
26  let nameList = new ArrayList();
27  fruits.forEach((v, i)=>{
28    if(nameList.indexOf(v.getName()) >= 0){ fruits.get(i).setName(""); return; }
29    nameList.add(v.getName());
30  });
31  
32  fruits.forEach(v=>{
33    console.log("name:" + v.getName() + ",no:" + v.getNo());
34  });
35
36})();