Как найти ближайшую не выходную, не выходную дату, вычитая один день за один раз?

Мне пришлось заполнить некоторые из недостающих компонентов, но, возможно, вы сможете почерпнуть необходимую логику из моего переписывания. Мой класс старается свести к минимуму рекурсию, изменив порядок массивов дисквалифицирующих объектов.

Код: (демонстрация)

class PayDay
{
public function __construct($Ymd = null) {
$this->date = $Ymd ?: date("Y-m-d");

$this->holiday_calendar = [
(object)["holiday" => "New Year's Day", "holidayDate" => "2018-01-01"],
(object)["holiday" => "Republic Day", "holidayDate" => "2018-01-26"],
(object)["holiday" => "Holi", "holidayDate" => "2018-03-02"]
];

$this->payroll_days_list = [
(object)["payrollDay" => "Mon", "isPayrollDay" => "Y"],
(object)["payrollDay" => "Tue", "isPayrollDay" => "Y"],
(object)["payrollDay" => "Wed", "isPayrollDay" => "Y"],
(object)["payrollDay" => "Thu", "isPayrollDay" => "Y"],
(object)["payrollDay" => "Fri", "isPayrollDay" => "Y"],
(object)["payrollDay" => "Sat", "isPayrollDay" => "N"],
(object)["payrollDay" => "Sun", "isPayrollDay" => "N"]
];

echo "Starting Date: ",$this->date,"\n";
}

public function getPayDate() {
$hc_match = false;
foreach (array_reverse($this->holiday_calendar) as $hc) {  // work in reverse to reduce recursions
if ($hc->holidayDate == $this->date) {
$hc_match = true;
//echo "Avoided: ", $hc->holiday,"\n";
$this->loseOneDay();
//echo "New Date: ", $this->date,"\n";
} elseif ($hc_match) {                            // break when consecutive adjustments cease, to reduce recursions
break;
}
}

$pdl_match = false;
foreach (array_reverse($this->payroll_days_list) as $pdl) {  // work in reverse to reduce recursions
if ($pdl->payrollDay == date("D", strtotime($this->date)) && $pdl->isPayrollDay == "N") {
$pdl_match = true;
//echo "Avoided: ", $pdl->payrollDay,"\n";
$this->loseOneDay();
//echo "New Date: ", $this->date,"\n";
} elseif ($pdl_match) {                           // break when consecutive adjustments cease, to reduce recursions
break;
}
}

if ($hc_match || $pdl_match) {
//echo "\tRECURSE\n";
$this->getPayDate();
}
return $this->date;
}

public function loseOneDay(){
return $this->date = date("Y-m-d", strtotime($this->date." -1 day"));
}
}

$date = new PayDay("2018-03-04");
echo "Adjusted Date: ", $date->getPayDate();

Выход:

Starting Date: 2018-03-04
Adjusted Date: 2018-03-01

Убедитесь, что ваши форматы даты единообразны, чтобы у вас не было 04/03/2018 сравнивать с 2018-03-04, Конечно, формат зависит от вас, просто выберите один для $dt а также $this->holiday_calendar->holidayDate для простоты.

Что касается вашего размещенного кода, потому что ваш foreach() петли не включают в себя break, вы делаете дополнительные / ненужные итерации после того, как совпадение было найдено.

Я рекомендую выполнять итерацию массивов объектов в обратном порядке (или лучше было бы объявить их в обратном порядке), чтобы вы могли выполнять последовательную настройку в одном и том же вызове функции.

Немного подумав, я думаю, что, вероятно, я бы поработал с объектами DateTime и нашел бы время для подготовки дисквалифицирующих данных, чтобы упростить итеративный процесс корректировки даты. Следующий класс вообще не использует рекурсию, а выполняет цикл с двумя условиями. (Опять же, как вы импортируете свои дисквалифицирующие массивы объектов, зависит от вас.)

Код: (демонстрация)

class PayDay
{
public function __construct($dmY = null) {
$this->date = (is_null($dmY) ? new DateTime("midnight") : DateTime::createFromFormat('!d/m/Y', $dmY));  // ! for midnight

$holiday_calendar = [                                          // import this data however you wish
(object)["holiday" => "New Year's Day", "holidayDate" => "2018-01-01"],
(object)["holiday" => "Republic Day", "holidayDate" => "2018-01-26"],
(object)["holiday" => "Holi", "holidayDate" => "2018-03-02"]
];
$this->blacklist_holidates = [];
if (!empty($holiday_calendar)) {
foreach ($holiday_calendar as $obj) {
$this->blacklist_holidates[$obj->holiday] = DateTime::createFromFormat('!Y-m-d', $obj->holidayDate);  // ! for midnight
}
}

$payroll_days_list = [                                          // import this data however you wish
(object)["payrollDay" => "Mon", "isPayrollDay" => "Y"],
(object)["payrollDay" => "Tue", "isPayrollDay" => "Y"],
(object)["payrollDay" => "Wed", "isPayrollDay" => "Y"],
(object)["payrollDay" => "Thu", "isPayrollDay" => "Y"],
(object)["payrollDay" => "Fri", "isPayrollDay" => "Y"],
(object)["payrollDay" => "Sat", "isPayrollDay" => "N"],
(object)["payrollDay" => "Sun", "isPayrollDay" => "N"]
];
$this->blacklist_daynames = [];
if (!empty($payroll_days_list)) {
foreach ($payroll_days_list as $obj) {
if ($obj->isPayrollDay == "N") {
$this->blacklist_daynames[] = $obj->payrollDay;
}
}
}
}

public function getPayDate() {
while (in_array($this->date, $this->blacklist_holidates) || in_array($this->date->format("D"), $this->blacklist_daynames)) {
$this->date->sub(new DateInterval('P1D'));
}
return $this->date->format("d/m/Y");
}
}

$input = "04/03/2018";
echo "Starting Date: $input\n";
$date = new PayDay($input);
echo "Qualifying PayDate: ",$date->getPayDate();