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

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

ただいまの
回答率

90.47%

  • C

    3825questions

    C言語は、1972年にAT&Tベル研究所の、デニス・リッチーが主体となって作成したプログラミング言語です。 B言語の後継言語として開発されたことからC言語と命名。そのため、表記法などはB言語やALGOLに近いとされています。 Cの拡張版であるC++言語とともに、現在世界中でもっとも普及されているプログラミング言語です。

2つの整数値を読み込み、前者が後者の何%であるかを表示するプログラム

解決済

回答 3

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 917

reotantan

score 247

#include<stdio.h>
    int main(void){
double na,nb;

puts("type two integrates");
printf("A:" ); scanf("%lf",&na);
printf("B:" ); scanf("%lf",&nb);

printf("heikin%f\n",(na/nb)*100);


    return 0;
}
という風につくりました。
int main(void){
    int x,y;
  printf("X:" ); scanf("%d",&x);
  printf("Y:" ); scanf("%d",&y);
    printf("X is %f percent of Y",100.0*x/y );



    return 0;
}
というのが模範解答でした。
実用的に考えて皆様はどのようにint,float,doubleを使い分けていますか?
intで定義してキャストでdoubleにしたら使う容量は初めからdoubleにしていたときと同じなのでしょうか?

皆様のご意見をいただきたいです。
  • 気になる質問をクリップする

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

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

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 3

+1

仕様によるとしか言えないと思います。
今回の仕様は課題ですよね。
・整数を入力
・小数点一桁まで表示する
等はかかれていなかったでしょうか。

ここ
に書かれているようにあとは精度の問題かと思います。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2015/08/23 13:26

    そうですか、ご回答ありがとうございます

    キャンセル

  • 2015/08/23 14:30

    「2つの整数値を読み込み」とあるのに、質問者さんの回答だと浮動小数点数を読み込んでいるから、間違いとは言えなくても、100点満点とは言えないかなと思います。

    キャンセル

  • 2015/08/23 18:56

    そうですね、丁寧に答えていただいてありがとうございました

    キャンセル

checkベストアンサー

0

printf("X:" ); scanf("%d",&x); 

 printf("A:" ); scanf("%lf",&na); 
は、設定される値に大きな差があります。
上では 0, 1, 2, .... の整数だけになりますが, 下では 0, 0,1, 00001, 2.1 など整数以外の小数値も設定できてしまいます。
課題の趣旨からいえば、入力値としては整数だけに制限すべきです。
(float で入力を受け付けてから、小数点以下を切り捨て / 切り上げなどして、整数に丸めるという方法もありますが)

この課題でのポイントの1つは、 演算結果を 整数で得るか? 小数で得るか?  だと思います。
模範回答例では、 分子を 100.0 倍してから、割り算をすることで float で計算結果を得ています。
(分子が float なので 整数で割り算しても、計算は float になります。)
これを 100.0 倍でなく、100 倍とすると、計算はすべて整数として行われます。(分子も分母も整数なので)
しかし、この方法では、0.3  % のように 1 % より小さい値の部分は切り捨てられてしまいます。
1 % より小さい値も扱うなら、  double / double のようにする必要があります。(分子、分母のどちらか一方が float なら、計算結果は float になります)

  100 * (3.0 / 1000.0)  -> 0.3    // double で割り算すれば、1 % 以下も求まる。 
  100 * (3 / 1000) -> 0            // 整数 同士で割り算すると、1 % 以下は切り捨てられる

つまり、求める値の有効精度に応じて、整数、float, double を使い分けることが必要です。

すこし違う数値例でも考えてみます。
  100 * (345 / 1000) ->  34
  100 * (345. 0 / 1000.0) -> 34.5 
となります。
 すべて整数だけで計算した場合と、 int から double (または float) にキャストしてから計算した場合では、値が異なってきます。
double (または float) で計算して、その結果の表示をする時に 小数以下を無視して表示するとしても、
小数以下を切り上げ、切り捨て、四捨五入するかによって, 34 % となったり 35 % となったりと差がでます。

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2015/08/23 13:26

    模範解答でx=3、y=10001としても0.02997と出るのですが、
    どこの部分が1%より小さい値が出ないんでしょうか
    100.0をかけただけでは不十分でしょうか

    キャンセル

  • 2015/08/23 14:16

    私が模範回答を読み間違えていました。
    模範回答では 100.0 * x / y としています。( 100 * x / y ではありませんでした)
    100.0 * x の結果は float になるので、 100 * x / y は小数を含む結果になります。
    (100 * x / y なら 結果は整数です)

    キャンセル

  • 2015/08/23 18:55

    丁寧な説明ありがとうございました。とても理解が進みました

    キャンセル

-1

実行速度的な観点からすると
(na/nb)*100
の場合は、100がdoubleにキャストされる無駄が生じていますね。
(na/nb)*100.0
が、より理想的です。(コンパイラによっては差がない場合はあるでしょうが・・・)
あと、割り算は一般的に実行速度が遅いです。逆数をとってかけると少し速くなります。
単に / y でなく、* (1.0 / y) とします。

もう一つ、分母がゼロになる場合が想定されていないのが気になります。
潜在的なバグの要因になりますので、割り算を利用するプログラム場合は、必ず分母がゼロになることを気を付ける必要があります。場合によってはコンパイラのオプションにも気を使う必要があります。

if (y == 0) エラー処理
100.0 * x * (1.0 / y)
が自分の回答ですね。
x,yのキャストはあらかじめ行うよりその場で値をとって捨ててもらう方が速いと思います。

後は、テスト駆動的にあらかじめ想定される値を列挙して、テストするプログラムを並行して作成し、ベンチマークもとるようにすれば、完璧ですね。ここまで求められる課題ではないと思いますがw

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2015/08/23 18:57

    分かりやすい解答ありがとうございました。
    参考にさせていただきます

    キャンセル

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

  • ただいまの回答率 90.47%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る

  • C

    3825questions

    C言語は、1972年にAT&Tベル研究所の、デニス・リッチーが主体となって作成したプログラミング言語です。 B言語の後継言語として開発されたことからC言語と命名。そのため、表記法などはB言語やALGOLに近いとされています。 Cの拡張版であるC++言語とともに、現在世界中でもっとも普及されているプログラミング言語です。