ツェラーの公式で曜日を求める。

Question

#include<stdio.h>
#include<stdlib.h>
#include<time.h>

time_t make_date(int year, int month, int date) {
	struct tm t;

	t.tm_year = year - 1900;
	t.tm_mon = month - 1;
	t.tm_mday = date;
	t.tm_hour = 0;
	t.tm_min = 0;
	t.tm_sec = 0;
	t.tm_isdst = -1;

	if (mktime(&t) == (time_t)-1)
		return -1;
	return t.tm_wday;
}

int day_of_zeller(int year, int month, int day) {
	if (month = 1 || month == 2) {
		year--;
		month += 12;
	}
	int u_year = (int)(year / 100);
	int d_year = (int)(year - ( u_year * 100)) + 1;

	printf("%d, %d\n", u_year, d_year);

	return (day + (int)((month + 1)*26 / 10) + d_year + (int)(d_year / 4) + (int)(u_year/4) -2*u_year) % 7;
}

int main() {
	int year, month, day, we, we2;
	char *ws[] = { "日", "月", "火", "水", "木", "金", "土" };
	char *wd[] = { "土", "日", "月", "火", "水", "木", "金"};

	printf("曜日を求めます。 \n");
	printf("年 : "); scanf("%d", &year);
	printf("月 : "); scanf("%d", &month);
	printf("日 : "); scanf("%d", &day);

	we = make_date(year, month, day);
	we2 = day_of_zeller(year, month, day);

	printf("それは%s曜日です。\n", ws[we]);  // こちらの表示は問題なし
	printf("それは%s曜日です。\n", wd[we2]);  //  こちらは合ってないです。
	
	return 0;
}

int u_year = (int)(year / 100);
int d_year = (int)(year - ( u_year * 100)) + 1;

return (day + (int)((month + 1)*26 / 10) + d_year + (int)(d_year / 4) + (int)(u_year/4) -2*u_year) % 7;

ツェラーの公式で、1997年、8/13が、何曜日だったか計算していただけませんか？

この３行で曜日を計算していますが・・・

１つ目の疑問です。
u_year はupという意味で上位２桁。d_yearはdownという意味で下位２桁を代入しています。

例えば、 1997 年なら、u_year = 19; d_year = 97; となるはずです。
しかし、d_yearの方は最後に ＋１しないと合いません。
なぜでしょうか？？？
キャストが原因でしょうか？

２つ目の疑問。
day + (int)((month + 1)26 / 10) + d_year + (int)(d_year / 4) + (int)(u_year/4) -2u_year) % 7
この数式、どこがおかしいのか分かりません。

(q+[(m+1)×26/10]+K+[K/4]+[J/4]-2J) (mod 7)　（リンク先の数式）

結果は、
それは水曜日です。
それは火曜日です。

どこがおかしいのでしょうか？

例えば、今日の日付、2017.11.26を入れてみると、
それは日曜日です。
それは金曜日です。
となります。

困っています。
分かる方います？？

ちなみに、有名なこちらの数式でもやってみましたが、合いません。(最初はこちらでやりましたが、合わなかったので、やり方を変えました。)
(y + y/4 - y/100 + y/400 + (13 * m + 8)/5 + d)%7;
ツェラー(Zeller)の公式をC言語で書いた場合

教えてください。

「追記」
今、確認したところ、関数の中のreturn に記述するとうまくいかないですね。
関数を作らずに直接、
we2 =(year + year / 4 - year / 100 + year / 400 + (13 * month + 8) / 5 + day) % 7;
とするとなぜかうまくいきます。

Answer 1

詳しくは追っていませんが下記が原因ではないでしょうか。

if (month = 1 || month == 2) {

monthは常に1となっています。

---

追記：

下記記述とすることで、両方の関数で同じ結果が返るようになるようでした。
また、d_yearの算出には剰余を使った方がよいかと思いましたのでそのようにしています。

int day_of_zeller(int year, int month, int day) {
    if (month == 1 || month == 2) {
        year--;
        month += 12;
    }
    int u_year = (int)(year / 100);
    int d_year = (int)(year % 100);

if文での代入処理はバグの素なので、コンパイラでチェックすることで発見できる可能性があります。
たとえばgccですと、下記のように警告オプションをつけることで、今回のような箇所は警告を受けることができます。

$ gcc -Wall a.c
a.c: In function ‘day_of_zeller’:
a.c:22:5: warning: suggest parentheses around assignment used as truth value [-Wparentheses]
     if (month = 1 || month == 2) {
     ^

Comments

[strike1217]

あああ！！
できました。

どうもありがとうございます。
ふぅぅ・・・