-1

Всем привет, подскажите пожалуйста почему мой цикл проходит всего один раз..

вот мой код

public function getDateHolidayArr($holidaysEmployee)
    {
        $allDates = array();

        $count = count($holidaysEmployee);

        if ($count == 0)
        {
            return $allDates;
        }

        for ($i = 0; $i <= $count; $i++)
        {
        $holidaysFrom = $holidaysEmployee[$i]->holidays_from;
        $holidaysBefore = $holidaysEmployee[$i]->holidays_before;

        $startDate = new Carbon($holidaysFrom);
        $endDate = new Carbon($holidaysBefore);

            while ($startDate->lte($endDate)) {
                $allDates[] = $startDate->toDateString();

                $startDate->addDay();
            }
           
        return $allDates;

        }

    }

введите сюда описание изображения

когда убираю return с цикла то появляется офсетная ошибка введите сюда описание изображения

введите сюда описание изображения

вынес ретурн за цикл но та же офсетная ошибка

введите сюда описание изображения

PS каунт 2 показывает, первый массив из коллекции перебирается, если ставлю допустим у $i значение 1 то перебирается второй массив.. получается сам цикл тормозится..

3
  • Проверьте переменную $count может она пуста. Он выполняет условие if и выводит return. На этом цикл заканчивается.
    – Miomoor
    21 окт 2019 в 12:25
  • каунт 2 показывает, первый массив из коллекции перебирается, если ставлю допустим у $i значение 1 то перебирается второй массив.. получается сам цикл тормозится..
    – Vlad
    21 окт 2019 в 12:27
  • У вас с offset ошибка т.к. цикл вы с 0 начинаете и до <= count, в то время когда count считается с 1. То есть, вы раз проходитесь по не существующему эл-ту. либо сделайте < count, либо c 1 цикл запускайте
    – InDevX
    21 окт 2019 в 13:09

3 ответа 3

2

Уберите return с цикла... и все заработает

public function getDateHolidayArr($holidaysEmployee)
{
    $allDates = array();

    $count = count($holidaysEmployee);

    if ($count == 0)
    {
        return $allDates;
    }

    for ($i = 0; $i <= $count; $i++)
    {
    $holidaysFrom = $holidaysEmployee[$i]->holidays_from;
    $holidaysBefore = $holidaysEmployee[$i]->holidays_before;

    $startDate = new Carbon($holidaysFrom);
    $endDate = new Carbon($holidaysBefore);

        while ($startDate->lte($endDate)) {
            $allDates[] = $startDate->toDateString();

            $startDate->addDay();
        }


        //return $allDates;
    }
    return $allDates;
}
3
  • вынес ретурн за цикл но та же офсетная ошибка
    – Vlad
    21 окт 2019 в 12:25
  • dd уберите , зачем он там
    – Kotikigor
    21 окт 2019 в 12:26
  • ддешкой смотрю чтобы нужный массив был на выходе
    – Vlad
    21 окт 2019 в 12:28
0

Прошу заметить, у вас стоит оператор return в теле цикла ;) Этот оператор моментально прерывает выполнение тела функции...


Вы можете каждую итерацию цикла прибавлять значение к переменной, а затем сделать return.

public function getDateHolidayArr($holidaysEmployee)
{
    $allDates = array();

    $count = count($holidaysEmployee);

    if ($count == 0)
    {
        return $allDates;
    }

    for ($i = 0; $i <= $count; $i++)
    {
    $holidaysFrom = $holidaysEmployee[$i]->holidays_from;
    $holidaysBefore = $holidaysEmployee[$i]->holidays_before;

    $startDate = new Carbon($holidaysFrom);
    $endDate = new Carbon($holidaysBefore);

        while ($startDate->lte($endDate)) {
            $allDates[] = $startDate->toDateString();

            $startDate->addDay();
        }


        //return $allDates;
    }
    return $allDates;
}
4
  • без return офсетная ошибка вылетает.. i.imgur.com/eogf3Kt.png i.imgur.com/eogf3Kt.png
    – Vlad
    21 окт 2019 в 12:15
  • можно пример пожалуйста.. а то мозг уже плывет..
    – Vlad
    21 окт 2019 в 12:16
  • Нельзя убирать return, ведь функции должны возвращать какое-то значение. Попробуйте опустить return ниже предпоследней скобки, но выше последней...
    – user355130
    21 окт 2019 в 12:22
  • вынес ретурн за цикл но та же офсетная ошибка
    – Vlad
    21 окт 2019 в 12:25
0

разобрался спасибо всем! Офсет возникал из за то что в условии $i <= $count надо убрать равно. Теперь все ок

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.