JavaのArrayListでremoveが上手くいかない

Question

前提・実現したいこと

独自のクラスを引数にインスタンス化したArrayListで、removeをしたい

発生している問題・エラーメッセージ

falseで削除できない

該当のソースコード

package dictionary;

import java.util.ArrayList;
import java.util.Collections;
import java.util.Iterator;
import java.util.List;

public class Dictionary {
	@SuppressWarnings("rawtypes")
	private List dictionaryList;
	private int displayNumber;

	public int getDisplayNumber() {
		return displayNumber;
	}

	public void setDisplayNumber(int displayNumber) {
		this.displayNumber = displayNumber;
	}

	public Dictionary() {
		dictionaryList = new ArrayList<>();
	}

	@SuppressWarnings("unchecked")
	public void put(String word, String explanation)throws NullPointerException{
		dictionarySet dSet = new dictionarySet(word,explanation);
		dictionaryList.add(dSet);
		Iterator<dictionarySet> iterator = dictionaryList.iterator();
		/*for(@SuppressWarnings("rawtypes")
		ListIterator it = dictionaryList.listIterator(dictionaryList.size()); it.hasPrevious();) {
			if(it.previous().equals(word)) {
				dictionaryList.remove(iterator.next().getWord());
			}
		}*/
		while(iterator.hasNext()) {
			//String removeStr = ;
			if(iterator.next().getWord().equals(word) && iterator.hasNext() == true){
				System.out.println(dictionaryList.remove(word));
				break;
			}
		}
		/*iterator = dictionaryList.iterator();
		while (iterator.hasNext()) {
			System.out.println(iterator.next().getWord());
		}*/

		//dictionaryList.add(dSet);
	}

	public void getAll() {
		sort();
		@SuppressWarnings("unchecked")
		Iterator<dictionarySet> it = dictionaryList.iterator();
		while (it.hasNext()) {
        	dictionarySet  data = it.next();
            System.out.println(data.getWord() + ":" + data.getExplanation());
		}
	}

	public void getOnePage(int page) {

	}

	@SuppressWarnings("unchecked")
	public void sort() {
		Collections.sort(dictionaryList);
	}

	public void clear() {
		dictionaryList.clear();
	}
}

package dictionary;

public class dictionarySet implements Comparable<dictionarySet>{
	private String word;
	private String explanation;


	public dictionarySet(String word, String explanation) {
		super();
		this.word = word;
		this.explanation = explanation;
	}
	public String getWord() {
		return word;
	}
	public void setWord(String word) {
		this.word = word;
	}
	public String getExplanation() {
		return explanation;
	}
	public void setExplanation(String explanation) {
		this.explanation = explanation;
	}

	@Override
    public int compareTo(dictionarySet dictionarySet){
        return word.compareTo(dictionarySet.word);
    }



}

試したこと

勉強中なのでできる限りjavadocなどドキュメントを読み漁りましたが分かりません。

補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

Comments

[swordone]

実際に実行しているコードを提示してください。

Answer 1

自己解決したました。
ご協力下さった皆さんありがとうございます。
調べたところIteratorは受け取ったコレクションのアドレスを参照すると言うことで、
if(iterator.next().getWord().equals(word) && iterator.hasNext() == true){
iterator.remove()
}
これで指定した要素を含む場合(重複する場合)だけremoveすることができました。
次回から質問の内容はより細かく記載したいと思います。

Answer 2

iterator を使用中にその元である List から直接 remove することはできません.
iterator の remove メソッドを使うか, iterator を使わずにループしてください.

失礼しました.
false というのは

System.out.println(dictionaryList.remove(word));

のことですね

remove

(o==null ? get(i)==null : o.equals(get(i)))となる、最小のインデックス値iを持つ要素を削除します

ですので, (word が null で無ければ) word.equals(dictionaryList.get(i)) が成立しなければ削除にはなりません.
word は String であり, dictionaryList.get(i) は dictionarySet ですから, どうやっても削除されないでしょう. ( dictionarySet には equals や hashCode の定義もありません.)

なお, この状態は @SuppressWarnings など付けずに List<dictionarySet> dictionaryList ときちんと型を書いてあれば, Eclipse において該当行で「ありそうもない Collenction<dictionarySet> の remove(Object) の引数型 String」(Plaiades訳) という警告が出て気付けます.

#追記
put が「word が重複したら上書き」するというのであれば, そして内部では word でソートするのであれば, TreeMap を使用しては如何でしょうか.
自分で探して削除したりソートしたりは必要無くなります.

package teratail.q252580;
//package dictionary;

import java.util.*;

public class Dictionary {
  private Map<String,Item> map = new TreeMap<>();
  private int displayNumber;

  public static void main(String args[]) {
    Dictionary dic = new Dictionary();
    dic.put("B", "BBB");
    dic.put("C", "CCC");
    dic.put("A", "AAA");
    dic.printAll();
    System.out.println();
    dic.put("B", "BBBbbbbb");
    dic.printAll();
  }
  public int getDisplayNumber() {
    return displayNumber;
  }
  public void setDisplayNumber(int displayNumber) {
    this.displayNumber = displayNumber;
  }
  public void put(String word, String explanation) {
    map.put(word, new Item(word, explanation));
  }
  public void printAll() {
    map.forEach((k,v) -> System.out.println(v));
  }
  public void getOnePage(int page) {
  }
  public void clear() {
    map.clear();
  }

  private static class Item implements Comparable<Item>{
    private String word;
    private String explanation;

    public Item(String word, String explanation) {
      this.word = word;
      this.explanation = explanation;
    }
    @Override
    public int compareTo(Item item){
      return word.compareTo(item.word);
    }
    @Override
    public String toString() {
      return word + ":" + explanation;
    }
  }
}

A:AAA
B:BBB
C:CCC

A:AAA
B:BBBbbbbb
C:CCC

Comments

[tetra]

早速のご返信ありがとうございます。
メンバー変数をList< dictionarySet>に変更することで警告が消えました。
ありがとうございます。