Expected ';' after top level declaratorというエラーを解決したい

Question

前提・実現したいこと

2021年のある月日を入力して、1月1日から何日後かを計算するプログラムをC++で作りたいです。Xcode13.1を使用しています。

色々なテキストを読んで試行錯誤してみたのですが、どうしても上手くいかないので教えて頂きたいです。
宜しくお願い致します。

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

関数DofMsを定義しようとすると、Expected ';' after top level declaratorというエラーが出て来てしまいます。
(DofM関数ではこのようなエラーは出ませんが、DofMs関数を定義しようとするときのみこのエラーが出てきます。)

該当のソースコード

#include <iostream>

using namespace std;

int m;
int d;

int DofM(int m){
       int days = 0;
            if (m == 4 || m == 6 || m == 9 || m == 11)
                days = 30;
            else if (m == 2)
                days = 29;
            else
                days = 31;

         return days;
    }

int DofMs((m - 1)){ //ここにエラーが出ます。
        int DofM(m);
        for (int (m - 1) = 0; (m - 1) <= 11; (m - 1)++){
                int sum = 0;
                sum += DofM(m - 1);
               
                return sum;
            }


int main (){
    cout << "1月1日からの日数を計算します。\n";
    cout << "月を入力して下さい。\n";
    cin >> m;
    cout << "日を入力して下さい。\n";
    cin >> d;
   
    int result = 0;
    result = DofMs;
    cout << "1月1日からの日数は、" << result + d << "です。\n" << endl;
    
    return 0;
        
    }
```### 試したこと

宣言と定義に分けてみましたが、同じエラーが出てきてさらにRedefinitionというエラーも出てきてしまいました。

### 補足情報（FW/ツールのバージョンなど）

Xcodeのバージョンは13.1 です。

Comments

[takasima20]

C++ って引数の定義に式を書けるんスか?

[otn]

> int DofMs((m - 1)){
これは具体的にどういう意図なのでしょう？関数定義のつもりですか？あるいはそれ以外？

[YU_p]

int DofMs((m - 1)){
は関数定義のつもりで書きました。1月1日からm月n日までの日数を求めるために、(m-1)月分の日数を求める関数を作りたいです。

[otn]

関数名の後の括弧の中には引数宣言を書きます。

[YU_p]

>関数名の後の括弧の中には引数宣言を書きます。
関数名の後の括弧の中を、int xのような形にすれば良いということでしょうか。

[otn]

まあそういうことですが、そのレベルも人に聞かないと進めないようであれば、もう一度、言語の復習してから取り組んだ方が良いですね。

[YU_p]

ありがとうございます。引数や変数や関数についての部分を本で読み直して取り組みたいと思います。

Answer 1

引数はさておき

int DofMs((m - 1)){

波括弧が足りないようにみえます。

ついでにいうと、for ループの中で return するのは(たぶん)意図とはちがいますよね。

あと、main から呼ぶ時に(括弧も含めて)引数を指定してないスね。

なんか (m - 1) をあちこちにコピペしている印象ですが、いちど考え方を整理してみては?
--- 追記 ---
とりあえず現状のコードに手を入れてエラーが出ないようにしましたが
ロジック的な部分はがんばって考えてください。

#include <iostream>
using namespace std;

int m;
int d;

int DofM(int m) {
    int days;
    if (m == 4 || m == 6 || m == 9 || m == 11)
        days = 30;
    else if (m == 2)
        days = 29;
    else
        days = 31;

     return days;
}

int DofMs(int m) {
    int sum = 0;
    for (int i=1; i<=m; i++) {
        sum += DofM(m);
    }

    return sum;
}

int main () {
    cout << "1月1日からの日数を計算します。\n";
    cout << "月を入力して下さい。\n";
    cin >> m;
    cout << "日を入力して下さい。\n";
    cin >> d;

    int result = DofMs(m);
    cout << "1月1日からの日数は、" << result + d << "です。\n" << endl;

    return 0;
}

Comments

[YU_p]

すみません。引数の形式については調べている最中です。
波括弧が足りないとは括弧の後に波括弧をさらに加えるということでしょうか。
forループ内でreturnしないとなるとvoid DofMsとなりますか？
mainから呼ぶ時に引数を指定とはどのように行えば良いのでしょうか。

宜しくお願いします。

[YU_p]

ありがとうございます。ロジック部分を整理し直してみます。