kotlinでシークバーから値を渡したい

Question

前提・実現したいこと

シークバーの値によってsin波の振動数を調節した後、startボタンを押した瞬間に音が鳴るようにしたいです。

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

80~95行目でシークバーを使ってgHz（sin波の振動数）を調節して97行目に渡そうとしているのですがgHzの値が100から変わらず、100Hzの音しか鳴りません。

該当のソースコード

package com.example.speaker

import android.app.Activity
import android.media.AudioFormat
import android.media.AudioManager
import android.media.AudioTrack
import android.os.Bundle
import android.view.Menu
import android.widget.SeekBar
import kotlinx.android.synthetic.main.activity_main.*
import java.lang.String
import java.util.*
import kotlin.math.ceil
import kotlin.math.sin


class DigitalSoundGenerator(
        private var sampleRate: Int, bufferSize: Int) {
    val audioTrack: AudioTrack

    private val bufferSize: Int

    fun getSound(frequency: Double, soundLength: Double): ByteArray {
        // byteバッファを作成
        val buffer = ByteArray(ceil(bufferSize * soundLength).toInt())
        for (i in buffer.indices) {
            var wave = i / (sampleRate / frequency) * (Math.PI * 2)
            wave = sin(wave)
            buffer[i] = (if (wave > 0.0) Byte.MAX_VALUE else Byte.MIN_VALUE)
        }
        return buffer
    }

    init {
        sampleRate = 44100
        this.bufferSize = bufferSize

        audioTrack = AudioTrack(
                AudioManager.STREAM_MUSIC,  // 音楽ストリームを設定
                sampleRate,  // サンプルレート
                AudioFormat.CHANNEL_OUT_MONO,  // モノラル
                AudioFormat.ENCODING_DEFAULT,  // オーディオデータフォーマットPCM16とかPCM8とか
                bufferSize,  // バッファ・サイズ
                AudioTrack.MODE_STREAM) // Streamモード。データを書きながら再生する
    }
}

class SoundDto(
        var sound: ByteArray,
        var length: Double) {

    init {
        length = length
    }
}

class MainActivity : Activity(), Runnable {
    // Sound生成クラス
    private var soundGenerator: DigitalSoundGenerator? = null
    
    private var audioTrack: AudioTrack? = null

    private val soundList: MutableList<SoundDto> = ArrayList()

    var gTime:Double = 0.5
    private var gHz = 100.0

    override fun onCreate(savedInstanceState: Bundle?) {
        super.onCreate(savedInstanceState)
        setContentView(R.layout.activity_main)

        soundGenerator = DigitalSoundGenerator(44100, 44100)

        audioTrack = soundGenerator!!.audioTrack
        start.setOnClickListener {
            val th = Thread(this@MainActivity)
            th.start()
        }

        seekbarhz.progress = 0
        seekbarhz.max = 255
        seekbarhz.setOnSeekBarChangeListener(
            object : SeekBar.OnSeekBarChangeListener {
                override fun onProgressChanged(seekBar: SeekBar, progress: Int, fromUser: Boolean) {
                    val strh = String.format(Locale.US, "%d", progress)
                    hzdown.setText(strh)
                    gHz = strh.toDouble()
                }

                override fun onStartTrackingTouch(seekBar: SeekBar) {
                }

                override fun onStopTrackingTouch(seekBar: SeekBar) {
                }
            })

        soundList.add(SoundDto(generateSound(soundGenerator, gHz, gTime), gTime))
    }

    override fun onDestroy() {
        super.onDestroy()

        if (audioTrack!!.playState == AudioTrack.PLAYSTATE_PLAYING) {
            audioTrack!!.stop()
            audioTrack!!.release()
        }
    }


    override fun onCreateOptionsMenu(menu: Menu?): Boolean {
        menuInflater.inflate(R.menu.option, menu)
        return true
    }

    private fun generateSound(gen: DigitalSoundGenerator?, freq: Double, length: Double): ByteArray {
        return gen!!.getSound(freq, length)
    }

    override fun run() {
        if (audioTrack!!.playState == AudioTrack.PLAYSTATE_PLAYING) {
            audioTrack!!.stop()
            audioTrack!!.reloadStaticData()
        }
        audioTrack!!.play()

        for (dto in soundList) {
            audioTrack!!.write(dto.sound, 0, dto.sound.size)
        }

        audioTrack!!.stop()
    }
}

試したこと

前回、前々回あたりの質問のことはしました。

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

Android Studioを使って開発しています。

Answer 1

97 行目の soundList.add は MainActivity が作られた時に 1 回だけ実行されるので、その後 seekbar の値が変わっても soundList の内容が変わらないからですね。

とりあえず onProgressChanged の中で soundList の内容を更新すればいいと思います。

                    override fun onProgressChanged(seekBar: SeekBar, progress: Int, fromUser: Boolean) {
                        val strh = String.format(Locale.US, "%d", progress)
                        hzdown.setText(strh)
                        gHz = strh.toDouble()

                        // soundList の内容を更新
                        soundList.clear()
                        soundList.add(SoundDto(generateSound(soundGenerator, gHz, gTime), gTime))
                    }

Comments

[ryu-sei]

ありがとうございます！音程変わるようになってくれました。
周波数を10000くらいまであげれるようにしたいのですが高くすると音が正確に出ないような気がします。
もし時間があれば解決方法教えていただきたいです。

[hoshi-takanori]

音程がおかしいのは、AudioFormat.ENCODING_DEFAULT がたぶん AudioFormat.ENCODING_PCM_16BIT (機種によって違うかも) のため、2 バイトでひとつのデータになっていて、音程が倍になってる気がします。また、矩形波なのでかなり極端な音ですね。
88 鍵のピアノの音の範囲が 27.5 〜 4186 Hz らしいので、10000 Hz はかなり危険な音になると思います。
余計なお世話かとは思いますが、この辺の記事などで音について調べた方がいいかもしれません。
https://info.shimamura.co.jp/digital/knowledge/2014/03/19260/2

[ryu-sei]

丁寧に教えていただいて助かりました。もう少し勉強してみます。
ありがとうございました。