実行すると、動作が止まる

Question

月を入力し、それに対応する昔の月の名前を表示するプログラムで、
これを実行すると「動作が停止しました」とプログラムが途中で止まってしまいます。

エラー内容は
「'GetOldMonthName' : 値を返さないコントロール パスがあります。」
と出ます。

main関数内の
cout << name << endl;の部分をコメント化すると動きました。（成功ではありませんが）
この行がおかしいのでしょうか？

言語はVC++です。

#include <iostream>

using namespace std;


const char* const MONTH_NAME[] = {
	"睦月", "如月", "弥生", "卯月", "皐月", "水無月", "文月", "葉月", "長月", "神無月", "霜月", "師走"
};


const char* GetOldMonthName(int month){
	if (1 <= month && month >= 12){
		return MONTH_NAME[month - 1];
	}
}


int main(){
	int month;

	cout << "今は何月ですか？ > " << flush;
	cin >> month;

	const char* name = GetOldMonthName(month);
	cout << name << endl;
}

Answer 1

'GetOldMonthName' : 値を返さないコントロール パスがあります。

これは、GetOldMonthName関数の中で値を返さないパスがあるということを示しているので、GetOldMonthName関数に注目してください。
if文の中でreturn MONTH_NAME[month - 1];としていますが、条件が合致しない場合はそれが実行されず、何も返さないまま関数が終了していますね。何か返す必要があります。
とりあえず関数の最後でreturn "monthが範囲外です。";のようにしておけば良いかと思います。

それと、main関数はintを返すことになっているので、最後でreturn 0;としてください。

ちなみに、

cout << name << endl;の部分をコメント化すると動きました。（成功ではありませんが）

これは、前の行でGetOldMonthName関数の戻り値をnameに入れたけど、条件が合致しない（monthが1～12の範囲でない）場合はnameに不正な値が入って、異常動作を起こしていたからです。

Answer 2

if (1 <= month && month >= 12){ -> if (1 <= month && month <= 12){

Comments

[catsforepaw]

よく見たら、あり得ない条件になっていますね。これでは必ず失敗してしまいますね。

[cateye]

１３以上を入れると何が起こるかわかりませんね＾＾；
補足：１以上and１２以上なので１２以上はすべて受け付けてしまいます。・・・範囲外のメモリデータをポインタとして扱ってしまうので・・・

Answer 3

GetOldMonthNameに13を渡すと値を返しません。