使用メモリがどんどん増えてしまう。

Question

前提・実現したいこと

別クラスで下記のSoundクラスのketteiメソッドを実行して音を鳴らしているのですが、何度も呼び出すとどんどん使用メモリが増えていってしまいます。実行直後にcloseしても増えてしまいます。
実行するために新しいインスタンスが作成されているから？という事しか分からず解決方法が見つかりません。
音を使いまわす場合メモリを増やさず使いまわすにはどうすればよいかご教授お願いいたします。

該当のソースコード

package package_sound;

import java.io.File;

import javax.sound.sampled.AudioInputStream;
import javax.sound.sampled.AudioSystem;
import javax.sound.sampled.Clip;

public class Sound{

	Clip clip;

	public void setFile(String soundFileName) {

		try {
			File file = new File(soundFileName);
			AudioInputStream sound = AudioSystem.getAudioInputStream(file);
			clip = AudioSystem.getClip();
			clip.open(sound);
		}
		catch(Exception e) {
		}
	}

	public void kettei() {
		setFile("./res/sound/xxx.wav");
		clip.start();
	}	

	public void stop() {
		clip.close();
		//clip.stop();
	}

}

Comments

[sazi]

VM起動パラメータのMaxTenuringThresholdの値は？

[clothblue]

すみません、上記内容の知識が全くないので一度調べてみます。
ありがとうございます。

[m.ts10806]

どこにもstop()を呼び出す記述がないようですが…

Answer 1

    public void setFile(String soundFileName) {
        try {
            if(clip != null) {
                clip.close();
            }

            //
            File file = new File(soundFileName);
            AudioInputStream sound = AudioSystem.getAudioInputStream(file);
            clip = AudioSystem.getClip();
            clip.open(sound);
            sound.close();
        }
        catch(Exception e) {
        }
    }

clipとsoundをきちんとcloseしてあげればいいようです。
Kotlin + JavaFXのサンプルですが、java -Xmx16m ...で、問題なく実行できます。closeを端折ると、速攻でOutOfMemoryError。

import javafx.application.Application
import javafx.application.Platform
import javafx.scene.Scene
import javafx.scene.control.Button
import javafx.scene.control.TextArea
import javafx.scene.layout.VBox
import javafx.stage.Stage
import java.io.File
import java.nio.file.Paths
import java.time.LocalTime
import javax.sound.sampled.AudioSystem
import javax.sound.sampled.Clip
import kotlin.concurrent.thread

object xxSound {

  @JvmStatic
  fun main(args: Array<String>) {
    Application.launch(App::class.java, *args)
  }

  class App : Application() {
    override fun start(stage: Stage) {
      var clip: Clip? = null

      fun setFile(file: File) {
        AudioSystem.getAudioInputStream(file).use { sound ->
          clip?.close()

          clip = AudioSystem.getClip()
          clip?.open(sound)
        }
      }

      val play = Button("Play").also { button ->
        button.setOnAction { ev ->
          val file = Paths.get("/tmp/ss.wav").toFile()
          setFile(file)
          clip?.start()
        }
      }
      val stop = Button("Stop").also { button ->
        button.setOnAction { ev ->
          clip?.stop()
        }
      }

      val text = TextArea()
      thread {
        while (true) {
          Platform.runLater {
            val free = Runtime.getRuntime().freeMemory()
            val total = Runtime.getRuntime().totalMemory()
            val used = free * 100.0 / total
            val msg = "${LocalTime.now()} // ${used} = ${free} / ${total}"
            text.text = "${msg}\n${text.text}"
          }
          Thread.sleep(1000)
        }
      }

      val box = VBox(play, stop, text)
      stage.scene = Scene(box)
      stage.title = xxSound::javaClass.name
      stage.show()
    }
  }
}

Answer 2

clipがnullかどうか判定して、setFileを複数回、呼ばないようにすればいいのでは？

Comments

[clothblue]

上記の方法で上手くいきました。
単純な方法が思いつかなくて申し訳なかったです。