Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Исправлено не корректное отоброжение дней недели #31

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mesolaries
Copy link

В номере дня недели для 1 числа не учитывалось с каким месяцем была вызвана функция. Скрипт всегда брал день недели для текущего месяца хотя изначально функция была вызвана с предыдущим месяцем.
Так же скрипт работал не корректно если первый день недели воскресенье (т.е. номер дня недели = 0)

В номере дня недели для 1 числа не учитывалось с каким месяцем была вызвана функция. Скрипт всегда брал день недели для текущего месяца хотя изначально функция была вызвана с предыдущим месяцем.
@Demiat
Copy link

Demiat commented Nov 20, 2021

скажите пожалуйста, по программе все понятно, интересует магическая математическая двойка - 2, в строке $n = - ($wday - 2); Из какой логики она следует? Всё работает, а вот почему, пока не понял.

@mesolaries
Copy link
Author

скажите пожалуйста, по программе все понятно, интересует магическая математическая двойка - 2, в строке $n = - ($wday - 2); Из какой логики она следует? Всё работает, а вот почему, пока не понял.

Таким образом определяется сколько пропусков нужно делать на первой неделе месяца. Например, если 1-ое число это среда ($wday = 3), то - (3 - 2) = -1. Цикл будет добавлять пустые ячейки пока не дойдет до 1. Соответственно в первой неделе месяца будет 2 пропуска (-1, 0). И первое число мы увидим на третьем дне недели.

@Demiat
Copy link

Demiat commented Dec 10, 2021 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants