C++で、日付をプログラミング

C++勉強中です。課題が出たので質問したいです。よろしくお願いします。

日付のエラー処理を完全にするというもので、
次のような仕様を満たすようにMyDateクラスを修正せよ。というものです。

１．月、日共に初期値は１とする。
２．月、日の状態を出力するメンバ関数print()を追加する。
３．changeDay関数において、monthに入っている値に対応して適切なエラー処理を行う。
　　　４月の時は日は30まで、５月の時は31まで、みたいな感じです。
４．changeMonth関数において、変更しようとする月に対しdayに入っている値が大きすぎる場合、dayを変更後の最後の日に修正するようにする。例えば、５月３１日に設定されている時に、changeMonth(2)が呼ばれた時、monthに２を、dayに２８を代入する。
５．うるう年は考慮しなくていい。


#include <stdio.h>

class MyDate{
private:
int day, month;
public:
MyDate(int =1, int = 1);
void changeMonth(int); //「月」変更用関数
void changeDay(int); //「日」変更用関数
void print();
};

void MyDate::changeMonth(int argMonth)//メンバ関数(changeMonth)の定義
{

if(argMonth >= 1 && argMonth <= 12){
//１～１２のときだけ値を更新
month = argMonth;
}
}

void MyDate::changeDay(int argDay)//メンバ関数(changeDay)の定義
{

//１～３１が入力された場合のみ値を更新
if(argDay >= 1 && argDay <= 31){
day = argDay;
}
}
void MyDate::print()
{
if(month = 2 && day > 28){
printf("DATE = %d/%d\n",1,31);
}else if(month = 4,6,9,11 && day > 30){
printf("DATE = %d/%d\n",month-1,31);
}else{
printf("DATE = %d/%d\n",month,day);
}
}

int main(int, char **){

MyDate today;
today.changeDay(30);
today.changeMonth(9);

return 0;
}

途中まで考えたのですが、煮詰まってしまってうまくいきません。
ifは使いますよね？

void MyDate::print()
の所はこれでいいのでしょうか？
よろしくお願いします。

No.2 ベストアンサー 回答者： Wr5 回答日時： 2011/05/18 11:33

仕様の２と３を見るに…

>void MyDate::print()
>{
>if(month = 2 && day > 28){
>printf("DATE = %d/%d\n",1,31);
>}else if(month = 4,6,9,11 && day > 30){
>　：

MyDate::print()でごにょごにょするのではなく、
changeDay()とchangeMonth()で処理しろ……
ということかと思われますが。

あと、上記のif文の判定式は正しくないです。
if(month = 2 && day > 28)
は…
monthに2を「代入」し、dayが28より大きい場合に真となります。
「monthが2、かつdayが28より大きい」という判定ではないです。
if(month == 2 && day > 28)
と記述していませんから。

if(month = 4,6,9,11 && day > 30)
は…
monthに4を「代入」し、dayが30より大きい場合に真となります。
「monthが４または６または９または１１で、dayが30より大きい」を期待されているようでしょうが違います。

もう少し、基礎からやり直した方がいいと思いますが。

この回答へのお礼

わかりやすい回答ありがとうございました。

お礼日時：2011/05/25 08:41

No.3 回答者： episteme 回答日時： 2011/05/18 12:10

質問になっていません。

> 途中まで考えたのですが、

ではその「途中まで」を示してください

> 煮詰まってしまってうまくいきません。

どこで/なにに煮詰まったのですか?
なにが/どううまくいきませんか?

この回答へのお礼

確かにこれじゃ丸投げみたいなものですよね。
指摘ありがとうございます。
今後気を付けます。

お礼日時：2011/05/25 08:40

No.1 回答者： m-take0220 回答日時： 2011/05/18 11:26

コンストラクタの実体は?

これがないと初期化ができないはずですが。

また、3.と4.の条件はchangeDay関数とchangeMonth関数内の処理を説明しているはずですが、なぜprint関数で処理してるの? しかもやってる処理自体も提示された条件とは違ってるし。

さらに、

＞ month = 4,6,9,11

これはエラーにはならないでしょうが、あなたが考えている「monthが4,6,9,11のどれかと一致したら」という条件にはなりません。C++のコンマ演算子について調べてみてください。

この回答へのお礼

わかりました
調べてみます!!
細かく教えてくれてありがとうございました

お礼日時：2011/05/18 11:44