シークバーで設定した値をバイブレーションを制御したい

Question

前提・実現したいこと

シークバーの値によってバイブレーションの強さ、継続時間を調節した後、startボタンを押した瞬間にバイブレーションが実行されるようにしたいです。

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

Type mismatch: inferred type is Long? but Unit was expected(37行目)
Type mismatch: inferred type is Int? but Unit was expected(58行目)というエラーが出ます。

該当のソースコード

package com.example.vibration

import android.content.Context
import android.os.Bundle
import android.os.VibrationEffect
import android.os.Vibrator
import android.view.View
import android.widget.SeekBar
import androidx.appcompat.app.AppCompatActivity
import kotlinx.android.synthetic.main.activity_main.*
import java.lang.String
import java.util.*

class MainActivity : AppCompatActivity() {
    override fun onCreate(savedInstanceState: Bundle?) {
        super.onCreate(savedInstanceState)
        setContentView(com.example.vibration.R.layout.activity_main)
        var time: Long= 1
        var hz: Int = 1
        setupSeekBart(time)
        setupSeekBarh(hz)
        val vibrator = getSystemService(Context.VIBRATOR_SERVICE) as Vibrator
        val vibrationEffect = VibrationEffect.createOneShot(time, hz)
        vibrator.vibrate(vibrationEffect)
    }

    private fun setupSeekBart(num1: Long)
    {
        seekbartime.progress = 0
        seekbartime.max = 50
        seekbartime.setOnSeekBarChangeListener(
            object : SeekBar.OnSeekBarChangeListener {
                override fun onProgressChanged(seekBar: SeekBar, progress: Int, fromUser: Boolean) {
                    val strt = String.format(Locale.US, "%d ", progress)
                    timedown.setText(strt)
                    var num1 = strt.toLongOrNull()
                    return num1
                }

                override fun onStartTrackingTouch(seekBar: SeekBar) {
                }

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

    private fun setupSeekBarh(num1: Int)
    {
        seekbarhz.progress = 0
        seekbarhz.max = 50
        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)
                    var num1 = strh.toIntOrNull()
                    return num1
                }

                override fun onStartTrackingTouch(seekBar: SeekBar) {
                }

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

Answer 1

OnSeekBarChangeListenerのonProgressChangedに返り値はないのでreturn num1を削除してください

===回答を追記
引数にnum1があったんですね、見落としていました。ちょっとだけやりたいことがわかった気もします。
「startボタンを押した瞬間にバイブレーションが実行されるようにしたいです。」
とのことなので、単純にメンバ変数に持っておいてその値を使ってみてはどうでしょうか。

class MainActivity : AppCompatActivity() {

    // ここで持っておく
    private var gTime: Long = 1
    private var gHz: Int = 1

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

        // ボタンが押されたときの処理
        button.setOnClickListener {
            val vibrator = getSystemService(Context.VIBRATOR_SERVICE) as Vibrator
            val vibrationEffect = VibrationEffect.createOneShot(gTime, gHz)
            vibrator.vibrate(vibrationEffect)
        }
    }

    private fun setupSeekBart()
    {
        seekbartime.progress = 0
        seekbartime.max = 50
        seekbartime.setOnSeekBarChangeListener(
            object : SeekBar.OnSeekBarChangeListener {
                override fun onProgressChanged(seekBar: SeekBar, progress: Int, fromUser: Boolean) {
                    val strt = String.format(Locale.US, "%d", progress)
                    timedown.setText(strt)
                    gTime = strt.toLong()
                }

                override fun onStartTrackingTouch(seekBar: SeekBar) {
                }

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

    private fun setupSeekBarh()
    {
        seekbarhz.progress = 0
        seekbarhz.max = 50
        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.toInt()
                }

                override fun onStartTrackingTouch(seekBar: SeekBar) {
                }

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

またはスタートボタンが押されたときに現在のプログレスの値を取ってきてその値を使うこともできるかと思います。そうすれば特に変数など用意する必要はありません。

// ボタンが押されたときの処理
        button.setOnClickListener {
            val vibrator = getSystemService(Context.VIBRATOR_SERVICE) as Vibrator

            val setTime = seekbartime.progress.toLong()
            val setHz = seekbarhz.progress

            val vibrationEffect = VibrationEffect.createOneShot(setTime, setHz)
            vibrator.vibrate(vibrationEffect)
        }

以上、よろしくお願いいたします。

Comments

[ryu-sei]

ありがとうございます。
エラーはなくなったのですがうまく動いてくれませんでした。Object内でnum1の値を操作するのは無理な感じですかね。

[razuma]

num1の値を操作と言うのがいまいちよくわかりません。
具体的には何がしたいのでしょうか？

[ryu-sei]

setupseekbartの関数に変数timeを代入する→シークバーの出した値をtimeに代入してその値を返す→変数vibrationeffectにtimeを代入する
というながれのプログラムを組みたいのですが今object：SeekBar.OnSeekBarChangeListenerの部分にtime（num1）を代入できない感じになってます。説明下手ですみません。

[razuma]

なるほど、引数にnum1があるのを見落としていました。なんとなくやりたいようなことはわかったかもしれません。
やりたいことは「シークバーの値によってバイブレーションの強さ、継続時間を調節した後、startボタンを押した瞬間にバイブレーションが実行されるようにしたいです。」かと思いますので、単純にメンバ変数として値を持つか、スタートボタンを押したときにプログレスの値を持ってくる、で良いかと思います。回答を追記したのでご確認ください。

以上、よろしくお願いいたします。

[ryu-sei]

ご丁寧にありがとうございました。まさに自分のやりたかったことができました。