VC++でif文が処理されません

質問

万年カレンダーを作っていて、先日こちらで日付の取得方法を質問して解決法を教えていただき、その後制作を進めていました。
が、今度はコンパイルは成功しても一部コードが処理されずに行き詰ってしまいました。
以下のようにlabel2に日付を表示させるコードを書いてみたのですが、/*日付の表示*/の
if(wd%7 == 0){
label2->Text = Cal + "\r\n";
}
が読み込まれていないようで、改行されません。
試しに"\r\n"ではなく別の文字列を入れてみたり、if(wd == 6)に換えてみたりしました。
が、どちらも成果はありませんでした。

どこかおかしい部分があるのでしょうか？
コンパイルは成功するので、構文としては間違っていないと思います。
無駄に長いコードになってしまっているかも知れませんが、よろしくお願いします。

＜↓ここからソースです＞
private: System::Void Form1_Load(System::Object^ sender, System::EventArgs^ e) {
DateTime d1 = DateTime::Now;
textBox1->Text = d1.ToString("yyyy");
textBox2->Text = d1.ToString("MM");
int year,month,day ; /* 年 月 日 */
int wd; /* 曜日 */
int dm ; /* 月の日数 */
int L0 ; /* カウンタ */
String^ Cal = "";

/*年・月・日を代入*/
year = System::Int32::Parse( d1.ToString("yyyy") );
month = System::Int32::Parse( d1.ToString("MM") );
day = System::Int32::Parse( d1.ToString("dd") );

/*現在の年・月が何日あるか*/
switch(month) {
case 2:
if ( year%4 == 0 && (year%100 != 0 || year%400 ==0)){
dm = 29 ; /* うるう年 */
} else {
dm = 28 ;
break ;
}
case 4:
case 6:
case 9:
case 11:
dm = 30 ;
break ;
default:
dm = 31 ;
break ;
}

/* 今年・今月１日の曜日を求める */
if (month == 1 || month == 2) {
year-- ;
month += 12 ;
}
wd = (year + year/4 - year/100 + year/400 + (13*month+8)/5 +1) % 7 ;

/* スペースの表示 */
for (L0=0;L0<wd;L0++) {
label2->Text = Cal + " ";
Cal = label2->Text;
}

/* 日付の表示 */
for(L0=1;L0<=dm;L0++){
if(wd == 6){
label2->Text = Cal + "\r\n";
}
label2->Text = Cal + L0;
Cal = label2->Text;
wd = wd + 1;
}

label1->Text = "日　　月　　火　　水　　木　　金　　土" + "\r\n";
}

回答No.3

mtaka2 · ベストアンサー

> if(wd == 6){
> label2->Text = Cal + "\r\n";
> }
ここで label2->Text に代入していますが、その直後の
> label2->Text = Cal + L0;
で、また別の文字列を代入してますね。
そのため、最初のTextへの代入はまったく反映されないことになります。
他の記述に合わせるなら、
if文のブロック内に Cal = Label2->Text が必要かな。

でも、そもそも「毎回 Label2->Text に代入しては Cal に代入し戻す」のはかなりの無駄です。
カレンダー文字列の設定は全て
Cal = Cal + "\r\n"
って感じで、変数Calにどんどん追加していって、最後に一回だけ
Label2->Text = Cal
と代入するので十分でしょう。

お礼

＞そもそも「毎回 Label2->Text に代入しては Cal に代入し戻す」のはかなりの無駄です。

落ち着いて考えると確かに無駄ですね。
どうやら途中で代入処理の記述を書き落としていたため、改行がCalに反映されていなかったようです。
ご指摘されたように最後に一回だけ出力するようにしたら、解決しました。

ご回答くださった方々、ありがとうございます。

回答No.2

asuncion

＞wd = (year + year/4 - year/100 + year/400 + (13*month+8)/5 +1) % 7 ;

ツェラーの公式で求まる曜日は、
0：日曜日、1：月曜日、...、6：土曜日のはずです。
したがって、

＞if(wd == 6){

１週間分のラベル文字列を出力した後で改行を入れたいのであれば、
wdが6のとき「だけ」ではなく、wdを7で割ったあまりが6のとき、
である必要があると思います。
また、

＞label2->Text = Cal + L0;

文字列と整数の連結は成立するのでしょうか？
Calと「L0を文字列に変換した内容」とを連結しなければならないのではないでしょうか。
さらに、１週間分のラベル文字列を出力し終わったら、
Calを空に初期化する必要があるのではないでしょうか。

以上、VC++ を全く知らない者からのたわごとでした。

お礼

＞if(wd == 6){
これに関しては、「%」がマズいのかと確認するためのテスト用です。
言葉足らずですみません；

＞label2->Text = Cal + L0;
今回は問題ないようです。
が、よくその内容でエラーを出してしまうので、改めて気をつけてみます。

回答No.1

SAYKA

とりあえずwd自体をdumpしてみたらどうかな？
目的の値を保持している？

お礼

/*日付の表示*/内のfor文でlabelに表示させてみたところ、wdの値は保持しているようです。
今後それについても気をつけてみたいと思います。