関数の中に呼び出した関数においてシグマの計算をしたい (代入して加算したい)

Question

前提・実現したいこと

Xcode13.1, C++で2021年の1月1日からある日までの経過日数を計算するプログラムを書きたいです。

発生している問題

int DofMs(int m){
    int DofM(int m);
    int sum = 0;
    for (int i = 0; i <= 11; i++){
        if (i == (m - 1)) break;
        sum += DofM(i);
    
    }
    
        return sum;
    }

とすることによって、DofM(1) + DofM(2) +...+ DofM(m - 1)をreturnすることができると考えていました。

しかし、1月と2月の場合は計算が上手くいくのですが、3月以降の計算が合いません。

3月1日が63日
4月1日が91日
5月1日が122日
6月1日が152日
7月1日が183日
8月1日が213日
9月1日が244日
10月1日が275日
11月1日が305日
12月1日が336日

となってしまいます。

該当のソースコード

#include <iostream>
using namespace std;

int m;
int d;
int i;

/*
 月ごとの日数を計算する関数
 */
int DofM(int m){
    int days = 0;
    if (m == 4 || m == 6 || m == 9 || m == 11)
        days = 30;
    else if (m == 2)
        days = 28;
    else
        days = 31;
    
    return days;
}

/*
 求めたい月までの日数を計算する関数
 (m - 1)月までの月の日数を加算する
 */
int DofMs(int m){
    int DofM(int m);
    int sum = 0;
    for (int i = 0; i <= 11; i++){
        if (i == (m - 1)) break;
        sum += DofM(i);
    
    }
    
        return sum;
    }
    
    
    int main(){
        cout << "1月1日からの日数を計算します。\n";
        cout << "月を入力して下さい。\n";
        cin >> m;
        cout << "日を入力して下さい。\n";
        cin >> d;
        
        int result = DofMs(m) + d;
        cout << "1月1日からの日数は、" << result << "日です。\n" << endl;
        
    }

試したこと

for文の中身を変えてみたりしましたが上手くいきませんでした。

Comments

[y_waiwai]

となっていますはわかるんですが、その数値は本来どういうもんになるもんなんでしょう。

[YU_p]

Σ[i=0, (m - 1)] DofM (i) を計算したものになると想定していました。

Answer 1

解決したようなので別解を示します。

#include <iostream>
using namespace std;

int dom[] = {
	0,
	31,
	31 + 28,
	31 + 28 + 31,
	31 + 28 + 31 + 30,
	31 + 28 + 31 + 30 + 31,
	31 + 28 + 31 + 30 + 31 + 30,
	31 + 28 + 31 + 30 + 31 + 30 + 31,
	31 + 28 + 31 + 30 + 31 + 30 + 31 + 31,
	31 + 28 + 31 + 30 + 31 + 30 + 31 + 31 + 30,
	31 + 28 + 31 + 30 + 31 + 30 + 31 + 31 + 30 + 31,
	31 + 28 + 31 + 30 + 31 + 30 + 31 + 31 + 30 + 31 + 30
};
	
int main()
{
	int m, d;
	cout << "1月1日からの日数を計算します。\n"
	        "月を入力して下さい。\n";
	cin >> m;
	cout << "日を入力して下さい。\n";
	cin >> d;
	cout << "1月1日からの日数は、" << dom[m-1] + d << "日です。\n";
}

Comments

[YU_p]

角括弧を利用すればそのように書けるのですね！教えてくださってありがとうございます。

[kazuma-s]

配列を知らないのですか？

Answer 2

1月は０、２月は１、と表現しているなら、

if (m == 4 || m == 6 || m == 9 || m == 11)

はおかしいでしょう

Comments

[YU_p]

ご指摘ありがとうございます。どのようにmとiを連動させれば良いのかで混乱しておりましたが、何とか分かりました。

Answer 3

int DofM(int m)

の引数 m の値域はどのような想定でしょう？
内側の記述から，
1月の場合，m の値は 1,
2月の場合，m の値は 2
といった感じで使うもの（つまり，m の有効な入力値とは 1,2,3,...12 ）であるように見えます．

対して，この関数を用いる側では

for (int i = 0; i <= 11; i++)

の i の値（ 0-based な値）を引数として用いています．
ここに食い違いがあります．

Comments

[YU_p]

ありがとうございます。
mの値域は1以上12以下の想定で、iの値域は0以上11以下にしています。

i = m - 1でbreakというところでmとiの値域が混乱しているのですが、関数を定義した時の引数と用いる時の引数をまずは合わせられるようにします。