プログラムの修正お願いします。

質問

レンタルビデオを借りる。
１本１５０円。
５本借りると１００円引き。
５本を越えると５本目以降は１本１００円。
どこが間違ってるでしょうか？？
教えてください。

#include<stdio.h>
void main(void)
{
int i,money;

printf("借りる本数を入力 ---> ");
scanf("%d",&i);

if(i < 5){
money = i * 150;
}
else if(i == 5){
money = money - 100;
}
else{
money = money + (i-5)*100;
}
printf("合計金額は%d円です。\n",money);
}

回答No.15

Largo_sp

>> こんなのはどうかな？
>→意味がわかりません。
俺にいってるのかな？
こんな考え方（書き方）もあるかなぁという事ですが……
もちっと書くと
main()
{
int i,j,money;

printf("借りる本数を入力 ---> ");
scanf("%d",&i);

j=money=0;
while(j<i){
money +=150
if ( j==4 )　　　// ５本の時
money -=100;　　//５本目から\100ならここは150
if (j>4)
money -=50;
j++;
}
printf("合計金額は%d円です。\n",money);
}
としたほうがきれいかな…
if でなくて case のほうがきれいかなぁ

回答No.14

78tch

> こんなのはどうかな？
→意味がわかりません。

回答No.13

Largo_sp

ちょっと茶々いれ
こんなのはどうかな？
for(j=0;j>i;j++)
{
money = money + 150
if (j>=5)
money = money - 50;
if (j=5)
money = money -50;
}

仕様書の書き方とは違いますが、結果は同じになりますよね……
５本目以降一本当たり50円引き
但し５本目は100円引き（もう50円引き）
と読み替えたかきかたです

回答No.12

Mizyu

あ、すいません、、、問題読み違えてました…。
仕事片手間に回答するもんじゃないですね…。

> 専門家やら経験者やらがこれほど集まって間違いだらけの回答…(^^;
これは私のこととして肝に銘じます…。

しかし
> ５本借りると１００円引き。
> ５本を越えると５本目以降は１本１００円。
この条件は同時に満たされるのでしょうか？
また10本借りた時は2回目の「100円引き」はあるのでしょうか？
その仕様が疑問ではありますが
同時に満たされる、かつ5本毎に100円引きというのであれば

if (i<5){
　money = i * 150;
}else{
　money = ( 5 * 150 ) + ((i-5) * 100) - (100*((i-(i%5))/5));
}

もしくは
int j;
for(j=0;j<i;j++){
　if (j<4){
　　money = money + 150;
　}else{
　　money = money + 100;
　}
　if (i%5==0){
　　money = money -100;
　}
}

(もう、PG引退したほうがいいかもなぁ…。)

回答No.11

Mizyu

#2です…。#5の訂正も間違ってたようで…。
しかし、専門家やら経験者やらがこれほど集まって間違いだらけの回答…(^^;
ちょっと苦笑いしてしまいました。
（そういう私もコレで飯食ってるはずなのに…ｗ）

正解（今度こそ）は
money = (i*150) + (i-4)*100;
でしょうか。

でも私はこの課題があったら個人的には
------------
int j;
money = 0;
for (j=0;j<i;j++){

money = money + 150;
if (j>4){
money = money -100;
}

}
------------
こう組んでしまうでしょう。
実際の手順と処理順が等しいのでこうすれば保守する人もわかりやすいかと。
演算回数が増えるので、時と場合によりけりですが。

回答No.10

yotta

＃６　です,間違いました

else if(i == 5){
money = money - 100;
}
else{
money = money + (i-5)*100;
}

の部分を下記に置き換える

else {
money = 4 * 150 - 100 + ( i - 4 ) * 100;
}

に訂正して下さい

回答No.9

78tch

ああっ、#8さんまで・・・
#4さんの後半のが正解。

回答No.8

azicyan

iに具体的な数字を入れて考えるといいでしょう。

まずi=5本だったとします。
すると・・・
ｉ＝＝５の文しか実行されません。
つまりMONEYは－１００円です。
あくまでも５本でかける１５０円払った上で１００円引きなんですから、
i＊１５０-100
ですかな・・？？
となります。
５本より大きい場合も
４本は１５０円それ以降を１００円と考えて
4*150+(i-4)*100
でいいんじゃないでしょうか？

回答No.7

78tch

○　money = (4*150) + (i-4)*100;
あぁ・・・発想はすごいっ！と思ったらｉ＝５の時違わないですか？

回答No.6

yotta

else if(i == 5){
money = money - 100;
}
else{
money = money + (i-5)*100;
}

の部分を下記に置き換える

else {
money = 5 * 150 - 100;
if(i > 5){
money = money + (i-5)*100;
}
}

理由は,５本以上の時,１００円引きされない