Ｃ言語でfor文を用いたプログラミングについてです。

Question

Ｃ言語のプログラミングの課題で以下のような表をfor文を用いて作るというものが出ました。

商品名　単価（円）個数（円）　計

　Ａ　　５０　　　２０　　　＊＊＊
　Ｂ　　９０　　　３０　　　＊＊＊
　Ｃ　　３０　　　８０　　　＊＊＊
　　　　　　　合計金額　　　＊＊＊

Ａ，Ｂ，Ｃ及びそれぞれの単価、個数の値はキーボードからの入力で、＊＊＊は演算によりその結果を出力するようにします。つまり、商品名（１文字）と単価、そしてその個数をキーボードから入力すると、単価と個数をかけた値と、それら全ての合計が出力されるようなプログラムを作るということです。
私は以下のようなプログラムを作りました

#include<stdio.h>
int main(void)
{
 int a,b,c,sum,allsum,i;
 printf("商品名　単価　個数　計
");
 for(i=1;i<=3;i++)
 {
  a=getchar();
  scanf("%d %d",&b,&c);
　sum=b*c;
  allsum +=sum;
  putchar(a);
  printf(" %d %d %d 
",b,c,sum);
 }
 printf("合計金額 %d
",allsum);
 return 0;
}

しかしこれでは合計金額がちゃんと表示されなかったりしてうまくいきません。おそらくfor文の中身に問題があると思うんですが…。どうか正しいプログラムと、なぜこのプログラムではうまくいかないのかを教えていただきたいです。なにぶん初心者なもんで、よろしくお願いします。

yama5140 · Accepted Answer

＞でも、このプログラムには fflush という授業で習っていないものがありますね。

★質問文にある getchar();　で 
 を捨てよう。

#include <stdio.h>

int　main( void )
{
　int　iGoods[3], iPrice[3], iNumber[3], iSum[3], iAllSum = 0, i;

　for( i = 0; i < 3; i++ ){　// Ｃの配列は [0] から

　　scanf( "%c %d %d", &iGoods[i], &iPrice[i], &iNumber[i] );

　　getchar();　// 
 (Enter) 読み捨て

　　iSum[i] = iPrice[i] * iNumber[i];

　　iAllSum += iSum[i];
　}
　printf( "商品名--単価--個数----計
" );　// 投稿上の都合で・・

　for( i = 0; i < 3; i++ ){

　　printf( "%-6c%6d%6d%6d
", iGoods[i], iPrice[i], iNumber[i], iSum[i] );
　}
　printf( "合計金額%16d
", iAllSum );

　return( 0 );
}
★万が一「提出」とする場合、見る人が見ると「加齢臭」のキツ～イ、ソースと判りますので若作りに改変して下さい（笑）。

　　int 変数名の頭の i を取るとか。

yama5140 · Answer

＞なぜこのプログラムではうまくいかないのか

　不具合１
　・加算前の allsum は、宣言しただけでは値は不定で、No.1, 2 さんの言われる初期化が必要です。

　不具合２
　・a, b, c を入力即出力では、入力過程が「表」に混在してしまいます。

　不具合３
　・入力ストリームのクリアが必要（質問者様のを実行して気づきました）
-------------------------------------
《改良》

　・入力時に「配列」に格納し、Ａ，Ｂ，Ｃの全てを入力後、出力。
　・入力ストリームのクリア。
　・出力書式の調整。

　　　　http://www.k-cube.co.jp/wakaba/server/format.html

《入力の仕方》

　例）A 50 20（Enter）　← 全て半角。これを３行入力。
--------------------------------------
#include <stdio.h>

int　main( void )
{
　int　a[3], b[3], c[3], sum[3], allsum = 0, i;

　for( i = 0; i < 3; i++ ){　// Ｃの配列は [0] から

　　scanf( "%c %d %d", &a[i], &b[i], &c[i] );

　　fflush( stdin );

　　sum[i] = b[i] * c[i];

　　allsum += sum[i];
　}
　printf( "商品名--単価--個数----計
" );　// 投稿上の都合で・・

　for( i = 0; i < 3; i++ ){

　　printf( "%-6c%6d%6d%6d
", a[i], b[i], c[i], sum[i] );
　}
　printf( "合計金額%16d
", allsum );

　return( 0 );
}
注：インデントに全角空白を用いています。タブに一括変換して下さい。
　　「商品名」は、半角１文字限定。

asuncion · Answer

変数の名前は、意味のあるものを付けましょう。
a, b, cでは、単なる記号です。

noname#75489 · Answer

allsum は初期化した方がいいです。（VC++とか使って値を見ながらトレースするとよいと思います）

auty · Answer

・　変数allsum が初期化されていませんね。

int allsum = 0;

Ｃ言語でfor文を用いたプログラミングについてです。

＞でも、このプログラムには fflush という授業で習っていないものがありますね。

＞なぜこのプログラムではうまくいかないのか

この回答への補足

変数の名前は、意味のあるものを付けましょう。

allsum は初期化した方がいいです。

この回答への補足

・ 変数allsum が初期化されていませんね。

この回答への補足

似たような質問が見つかりました

関連するカテゴリからQ&Aを探す

デイリーランキングこのカテゴリの人気デイリーQ&Aランキング

マンスリーランキングこのカテゴリの人気マンスリーQ&Aランキング

・　変数allsum が初期化されていませんね。