Q&A
コードやエラーはマークダウンのcode機能を利用してご提示ください。
https://teratail.com/questions/238564
特に
行頭の# やタブインデントが意味を持つ言語ではそれが伝わらないのは致命的です。
下のようなプログラミングをしています。
最安値(saiyasu)と所持金(syoji)を入力した場合に最安値より所持金が少なかった場合のifのとこまではうまくいったのですが、最安値より所持金のほうが高かった場合のelseのほうの計算や出力に行きません。
当人、学校でC言語を勉強したものの筆記でしか教えてもらわず実際に打ち込んでプログラムを実行したことが今までになくわからないことだらけなので、この場合のここがダメなど細かく教えていただけると大変うれしく思います。
解決策を教えていただきたいです。
お願いします。
#include <stdio.h>
int main()
{
int syoji, kosu, zan,tarinai, saiyasu;
printf("最安値を入力してください\n");
scanf_s("%d", &saiyasu);
printf("所持金を入力\n");
scanf_s("%d", &syoji);
if (syoji < saiyasu) { tarinai = saiyasu - syoji; printf("あなたの所持金は%d円で、%d円足りないので買えません", syoji,tarinai); } else{ for (kosu = 1; syoji <= saiyasu; kosu++) { printf("おでん%d\n個めを買いました", kosu); zan = syoji - saiyasu; printf("残金は%d\n円です", zan); printf("あなたはおでんを%d\n個買い残金は%d\n円になりました。", kosu, zan); } }
}
回答3件
0
ベストアンサー
elseの処理には行っています。
しかし、for文の条件式(継続条件)が間違っており、for文の繰り返し処理が行われるのは、saiyasuとsyojiに同じ金額を入力した場合だけです。しかも、その場合にはsyojiが減らないために条件式がいつでも成立してしまい、繰り返しが終わりません。
せっかく残金の計算をしているのですから、これを活かすようにしましょう。
C
1else { 2 zan = syoji; 3 for (kosu = 1; zan >= saiyasu; kosu++) {
とすれば、残金が最安値以上なら繰り返すようになります。残金の初期値を設定してから繰り返し処理を行なう点に注意してください。
投稿2020/07/03 21:47
総合スコア11990
0
他のサイトではちゃかすの回答 担当でした。まだまじめにもどりきれていないので
おふざけがあったらごめんなさい
すこし古い話にはなってしまいますが
int main(int argc. char argv)というふうにmain関数の戻り値がintとか引数がint,charなどを省略しないほうが初心者っぽいので回答がおおくあつまるかなとおもいました。
いろいろ省略されていますが
char buffer[1024];（←省略されてる）
int main(省略されてる)（←省略されてる int argc,char **argv）
sscanf_s(buffer,"%s",&value);
などのように１つ１つ省略せずに まずは お手本をよく見て 省略しない書き方を使うほうが 僕たちにはわかりやすいので 人にコードを見せるときで質問をする側のときは、そうかいてもらえると助かります。
投稿2020/07/03 23:24
編集2020/07/03 23:32
総合スコア190
> int main(int argc. char argv)というふうにmain関数の戻り値がintとか引数がint,charなどを省略しないほうが初心者っぽいので回答がおおくあつまるかなとおもいました。
異議あり。mainの引数を省略しようがしまいがこの問題とは何の関係もないから。
# 実際回答ついてるし。
意義というのがちょっとニュアンスが取れませんでした。プログラムの一般論で言えば様々なアルゴリズムがあるように、答えが１つではない というのはわかるので 異なる議論つまりオルタナティブアンサーという意味でしょうか？だとすると あなたもこういう回答のほうがいいと持論をご展開いただくべきで 私の考え方とは違うだけだと 異議というよりはコメントかなと思いましたがコメント機能なのであってるし 用語の使い方がわかってなくてすみません
まず、この問題だとコマンドライン引数使ってないから、提案するにしても「int main(void)」でいい。ついでに、提案している引数の型の「**」がマークダウンに喰われて質問者には意味不明。さらに、scanf_sなのを勝手にsscanf_sにして「省略されてる」という無茶振り。全体として、問題の解決に結びつかない回答なのでは。
異議 : 「一つの意見に対して、反対または不服であるという意見」
> あなたもこういう回答のほうがいいと持論をご展開いただくべきで
「mainの引数なんか書く必要ないよ。この問題には関係ないから。」が僕の意見。そう述べています。
いや 反論どうも ＊＊がマークダウンだからと ご理解いただけているので まぁ わかるるなら 大丈夫 とおもいました。参考になりました。
回答にて、なぜに「argv)というふうにmain関数の戻り値がintとか引数がint,char」という部分を強調しているのでしょうか？
質問の解決に全く資さない回答ですのでマイナスしました。
**〜** のせい。
「質問者」にわからなければ大丈夫じゃあないでしょうに。
低評価の理由：質問と全く関係ない持論を展開しているだけである．
`int main()`は`int main(int argc, char **argv)`の省略形ではないのに、「省略されてる」という表現を使っているため、低評価しました。
buffer なんてどこにも使われていませんね。何をお手本にしているのかわかりませんが、お手本を写すのではなくプログラミングできるようになってください。
低評価が積もると信用がなくなるので、最低限のプログラミングができるようになってから回答することをお勧めします。
＞最低限のプログラミングができるようになってから
あとプラスで、サイトの機能(特にマークダウン)は使えるようになってから。
質問も回答も書いてる通りにしか伝わらないので、単に太字が書かれてるだけです。
質問者およびその他、同様の問題を抱えた人からは「何のための強調？」という疑問しか、ありません。
エスパーを強要しています。複数名からの再三の真っ当な指摘を応じられないので通報しました。
あなたの回答
tips
プレビュー
