JAVA:ArrayListでの正しいソースの書き方

Question

前提・実現したいこと

独学でjavaを初めてまだ数か月です。ArrayListの使い方を自習しようと、ArrayでかかれたプログラムをArrayListで書き直していますが、独学のためどこがおかしいのか分かりません。正しいコードを示していただけると非常に助かります。

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

一通りArrayListを使ったソースに書き直しましたがコンソールには表示が何も出ません。(エラーも出ません)
独学者のためどこが間違っているのかわかりません。

該当のソースコード

package Ü2_2_1;
import java.util.ArrayList;
public class Album {

public String albumTitle;

public  ArrayList<Song> songs;

public Album(String albumTitle) {
	this.albumTitle = albumTitle;
	songs = new ArrayList<Song>();
}

public void addSong(Song song) {
	for(Song s : songs) {
		if(s == null) {
		  songs.add(song);
		  return;
		}
	}
}

public int getSongCount() {
	for(int i=0; i < songs.size(); i++) {
		if(songs == null ) {
			return i;
	    }
    }
	return songs.size();
}

public int getAlbumDuration() {
	int sum = 0;
	for(Song s : songs) {
		if(s != null) {
			sum += s.getDuration();
		}
	}
	return sum;
}

public void printSong() {
	for(Song s : songs) {
		if(s != null) {
		System.out.println(s.toString());
		}
	}
}

}

試したこと

ここに問題に対して試したことを記載してください。

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

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

Comments

[退会済みユーザー]

・コードは```で囲んでください。
・そもそもこのクラスはどこから使っていますか？このクラスだけコンパイルしても何も起きないと思いますが。使用箇所も含め全文載せてください。
・最低限何をするクラスなのか説明してください。

[m.ts10806]

コードはマークダウンのcode機能を利用してご提示ください。
「正しいコード」はあくまで要件を満たすものである必要があります。
要件やゴール、仕様をきちんと具体的に記載してもらう必要があります。
つまり、現時点の内容では「正しいコード」を他者が提供するのは無理ということです。きちんと説明追記してください。
（それにおおっぴらに「コードくれ」って作業依頼のように受け取れますが・・・？）

[m.ts10806]

それに単にmainメソッドがないから実行できないだけでは・・・？
エラー、出るのでは。
どういう手順でコードを作成し、実行しているのでしょうか？

[m.ts10806]

あと「Arrayでかかれたプログラム」をご提示ください。要は動いてたものですね。
単にArrayListで書き換えただけなら動かないはずです。

[ShinShin82]

>yukke様, m.ts10806様
申し訳ありません。はじめて、こういった質問をさせてもらったので、ルールがいまいち分かっていませんでした。
ちなみにメインメソッドはあります。クラス毎でタブを分けているため別タブに分けています。（Eclipseを使用しています）。
はい、Arrayで書かれたプログラムでは正常に動いていました。ある課題でArrayListを学んだのでArrayで書かれたコードをArrayListで組み替えれば良い練習になると思い取り組んだ次第です。

[jimbe]

このご質問自体を修正出来ます.

Answer 1

配列から ArrayList に書き直してるとのことで、元のコードは適当な長さの配列を用意して空いてる（null がある）場所に新しい項目を追加してたと思いますが、ArrayList では基本的に null は入れないので、null チェックは不要です。
（厳密には null を add することもできなくはありませんが、普通はしません。）

    public void addSong(Song song) {
        for(Song s : songs) {
            if(s == null) {
                songs.add(song);
                return;
            }   
        }
    }

例えば Album に 3 曲入ってるとすると、元のコードでは songs は

+---------+---------+---------+---------+---------+---------+---------+
| Song #1 | Song #2 | Song #3 |  null   |  null   |  null   |  null   |
+---------+---------+---------+---------+---------+---------+---------+

みたいな配列で、最初の null の場所を探してそこに新しく Song #4 を入れてたんだと思いますが、ArrayList では入ってる Song の分しか領域はないと思って構いません。ので、

+---------+---------+---------+
| Song #1 | Song #2 | Song #3 |
+---------+---------+---------+

という状態です。で、Song #4 を追加するには単に songs.add(song) するだけで以下のようになります。

+---------+---------+---------+---------+
| Song #1 | Song #2 | Song #3 | Song #4 |
+---------+---------+---------+---------+

他のメソッドでも、基本的に null チェックは不要です。

あと、albumTitle や songs を public にしていると外からいじれてしまう（勝手に null を入れられるかも）なので、private にして、必要ならば getter や setter を定義することをお勧めします。

public class Album {
    private String albumTitle;
    private ArrayList<Song> songs;

    // コンストラクタ略

    public String getAlbumTitle() {
        return albumTitle;
    }

    // 以下略

Comments

[momon-ga]

あ、根底にあるのは配列(array)とArrayListを同じように扱っているから、nullのチェックとかしてそうですね。

[hoshi-takanori]

他の言語と違って、Java の配列は長さが変えられないですからね。

Answer 2

songsにnullって、入れる前提ですか？
入っていない場合、songs.addは、呼ばれません。

songs.add(song);が、呼ばれる条件は。
１．songsに１件以上Songは入っていること。(0件だとforループの中を通らない)
２．songsにnullが入っていること(Songがnullでないと、ifの中を通らない)
の２つを満たす場合です。

	public void addSong(Song song) {
		for (Song s : songs) {
			if (s == null) {
				songs.add(song);
				return;
			}
		}
	}

下記のように条件を満たすように、nullを追加すれば表示されますが。
期待の動作ではないのでは？

	public static void main(String[] args) {
		Album myAlbum = new Album("title");

		myAlbum.addSong(new Song());
		myAlbum.printSong();	// 何も表示しない(songが追加されない)

		System.out.println("null 追加");
		myAlbum.songs.add(null);

		myAlbum.addSong(new Song());
		myAlbum.printSong();	// 表示される


		Album myAlbum2 = new Album("title");

		System.out.println("Songを直接追加");
		myAlbum2.songs.add(new Song());

		myAlbum2.addSong(new Song()); // このSongは追加されない
		myAlbum2.printSong();	// 直接追加したSongのみ表示される
	}

Answer 3

mainメソッドがどこにもないようですが？