[C++]値の引渡しについて。

質問

教えて下さい。
mainの中においてasctimeで現在の日付を所得します。
その中の年だけをclassに渡し、classの中のメンバ関数で
使いたいのですが、mainからclassへはどう引き渡せばいいでしょうか？

汚いプログラムで申し訳ないのですが、mainのtm_yearの値を
うるう年判定の関数で使いたいのです。

#include <iostream>
#include<time.h>
using namespace std;
class TDate {

public:
int dt_month;
int dt_day;
int dt_year;
bool leapYear;
int year1;

TDate(int month, int day, int year);

void Display();

void Leap();　/*うるう年判定の関数*/
};

int main()
{
TDate dt_obj(11,1,2005);
dt_obj.Display();
dt_obj.Leap();

struct tm *newtime;
time_t aclock;
char *now;
time(&aclock); /* 現在時刻の獲得 */

newtime = localtime(&aclock); /* tm構造体への変換 */
now = asctime(newtime); /* 文字列への変換 */
cout << "現在の時刻" << now << endl;
cout << "YEAR" << newtime -> tm_year << endl;

return 0;
}
TDate::TDate(int month, int day, int year)
{
dt_month = month;
dt_day = day;
dt_year = year;
}

void TDate::Display()
{
cout << dt_month << "/" << dt_day << "/" << dt_year << endl;
}

void TDate::Leap()
{
year1 = n+1900;
if ( year1 <= 0)
cout << " Wrong year : Greater or equal to 0 " << endl;
else {
if ((year1 % 100) == 0 )
leapYear = ((year1 % 400) == 0 );
else
leapYear = ((year1 % 4) == 0 );
if (leapYear)
cout << year1 << " is a leap year " << endl;
else
cout << year1 << "is not a leap year" << endl;

}
}

よろしくお願いします。

回答No.1

charmer29-2

・クラス内のメンバ変数(Ex:dt_year)をpublicにしてしまっては、折角クラスで隠蔽する意義が薄れます。
・関数内でのみ使う変数(Ex:year1)は、ローカル変数で充分。メンバ変数にする必要がないでしょう。

それはさておきまして。
tm_yearをTDate::Leap()で使いたいのなら、TDate::Leap(int year)とでもして引き渡せば宜しいかと。
クラスの仕様が判らないのでなんとも言えませんが、Leap()はどんな関数なんでしょうか。
判定をする関数で出力もする、メンバ変数の更新をする、なんてやっていては使い勝手も悪いしそもそも設計の「せ」の字もありません。
「汚いプログラム」以前に基礎からきちんと実習することをお勧めしておきます。

お礼

回答ありがとうございます。

>>・クラス内のメンバ変数(Ex:dt_year)をpublicにしてしまっては、折角クラスで隠蔽する意義が薄れます。

これはこの場合どちらでも良いと言われたので、privateに入っていたものを
わかりやすくするためpublicに移しました。
お察しの通りですが、わかっております。

>>判定をする関数で出力もする、メンバ変数の更新をする、なんてやっていては使い勝手も悪いしそもそも設計の「せ」の字もありません。
「汚いプログラム」以前に基礎からきちんと実習することをお勧めしておきます。

今まさにその実習のときなんでお許し下さい。
色々試行錯誤の途中で、理解を深めています。

>>tm_yearをTDate::Leap()で使いたいのなら、TDate::Leap(int year)とでもして引き渡せば宜しいかと。

やってみます。