ArrayList データの削除

Question

###前提・実現したいこと
ArrayListで一番最後に追加されたデータを削除したいです。

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

###該当のソースコード

public String pop(){
		if(list.equals(0)){
			return null;
		}
		return list.get(list.size() - 1);
		list.remove(list.size() - 1);
	}```


###補足情報(言語/FW/ツール等のバージョンなど)
public String pop()
・フィールドlistのデータ個数が0ならばnullを返す
・一番最後に追加されたデータを返す
・返すデータ（一番最後に追加されたデータ）をフィールドlistから削除する
こんな感じの課題です。

Answer 1

public String pop() {
    return list.isEmpty() ? null : list.remove(list.size() - 1);
}

これだけで十分です。remove(int)は対象インデックスの要素をリストから削除すると同時に、その要素を返します。

Answer 2

まず、リストをLinkedListで宣言すればそもそも pop() メソッドが提供されますが、恐らく質問者さんの意向とは違っていると思うので省略します。

---

コードの誤りについて

if(list.equals(0)){
    return null;
}

equals()メソッドはオブジェクトとオブジェクトが等しいかどうかを判定するものでリスト内の数を表すものではありません。
要素数を調べたいときはsize()メソッドを使います。

return list.get(list.size() - 1);
list.remove(list.size() - 1);

これだとlist.get(list.size() - 1);を実行した時点でreturnされ処理が終わるのでlist.remove(list.size() - 1);が実行されません。

コードを修正すると次のようになります。

public String pop() {
    if (list.size() == 0) {
        return null;
    }

    String item = list.get(list.size() - 1);
    list.remove(list.size() - 1);
    return item;
}