時刻に合わせてダイアログ内のImageViewを非表示にしたい

Question

現在todoアプリを作成しています。
##実現したいこと
タスク編集画面のリマインダーボタンを押すと、自作したダイアログを表示することができ、現在時刻に合わせて、朝の6時から18時までの間ならreminderNightButtonを表示し、reminderMorningButtonを非表示に、それ以外の時刻なら表示・非表示を逆にするという処理を作成したいのですが、問題があります。
##問題
TaskAddActivityでreminderButton.setOnClickListenerでダイアログのインスタンスを取得した後、dialog.reminderMorningButton.visibility = View.GONEで現在時刻に合わせてボタンを非表示にするという処理を書いていますが、そこでjava.lang.IllegalStateException: dialog.reminderMorningButton must not be nullとなり強制終了してしまいます。

TaskAddActivity

class TaskAddActivity : AppCompatActivity(), ReminderDialog.Listener{

    override fun reminderUp(date: Date) {
        val calendar = Calendar.getInstance()
        calendar.time = date
        setAlarmManager(calendar)
        Toast.makeText(
            this, "リマインダーをセット",
            Toast.LENGTH_SHORT
        ).show()
    }
    private lateinit var realm: Realm
    var taskId:Long? = null
    private var taskTitle:String? = null

    override fun onCreate(savedInstanceState: Bundle?) {
        super.onCreate(savedInstanceState)
        setContentView(R.layout.activity_task_add)
        realm = Realm.getDefaultInstance()

        taskId = intent?.getLongExtra("task_id", -1L)

        val task = realm.where<Task>().equalTo("id", taskId).findFirst()
        titleEdit.setText(task?.title)
        taskEditText.setText(task?.detail)
        taskTitle = task?.title


        //リマインダーダイアログを呼び出す
        reminderButton.setOnClickListener {
            var dialog = ReminderDialog()
            val now = Calendar.getInstance()
            //時刻が夜の６時より前だったら太陽ボタンを消す
            if(now.get(Calendar.HOUR) <= 18 && now.get(Calendar.HOUR) >= 6){
                dialog.reminderMorningButton.visibility = View.GONE
            }else{
                dialog.reminderNightButton.visibility = View.GONE
            }
            dialog.show(supportFragmentManager, "alert_dialog")
        }

    }

ReminderDialog

class ReminderDialog : DialogFragment(),
    TimePickerDialog.OnTimeSetListener, com.wdullaer.materialdatetimepicker.date.DatePickerDialog.OnDateSetListener
{
    interface Listener{
        fun reminderUp(date:Date)
    }
    private var listener: Listener? = null
    private var dialogDateTime: String? = null
    private var dateTime:Date? = null

    //フラグメントがアクティビティから呼び出されたとき
    override fun onAttach(context: Context) {
        super.onAttach(context)
        when(context){
            is Listener -> listener = context
        }
    }

    //日付選択後時刻選択ダイアログを呼び出す
    override fun onDateSet(
        view: com.wdullaer.materialdatetimepicker.date.DatePickerDialog?,
        year: Int,
        monthOfYear: Int,
        dayOfMonth: Int
    ) {
        val c = Calendar.getInstance()
        c.set(year, monthOfYear, dayOfMonth)
        val date = DateFormat.format("yyyy/MM/dd", c).toString()
        dialogDateTime = date
        showTimePickerDialog(date)
    }

    //時刻を選択
    override fun onTimeSet(view: TimePickerDialog?, hourOfDay: Int, minute: Int, second: Int) {
        val time = "%1$02d:%2$02d".format(hourOfDay, minute)
        dialogDateTime += " ${time}"
        dateTime = dialogDateTime?.toDate()

    }



    //時刻選択ダイアログ
    fun showTimePickerDialog(date:String) {
        val now = Calendar.getInstance()
        val dpd = TimePickerDialog.newInstance(
            this,
            now.get(Calendar.HOUR),
            now.get(Calendar.MINUTE)+1,
            now.get(Calendar.SECOND),
            false
        )
        var test = now.get(Calendar.YEAR).toString() + "/0" + ((now.get(Calendar.MONTH)+1).toString()) + "/" + now.get(Calendar.DAY_OF_MONTH).toString()
        Log.d("HOUR","${now.get(Calendar.HOUR)}")
        //指定された日付が現在日付ならば現在時刻の五分以降より前の時刻は選択させない
        if(date == test) {
            dpd.setMinTime(Timepoint(now.get(Calendar.HOUR),(now.get(Calendar.MINUTE)+1)))
        }

        dpd.show(fragmentManager!!, "aaaa")
    }


    override fun onCreateDialog(savedInstanceState: Bundle?): Dialog {

        //ダイアログのインスタンス取得
        val dialog:Dialog = Dialog(activity!!)
        //タイトルバーなしのダイアログを表示
        dialog.window?.requestFeature(Window.FEATURE_NO_TITLE)
        //フルスクリーンでダイアログを表示
        dialog.window?.setFlags(WindowManager.LayoutParams.FLAG_FULLSCREEN,
            WindowManager.LayoutParams.FLAG_LAYOUT_IN_SCREEN)
        //レイアウト指定
        dialog.setContentView(R.layout.reminder_layout)
        val now = Calendar.getInstance()


        //背景色を透明に
        dialog.window?.setBackgroundDrawable(ColorDrawable(Color.TRANSPARENT))
        dialog.window?.setLayout(LayoutParams.MATCH_PARENT, LayoutParams.WRAP_CONTENT);


        //日付選択ダイアログ呼び出し
        dialog.reminderCustomButton.setOnClickListener {
            val dpd = com.wdullaer.materialdatetimepicker.date.DatePickerDialog.newInstance(
                this,
                now.get(Calendar.YEAR), // Initial year selection
                now.get(Calendar.MONTH), // Initial month selection
                now.get(Calendar.DAY_OF_MONTH) // Inital day selection
            )
            dpd.minDate = now
            dpd.show(fragmentManager!!, "Datepickerdialog")
        }

        //保存ボタンを押したら通知を指定時間にセット
        dialog.saveButton.setOnClickListener {
            when{
                dateTime != null ->{
                    val setDateTime = dateTime
                    if(setDateTime !=null)
                    listener?.reminderUp(setDateTime)
                }
            }
        }

        dialog.cancelButton.setOnClickListener {
            dialog.cancel()
        }

        return dialog
    }

    //文字をDateに変換
    private fun String.toDate(pattern: String = "yyyy/MM/dd HH:mm"): Date?{
        return try {
            SimpleDateFormat(pattern).parse(this)
        }catch (e: IllegalArgumentException){
            return null
        }catch (e: ParseException){
            return null
        }
    }
}

reminder_layout.xml

<?xml version="1.0" encoding="utf-8"?>
<androidx.constraintlayout.widget.ConstraintLayout xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:app="http://schemas.android.com/apk/res-auto"
    xmlns:tools="http://schemas.android.com/tools"
    android:id="@+id/linearLayout"
    android:layout_width="match_parent"
    android:layout_height="wrap_content"
    android:layout_margin="16dp"
    android:background="@drawable/shape_rounded_corners_5dp">

    <ImageView
        android:id="@+id/reminderMorningButton"
        android:layout_width="91dp"
        android:layout_height="91dp"
        android:background="@drawable/image_view_maru"
        android:scaleType="fitCenter"
        app:layout_constraintBottom_toTopOf="@+id/linearLayout2"
        app:layout_constraintEnd_toEndOf="parent"
        app:layout_constraintStart_toEndOf="@+id/reminderCustomButton"
        app:layout_constraintTop_toBottomOf="@+id/textView"
        app:srcCompat="@drawable/ic_wb_sunny_black_24dp" />

    <ImageButton
        android:id="@+id/reminderCustomButton"
        android:layout_width="91dp"
        android:layout_height="91dp"
        android:layout_marginStart="24dp"
        android:layout_marginTop="8dp"
        android:background="@drawable/image_view_maru"
        android:scaleType="centerCrop"
        app:layout_constraintStart_toEndOf="@+id/reminderNightButton"
        app:layout_constraintTop_toBottomOf="@+id/textView"
        app:srcCompat="@drawable/ic_watch_later_black_24dp" />

    <ImageButton
        android:id="@+id/reminderNightButton"
        android:layout_width="91dp"
        android:layout_height="91dp"
        android:layout_marginStart="24dp"
        android:layout_marginTop="8dp"
        android:background="@drawable/image_view_maru"
        android:scaleType="centerCrop"
        app:layout_constraintStart_toStartOf="parent"
        app:layout_constraintTop_toBottomOf="@+id/textView"
        app:srcCompat="@drawable/ic_brightness_3_black_24dp" />

    <TextView
        android:id="@+id/textView"
        android:layout_width="0dp"
        android:layout_height="wrap_content"
        android:background="@drawable/text_view_rounded"
        android:text="リマインダー追加"
        android:textColor="#000000"
        android:textSize="35sp"
        android:textStyle="bold"
        app:layout_constraintEnd_toEndOf="parent"
        app:layout_constraintStart_toStartOf="parent"
        app:layout_constraintTop_toTopOf="parent" />

    <LinearLayout
        android:id="@+id/linearLayout2"
        android:layout_width="match_parent"
        android:layout_height="0dp"
        android:layout_marginTop="16dp"
        android:orientation="horizontal"
        app:layout_constraintEnd_toEndOf="parent"
        app:layout_constraintStart_toStartOf="parent"
        app:layout_constraintTop_toBottomOf="@+id/reminderCustomButton">

        <Button
            android:id="@+id/cancelButton"
            android:layout_width="match_parent"
            android:layout_height="match_parent"
            android:layout_weight="1"
            android:background="@drawable/botton_border"
            android:text="キャンセル"
            android:textSize="25sp"
            app:layout_constraintStart_toStartOf="parent" />

        <Button
            android:id="@+id/saveButton"
            android:layout_width="match_parent"
            android:layout_height="match_parent"
            android:layout_weight="1"
            android:background="@drawable/botton_border"
            android:text="保存"
            android:textSize="25sp" />
    </LinearLayout>

</androidx.constraintlayout.widget.ConstraintLayout>

画面デザイン

Comments

[jimbe]

kotlin としてはどうでもよいことですが, 「Morinig」とは?
「Morning」のことでしょうか.

[jimbe]

マークダウンで ``` の右に書くのはファイル名では無く言語名("kotlin","xml"等)としてください.
ファイル名は ```の行の直前にでも通常の文としてお書きください.

[kbayashi]

ご指摘ありがとうございます。Morinigはタイピングミスをしていました。気づきませんでした。また、マークダウンの方も本来の書き方に修正させていただきました。

[jimbe]

修正ありがとうございます

Answer 1

ダイアログに ReminderMorinigImage が見当たりません.

#追記
ダイアログのレイアウト設定は onCreateDialog() 内で行われていますので, show() の前にはビューはまだ生成されていません.
形としては, ダイアログに(newInstance 経由で setArguments を用いて「どちらのボタンを表示するのか」だけを渡し, onCreateDialog 内でそのパラメータから両ボタンの GONE/VISIBLE を設定しては如何でしょうか.

...再度見ていたのですが, onCreateDialog 内で val now = Calendar.getInstance() して処理している個所がありますので, そこに //時刻が夜の６時より前だったら ～ の if 文部分を移してしまえばよいのではないでしょうか.
そうすれば TaskAddActivity の onCreate ではボタンが押されたらダイアログを生成して表示するだけになると思います.

Comments

[jimbe]

少々気になったのは, このようにコードの知らないところで状況が変わるもの(時刻)によってインターフェースが変わる場合, その変わる前に表示した状態が変わった後にも問題無いか...ということです.
具体的には, 18時前にダイアログを表示して reminderMorningButton が VISIBLE, reminderNightButton が GONE となった状態で18時を過ぎた場合, 18時以降であれば押せないはずの reminderMorningButton を押せてしまうことになりますが, 大丈夫でしょうか.
わざわざ消しているということは, その時間帯には押せないことを前提にコードが書かれている可能性があります.

[kbayashi]

回答ありがとうございます。
reminderNightButtonを使用して実装したい処理は、それが押された場合は今日の20時に、アプリから通知を行うという処理で、仮に20時より後に押されてしまっても通知が作動しないだけなので、この問題は許容しています。

[kbayashi]

onCreateDialog内でボタンの非表示処理を書いたところ正常に動作しました。ありがとうございます。

[jimbe]

> 今日の20時に、アプリから通知を行うという処理
なるほど, プリセットボタンなんですね. 便利そうです.