質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.35%
C#

C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。

Q&A

解決済

2回答

895閲覧

C#での電卓作成について

terayabasu

総合スコア4

C#

C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。

0グッド

0クリップ

投稿2021/02/01 08:18

前提・実現したいこと

C#初心者です。Visual C#で電卓を作成しています。
一桁の簡単な四則演算ができるものを作ろうとしているのですが、

発生している問題・エラーメッセージ

デバックをすると、計算結果が間違っていたり「値を Null にすることはできません。
パラメータ名: String」という文言が出てきてしまい、思うように動きません。
自分ではどこのデータ型が間違っているのか理解できないのでご教示いただきたいです。

該当のソースコード

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 Dentaku
{
public partial class Form1 : Form
{
public Form1()
{
InitializeComponent();
}

string aaa; string bbb; string symbol; int keisankekka ; // =ボタン private void button6_Click(object sender, EventArgs e) { int a = int.Parse(aaa); int b = int.Parse(bbb); int keisankekka; keisankekka = 0; if (symbol == "+") { keisankekka = a + b; } else if (symbol == "-") { keisankekka = a - b; } else if (symbol == "*") { keisankekka = a * b; } else if (symbol == "/") { keisankekka = a / b; } textBoxBottom.Text = keisankekka.ToString(); } // Cボタン private void button5_Click(object sender, EventArgs e) { textBoxTop.ResetText(); textBoxBottom.ResetText(); } // 計算記号(+~/) public void button1_Click(object sender, EventArgs e) { this.textBoxTop.AppendText(buttonPlus.Text); //テキストボックスに反映 textBoxBottom.ResetText(); symbol = "+"; } public void button2_Click(object sender, EventArgs e) { this.textBoxTop.AppendText(buttonMinus.Text); textBoxBottom.ResetText(); symbol = "-"; } public void button3_Click(object sender, EventArgs e) { this.textBoxTop.AppendText(buttonTimes.Text); textBoxBottom.ResetText(); symbol = "*"; } public void button4_Click(object sender, EventArgs e) { this.textBoxTop.AppendText(buttonDivided.Text); textBoxBottom.ResetText(); symbol = "/"; } //数字キー(0→.→1~9) public void buttonN0_Click(object sender, EventArgs e) { this.textBoxTop.AppendText(buttonN0.Text); this.textBoxBottom.AppendText(buttonN0.Text); if (aaa != "") { bbb = "0"; } else { aaa = "0"; } } // 数字(1~9) public void buttonN1_Click(object sender, EventArgs e) { this.textBoxTop.AppendText(buttonN1.Text); this.textBoxBottom.AppendText(buttonN1.Text); //aaa =aaa + "1"; if (aaa != "") { bbb = "1"; } else { aaa = "1"; } } public void buttonN2_Click(object sender, EventArgs e) { this.textBoxTop.AppendText(buttonN2.Text); this.textBoxBottom.AppendText(buttonN2.Text); aaa = aaa + "2"; if (aaa != "") { bbb = "2"; } } public void buttonN3_Click(object sender, EventArgs e) { this.textBoxTop.AppendText(buttonN3.Text); this.textBoxBottom.AppendText(buttonN3.Text); if (aaa != "") { bbb = "3"; } else { aaa = "3"; } } public void buttonN4_Click(object sender, EventArgs e) { this.textBoxTop.AppendText(buttonN4.Text); this.textBoxBottom.AppendText(buttonN4.Text); if (aaa != "") { bbb = "4"; } else { aaa = "4"; } } public void buttonN5_Click(object sender, EventArgs e) { this.textBoxTop.AppendText(buttonN5.Text); this.textBoxBottom.AppendText(buttonN5.Text); if (aaa != "") { bbb = "5"; } else { aaa = "5"; } } public void buttonN6_Click(object sender, EventArgs e) { this.textBoxTop.AppendText(buttonN6.Text); this.textBoxBottom.AppendText(buttonN5.Text); if (aaa != "") { bbb = "6"; } else { aaa = "6"; } } public void buttonN7_Click(object sender, EventArgs e) { this.textBoxTop.AppendText(buttonN7.Text); this.textBoxBottom.AppendText(buttonN7.Text); if (aaa != "") { bbb = "7"; } else { aaa = "7"; } } public void buttonN8_Click(object sender, EventArgs e) { this.textBoxTop.AppendText(buttonN8.Text); this.textBoxBottom.AppendText(buttonN8.Text); if (aaa != "") { bbb = "8"; } else { aaa = "8"; } } public void buttonN9_Click(object sender, EventArgs e) { this.textBoxTop.AppendText(buttonN9.Text); this.textBoxBottom.AppendText(buttonN9.Text); if (aaa != "") { bbb = "9"; } else { aaa = "9"; } } private void Form1_Load(object sender, EventArgs e) { } }

}

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

退会済みユーザー

退会済みユーザー

2021/02/01 08:30

・ソースコードをそのまま記載するとインデントが崩れるので、「コードの挿入」で記入してください。 ・フレームワークのバージョン(.NET Framework X.XX, .NETCore X.X等)、プロジェクトの種類(WindowsForms, WPF, コンソール等)を記載してください
terayabasu

2021/02/01 09:23

見落としてしまっていて返信を忘れてしまってすみません。以後気をつけたいと思います。
guest

回答2

0

タイトルの問題は変数を格納する場所を作らずに宣言をしていたことが原因で、計算の間違いはコードの改変し忘れでした。回答をいただいたy_waiwaiさん本当にありがとうございます!

投稿2021/02/01 09:21

terayabasu

総合スコア4

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

0

ベストアンサー

// =ボタン

private void button6_Click(object sender, EventArgs e)
{
int a = int.Parse(aaa);
int b = int.Parse(bbb);

いきなり=ボタンを押したらどうなるのか、ってのを考えてみましょう。
aaaとbbbにはなにが入ってるでしょうか

投稿2021/02/01 08:25

編集2021/02/01 08:26
y_waiwai

総合スコア88042

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

terayabasu

2021/02/01 08:33

ご回答ありがとうございます。 最初にイコールボタンを押した場合、aaaとbbbに値が入っていないと思います。
y_waiwai

2021/02/01 08:37

そこでエラーが出てしまいますね 値が入ってないときにエラーが出ないようにする必要があります まずは、 string aaa=""; string bbb=""; として、初期値に空文字列を入れときましょう
terayabasu

2021/02/01 08:44

ありがとうございます。定義しただけではダメという初歩的なこともわかっておらずすみません。助言いただいたおかげで足し算、引き算、かけ算は正しい結果が出るようになったのですが、割り算だけ正しい結果が出力されません。同じ質問内で二度質問して申し訳ありません。割り算だけうまくいかないのはどのような点に問題があるのでしょうか。
y_waiwai

2021/02/01 08:51

どううまくかないのかを書いてください 割り算は、0で割るとエラーになります。 なので、割る数が0の場合にエラーにならないようにする必要がありますね
terayabasu

2021/02/01 08:56

9/9や5/5のような答えが1になる計算以外は誤った値(8/2が41になるなど)が返されてしまいます。0はまだエラーよけをしていないのでデバッグの際に入力しておりません。
y_waiwai

2021/02/01 09:04

> keisankekka = a / b; の行の左側の灰色のところをクリックすると、そこに赤い印が付きます そうしておくとそこで実行が止まりますんで、aとかbの値を見てみてください。 https://itsakura.com/visualstudio-debug
terayabasu

2021/02/01 09:19

ブレイクポイントを付けてデバッグしたところ、8/2のaの値が82、bの値が2になっていて、2のボタンのところのコードに消し忘れたものがあることに気づけました。 答えを直接教える形ではなく、ミスの見つけ方を丁寧に教えてくださってありがとうございます。本当に助かりました。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.35%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問