8bitの2進数でタイムリーに0を検知するプログラムについて

Question

試してみたこと

いつもお世話になっております。
現在、C#にて8bit(8入力)のデジタル入力を検知するプログラムを作成しています。

TOP16というI/Oボードを使用しており、提供されたDLLからI/Oの入力状態を監視し（該当の入力があったらアクションを）させようとしています。

入力される値は以下のように10進数で取得することができます。（悩んで悩んで・・やっと分かりました）
例として一部を記載します。

無入力であれば　255 (11111111)
Input1が入れば　254(11111110)
Input2のみが入れば　253(11111101)
Input4のみが入れば　247(11110111)
Input1と3が入っていれば　250(11111010)

8入力なので2進数の1桁目がInput1で8桁目がInput8となります。

同時に複数のInputが入っていることも想定し、若いInput入力でbreak;をせず全Inputの状態を取得したいです。
実際に複数の入力があった場合、Input1が入っていることを条件にInput3はアクション可能というような使い方も想定しています。

そこでここ3日ほど、乏しい経験と無い頭をフル稼働し自分でプログラムを作ってみて一段落したと思ったのですが、

ふと。もっと良い書き方、例えば配列を使ったりする方法もあるのではないか？と思った次第です・・・

ここで別の手法を使って同じ結果を出せるようになれば一皮むける気がしているのですが、
苦渋の決断で経験豊富な先輩方の知恵をお借りできればと思い、こちらで相談させて頂くことにしました。

稚拙なプログラムで公開するのは恥ずかしいのですが私の考えたものを記載したいと思います。
どうか皆様のお知恵、アイデア、アドバイスを頂けませんでしょうか？
どうぞ宜しくお願い致します。

作成したソースコード

        //タイマーを使って入力を監視します。
        private void timer1_Tick(object sender, EventArgs e)
        {
            //デジタルインプット(8bit)を取得
            int xInput = Top16.GetInputs(handle);

            //10進数を2進数に変換する
            string binaryStr = Convert.ToString(xInput, 2);

            //カウンターを初期化
            int count = 0;

            //binaryStrの文字列でカウントダウンをさせる
            for (int i = binaryStr.Length - 1; i >= 0; i--)
            {
                char inputBinaryNum = binaryStr[i]; //binaryStrの[i]番目の文字列の切り出し
                count++;

                if (inputBinaryNum == '0') //inputBinaryNumが'0'だった時
                {
                    if (count == 1)
                    {
                     //Input1が入ったときのアクション内容を書く
                    }
                    else if (count == 2)
                    {
                     //Input2が入ったときのアクション内容を書く
                    }
                    else if (count == 3)
                    {
                     //Input3が入ったときのアクション内容を書く
                    }
                    else if(count == 4)
                    {
                     //Input4が入ったときのアクション内容を書く
                    }
                    else if(count == 5)
                    {
                     //Input5が入ったときのアクション内容を書く
                    }
                    else if(count == 6)
                    {
                     //Input6が入ったときのアクション内容を書く
                    }
                    else if(count == 7)
                    {
                     //Input7が入ったときのアクション内容を書く
                    }
                    else if (count == 8)
                    {
                     //Input8が入ったときのアクション内容を書く
                    }
                }
            }
         }

Comments

[changryu]

ご指摘頂きました点について、自分のteratailの使い方の認識不足及び間違いがあるかと思います。改めて「質問するときのヒント」を読み直して問題点について修正させて頂きたいと思います。この度は、ご指摘ありがとうございました。

[TN8001]

> 週末バタバタしており返信が遅くなりました

急な予定が入って回答の吟味ができないことはわかります。
ただそれを表明する（「返事が遅れます」等のコメント）くらいの時間はあるでしょう。

わたしがせっかちすぎですかね？ｗ
あんまり開くと自分で書いたことも忘れてしまうんですよね^^;

[TN8001]

「調査したこと・試したことが記載されていない質問」
という指摘はわたしです。取り下げました。

修正依頼の内容としては間違っている（拡大解釈をしている）と思うのですが、回答やコメントしてから３日以上何のリアクションもない質問に一律付けています（事前に遅れることを書いてあればその期間以上開けます）
「望んだ回答でないと返事しない上その希望すら表明していない」といった解釈。ほかにいい内容があればいいのですが...

目的としては
* （通知が行くので）最後の希望として返事の期待
* （稀にいる）回答への返事を質問の編集あるいは編集履歴タイトルでする人への対策（回答しただけでは編集の通知が来ない）
* （せっかく丁寧に回答したのに返事がなく内心ムカついてはいるので）ささやかな抵抗としてｗ

当初は１週間としていたのですが、５日・３日と短縮してきました（４年1000回答以上してきた感触）
（リニューアル後）通知が来ないことがよくあるので、（回答した未解決質問は）最低２４時間に１回は実際に開いて確認するのですが面倒になってくるんですよね。
修正依頼後は（遅れてくるかもしれないが）通知に期待して確認をやめています。

ノーリアクションの方が多すぎるんですよね（体感１割弱）
難しすぎるのか望んだ内容でないのかわかりませんが、言ってもらわないと直しようもないのに...

changryuさんには関係がないことを長々失礼しました。

---

「こちらの質問が他のユーザーから「調査したこと・試したことが記載されていない質問」という指摘を受けました。」

「他のユーザー」となっている場合は、指摘者は（おそらく）ひとりです。
この場合上記注釈が見えるのは、質問者と指摘者のみになります（ログインせずに質問を見ると注釈が見えません）

「複数のユーザー」となっている場合は、全員に注釈が表示されています。

[changryu]

TN8001様

コメントの追記ありがとうございます。

>>ただそれを表明する（「返事が遅れます」等のコメント）くらいの時間はあるでしょう。

この一文を見て確かにその通りだな。と思い反省しました。
回答してくださっている方は人間だということを少し忘れがちだった気がしました。
如何なる事情があったとしても、人として一言お礼と、遅れる旨を書くのが筋でした。

TN8001様他、ご回答を頂いた皆様

せっかく回答をしていただいたのにも関わらず、気分を害するような対応をしたことをお詫び致します。

[changryu]

上記のことを昨夜、布団に入ってから考えていて、昨今身近なサービスがシステム化し便利になったことは嬉しいことですが

例えば、ヤフーオークションなんか四半世紀前なんて見ず知らずの相手にメールアドレスを晒していちいちメールを送り合っていたのが、今ではやり取りのオペレーションがほぼ確立し、商品の善し悪しは大事ですが、それ以外はほぼシステム間のやりとりだけ（なんならコメントもテンプレ）で済み便利になったと思います。

一方で、人、AI関わらずオンラインでのやり取りに慣れてきてしまったのか、私だけかも知れませんがモニターの向こう側に人が居ることを忘れだしている気がしました。

若者にネットリテラシーなんかを説教する前に、自分の意識を変えないと、もしかしたら今どきの若者の方が現在の世界（ネット含め）をもっと上手に使っている気がし、まだまだ修行が足りないな。と眠りについた次第です。

何の話やらw

Answer 1

実際に複数の入力があった場合、Input1が入っていることを条件にInput3はアクション可能というような使い方も想定しています。

そういう複合条件みたいなのがどれだけあるのかわかりませんが，とりあえず switch で良いのでは？　という気がします．
（むしろそれだと困るような話があるならば質問で明言すべきでしょう）

//入力１と３だけが成されているときの値．
//プログラムの意味的に「それが何の入力に相当するのか」という話があると思うので，
//なんかそういう名前を付けておけば良いのでは？　と思う．
const int CatNapInput = 0b1111_1010;

//こっちは入力３だけが成されているときの値．
const int WildMeowInput = 0b1111_1011;

...

//入力状態に応じた処理
switch( xinput )
{
case CatNapInput: CatNap(); break;
case WildMeowInput: WildMeow(); break;
default: break;
}

Comments

[changryu]

fana様

この度は追加でのアドバイスをしていただき感謝致します。ありがとうございます。

>>そういう複合条件みたいなのがどれだけあるのかわかりませんが，とりあえず switch で良いのでは？　という気がします．
>>（むしろそれだと困るような話があるならば質問で明言すべきでしょう）

すみません。ちゃんとどの入力入っているとき（条件）、何をするのかを整理出来ていないことに気づきました。

例えば
Input1：安全センサー
Input2：非常停止
Input3：実行ボタン
だとしたときに、

Input3はInput1がONのときに実行可
Input1がOFFのときはInput3は入力入っても無視される
などなど

最初にちゃんと条件が分かっていれば（設計ができていれば）、教えて頂いたswitch文はスマートな気がしました。

意味合いはあってますでしょうか？

その前にコメントしていただいたビット演算も試してみたいので、調べる時間、試すお時間を頂けたらと思います。
別の方のところにも記載しましたが、別の業務との脳の切り替えが出来ないもので・・・本当にすみません

[fana]

他の回答でのコメント欄でも触れられていますが，
いくつかの考えられる方法があるときにそのどれが適しているのか？　というのは｛やること，状況，背景事情，あなたの好み，etc｝に多分に依存するものなので，いろいろと比較検討する必要があるでしょう．
（回答を書いている側にはそこらへんの実情が正確にはわからないので，いろいろと列挙しているにすぎない）

そこに時間を要するのは当たり前のことであり，本来わざわざ断りを入れるようなことでもないと思います．
「質問したならば即座に解決しなければならない！」みたいな謎ルールがあるわけでもないのですし．
（……とはいえ，あまりにも放置が蔓延している場であるがために，「時間がかかりそう」とか明示的に述べておく必要性が生じてしまっている気はします．）

[fana]

> 意味合いはあってますでしょうか？

* 条件が完全に定まっている（固定である．実行中に動的に変化しない）し，
* 結局そんなにパターンが多いわけでもないし（実際上，数十パターンとかの処理があるわけじゃないですよね，きっと）
* 入力のON/OFFに依存する処理が複数個所にあるわけではない

…みたいな場合，switch で xInput の値そのもので処理分岐を書くのが｛手っ取り早い，それで十分である，わかりやすい，etc｝な可能性もあるよね，という話です．

逆に言えば，
動作中に入力に応じて行うべき処理内容が変わってくるとか，Input1のON/OFF状況を他所でも判断せねばならないとか，そういう場合には別の方法を考えた方が良いかもしれません．

[changryu]

この度はお世話になりました。回答をいただき誠にありがとうございました。
それと回答に時間がかかり申し訳ございませんでした。

今回ベストアンサーとさせて頂いた理由として、以下の内容があります。
1.可読性の良さ
2.今回の用途に合っている
3.限られた時間で実装できたこと

結局は全て自分の知識レベルによるものですが、修正などで時間が経ってから見直したときに分からないようでは問題があるので、現在の自分で理解して、使いこなし、最初に考えた案より改善されたからです。

ひとまず今回はここをゴールとしますが、引き続き電気設計も含めて見直し、またプログラミング技術向上にも努め、本当の意味での最適化を目指したいと思います。

改めてお礼申し上げます。ありがとうございました。

Answer 2

ここ長いこと、C系のcode書いてないので考え方だけですが。ポインタ扱いの表現間違えてるかも、なのでそこ注意してください。

長さ256の 「functionへのポインタ」の配列を用意します。
配列[0] = *全部onの時の処理関数
配列[1] = *Input1以外がonの時の処理関数
：
：
配列[254] = *input2のみonの時の処理関数
配列[255] = *inout1のみonの時の処理関数
( 装置の性質からしてありえない入力の組み合わせが有ったらそこは省略可能）

でもって *配列[入力の値] を実行

Comments

[changryu]

winterboum様

早速の回答に感謝致します。週末バタバタしており返信が遅くなりましたことをお許しください。
回答を読ませて頂き、自分がなんとなく出来るのではないかという事のヒントにすごくなりました。
これをどう実装するのか？ここから成長できる気がします。
これをヒントに自分でもどう解決するか検討してみますので、少しお時間いただければと思います。

[winterboum]

ポインタ ってややこしいので楽しんでください。
関数へのポインタは変数への よりも楽しめるかな。
ポインタ使いこなせれば２ランクアップです

[fana]

質問は C# の話ですよね？
（あえて別言語な雰囲気で回答せずとも良いのでは…？）

それはそれとして，
本当にそんな膨大なパターン毎にやることが違うというような話であれば，そのテーブルを用意するのと巨大 switch 書くのとで労力変わらないような気もしますね．
（まぁパターン毎の処理が動的に変わる！　とか，同じような分岐箇所がたくさんあるとかなってくるならば別ですが）
→　というわけで，「switchでよくね？　（逆に何がだめなん？）」っていう回答を別途作っておきました．

[winterboum]

書く労力は変わらんですね。caseがまばらだとswitchの方が見通し良いかも。
色々な書き方のなかで、こういうのもありますよ、という例としてだしました。
で、
実行速度が早くなります。
というか
switchだとcaseの最初と最後ではcaseの評価回数分だけ実行着手が遅れますが配列だとそういう差は出てきません。（caseの評価にどんだけ時間がかかるんだ てのはありますが。）
あと、caseの順番を吟味しないとまずいのでその負荷も減るかな。
１の場合と、１と３の場合と、１と３と５の場合があるとき、10101010 なパターンのときはどれを選ぶの？というのを考えて順番を整えないと。

[fana]

> switchだとcaseの最初と最後ではcaseの評価回数分だけ実行着手が遅れますが

もうちょっと効率がよい形にコンパイラ様が頑張ってくれるのではないか（２分木的にとか），と根拠なく期待しています（実際どうなるのか知りません）．

> caseの順番を吟味

順番とは，fall through を想定しているのでしょうか？
そこは割り切って｛１の場合，１と３の場合，１と３と５の場合，…｝は全て独立した case で個別に実装する（最大で256分岐）ということにすれば，（caseの順番には意味がなくなるので）配列案と差は無いかと．

[winterboum]

え！？
私の古い記憶では、最初にマッチするcaseを実行して終わってしまうのですが。違いましたっけ？
switch flag のflagの値が 1 かつ 12 なんてことはありえないから、全てのcaseを舐める必要ないですよね。

fall through は想定してません。というか、それを使ってcodeを短くしたり可読性上がるなら使ってもよいですが。
２分木的に は無理でしょうね、場合が樹枝状になるわけでは無いから。
私がコンパイラなら 分岐先のアドレスを並べておいて(配列だ！)switchの値をレジスタに入れ、間接参照JUMPですね。

あと、可読性がちがうな。
switchだと文脈の中に置きますよね、ですので そこに256〜512行の固まりが入る。
配列の場合は、配列の定義と関数の定義は文脈の流れと関係ない所におけますから、1行で済むのでcode読むのを邪魔しません。配列の命名センス次第ですが。
場合が256もなく、1画面に収まるなら switchの方が読みやすいかな、と思います。

[winterboum]

そか、 C#のコンパイラが
＞分岐先のアドレスを並べておいてswitchの値をレジスタに入れ、間接参照JUMP
させるなら、関数のポインタへの配列 使うのと実質同じですね。caseの記述順には影響しない。

すると可読性だけの違いか

[fana]

> 1画面に収まるなら

実際上，そういう判断基準みたいなのがありますね．
自分で switch を挙げておいてなんですが，switch-case って無駄に行数食いがちな形してるところがイマイチ好きじゃない感．

case XXX: 処理; break;
case YYY: 処理; break;

という感じで各 case を１行に書いても見難くないようなやつじゃないと，ちょっとつらい．
「処理;」の前後に改行入れて各caseが３行になるとか，各 case の間に空行入れたいとか，処理のところを {} で囲んでブロックにしたいとか言い始めると，大したことしてないくせにやたらとスペースだけ食う塊が爆誕するので．

[winterboum]

ですね。
で、実を言うと私 if elsif、、、もswitch case も避けたい方なんです。
if が来ると 「これは例外処理か単なる場合分けか」とちょっと緊張して文脈からそれてしまう、ってのもあるのですが、両者とも
全ての場合が網羅されているか
欠けているのは 無いからなのか見落としなのか
を考えなければならなくなって。
場合分けを他で定義できれば、その書き方の工夫で場合分けの網羅性の確認がしやすいので。

[changryu]

この度はお世話になりました。回答をいただき誠にありがとうございました。
それと回答に時間がかかり申し訳ございませんでした。

ポインタ。なんとなく良さそうな気がしたのですが、今の自分にはまだ早かったようです。
こちらの内容も検討していると１年位平気で時間が経ってしまいそうなのでこちらの質問についてはクローズとさせて頂きたいと思います。
どうぞご理解のほど宜しくお願い致します。

いつかは２ランクアップを目指して頑張りたいと思います。

改めてお礼を申し上げます。ありがとうございました。

Answer 3

同時に複数のInputが入っていることも想定し、若いInput入力でbreak;をせず全Inputの状態を取得したいです。

現状上位桁が0の時、取り逃してしまいますね。

Input1が入っていることを条件にInput3はアクション可能というような使い方も想定しています。

enumを使うと判定をわかりやすく書けます（.NET Frameworkの場合は、HasFlagは遅いで有名なので注意してください）
FlagsAttribute クラス (System) | Microsoft Learn
Enum.HasFlag(Enum) メソッド (System) | Microsoft Learn

ふと。もっと良い書き方、例えば配列を使ったりする方法もあるのではないか？と思った次第です

例えばbinaryStr[i]のようにインデクサをお使いなので、配列を使っているのとほぼ同じことかと思います。
イヤなのはその後の、if羅列のほうですよね。

iやcountを使って処理を共通化できるなら、if羅列を消すことができるでしょう。

---

前回の質問の続きでしょうから寄せました。
WindowsフォームアプリケーションにてDllを使いたい

using System;
using System.Diagnostics;
using System.Drawing;
using System.Linq;
using System.Windows.Forms;

namespace Qvcazys1ngsxj93
{
    public partial class Form1 : Form
    {
        private DataGridView dataGridView1;
        private Timer timer1;
        private object handle;

        public Form1()
        {
            InitializeComponent();

            dataGridView1 = new DataGridView
            {
                AutoSizeColumnsMode = DataGridViewAutoSizeColumnsMode.Fill,
                Dock = DockStyle.Fill,
                Parent = this,
            };

            dataGridView1.Columns.Add(new DataGridViewTextBoxColumn { HeaderText = "Board", });

            for (int i = 1; i <= 8; i++)
                dataGridView1.Columns.Add(new DataGridViewTextBoxColumn { HeaderText = $"in {i}", });

            dataGridView1.Rows.Add("hoge");
            dataGridView1.Rows.Add("fuga");
            dataGridView1.Rows.Add("piyo");


            timer1 = new Timer { Interval = 1000, };
            timer1.Tick += Timer1_Tick;
            timer1.Start();
        }

        private void Timer1_Tick(object sender, EventArgs e)
        {
            int xInput = Top16.GetInputs(handle);

            // 0埋めで8桁取得（上位桁の0を取り逃さないように）
            string binaryStr = Convert.ToString(xInput, 2).PadLeft(8, '0');

            // 確認用（出力ウィンドウ）
            Debug.Write($"\n{binaryStr}");
            Debug.Write($" →逆順 {string.Join("", binaryStr.Reverse())}");
            Debug.WriteLine($" →反転 {string.Join("", binaryStr.Reverse().Select(x => x == '0' ? '1' : '0'))}");


            // hoge行の処理
            int count = 0;
            foreach (char inputBinaryNum in binaryStr.Reverse()) // 右が1なのでひっくり返す
            {
                count++;
                if (inputBinaryNum == '0') // 0だったら入力有
                {
                    Debug.Write($"Input{count}, ");
                    dataGridView1.Rows[0].Cells[count].Style.BackColor = Color.Blue;
                }
                else
                {
                    dataGridView1.Rows[0].Cells[count].Style.BackColor = Color.White;
                }
            }
            Debug.WriteLine("");


            // fuga行の処理
            Flag flag = (Flag)~xInput; // 各ビット反転してenumにキャスト
            Debug.WriteLine(flag);

            var row1 = dataGridView1.Rows[1];

            // ループにしずらいのでこういうのは向かないかなｗ
            if (flag.HasFlag(Flag.Input1)) row1.Cells[1].Style.BackColor = Color.Blue;
            else row1.Cells[1].Style.BackColor = Color.White;
            // 以下同じ意味
            row1.Cells[2].Style.BackColor = flag.HasFlag(Flag.Input2) ? Color.Blue : Color.White;
            row1.Cells[3].Style.BackColor = flag.HasFlag(Flag.Input3) ? Color.Blue : Color.White;
            row1.Cells[4].Style.BackColor = flag.HasFlag(Flag.Input4) ? Color.Blue : Color.White;
            row1.Cells[5].Style.BackColor = flag.HasFlag(Flag.Input5) ? Color.Blue : Color.White;
            row1.Cells[6].Style.BackColor = flag.HasFlag(Flag.Input6) ? Color.Blue : Color.White;
            row1.Cells[7].Style.BackColor = flag.HasFlag(Flag.Input7) ? Color.Blue : Color.White;
            row1.Cells[8].Style.BackColor = flag.HasFlag(Flag.Input8) ? Color.Blue : Color.White;


            // 複数条件はわかりやすい
            //if (flag.HasFlag(Flag.Input1) && flag.HasFlag(Flag.Input3)) row.Cells[3].Style.BackColor = Color.Red;
            // ↑↓意味は同じ
            if (flag.HasFlag(Flag.Input1 | Flag.Input3)) row1.Cells[3].Style.BackColor = Color.Red;

            // 意味のある名前を付けたければこうでもいい
            //if (flag.HasFlag(Flag.Input1_and_3)) row.Cells[3].Style.BackColor = Color.Red;


            //// piyo行の処理
            //var row2 = dataGridView1.Rows[2];

            //for (var i = 1; i <= 8; i++)
            //{
            //    row2.Cells[i].Style.BackColor = Color.White;

            //    // インデックス（C# 8.0以上）
            //    // [インデックス/範囲処理 - C# によるプログラミング入門 | ++C++; // 未確認飛行 C](https://ufcpp.net/study/csharp/data/dataranges/)
            //    if (binaryStr[^i] == '0') row2.Cells[i].Style.BackColor = Color.Blue;
            //}

            //// リスト パターン（C# 11以上）
            //// [パターン マッチング - C# によるプログラミング入門 | ++C++; // 未確認飛行 C](https://ufcpp.net/study/csharp/datatype/patterns/?p=2#list)
            //row2.Cells[3].Style.BackColor = binaryStr switch
            //{
            //    // Input1・Input3の入力があり、Input8の入力がなければ。。。
            //    ['1', .., '0', _, '0'] => Color.Green,

            //    // Input1・Input3（・Input8）の入力があれば。。。
            //    // ↑に当てはまらない時なのでここに入るのはInput8もONなことになる
            //    [.., '0', _, '0'] => Color.Red,

            //    [.., '0', _, _] => Color.Blue, // Input3の入力があれば。。。

            //    _ => Color.White, // default
            //};

            //// ifを並べたほうがわかりやすい場合も？
            ////if (binaryStr is [.., '0', _, '0']) row2.Cells[3].Style.BackColor = Color.Red;
            ////if (binaryStr is ['1', .., '0', _, '0']) row2.Cells[3].Style.BackColor = Color.Green;
        }

        [Flags]
        enum Flag : byte
        {
            None = 0,
            Input1 = 1,
            //Input1 = 0b0000_0001, // こうのほうがわかりやすければそれでもいい
            //Input1 = 0b1111_1110, // こうはダメ！
            Input2 = 2,
            Input3 = 4,
            Input4 = 8,
            Input5 = 16,
            Input6 = 32,
            Input7 = 64,
            Input8 = 128,

            Input1_and_3 = Input1 | Input3, // 複数条件に意味のある名前を付けたければ
        }

        private class Top16 // 雑いｗ
        {
            private static Random random = new Random();
            public static int GetInputs(object handle) => random.Next(256);
        }
    }
}

Comments

[TN8001]

結局「ビット演算する」がベストな気もしますが...^^;

[changryu]

TN8001様

早々にご回答頂いていたのにも関わらず、別の仕事が立て込んでおり返信が遅くなり大変申し訳ございません。
この度はご回答ありがとうございました。

ご回答を見させて頂き、第一に前回の質問との関連まで読み取って頂き本当に感謝致します。
その通りでございまして、前回まではTop16の使い方や特性を探っておりまして、実際にTop16で装置を制御しようとした際に、装置のそれぞれのフェーズ（状況）でInput/Outputを使う必要があり（この辺もクラスで部品化できればいいのですが・・・）、もうちょっとちゃんと考えなければいけないことに気付いた次第です。

次に、ご指摘の『イヤなのはその後の、if羅列のほうですよね。』
ほんとそうなんです。感覚的に美しくない（ダサい）ということは分かっているのですが、おそらく"最低限"やりたい事はできるかも知れないが、最高（最適）ではない気がしておりまして・・・言語化できない知識量ですみません。

最後に自分の今持っている知識を遥かに上回ったアドバイスをして頂いたことと同時に、やはりもっと違うやり方はあるんだな！（当たり前かw）ということを知ることができて今回思い切って質問して良かったと思いました。
私自身、だいぶ昔に大学でちょっとプログラミングをかじっただけで、リアルでは先輩プログラマーがいないため、こんな時に先輩プログラマーがいたらどんなアドバイスされるのかなーと想像して今回質問させて頂いたのもあり、感動しました。

せっかくなので今回頂いたアドバイスを元にもう少し自分でも考えて試して見たいと思います。
少しお時間を頂くことになるかと思いますが、もう少しお時間をいただければと思います。

[TN8001]

> 現状上位桁が0の時、取り逃してしまいますね。

まずこの「バグ」の指摘が第一義なのですが、そこは通じてますでしょうか。

例えば入力が51（0011_0011）の時は、Input3,Input4,Input7,Input8です。
Convert.ToString(xInput, 2) ですと、文字列は"110011"になりInput7,Input8を取り逃します。
どの数値も確実に8桁確保するため、回答では .PadLeft(8, '0') としました。

問題点は見た瞬間わかりましたが、実際どうするのかは（わたしも）都度ｇｇっています。
[c# 2進数 ゼロ埋め - Google 検索](https://www.google.co.jp/search?q=c%23+2%E9%80%B2%E6%95%B0+%E3%82%BC%E3%83%AD%E5%9F%8B%E3%82%81)

> 次に、ご指摘の『イヤなのはその後の、if羅列のほうですよね。』
> 最後に自分の今持っている知識を遥かに上回ったアドバイスをして頂いた

できるだけ平易に書いたつもりですが、わかりにくい点があるなら遠慮なく聞いてください。

foreachに直したのが分かりにくいですか？（forに戻しても動作は同じです）

//Input1が入ったときのアクション内容を書く

の内容を前回質問から推測して、↓としましたがこれ自体はわかりますよね？
dataGridView1.Rows[0].Cells[count].Style.BackColor = Color.Blue;

それとも全然違う内容ということでしょうか。
実行内容がてんでばらばらとなると、どこかしらに値と処理内容は書くことにはなるでしょう（その書き方は処理内容や個人の好み等でいろいろありえます）

>> Input1が入っていることを条件にInput3はアクション可能というような使い方

のわかりやすい判定としてFlags enumをご紹介しました。

機器からの入力は250（1111_1010）だそうですが、プログラムでは扱いにくいのでビットを反転して5（0000_0101）として扱います（「~（チルダ）」のところ）

if (flag.HasFlag(Flag.Input1) && flag.HasFlag(Flag.Input3))
enumを使用して「どちらもオンの時」という条件が分かりやすく書けました。

ただし本来はORを取って（「|（パイプ）」のところ）こう書いたほうがいいです（ビット演算を説明したくなかったので^^;
if (flag.HasFlag(Flag.Input1 | Flag.Input3))

詳しくは以下や以下の中で出てきたキーワードでさらにｇｇってください。
[ビットごとの演算子とシフト演算子 - 整数型の個々のビットに対して、ブール演算 (AND、NOT、OR、XOR) とシフト演算を実行します - C# | Microsoft Learn](https://learn.microsoft.com/ja-jp/dotnet/csharp/language-reference/operators/bitwise-and-shift-operators)

> "最低限"やりたい事はできるかも知れないが、最高（最適）ではない気がして

やりたいことができるのがまず第一で美しさは2の次ですよね（特にお仕事なら）
わたしは趣味グラマなので「最高」のものができなかったら、「しばらく寝かしておく」ようなことができますがｗ
「最高のものができた」と思っても、１年後に見たら書き直したくなるというのもよくある話です^^;

> せっかくなので今回頂いたアドバイスを元にもう少し自分でも考えて試して見たいと思います。
> 少しお時間を頂くことになるかと思いますが、もう少しお時間をいただければと思います。

はい。別に１週間でも１か月でも構いませんが、個人的には１か月以内くらいには（自己解決でも誰かのBAでもいいですが）何らかの結論が欲しいですね。

解決後でもコメントいただければ返信します（内容によっては新たな質問として建てるよう誘導することもあります）

[changryu]

フォロー含め、ご配慮に感謝致します。

私事ですが、零細企業の雇われ人の為、装置の設計から制御、納品、サポートまでしている関係で、また現在別の装置の設計フェーズに入っており、私の能力では設計業務と制御（プログラミング）を同時期に考えることができな為、お時間を下さい。

それは時間的な問題ですが、たぶんプログラミングが好きだったりするのか、または性格なのか経験的な問題なのか、理解していないこと、分かっていないことに対して「分かりました」と言えない性分でして、ちゃんと理解したい、使いこなしたい、という意識も働いて片手間でやりたくない意識も働いています。

また来週末にでもゆっくりと読ませて頂き、試させて頂きたいと思います。きっとまた躓くと思いますwその際は・・・

そんな訳でまずはお時間を頂いたこと、アドバイスをして頂いたことに感謝致します。ありがとうございました。

[TN8001]

はい。わたし達（回答者）は誰も急いでいませんのでごゆっくりどうぞ^^

[changryu]

この度はお世話になりました。回答をいただき誠にありがとうございました。
それと回答に時間がかかり申し訳ございませんでした。

僅かなスキマ時間にFlags enumについて調べていたりしたのですが、
「あ、そうか！このソースそのままコピペして試してみればいいんだ」と２週間位前に気づき（遅っ・・・）、試してみたのですが、

37行目、41行目で［インデックスが範囲を超えています。負でない値で、コレクションのサイズよりも小さくなければなりません。］

というエラーが出て、調べようとしてまた時間が経ってしまいました。

順番代わりますが、

> 現状上位桁が0の時、取り逃してしまいますね。

そうですね。気づかせていただきありがとうございます。

それで話は代わりますが、今回は大変申し訳ございませんがfana様の回答にベストアンサーをさせていただきました。
気持ち的にはTN8001様の案のFlags enumとfana様の案のSwitchを組み合わせるとどうなんだろう？と思ったのですが、もう少し時間がかかりそうなのでSwitch案を今回のゴールとさせて頂きました。すみません。

おそらくもう少し時間がかかると思いますが、上記のエラーの件引き続き調べてみたいと思います。

[TN8001]

> というエラーが出て、調べようとしてまた時間が経ってしまいました。

すぐ言ってくれたらよかったのに。。ｗ
画像を付けたのでわかるかなと省略してしまいましたが、デザイナでDataGridViewと９列分のDataGridViewTextBoxColumn（あとTimer）を追加済みという想定でした。すいません。
コピペだけで実行できる状態に回答コードを更新しました^^

> 気持ち的にはTN8001様の案のFlags enumとfana様の案のSwitchを組み合わせるとどうなんだろう？と思った

switchは結局 if() {} else if() {} else{} と同義ですからひとつしか選べません。
if() {} if() {} if() {} と同じ意味でないことに注意が必要です（それはわかっていてこの質問ということでしょうけど^^;
条件が複雑なら複数のifやswitchが出てくるのは仕方がないでしょう。

switchで地味に困るのは、任意の値でいい部分があるときですね。
パターンマッチング等を使えれば、場合によってはわかりやすく書けるかもしれません。
.NET Frameworkでも使えないこともないですがいくつか準備が必要になります（ので回答コードはとりあえずコメント化しています）

Answer 4

for(int i=1;i<0x100;i<<1){
  if(0==(xinput&i)){ なんやかや }
}

でどうでしょう

Comments

[fana]

これだけだとすぐに分からない場合には
「ビット演算」とか「ビットフラグ」みたいな言葉を検索してみると良いかも．

[changryu]

y_waiwai様
funa様

早々にご回答頂いていたのにも関わらず、別の仕事が立て込んでおり返信が遅くなり大変申し訳ございません。
この度はアドバイス頂きましてありがとうございました。

知識がなく大変恐縮なのですが、for文が何をしているのか分からず、検索してみたらfuna様の書かれていたビット演算がひっかかったのですが、今回の件についてはこう使うのが正解な気がしてきました。

ちょっとまだ理解しきれていないので、ちょっと自分でも試して血肉にしたいと思います。
理解するまでにもう少し時間が必要そうなので、先にお礼をさせて頂きます。

[changryu]

この度はお世話になりました。回答をいただき誠にありがとうございました。
それと回答に時間がかかり申し訳ございませんでした。

もう少し勉強して出直して来ます。ありがとうございました。