質問編集履歴

タイトルの一部を変更しました。（「メモリ解放したい」→「closeしたい」）

2021/01/04 14:36

投稿

pami

スコア4

test CHANGED Viewed

	@@ -1 +1 @@
1	- 使い終えたjavax.sound.sampled.Clipを自動で~~メモリ解放~~したい。
1	+ 使い終えたjavax.sound.sampled.Clipを自動でcloseしたい。

test CHANGED Viewed

File without changes

MusicPlayerクラスを呼び出すMusicPlayerManagerクラスのコードを追記しました。また、助言を受けて試したことの結果を追記しました。

2021/01/04 14:36

投稿

pami

スコア4

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -186,7 +186,149 @@
-MusicPlayerのインスタンスは他のクラスで生成され、そのクラスのArrayListに格納・保持されています。
+MusicPlayerのインスタンスはMusicPlayerManagerクラスで生成され、このクラスのArrayListに格納・保持されています。
+MusicPlayerManager.java
+```Java
+import java.util.ArrayList;
+import java.util.Queue;
+import java.util.ArrayDeque;
+import javax.sound.sampled.Clip;
+public class MusicPlayerManager {
+    private static ArrayList<String> filePathList;
+    private static ArrayList<MusicPlayer> musicPlayers;
+    private static RhythmManager rhythmManager;
+    // 何番にPlay指示が出たかを一時保管し、リズムに合わせて処理するためのバッファ
+    private static Queue<Integer> orderBuffer;
+    public MusicPlayerManager() {
+        filePathList = new ArrayList<String>();
+        musicPlayers = new ArrayList<MusicPlayer>();
+        rhythmManager = new RhythmManager(150);
+        orderBuffer = new ArrayDeque<>();
+    }
+    public MusicPlayerManager(String[] filePath) {
+        this();
+        for(String filePath_ : filePath) {
+            filePathList.add(filePath_);
+        }
+    }
+    public MusicPlayerManager(ArrayList<String> filePath) {
+        this();
+        for(String filePath_ : filePath) {
+            filePathList.add(filePath_);
+        }
+    }
+    public void Play(int fileNum) {
+        // 処理待機バッファ内のfileNumの重複を避ける
+        if(!orderBuffer.contains(fileNum))
+            orderBuffer.add(fileNum);
+    }
+    public void AddFile(String filePath) {
+        filePathList.add(filePath);
+    }
+    public int GetFileTotalNum() {
+        return filePathList.size();
+    }
+    // RhythmManagerに呼び出されるメソッド
+    // リズムに合わせて一定間隔で呼び出される
+    public static void PlayAllInOrderBuffer()
+    {
+        // キューの先頭を取得し削除しながらループする
+        while(orderBuffer.peek() != null)
+        {
+            int buf = orderBuffer.poll();
+            musicPlayers.add(new MusicPlayer(filePathList.get(buf)));
+        }
+    }
+}
+```
@@ -198,6 +340,92 @@
 - javax.swing.TimerとActionListenerを用いて同様のプログラムを作成しましたが、動作は大差ありませんでした。
+- m-oguraさんの助言を受け、try-with-resourcesを用いてMusicPlayer.javaを下のように修正しましたが、タスクマネージャーでOpenJDK Platform binaryアプリ（起動中のアプリ）のメモリを監視したところ、音が鳴るたびにメモリ量は増え続けており、ストリームはclose出来ていないようでした。
+```Java
+import java.io.File;
+import javax.sound.sampled.AudioFormat;
+import javax.sound.sampled.AudioInputStream;
+import javax.sound.sampled.AudioSystem;
+import javax.sound.sampled.DataLine;
+import javax.sound.sampled.Clip;
+// Exceptions
+import java.io.IOException;
+import javax.sound.sampled.LineUnavailableException;
+import javax.sound.sampled.UnsupportedAudioFileException;
+public class MusicPlayer
+{
+    public MusicPlayer(String filePath)
+    {
+        LoadClipPlayer(filePath);
+    }
+    private static void LoadClipPlayer(String filePath) {
+        try(AudioInputStream stream = AudioSystem.getAudioInputStream(new File(filePath));)
+        {
+            AudioFormat format = stream.getFormat();
+            DataLine.Info dataLine = new DataLine.Info(Clip.class, format);
+            Clip clip = (Clip)AudioSystem.getLine(dataLine);
+            clip.open(stream);
+            clip.start();
+        }
+        catch(UnsupportedAudioFileException e) {
+            e.printStackTrace();
+        }
+        catch(LineUnavailableException e) {
+            e.printStackTrace();
+        }
+        catch(IOException e) {
+            e.printStackTrace();
+        }
+    }
+}
+```
 ### 補足情報（FW/ツールのバージョンなど）