ｃ言語のプログラムについて

質問

今年からＣ言語を始めた大学１年生です。レポートで四則計算を行うプログラムを6/19までに作らないといけないです。使えるのは今までに習った選択型、反復型、関数scanfです。このプログラムの留意点は次の４つです。

１．使う演算を選択してもらう
　　例えば
　　どんな演算をしますか？　1:+,2:-,3:*,4:/　(選択)
２．演算を行う数値を入力してもらう
　　例えば
　　a + b の計算をします。aを入力してください。
３．演算結果を表示する
　　○＋○の答えは　○です
４．計算終了か、それともほかの演算を行うか　決める
　　０：計算終了　１：他の計算を行う　を選択してください　

以下のプログラムはこのことを留意してプログラムをつくりました。どこがおかしいのでしょうか。
#include<stdio.h>

main()

{
short a,b,m,n;
long c,d,e,f;

c = a + b;
d = a - b;
e = a * b;
f = a / b;
m = 1;

printf("整数を入力してください:");
scanf("%d",&a);

printf("どんな演算をしますか？:");
printf("1: +,2: -,3: ×,4: ÷ :");
scanf("%d",&n);

while(m=1){
if(n<=4){
while(m=1){
switch (n<=4){
case 1:
printf("整数を入力してください:");
scanf("%d",&b);
printf("%d + %d = %d \n",a,b,c);
break;

case 2:
printf("整数を入力してください:");
scanf("%d",&b);
printf("%d - %d = %d \n",a,b,d);
break;

case 3:
printf("整数を入力してください:");
scanf("%d",&b);
printf("%d × %d = %d \n",a,b,e);
break;

case 4:
printf("整数を入力してください:");
scanf("%d",&b);
printf("%d ÷ %d = %d \n",a,b,f);
break;
}
}
}
else {
printf("計算できません。");
printf("演算をやり直してください。\n:");
}
}

printf("計算を終了しますか、それとも他の計算を行いますか？:");
printf("0:計算終了,1:他の計算を行う:");
scanf("%d",&m);
printf("計算を終了します。\n");
}

提出期限があるのでなるべく早めのご回答待ってます。生意気言ってすいません。

回答No.2

mitoneko · ベストアンサー

ここまで書けるのでしたら、まず、自分の頭で、各変数の値がどうなっているか、どのようにプログラムが流れていくかのシミュレートをしましょう。

c = a + b;
d = a - b;
e = a * b;
f = a / b;

冒頭近くの、この演算。何をしていますか？ここまで終わった時、登場している各変数の値は何ですか？

　ループの始まり。
while(m=1){

　これ、Ｃ言語を始めたばかりの人が良くやるミスが含まれます。（往々にして、ここに打つ時に間違えることもあります。もしそうだったらごめんなさい。）
　ここに書いてあるとおりだとすると、このループは無限ループとなります。何故か解りますか？
　Ｃ言語の構文としては合っています。でも、おそらくやりたいことは違います。という初心者にとっていやらしいエラーです。

switch (n<=4){

　これも、構文としては合っています。でも、きっとやりたいことは違います。
　switch文の括弧の中の式の結果は、case文の後ろの数字と比較され、同じ数値の所に分岐します。では、（）内の式の結果が何かわかりますか？

　Ｃ言語においては、代入・比較を含む全ての式に、結果があります。
　というわけで、式を受け入れるとこには、大概の場合においてどんな式を書くことも出来ます。比較のつもりで代入しても、構文的には何ら問題はありません。これは、時としてテクニックとなりますが・・・普通は間違いを誘発します（苦笑）

case 1:
printf("整数を入力してください:");
scanf("%d",&b);
printf("%d + %d = %d \n",a,b,c);
break;

　case文の中全てに当てはまります。printfが実行される時、a,b,cの各変数に入っている数字を考えてください。特にcには何が入っていますか？
　なにせ、bを入力した直後のprintf文です。まさか、cに計算結果は入っていませんよね。

　while(m=1) {
で表記されるループが二重になっていますが、意図は何ですか？

　ここまで、ちゃんと直したとします。

printf("計算を終了しますか、それとも他の計算を行いますか？:");
printf("0:計算終了,1:他の計算を行う:");
scanf("%d",&m);

　末尾近くのこの文は、いつ実行されますか？プログラムの動きを頭で追いながら、よく考えましょう。これが実行されないと、計算が終了できません。

　これもちゃんと直したとします。
　ここからは、プログラムの仕様には書いていませんが、普通常識としてそうだろうなという指摘です。
　「他の計算を行う」を指定した時、次に実行されるscanfはどの行の物ですか？他の計算というのは、第一引数と演算子が同じで第二引数だけ違う計算をすることなのかな？

　ここまで直せば、プログラムは、仕様通りの動きをするでしょう。

　最後です。
　各case文は、全てが

printf("整数を入力してください:");
scanf("%d",&b);

　の二行で始まります。
　このように、カット＆ペーストでプログラムを入力したくなったら、流れを疑いましょう。共通化できませんか？場所を変えるだけで、一つで済みませんか？

　本来は、この質問は、カット＆ペーストをして、中身を少しだけチョコチョコいじるという時に発生します。関数を学べば、理解できますが・・・ここまではまだ進んでいないかな？

　無駄な行という意味では、もう一つ。何故、四則演算の各々の計算式用に結果変数を用意したのでしょう。このロジックであれば、同じ変数でも良いはずです。（足し算と引き算の結果を同時に使うことはありませんよね。）

　まぁ、答えすれすれという感じではありますが、一応、考えいただく余地を残しておきました。
　後は、がんばって期限に間に合わせてくださいね。

回答No.5

kmee

> short a,b,m,n;
> scanf("%d",&n);
> scanf("%d",&a);
> scanf("%d",&b);
> scanf("%d",&m);

これは、sizeof(short) == sizeof(int)の環境でないと期待通りに動作しません。

scanfの書式文字列にある%～と、それに対応する引数の型は一致させる必要があります。
** C言語の仕様上、コンパイルエラーにはなりません ** 。ですので、プログラマ側で正しく記述する必要があります。
# 一部コンパイラでは、このプログラムのように直接記述してある場合に限って、書式と引数が一致しない、という「警告」を出すものはあります。
# ただ、詳しい警告を出す設定が必要です。また、警告はエラーでは無いので、コンパイル自体は正常終了します(警告をエラーとして扱う設定をしていない場合)

scanfのマニュアルで、書式のところをよく読んで
・ short a,b,m,n ;をそのままにして、"%d"ではなく正しい書式に
・ "%d"をそのままにして、 short a,b,m,n ;ではなく、正しい型に
変更してください。
# このプログラムなら、shortを使う意図がまったくわかりません。"%d"をそのまま、型を変えるのが妥当と思います。

> while(m=1){
> switch (n<=4){
これらと同様、「言語仕様としては正しく、コンパイルもできるのに、実際の動作が期待通りでない」という、やっかいな間違いです。

他のツッコミとして

> 選択型、反復型

ってなに?
# この並びなら、「選択構文」「反復構文」の意味だろうけど。

> main()

main関数の定義にこんな書き方をしている参考書は捨てましょう。
大学の教科書なら「昔から使い回していて、新しいのにするつもりが無いんだな」と思いましょう。

> long c,d,e,f;

これもlongを使う意図がわかりません。
4つ用意することも、意図はわかりますが、このプログラムでは意味はありません。
このプログラム限定で言えば、そもそも、変数を用意する必要もありません。

> printf("整数を入力してください:");
> scanf("%d",&b);

間違いではないですが、同じ処理を何度も書くのは「間違い」のもとです。
なにより、「面倒臭い」です。(コピペで簡単にできる?いや、そのコピペ自体が面倒臭い)
・nを入力
・nが正しい(1≦n≦4)か判定、正しい時のみ続きを処理
・a,bを入力
・nによって指定された計算と出力
とすれば、同じ処理が一つにまとまります。
# 「面倒臭いからなんとか楽できないか?そのための努力ならなんでもする」
# と考えるのが、プログラミング上達のコツ、だと私は考えます。

回答No.4

black2005

何で素直に1～4の順に処理しないの？
コーディングでは3から処理しようとしてるでしょ。
それが間違いの原因。

回答No.3

επιστημη（@episteme）

> 以下のプログラムはこのことを留意してプログラムをつくりました。どこがおかしいのでしょうか。

まるでデタラメすね。

> #include<stdio.h>
> main() {
> short a,b,m,n;
> ｌong c,d,e,f;
>
> c = a + b;
> d = a - b;
> e = a * b;
> f = a / b;

a,b に入力もしてない段階でc,d,e,fを求めても意味ないでしょ。

回答No.1

unacyo

計算結果はどうなったんでしょう？おそらく、意味不明な値になってますよね？

とりあえず、あれ？っと思った部分。

まず１つ目、switch()へ渡す値がおかしいです。
switch()は値をcaseの値と同じかどうか判断するので、switch()内で評価させてはいけません。
きっと、＋の計算しかできていないのでは？

２つ目。なんでａ，ｂの値入力前にｃ，ｄ，ｅ，ｆを計算しているんでしょう？
ａとｂに値が入っていない（というより、不定の値が入ってる）状態でｃ，ｄ，ｅ，ｆを計算しちゃってるので、入力した値と違う結果になりますよね？
scanf()でbを入力したあとで、ｃ，ｄ，ｅ，ｆの計算をしないとダメです。

３つ目。計算が終了できませんよね、これ？
計算を終了するかどうか問い合わせるならば、while( m == 1 ){ /* ここでやるべきですよ？ */ ｝

４つ目。while( m == 1 )がなんで２つあるんでしょうか？
この内容であれば、１つあれば十分のはずですが。