У меня есть две строки, которые повторяются много раз и между ними есть код. Как избавиться от дублирования?

Question

[Александр]

У меня есть две строки, которые повторяются всегда и между ними есть код. Как избавиться от дублирования ?
Код между строками всегда разный
Вот код, который повторяется много раз:

$this->apiUrl = self::DEFAULT_API_URL;
            
    do {
       Here is different code every time
    } while ($this->apiUrl = $jsonData['next_page_url'] ?? null);

Пример реального кода:

public function parse()
            {
                $this->apiUrl = self::DEFAULT_API_URL;
        
                do {
                    $this->importProductWithAttributes($this->apiUrl);
                } while ($this->apiUrl = $jsonData['next_page_url'] ?? null);
            }

    public function addToQueue() : void
            {
                $this->apiUrl = self::DEFAULT_API_URL;
        
                do {
                    $this->getDataJson($this->apiUrl);
                    $jsonData = self::$data;
                    $tenPartJsonDate = array_chunk($jsonData['data'], 10, true);
        
                    foreach ($tenPartJsonDate as $key => $products) {
                        $skuArray = [];
        
                        foreach ($products as $sku => $product) {
                            $skuArray[] = $sku;
                        }
        
                        ProcessImportB2B::dispatch($skuArray)->onQueue('b2bImport');
                    }
        
                } while ($this->apiUrl = $jsonData['next_page_url'] ?? null);
            }

Answer 1

[Adamos]

private function check($jsonData) {
  return $this->apiUrl = $jsonData['next_page_url'] ?? null;
}

Comments

[Александр]

Вы не поняли. ВОТ ЭТОТ КОД ПОВТОРЯЕТСЯ. Я не хочу чтобы он дублировался. И хочу его вынести. Есть вариант вынести в Метод, который принимает callback, но как по мне не лучшее решение.
$this->apiUrl = self::DEFAULT_API_URL;

do {
ЗДЕСЬ КАЖДЫЙ РАЗ РАЗНЫЙ КОД
} while ($this->apiUrl = $jsonData['next_page_url'] ?? null);

[BoShurik]

Есть вариант вынести в Метод, который принимает callback, но как по мне не лучшее решение.

Почему?

[Adamos]

Александр, я понял. И предложил все, что имеет смысл сделать по этому поводу.
А заворачивать код в колбэки только ради того, чтобы не повторить три строчки - перфекционистская дурь, которая только испортит и читаемость, и сопровождаемость кода.

[Александр]

Adamos, эти 3 строчки повторяются около 20 раз. А это 60 лишних строк....

[Александр]

BoShurik, как минимум ухудшает скорость работы прилично на больших обьемах. Во вторых читаемость кода ухудшается. Другому программисту будет не просто понять, что там за callback , что нужно передать и прочее.

[Adamos]

Александр, строки сами по себе - не проблема. Проблема может быть в том, что их однажды придется исправить - или в них можно допустить ошибку. Тут же все настолько тривиально, что просто нет смысла заморачиваться.

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

Answer 2

[Антон Антон]

Сделать функцию с циклом, в которую передавать лямбду с меняющимся кодом.