Как добавить условие, при котором которое будет исполнять функцию через раз?

Question

[Руслан Макаров]

Доброго!
Для показа рекламы в статье (вордпрес) я использую код, который вставляет ее перед каждым заголовком H2:

function ads_h2($the_content) {
$ads='
<!--КОД РЕКЛАМЫ-->
';
if (is_single()) {
 $the_content=preg_replace('#<h2(.*?)</h2>#','<div class="ads_content"><h2\1</h2>'.$ads.'</div>',$the_content);
 }
 else {
 }
return $the_content;
}
add_filter('the_content', 'ads_h2');

Он отлично работает, но в некоторых статьях рекламные блоки очень блиско расположены к друг другу: пример...

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

Answer 1

[Игорь Воротнёв]

function ads_h2( $content ) {

	// Хорошая практика, называется "return early"
	if ( ! is_single() ) {
		return $content;
	}

	// Заготовили код рекламы
	$ad = '[AD]';

	// Инициализировали счетчик
	$count = 1; // 0 или 1, по ситуации

	// Делаем замену через коллбек
    $content = preg_replace_callback(
        '#<h2(.*?)</h2>#',
        // С помощью use передаем в замыкание дополнительные параметры
        function( $matches ) use ( &$count, $ad ) {
            // Увеличиваем счетчик
            $count++;
            // Если число четное, добавляем рекламу
            if ( $count % 2 == 0 ) {
	            return '<div class="ads_content">' . $matches[0] . $ad . '</div>';
            // Если нечетное - ничего не делаем
            } else {
                return $matches[0];
            }
        },
        $content
    );

	// Вернули измененный контент
	return $content;
}
add_filter( 'the_content', 'ads_h2' );

Бонус

Если менять цифру в if ( $count % 2 == 0 ) { вместо 2 на, скажем, 3 или 4, то реклама будет выводиться, соответственно, в каждом 3м или 4м подзаголовке.

Comments

[Павел Корнилов]

Согласен, удаляю.

[Руслан Макаров]

Работает! Спасибо большое!

Answer 2

[Алексей]

$odd = true;
if ($odd) {
 add_filter('the_content', 'ads_h2');
 $odd = false;
} else {
 $odd = true;
}

Comments

[GavriKos]

Ну тогда уж $odd = !$odd. PHP так умеет?

[Алексей]

GavriKos: там ошибка была, исправил.

[Руслан Макаров]

Алексей: Подскажите этот kод нужно вставлять в фаил functions.php?

[Алексей]

Руслан Макаров: Вот тут я не подскажу, так как не знаю всех деталей.
Возможно будет удобнее, что functions.php будет вызываться не пред каждым Н2, а через раз.

$odd = true;
if ($odd) {
 тут вызов functions.php;
 $odd = false;
} else {
 $odd = true;
}

Но только вы сами можете сказать как правильно, так как только вы знаете структуру и знаете как всё работает.

[Игорь Воротнёв]

Алексей: Руслан Макаров: оберните в эту конструкцию ваш preg_replace (то есть, внутри вашей функции ads_h2

[Алексей]

Игорь Воротнёв:
тогда будет бесконечная рекурсия если внутри ads_h2 вызывать
add_filter('the_content', 'ads_h2');

[Игорь Воротнёв]

Алексей: да не add_filter вызывать, боже ж ты мой) Сейчас ответ подготовлю

[Игорь Воротнёв]

Алексей: добавил в виде ответа

[Алексей]

Игорь Воротнёв: Одобряю! Хорошее решение. Вы выполнили работу в полном объеме за человека :)

[Игорь Воротнёв]

Алексей: та когда-то делал это уже, была у клиента аналогичная задача. Четко помню, что перепробовал тогда разные методы, и такой - единственный рабочий. Порылся, но проект тот старый не нашел, пришлось написать заново :) Заодно себе в шпаргалку сохранил, на будущее может пригодиться еще.