演算子が複数含まれている式の計算

Question

電卓を作成していて、演算子が複数出てくる式の計算の処理で躓いています。
数字や演算子ボタンを押下するとTextBoxに式が表示されるようにはなったのですが、
BtnEqual_Clickメソッドでの処理で
演算子の後にオペランドがありませんとエラーが出てしまいます。

例）　3+5*20-7
と表示されているものを計算させたいと考えております。（293と表示させる）
現段階では積、徐を優先的にさせる必要はなく、左から順に計算させるところまでで十分です。

public partical class Form1 : Form
{
  private String _TxtNumber = "0";

  public Form1()
  {
    IntializeComponent();
  }

  //数字を押したときの処理
  private void BtnNumber_Click(object sender, EventArgs e)
  {
    //(Button)senderの意味が分からない
　　Button BtnNumber = (Button)sender;
    textBox1.Text = textBox1.Text + BtnNumber.Text;
    _TxtNumber = textBox1.text;
  }

  //演算子を押したときの処理
  private void BtnOperator_Click(object sender, EventArgs e)
  {
　　Button BtnOperator = (Button)sender;
    textBox1.Text = _TxtNumber + BtnOperator.Text;
  }

  //イコールを押したときの処理
  private void BtnEqual_Click(object sender, EventArgs e)
  {
    dynamic valResult = new Datable().Compute(textBox.Text, null);
    textBox.Text = valResult.ToString();
  }
}

Answer 1

電卓の演算子ボタンのTextを全角で＋,ー,×,÷と表示させてしまっていたので、
半角で+,-,*,/と表示させたところ無事にイコールボタン押下時に計算結果が表示されるようになりました。

ただ、回答のコメントにもあるようにDataTable().Compute("", null)の正しい使い方ではないと思われます。
演算子が複数含まれる状態の計算、連続計算の機能を実装させる一般的な方法としては逆ポーランド記法が一般的な様です。

Answer 2

イコールボタン押下時に計算させる処理が分からない

とは、 BtnEqual_Click()での処理が分からないという事でしょうか?
"TextBoxに式が表示されるようにはなった" という事は、textBox1.Textに (例えば)"３＋５×２０ー７"が入っているという事ですね。
簡単には、

    foreach (char c in textBox1.Text) {
        switch (c) {
            // 一文字ずつ処理 ...
        }
    }

という感じでしょうか。

なお、

//(Button)senderの意味が分からない

特定のボタンに紐づいているイベントハンドラならば、気にする必要はありません。複数のボタンに一つのイベントを紐づけられるので、その時に、どのボタンから呼ばれたかの判断などに参照します。

Comments

[karin10]

BtnEqual_Click()内での処理が分からないという感じです。
dynamic valResult = new Datable().Compute(textBox.Text, null);
textBox.Text = valResult.ToString();
と処理を入れているのですが、演算子の後にオペランドがありませんとエラーがでている状態です。

[pepperleaf]

> Datable().Compute()
もしかして、DataTable ? <https://docs.microsoft.com/ja-jp/dotnet/api/system.data.datatable?view=netframework-4.7.2>でしょうか。
それならば、使い方を間違えてます。詳しくありませんが、多分、使えないと思います。
この手の処理は、一文字ずつ見るしかないでしょう。(C#では)
まずは、一文字ずつ処理する事を考えるべきと思います。
また、テキスト表示を全て全角にしているようですが、半角数字等にした方が、扱いは楽です。(本質でないですが)

[karin10]

そもそも使い方が間違っていたんですね。
もし可能であれば一文字ずつ処理する上で参考になりそうな記事を教えて頂けますでしょうか。
何故か全角で入力していました。すみません。修正しておきます。

[Zuishin]

使える使えないで言えば使えます。ただし乗算の記号が違います。「×」ではなく「*」でないといけません。他の記号も全角のようなので、すべて半角にする必要がありますね。

Console.WriteLine(new DataTable().Compute("3+5*20-7", null));

また演算子の優先順位は左からではなく通常のものになります。

このように計算ができても、本来の用途ではないので使わない方がいいでしょう。

[pepperleaf]

Zuishinさん、コメントありがとうございます。
確かに使えました。

Answer 3

Button BtnNumber = (Button)sender;

()の部分はキャスト（型変換）です。このメソッドの第一引数として受け取ったObject型のsenderをButton型に型変換しています。

Answer 4

＋が来たら両側の数値を加算し、
－が来たら両側の数値を減算し、
＊が来たら掛けて、
／が来たら割ればいいだけです。

がんばってコードを書きましょう