Почему использование Limit и Offset не позволяет получить все записи в цикле?

Question

EmberTide

laravel

Почему использование Limit и Offset не позволяет получить все записи в цикле?

Каково различие в результате выполнения кода, если использовать следующий подход: 

```php
$count = Ship::where("updated_at", ">", "2024-01-01 00:00:00")->count();
$skip = 0;
$take = 10;

while ($count > 0) {
    $count = $count - $skip;
    $dem = Ship::where("updated_at", ">", "2024-01-01 00:00:00")
                ->skip($skip)
                ->take($take)
                ->get();

    foreach ($dem as $item) {
    }
    $skip = $skip + $take;
}
```

по сравнению с использованием такого кода:

```php
$dem = Ship::where("updated_at", ">", "2024-01-01 00:00:00")->get();
foreach ($dem as $item) {
}
```

В чем заключается разница в обработке данных между этими двумя подходами?

13 сентября 2024 г. 8:45

Ответы на вопрос 2

Похожие вопросы

Answer 1

Использование `LIMIT` и `OFFSET` в приведенном коде может привести к проблемам в получении всех записей из базы данных, особенно если количество записей превышает 10, и если вы не корректно обновляете счетчики `skip` и `take`. Рассмотрим оба подхода подробнее.

### Проблемы с первым подходом

В вашем первом коде вы пытаетесь получить записи из базы данных пакетами по 10 (из-за `take(10)`), используя `skip` для пропуска уже обработанных записей. Однако есть несколько ошибок:

1. **Неправильное обновление $count**: В вашем цикле вы обновляете `$count` с помощью `count - $skip`, что делает логику цикла некорректной. Вам нужно оставить `$count` неизменным, а вместо этого обновлять счётчик на количество обрабатываемых записей. В результате, когда вы получите записи, переменная `$count` не будет отражать текущее количество оставшихся записей.

2. **Проверка условия выхода из цикла**: Условие `while ($count > 0)` не учитывает, сколько записей вы на самом деле получили в каждой итерации. Вам нужно проверять, были ли возвращены записи, и если не было — выходить из цикла.

3. **Возможный пропуск записей**: Если вы пропускаете записи с помощью `skip`, это может стать проблемой, если база данных обновляется одновременно с выполнением вашего кода. Записи могут добавляться или удаляться, что станет причиной возможного пропуска записей.

### Второй подход

Во втором подходе вы используете метод `get()`, который получает все записи сразу. Это проще и не требует управления пакетами:

1. **Простота и ясность**: Вы просто запрашиваете все записи, соответствующие условию. Нет необходимости управлять `skip` или `take`.

2. **Нет риска управления состоянием**: Так как вы загружаете все записи за один раз, вам не нужно беспокоиться о том, правильно ли вы отследили, сколько записей уже обработано.

### Разница в обработке данных

- **Производительность**: В зависимости от объема данных, первый подход может быть более производительным, если речь идет о больших таблицах, так как он загружает данные порциями и уменьшает нагрузку на память. Но следует учитывать, что при этом сложность кода возрастает.

- **Целостность данных**: Второй подход всегда получает актуальные данные на момент времени, когда выполняется запрос. Таким образом, вы минимизируете риск того, что не получите все записи из-за изменения данных между итерациями.

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

Answer 2

вы отнимаете от $count числа кратные 10ти,  для того, чтобы пройти гарантированно всё у вас и $count должен быть кратен 10ти. в какой-то момент вконце $count у вас становится отрицательным и вы теряете то кол-во записей, чему он равен. 
 
 сделайте while(true) и выход из цикла по условию count($dem) === 0