if文を使った面積算出プログラミング

#include<stdio.h>
#include<math.h>
int main(void){

int n;
printf("面積を計算します。\n 1.円\n 2.正方形\n 3.三角形\n 何番の図形にしますか？\n");

scanf("%d",&n);

if(n == 1)
{
    double a;
    float S;
    const double PI=3.141592653589793;   
    printf("半径aを入力してください\n");
    scanf("%lf",&a);
    S=a*a*PI;
    printf("面積は%fです",S);
}
else if(n == 2)
{
    double a;
    float S;
    printf("正方形の一辺aを入力してください\n");
    scanf("%lf",&a);
    S=a*a;
    printf("面積は%fです",S);
}
else if(n == 3)
{
    double a;
    float S;  
    printf("正三角形の一辺aを入力してください\n");
    scanf("%lf",&a);
    S=sqrt(3)/4*a*a;
    printf("面積は%fです",S); 
}
else
{
     printf("入力が不適切でした\n");
}

return 0;

}
面積が０になってしまいます。解決方法を教えてください。

AMK

2020/05/25 00:19

ソースコードは下記の方法で入力しましょう https://teratail.com/help/question-tips#questionTips3-5-1 Cあまり詳しくないけど・・・ C＃の知識で行くと、変数宣言がおかしくない？同じ変数宣言を何度もしている・・・全部面積がゼロになるのか？1つだけなのか？を明確にしてください。

Sade_ni

2020/05/25 00:38

ソースコード入力方法教えていただきありがとうございます。全部面積が0になります。

elsewhere

2020/05/25 01:04

コードをコピペして実行したところ、一応、私の環境では0にはならずに正しい計算結果を出しています。 windows10 (1903 osビルド18362.36) gcc 6.3.0 コマンドプロンプトからexeファイルを実行しました。

行動規範の内容に同意します

回答3件

とりあえず、プリント文を使って入力された変数の値を表示してみましょう
１つ１つ変数の内容が合っているのか？確認してみてください。

投稿2020/05/25 00:45

AMK

総合スコア765

C
1double a;
2float S;

3つある分岐全てにおいて、ここが３回登場しているのが原因ではないでしょうか？
環境が分からないので、コンパイラにもよりますが、このままでは多重定義と判断され実行出来ないケースがあります。

C
1printf("面積は%fです",S);

それともう一つ、%fではなく**%lf**のような気がします。

投稿2020/05/25 00:42

stdio

総合スコア3307

LouiS0616

2020/05/25 00:44

1. スコープが違うように見えます。 2. doubleの出力指定子は%fで合っています。%lfにするのは入力を受け取る場合です。

SHOMI

2020/05/25 01:14 編集

>3つある分岐全てにおいて、ここが３回登場しているのが原因ではないでしょうか？ K&Rより前の大昔のコンパイラならコンパイルエラーになりますが、前の質問でgccと書いているのでブロック先頭で宣言できるため関係ないですね。 >%fではなく%lfのような気がします floatもdoubleも"%f"であっています。scanfのdoubleに対しての書式と混同しprintfにも"%lf"と書く人が多かったためC99で追加で"%lf"が許容されました。

SHOMI

2020/05/25 00:57

すみませんリロードせずに書いて指摘がかぶってしまいました

stdio

2020/05/25 01:00 編集

1は単に同じ変数が関数内にいる事が、なんとなく気になるだけですね。 2は前行の時点で、doubleからfloatに無理に変換しているように見えました。VSの場合警告が発生するので変数Sをdoubleに変更して下さい。別に動作に問題ないと切り捨てるなら、構いませんが...

Zuishin

2020/05/25 04:22 編集

それは %lf とは無関係ですね。S は float なので double の値を明示的なキャスト無しに代入していることへの警告です。double の書式指定子が仮に %lf でなければならなかったとしても表示しようとしているのは float なので的外れです。実際には %f で問題ないので、二重に間違っています。