if(!c7)より短く書く方法が分からなくて困っています。

Question

前提・実現したいこと

原始的なやり方なのでコードが長くなるので短くしたいです。
水分補給を管理するアプリを作っています。8時なると7時のチェックボックスにチェックが入ります。自分でチェックを入れる(水分補給したらチェックする)と加点されますが、プログラムにチェックを入れられると減点するアプリです。

"c7"は7時のチェックボックスを管理するためのフラグです。<　演算子によって8時になったら"c7"がfalseになります。
c7がfalseの時にチェックを入れたいです。if(!c7)だとifを18個書かないといけないのでみっともないです。短くする方法を教えてください。
最近学び始めた独学高校生なので知識が乏しいですが頑張りたいと思っています。回答よろしくお願いします。

C#

        public MainWindow()
        {
            InitializeComponent();

            DispatcherTimer timer = new DispatcherTimer();
            timer.Tick += timer_Tick;
            timer.Interval = new TimeSpan(0, 0, 1); //1000ミリ秒に一度呼び出す。
            timer.Start();
        }

        private  void timer_Tick(object sender, EventArgs e)
        {

            DateTime dt = DateTime.Now;
            int H = dt.Hour;    //時
            int M = dt.Minute;　//分
            Hfirudo = H;　//時
            jikoku.Text = H + "時" + M + "分";

            bool c7 = 8 < Hfirudo;　/c7 = チェックボックス(7時)の管理に使うフラグ
            bool c8 = 9 < Hfirudo;
            bool c9 = 10 < Hfirudo;
            bool c10 = 11 < Hfirudo;
            bool c11 = 12 < Hfirudo;
            bool c12 = 13 < Hfirudo;
            bool c13 = 14 < Hfirudo;
            bool c14 = 15 < Hfirudo;
            bool c15 = 16 < Hfirudo;
            bool c16 = 17 < Hfirudo;
            bool c17 = 18 < Hfirudo;
            bool c18 = 19 < Hfirudo;
            bool c19 = 20 < Hfirudo;
            bool c20 = 21 < Hfirudo;
            bool c21 = 22 < Hfirudo;
            bool c22 = 23 < Hfirudo;
            bool c23 = 24 < Hfirudo;
            bool c24 = 25 < Hfirudo;

            if (!c7)
            {
              チェックボックスにチェック
            }

チェックボックス

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

vs2017 WPF

Comments

[papinianus]

配列を使いたいのですが、回答するにあたって、もうちょっと全体像がわかりたいです。コードも動きも。タイマーかなにかで動く部分なんですかね？説明を補足してほしいです。

[len_souko]

↑それは質問を編集して書いてください。こちらは既定では閉じられているというのと、コード表示ができないので気づかない人が多く、気づいても見づらくて回答者が見間違いやすいです。

[kodokudane]

編集しました。ありがとうございます。

Answer 1

CheckBox に限らず、FrameworkElement を継承するコントロールは Tag プロパティを持っています。このプロパティは Object 型なので、何でも入れておくことができます。7時の CheckBox の Tag に、「8時になったら検査する」ことを覚えさせておくとします。すると、毎正時に、すべての CheckBox の Tag を見て、「今、検査するべきか？」を取得し、検査することができるのではないでしょうか。

Comments

[kodokudane]

回答ありがとうございます。回答いただいた情報をもとにobjectの使い方を調べます。

Answer 2

単純に配列にしては？
（ビルドしていないので、コンパイルがエラーになるかも）

            // c7～c24を配列にする
            bool[] c7to24 = new bool[18];
            int index = 0;
            for (int intHour = 8; intHour <= 25; intHour++)
            {
                // 配列の添え字(index)は、0から17の18個
                index = intHour - 8;
                c7to24[index] = intHour < Hfirudo;
            }

            // チェックボックスも配列にする
            Checbox[] ChecboxArray = new Checbox[] 
            {
                チェックボックス7時 ,
                チェックボックス8時 ,
                ・
                ・
                ・
            };


            for (index = 0; index < c7to24.length; index++)
            {
                if (c7to24[index] == false) // !c7to24[index] と同じ
                {
                    // チェックボックスにチェック
                    ChecboxArray[index].Checked = true;
                }
            }

c7からc24はチェックボックスにチェックする以外に使わないのであれば、
チェックボックスの配列だけで、できそう。

            // チェックボックスの配列にする
            Checbox[] ChecboxArray = new Checbox[] 
            {
                チェックボックス7時 ,
                チェックボックス8時 ,
                ・
                ・
                ・
            };

            int index = 0;
            for (int intHour = 8; intHour <= 25; intHour++)
            {
                // 配列の添え字(index)は、0から17の18個
                index = intHour - 8;
                bool c = intHour < Hfirudo;
                if (c == false)
                {
                    // チェックボックスにチェック
                    ChecboxArray[index].Checked = true;
                }
            }

Answer 3

どういう組み方をするのか分からないけれど・・・
自分ならクラスを定義してその中で時刻（～時のみ）と自分が推したかどうかと時間切れでチェックしたかどうかとまだ押してないし時間も来てないの状態を保持するようするかなぁ
んで、それをListに格納して初期化時は開始時刻（質問のソースだと７時なので7）から終了時刻（質（ｒｙだと24）でループして時刻情報を設定かな
enumあたりから名前がおかしいけどコメント省略のためにやった、反省はしてない

// enum名や値は自分で好みに合わせて
public enum 水やったんか
{
    まだやで,
    じぶんでやったで,
    間に合わんかった,
}

public class 水やりState
{
    public int Hour {get;}
    public 水やったんか 水やりどない {get; set; } = 水やったんか.まだやで;
    public 水やりState(int h)
    {
        Hour = h;
    }
}

// windowクラスのメンバ
private List<水やりState> 水やりStatus = new List<水やりState>();

// 初期化時に
// ７と２４は定数切るなり画面入力にするなりご自由に
for(int i = 7; i <= 24; i++)
{
    水やりStatus.Add(new 水やりState(i);
}

// タイマーイベントでのチェック時
var nowH = DateTime.Now.Hour;
if(水やりStatus.Any(m => m.Hour == nowH && m.水やりどない == 水やったんか.まだやで))
{
    // 時間が過ぎてて水やりしたかどうかの指定が無い場合は時間切れで乙る
    水やりStatus.First(m => m.Hour <= nowH && m.水やりどない == 水やったんか.まだやで).水やりどない = 水やったんか.間に合わんかった;
}

Answer 4

Q71さんの回答よりTagプロパティなる便利なものを知ったので、勉強がてら使ってみました。

特定のコンポーネント内のすべての子要素処理をするときは、その入れ物にx:NameをつけてChildrenプロパティから取得するのをよく使います。
これにTagを組み合わせると、自動生成した場合でもそれぞれを容易に識別できるので便利ですね。

// MainWIndow.xaml.cs
public partial class MainWindow : Window
    {
        private int MinTime = 7;
        private int MaxTime = 24;
        private DispatcherTimer CheckTimer;

        public MainWindow()
        {
            this.InitializeComponent();

            // タイマーの初期化
            this.CheckTimer = new DispatcherTimer();
            this.CheckTimer.Tick += this.CheckTimer_Tick;
            this.CheckTimer.Interval = new TimeSpan(0, 0, 1);
            this.CheckTimer.Start();
        }

        private void Window_Loaded(object sender, RoutedEventArgs e)
        {
            // チェックボックスの自動生成
            for (int i = this.MinTime; i <= this.MaxTime; i++)
            {
                CheckBox chekcbox = new CheckBox
                {
                    Content = string.Format("{0}時", i), // 表示テキスト
                    Tag = i // 時間のタグ付け
                };

                // チェックボックスをCheckBoxContainerに追加
                this.CheckBoxContainer.Children.Add(chekcbox);
            }
        }

        private void CheckTimer_Tick(object sender, EventArgs e)
        {
            DateTime now = DateTime.Now;

            // 時刻表示更新
            this.Clock.Text = now.ToString("H時m分");

            // CheckBoxContainer内のチェックボックスそれぞれに処理
            foreach (object child in this.CheckBoxContainer.Children)
            {
                if (child is CheckBox checkBox) // objectからCheckBoxに型変換
                {
                    if (checkBox.Tag is int time) // タグ付けした時間を取得
　　　　　　　　　　　{
                        if (time < now.Hour)
                        {
                            if(checkBox.IsChecked == true)
                            {
                                // ポイント増加
                            }
                            else
                            {
                                checkBox.IsChecked = true;
                                checkBox.IsEnabled = false;
                                // ポイント減少
                            }
                        }
                    }
                }
            }
        }
    }

<!-- MainWindow.xaml -->
<Window x:Class="CheckManager.MainWindow"
        xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
        xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
        xmlns:d="http://schemas.microsoft.com/expression/blend/2008"
        xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006"
        mc:Ignorable="d"
        Title="MainWindow" Height="400" Width="800" Loaded="Window_Loaded">
    <Grid>
        <Grid.RowDefinitions>
            <RowDefinition Height="Auto"/>
            <RowDefinition Height="300"/>
        </Grid.RowDefinitions>

        <TextBlock x:Name="Clock"
                   Margin="5" FontSize="20"/>

        <WrapPanel x:Name="CheckBoxContainer" Grid.Row="1"
                   Orientation="Vertical">
            <WrapPanel.Resources>
                <Style TargetType="CheckBox">
                    <Setter Property="Margin" Value="5"/>
                    <Setter Property="FontSize" Value="16"/>
                    <Setter Property="VerticalContentAlignment" Value="Center"/>
                </Style>
            </WrapPanel.Resources>
        </WrapPanel>
    </Grid>
</Window>