-2

電卓コードに問題があります。プラスとマイナスは期待どおりに機能しますが、除算と乗算は毎回ゼロの答えになります。コードは次のとおりです。

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Windows.Forms;

namespace Calculator
{
public partial class Form1 : Form
{
    public Form1()
    {
        InitializeComponent();
    }

    private void Form1_Load(object sender, EventArgs e)
    {

    }

    private void btn1_Click(object sender, EventArgs e)
    {
        txtDisplay.Text = txtDisplay.Text + btn1.Text;
    }

    private void btn2_Click(object sender, EventArgs e)
    {
        txtDisplay.Text = txtDisplay.Text + btn2.Text;
    }

    private void btn3_Click(object sender, EventArgs e)
    {
        txtDisplay.Text = txtDisplay.Text + btn3.Text;
    }

    private void btn4_Click(object sender, EventArgs e)
    {
        txtDisplay.Text = txtDisplay.Text + btn4.Text;
    }

    private void btn5_Click(object sender, EventArgs e)
    {
        txtDisplay.Text = txtDisplay.Text + btn5.Text;
    }

    private void btn6_Click(object sender, EventArgs e)
    {
        txtDisplay.Text = txtDisplay.Text + btn6.Text;
    }

    private void btn7_Click(object sender, EventArgs e)
    {
        txtDisplay.Text = txtDisplay.Text + btn7.Text;
    }

    private void btn8_Click(object sender, EventArgs e)
    {
        txtDisplay.Text = txtDisplay.Text + btn8.Text;
    }

    private void btn9_Click(object sender, EventArgs e)
    {
        txtDisplay.Text = txtDisplay.Text + btn9.Text;
    }

    private void btn0_Click(object sender, EventArgs e)
    {
        txtDisplay.Text = txtDisplay.Text + btn0.Text;
    }

    private void btnClear_Click(object sender, EventArgs e)
    {
        txtDisplay.Clear();
    }

    Double total1 = 0;
    Double total2 = 0;
    bool minusButtonClicked = false;
    bool plusButtonClicked = false;
    bool divideButtonClicked = false;
    bool multiplyButtonClicked = false;

    private void btnPlus_Click(object sender, EventArgs e)
    {
        total1 = total1 + Double.Parse(txtDisplay.Text);
        txtDisplay.Clear();

        plusButtonClicked = true;
        minusButtonClicked = false;
        divideButtonClicked = false;
        multiplyButtonClicked = false;
    }

    private void btnMinus_Click(object sender, EventArgs e)
    {
        total1 = total1 - Double.Parse(txtDisplay.Text);
        txtDisplay.Clear();

        plusButtonClicked = false;
        minusButtonClicked = true;
        divideButtonClicked = false;
        multiplyButtonClicked = false;
    }

    private void btnDivide_Click(object sender, EventArgs e)
    {
        total1 = total1 / Double.Parse(txtDisplay.Text);
        txtDisplay.Clear();

        plusButtonClicked = false;
        minusButtonClicked = false;
        divideButtonClicked = true;
        multiplyButtonClicked = false;
    }

    private void btnMultiply_Click(object sender, EventArgs e)
    {
        total1 = total1 * Double.Parse(txtDisplay.Text);
        txtDisplay.Clear();

        plusButtonClicked = false;
        minusButtonClicked = false;
        divideButtonClicked = false;
        multiplyButtonClicked = true;
    }

    private void btnEquals_Click(object sender, EventArgs e)
    {
        if (plusButtonClicked == true)
        {
            total2 = total1 + Double.Parse(txtDisplay.Text);
        }
        else if (minusButtonClicked == true)
        {
            total2 = total1 - Double.Parse(txtDisplay.Text);
        }
        else if (divideButtonClicked == true)
        {
            total2 = total1 / Double.Parse(txtDisplay.Text);
        }
        else if (multiplyButtonClicked == true)
        {
            total2 = total1 * Double.Parse(txtDisplay.Text);
        }

        txtDisplay.Text = total2.ToString();
        total1 = 0;
    }

}
}

if else ステートメントの何かですか、それとも他の一般的なエラーですか?

4

2 に答える 2

3

total1 と total2 を 0 に初期化します。

あなたのコード:

total1 = total1 * Double.Parse(txtDisplay.Text);

0 * #anynumber# = 0 であるため、常に 0 が返されます。

割り算も同じ: 0 / #anynumber# = 0.

あまり多くのブール値フラグを使用しないようにコードを整理することもできますが、とにかく、それは実際の問題ではありません。

于 2012-12-31T18:51:38.613 に答える
1

Dark Flacon が言ったように、等号ボタンが押されたときに計算を行う必要があります。total1また、 if fact が最初の数字を覚えているだけの場合は、合計を呼び出すべきではありません。

Double lastNumber = 0;
Double total = 0;
bool minusButtonClicked = false;
bool plusButtonClicked = false;
bool divideButtonClicked = false;
bool multiplyButtonClicked = false;

private void btnPlus_Click(object sender, EventArgs e)
{
    lastNumber = Double.Parse(txtDisplay.Text);
    txtDisplay.Clear();

    plusButtonClicked = true;
    minusButtonClicked = false;
    divideButtonClicked = false;
    multiplyButtonClicked = false;
}

private void btnMinus_Click(object sender, EventArgs e)
{
    lastNumber = Double.Parse(txtDisplay.Text);
    txtDisplay.Clear();

    plusButtonClicked = false;
    minusButtonClicked = true;
    divideButtonClicked = false;
    multiplyButtonClicked = false;
}

private void btnDivide_Click(object sender, EventArgs e)
{
    lastNumber = Double.Parse(txtDisplay.Text);
    txtDisplay.Clear();

    plusButtonClicked = false;
    minusButtonClicked = false;
    divideButtonClicked = true;
    multiplyButtonClicked = false;
}

private void btnMultiply_Click(object sender, EventArgs e)
{
    lastNumber = Double.Parse(txtDisplay.Text);
    txtDisplay.Clear();

    plusButtonClicked = false;
    minusButtonClicked = false;
    divideButtonClicked = false;
    multiplyButtonClicked = true;
}

private void btnEquals_Click(object sender, EventArgs e)
{
    if (plusButtonClicked == true)
    {
        total = lastNumber + Double.Parse(txtDisplay.Text);
    }
    else if (minusButtonClicked == true)
    {
        total = lastNumber - Double.Parse(txtDisplay.Text);
    }
    else if (divideButtonClicked == true)
    {
        total = lastNumber / Double.Parse(txtDisplay.Text);
    }
    else if (multiplyButtonClicked == true)
    {
        total = lastNumber * Double.Parse(txtDisplay.Text);
    }

    txtDisplay.Text = total.ToString();
    lastNumber = 0;
}
于 2012-12-31T18:52:30.757 に答える