Цикл создает лишнюю пустую строку?

Question

[MrFlatman]

Пробую написать обычный календарь на год. Столкнулся с проблемой, что в сентября цикл создает дополнительную лишнюю строку, хотя ее быть не должно, это ситуация встречается только в том случаи если конец месяца выпадает на воскресенье, тогда он повторно создает пустую строку, где ошибка в цикле?

Картинка для наглядности

Код

<?php
function draw_calendar($month, $year, $action = 'none') {
	$calendar = '<table cellpadding="0" cellspacing="0" class="b-calendar__tb">';
	
	// вывод дней недели
	$headings = array('Пн','Вт','Ср','Чт','Пт','Сб','Вс');
	$calendar.= '<tr class="b-calendar__row">';
	for($head_day = 0; $head_day <= 6; $head_day++) {
		$calendar.= '<th class="b-calendar__head';
		$calendar .= '">';
		$calendar.= '<div class="b-calendar__number">'.$headings[$head_day].'</div>';
		$calendar.= '</th>';
	}
	$calendar.= '</tr>';
	// выставляем начало недели на понедельник
	$running_day = date('w',mktime(0,0,0,$month,1,$year));
	$running_day = $running_day - 1;
	if ($running_day == -1) {
		$running_day = 6;
	}
	
	$days_in_month = date('t',mktime(0,0,0,$month,1,$year));
	$day_counter = 0;
	$days_in_this_week = 1;
	$dates_array = array();
	
	// первая строка календаря
	$calendar.= '<tr class="b-calendar__row">';
	
	// вывод пустых ячеек
	for ($x = 0; $x < $running_day; $x++) {
		$calendar.= '<td class="b-calendar__np"></td>';
		$days_in_this_week++;
	}
	
	// дошли до чисел, будем их писать в первую строку
	for($list_day = 1; $list_day <= $days_in_month; $list_day++) {
		$calendar.= '<td class="b-calendar__day';
		// выделяем выходные дни
		if ($running_day != 0) {
			if (($running_day % 5 == 0) || ($running_day % 6 == 0)) {
				$calendar .= ' b-calendar__weekend';
			}
		}
		$calendar .= '">';
		// пишем номер в ячейку
		$calendar.= '<div class="b-calendar__number">'.$list_day.'</div>';
		$calendar.= '</td>';
		// дошли до последнего дня недели
		if ($running_day == 6) {
			// закрываем строку
			$calendar.= '</tr>';
			// если день не последний в месяце, начинаем следующую строку
			if (($day_counter + 1) != $days_in_month) {
				$calendar.= '<tr class="b-calendar__row">';
			}
			// сбрасываем счетчики 
			$running_day = -1;
			$days_in_this_week = 0;
		}
		$days_in_this_week++; 
		$running_day++; 
		$day_counter++;
	}
	// выводим пустые ячейки в конце последней недели
    if ($days_in_this_week < 8) {
		for($x = 1; $x <= (8 - $days_in_this_week); $x++) {
			$calendar.= '<td class="b-calendar__np"> </td>';
		}
	}
	
	$calendar.= '</tr>';
	$calendar.= '</table>';
	return $calendar;
}
?>

Answer 1

[0xD34F]

if ($days_in_this_week < 8) {

И как называется этот восьмой день? Я, честно говоря, не понял, зачем тут лишняя единица. В любом случае, проверять надо не верхнюю границу (всё равно выйти за неё не позволит условие продолжения цикла), а нижнюю - что в последней строке уже что-то есть, что она не пустая:

if ($days_in_this_week > 1) {

Но вообще, пустая строка в конце - это вовсе не косяк. Сколько строк всего может быть? - от четырёх (не високосный февраль, начинающийся в понедельник) до шести (месяц из 30 дней начинается в воскресенье, из 31 дня в воскресенье или субботу). Как-то не единообразно. Пусть всегда будет шесть строк, чтобы размер календаря не зависел от месяца. Так что отступаем от начала месяца назад до ближайшего понедельника, затем выводим 42 дня (шесть недель):

function drawCalendar($year, $month) {
  $date = DateTime::createFromFormat('Y-m-d', "$year-$month-1");
  $day = DateInterval::createFromDateString('1 day');
  $weekdays = [ 'Пн', 'Вт', 'Ср', 'Чт', 'Пт', 'Сб', 'Вс' ];

  do {
    $date->sub($day);
  } while ($date->format('w') !== '1');

  $days = [];
  for ($i = 0; $i < 42; $i++, $date->add($day)) {
    $m = intval($date->format('m'));
    $days[] = $m === $month ? intval($date->format('d')) : '&nbsp;';
  }

  return "
<table cellpadding=\"5\" cellspacing=\"5\" border=\"1\">
  <thead>
    <tr class=\"b-calendar__row\">".implode('', array_map(function($n) { return "
      <th class=\"b-calendar__head\">$n</th>"; }, $weekdays))."
    </tr>
  </thead>
  <tbody>".implode('', array_map(function($week) { return "
    <tr class=\"b-calendar__row\">".implode('', array_map(function($n, $i) { return "
      <td class=\"b-calendar__day ".($i > 4 ? 'b-calendar__weekend' : '')."\">
        $n
      </td>"; }, $week, array_keys($week)))."
    </tr>"; }, array_chunk($days, 7)))."
  </tbody>
</table>";
}


echo drawCalendar(2018, 1);