C++：関数内で変数を操作し、メイン内に格納する方法

Question

前提

C++を独学している初学者です（タイトルが誤解を生むものだった場合はすみません）。
現在、「年月日（yyyymmdd）から月部分（mm）を取り出して表示する関数」というものを作成しています。
しかし”取り出した月部分（mm）を変数に格納”の部分が上手くいきません。

実現したいこと

• 取り出した月部分（mm）を変数（mon）に格納出来る

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

test.cpp: In function 'void getMonth(char&, char&)':
test.cpp:14:12: error: cannot convert 'std::string' {aka 'std::__cxx11::basic_string<char>'} to 'char' in assignment
   14 |     cmon = str;
      |            ^~~
      |            |
      |            std::string {aka std::__cxx11::basic_string<char>}

該当のソースコード

/* ヘッダ */
#include <iostream>
#include <string>

/* getMonth関数 */
void getMonth (char &cymd, char &cmon)
{
    std::string str = &cymd;
 
    std::string substr = str.substr(4, 2);
 
    std::cout << substr << std::endl;

    cmon = substr;
}

/* メイン処理 */
int main ()
{
    char ymd[8+1] = "20220727";
    char mon[2+1] = "";

    std::cout << ymd << "\n";
    std::cout << mon << "\n";

    getMonth(*ymd, *mon);

    std::cout << ymd << "\n";
    std::cout << mon << "\n";

    return 0;
}

試したこと

・getMonth関数を個別の機能ごとに確認
⇒月部分の抽出は出来ている

・ポインタや変数などの変更

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

VScode(ver: 1.69.2)

Answer 1

質問のコードで 14行目は、cmon = substr; なのに、
エラーメッセージでは、14 | cmon = str; です。変ですね。

cmon の型は char で、substr の型は std::string です。
代入でそんな型変換はできません、とエラーメッセージは言っています。

cmon = sbustr; の行だけを変更するとすれば、

	const char *p = substr.c_str();
	std::copy(p, p + 3, &cmon);

getMonth関数の中身全部を変更するとすれば、

void getMonth(char &cymd, char &cmon)
{
	char *p = &cymd, *q = &cmon;
	q[0] = p[4];
	q[1] = p[5];
	q[2] = '\0';
	std::cout << q << "\n";
}

getMonth関数の引数の型を変更し、main からの呼出しも変更していいのなら、

#include <iostream>

void getMonth(char *cymd, char *cmon)  // ★
{
    cmon[0] = cymd[4];
    cmon[1] = cymd[5];
    cmon[2] = '\0';
    std::cout << cmon << "\n";
}

int main()
{
    char ymd[8+1] = "20220727";
    char mon[2+1] = "";

    std::cout << ymd << "\n";
    std::cout << mon << "\n";

    getMonth(ymd, mon);  // ★

    std::cout << ymd << "\n";
    std::cout << mon << "\n";

    return 0;
}

Comments

[inle_rah]

ありがとうございます。
段階的にどこが間違っていたのか、どう修正されれば良いのかが素人目にも分かりやすかったので
ベストアンサーとさせて頂きました。

Answer 2

僕ならこう書く:

#include <iostream>
#include <string>

std::string getMonth (const std::string& ymd) {
    return ymd.substr(4, 2);
}

int main () {
    const char* ymd = "20220727";
    std::string mon = "(empty)";

    std::cout << ymd << "\n";
    std::cout << mon << "\n";

    mon = getMonth(ymd);

    std::cout << ymd << "\n";
    std::cout << mon << "\n";

}

Answer 3

エラーメッセージのとおりです。char &cmonに対してstd::string substrは代入が成立しません。

変にcharを混ぜず、全部std::stringで統一するのがやりやすいかと思います。