タクシメーター風の計算プログラムで料金が正しく表示されない

Question

実現したいこと

タクシーメーター風の計算をするプログラムを練習で作っています。
・タクシーの初乗り料金が680円、1.3キロまでは初乗り料金だけ支払えばよい。
・1.3キロを超えると350メートルごとに200円加算される。
これをC言語で実装したいです。

発生している問題・エラーメッセージ

880円までの料金は計算されます。
それを超える料金だと画面にも何も表示されません。
22行目がうまくいっていない気がします。

該当のソースコード

#include<stdio.h>

int main(void) {
	double kilo = 0.0;//距離km
	double meter = 0.0;//距離m
	int fare = 680;//運賃を初乗りで初期化

	//マイナスの入力距離を防ぐ
	do {
		printf("距離をキロメートルで入力：");
		scanf_s("%lf", &kilo);
	} while (0.0 >= kilo);

	//キロメートルをメートルに変換
	meter = kilo * 1000;

	if (meter >= 1300) {
		if (meter >= 1300 && meter <= 1650) {
			//初乗りからワンメーターUPのみ
			printf("運賃は%dです",fare + 200);
		}if(meter >= 1651) {//運賃計算
			fare = 680 + meter / 350 * 200;/*実行されない？*/
		}
	}
	else {//初乗り料金以内
		printf("運賃は%dです", fare);
	}

	return 0;
}

実行結果

距離をキロメートルで入力：1.0
運賃は680です

距離をキロメートルで入力：1.5
運賃は880です

距離をキロメートルで入力：2.0
（何も表示されません）

Comments

[tatsu99]

22行　fare = 680 + meter / 350 * 200;/*実行されない？*/
で料金を算出していますが、それを印字していません。
２２行の次に
printf("運賃は%dです", fare);
を追加すれば良いかと。

[hoshi-takanori]

計算はしてるけど、その結果を表示してないような…。
あと、1.6505 とか入れたら…。
それから、「1.3キロを超えると」なので 1.3 ちょうどの時は 680 円では？

[jimbe]

比較に = を含めないといけないような制限があるんですかね。
>マイナスの入力距離を防ぐ
なら 0.0 は入力できないといけないのでは。

必ず "運賃はxxです" を表示するのでしたら return 0 の直前に printf("運賃は%dです", fare) を入れておいて、そこに向けて条件に従って fare を更新する形にしたほうが良いのではないでしょうか。

仕様にはワンメータUP の時に何か特別なことをすることは無いのに、なぜ if でワンメータUPだけ特別に計算しているのでしょう。
>1.3キロを超えると350メートルごとに200円加算
の仕様に
>fare = 680 + meter / 350 * 200
の式であってますか？

[tmp]

算数の問題でなくとも「超える」や「未満」は、それに対応するさせる必要があると思います。
hoshi-takanoriさんがおっしゃている通り、
距離は、整数ではないので、1650m超え～1651m未満がif分がどこにもありません。
また、1300m丁度も超えてはいないので、680円です。
fare = 680 + meter / 350 * 200;の計算もmeterは、整数ではないので、２００円単位であがらない。
また、初乗り距離分も200円加算の計算に入ってます。対象は初乗り距離を超えた距離です。

あと、質問者の該当のソースコードを見ていると（料金計算やif分の+1しての比較などから）
距離を整数で演算したいように見えますが、特に問題文にはそういったことは、書いてありませんが、
そういう条件であれば、記載してください。

Answer 1

プログラムは上から下に流れますが、必ずしも作り方・考え方も上から下でなければならないわけではありません。
例えば、初乗りだけ表示する所から始めても良いのです。

#include <stdio.h>

int main(void) {
  int fare = 680;//運賃計算
  printf("運賃は%d円です", fare);
  return 0;
}

これを、区間によって200円ずつ加算されるような式に変えてみます。

#include <stdio.h>

int main(void) {
  int section = 0;//区間番号(0～1300→0, 1301～1650→1, 1651～2000→2,…)

  int fare = 680 + section * 200;//運賃計算
  printf("運賃は%d円です", fare);
  return 0;
}

変数 section を追加して、初乗り区間なら 0, それ以降は 350m 毎に 1, 2,…と変化するものと仮定した式にしました。
この section が仮定通りになる文を考えれば、仕様に沿った金額が表示されるはずです。

#include <stdio.h>

int main(void) {
  int meter = 2000;//距離[m]

  int section = 0;//区間番号(0～1300→0, 1301～1650→1, 1651～2000→2,…)
  if(meter > 1300) {
    section = (meter - 1301) / 350 + 1;
  }

  int fare = 680 + section * 200;//運賃計算
  printf("運賃は%d円です", fare);
  return 0;
}

2000m の料金は 1080 円であっているでしょうか。
後は、入力部分を追加するだけです。

#include <stdio.h>

int main(void) {
  double kilo = 0.0;//距離[km]
  do {
    printf("距離をキロメートルで入力：");
    scanf_s("%lf", &kilo);
  } while(0.0 > kilo);//マイナスの入力距離を防ぐ

  //キロメートルをメートルに変換
  int meter = (int)(kilo * 1000);//距離[m]

  int section = 0;//区間番号(0～1300→0, 1301～1650→1, 1651～2000→2,…)
  if(meter > 1300) {
    section = (meter - 1301) / 350 + 1;
  }

  int fare = 680 + section * 200;//運賃計算
  printf("運賃は%d円です", fare);
  return 0;
}

Comments

[ceg34ry]

考え方を順番に提示しての回答ありがとうございました。

Answer 2

meter = kilo * 1000;

で、kiloが２．０のときは、meterはいくらになるのかを考えよう。
で、そのときにそれ以降のif文で、どういう動作になるか追いかけてみよう。

（何も表示されません）

って言ってるけど、なにも表示してないんだからそれが正しい動作ですよ

Answer 3

「距離に対して料金が階段状に設定されており，境目までならそれ未満と同じ料金で超えた時に料金が上がる」という条件での記述例を示します。なお，この「切り上げ」を行う部分は関数 ceil() を使いました。

#include <stdio.h>
#include <math.h>

int main(void) {
    double kilo = 0.0;
    double meter, excess;
    int fare = 680;

    do {
        printf("距離をキロメートルで入力: ");
        scanf_s("%lf", &kilo);
        // scanf("%lf", &kilo);
    } while (0.0 >= kilo);

    meter = kilo * 1000;

    excess = meter > 1300 ? meter - 1300 : 0;
    fare += ceil(excess / 350) * 200 + 0.5;

    printf("運賃は %d円です\n", fare);

    return 0;
}

Comments

[ceg34ry]

math.hのceil()を使った方法を教えてくださりありがとうございました。