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

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

新規登録して質問してみよう
ただいま回答率
85.48%
Windows 10

Windows 10は、マイクロソフト社がリリースしたOSです。Modern UIを標準画面にした8.1から、10では再びデスクトップ主体に戻され、UIも変更されています。PCやスマホ、タブレットなど様々なデバイスに幅広く対応していることが特徴です。

C#

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

Q&A

2回答

4928閲覧

計算後、値が0に変わる。

fabriz

総合スコア4

Windows 10

Windows 10は、マイクロソフト社がリリースしたOSです。Modern UIを標準画面にした8.1から、10では再びデスクトップ主体に戻され、UIも変更されています。PCやスマホ、タブレットなど様々なデバイスに幅広く対応していることが特徴です。

C#

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

0グッド

0クリップ

投稿2020/07/22 14:06

編集2020/07/22 14:46

下記のコードで、ffBtn_Clickを実行すると、
fresultval = fnicepcnt / ftotalcnt * 100;
でfresultvalの値が0になってしまいます。
やりたいことは、fresultvalに小数付きの値を算出することです

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

namespace WindowsFormsApp2
{
public partial class NicePitching : Form
{

public static int ftotalcnt = 0; public static int fnicepcnt = 0; public static int etotalcnt = 0; public static int enicepcnt = 0; public static decimal fresultval = 0; public static decimal eresultval = 0; public NicePitching() { InitializeComponent(); } private void ftBtn_Click(object sender, EventArgs e) { fnicepcnt += 1; ftotalcnt += 1; fresultval = fnicepcnt / ftotalcnt * 100; this.fResult.Text = fnicepcnt + "/" + ftotalcnt + " " + fresultval.ToString() + "%" ; } private void ffBtn_Click(object sender, EventArgs e) { ftotalcnt += 1; fresultval = fnicepcnt / ftotalcnt * 100; this.fResult.Text = fnicepcnt + "/" + ftotalcnt + " " + fresultval.ToString() + "%"; } }

}

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

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

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

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

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

退会済みユーザー

退会済みユーザー

2020/07/22 14:32 編集

fnicepcnt, ftotalcnt 両方が int 型だからではないのですか? (初心者マークを付けた方が良いと思います)
fabriz

2020/07/22 14:53

計算は型を合わせないと上記のようなことが起こるというのがプログラミングの仕様という認識であっていますか? 「両方の型がintだったから」にしても値が0になるのは納得いきません
fabriz

2020/07/22 14:56 編集

>計算は型を合わせないと 代入する変数側に型を合わせる という意味です >値が0になるのは納得いきません 小数切り捨ての値が入るのが正しいと思いました
退会済みユーザー

退会済みユーザー

2020/07/22 15:11

int 型同士の割り算の結果がどうなるかは基本のキですので、ググれば参考になる記事がヒットするのではないかと思います。やってみましょう。
Daregada

2020/07/22 15:19

割り算は、代入処理が行われるより前に実行されるので、その時点では整数同士の割り算になります。分母のほうが大きい場合は間違いなく0になります。その後の100との掛け算も整数同士ですね。そうして右辺の値が0と決まってから代入処理が行われるんで、あなたの納得に関わりなく0になるのがこの式の正しい動作です。
guest

回答2

0

C#に限らず、明示的に変数の型を宣言する言語のほとんどは、計算時に「より大きい」型へ自動的にキャストしてから計算します。これを拡大変換と呼びます。
逆に、型が同じデータ同士の計算は同じ型で結果が返されます。

質問文の計算 fresultval = fnicepcnt / ftotalcnt * 100;については、fnicepcnt 、ftotalcnt、100 の3つ全てのデータがint型です。
算術演算の優先順位通り、まずfnicepcnt / ftotalcntが計算されます。両方int型なので結果がintで返ってきます。小数点以下の数値は切り捨てられます。
次に、上記計算の結果と 100 の乗算が計算されます。こちらも結果がintで返されます。

1つ目の計算においてfnicepcnt < ftotalcntであるならば(題意を考えると常にこうなるはず)この計算の結果は常に0になると思われます。0に何をかけても0なので、全体の計算結果も常に0になります。
先に乗算を行っていれば結果が0になることはおそらくないでしょうが、小数点以下が切り捨てられるのは変わりません。

計算結果を期待する値で返したい場合、fnicepcnt と ftotalcnt をdecimal型にキャストしてから計算するのが正解です。
あらかじめdecimal型にキャストしておけばdecimal型同士の計算となり、結果もdecimal型で返ってきます。

投稿2020/07/22 17:34

hope_mucci

総合スコア4447

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

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

0

やりたいことは、fresultvalに小数付きの値を算出することです

なら、Double型なりの小数点数を表現できる型の変数で計算しましょう。
int型では小数点数は表現できません

投稿2020/07/22 22:13

y_waiwai

総合スコア87747

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

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

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

まだベストアンサーが選ばれていません

会員登録して回答してみよう

アカウントをお持ちの方は

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

ただいまの回答率
85.48%

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

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

質問する

関連した質問