Как запретить выполнение функций дважды для текущего пользователя?

Question

[zeni1agent]

мне необходимо что бы функция выполнялась только 1 раз для одного пользователя.
при повторной попытке нечего бы не происходило бы.
Если зайдет другой пользователь то функция выполнится для него но только 1 раз
и так далее для всех других пользователей.
Необходимо, чтобы проверка повторялась на каждой странице отдельно.
И не переходил на другие страницы в single.php

моя функция

function setPostViews($postID) {
    $count = get_post_meta($postID, 'post_count', true);
    $count++;
    update_post_meta($postID, 'post_count', $count);
    echo $count.;
}

кто нибудь знает как это можно сделать?

Answer 1

[Дмитрий Добрышин]

В базе данных создать таблицу с пользователями. При запуске функции проверять есть ли в таблице пользователь. Если нет - выполнять основной алгоритм и дописывать текущего пользователя в базу.

P.S. Если кто-то смог ответить на Ваш вопрос, отметьте его решением. Так Вы дадите стимул давать ответы всем участникам сообщества.

Comments

[zeni1agent]

Вроде бы я понял как это должно работать
Вначале надо вывести из базы данных значение поля которое по умолчанию равно 0
после проверить с помощью if равно ли оно значению 0 если да то
увеличить на 1 и обновить в базе данных
после выполнить функцию закрыть скобки

Поправьте если я где то ошибаюсь.

но тогда появляется проблема в том что она будет работать только на одной странице сайта wordpress
а мне необходимо что бы она проверяла каждую страницу отдельно друг от друга в файле single.php
для того что бы эта функция срабатывала только один раз у одного пользователя на каждой странице

function setPostViews($postID) {
		$count = get_post_meta($postID, 'post_count', true);
		$count++;
		update_post_meta($postID, 'post_count', $count);
		echo $count.;
	}

[Пычев Анатолий]

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

только на архивных страницах, где отображается много постов, надо собирать все проверки в одно значение

[zeni1agent]

Пычев Анатолий,
Мне и нужно что бы эта функция работала на каждом посте отдельно

if( current_user_can('author') || current_user_can('editor') || current_user_can('administrator') ){  
function setPostViews($postID) {
    $count = get_post_meta($postID, 'post_count', true);
    $count++;
    update_post_meta($postID, 'post_count', $count);
    echo $count.;
}

она мне нужна для подсчета просмотров страницы зарегистрированным пользователям

А проверка мне нужна для того что бы не было накрутки просмотров
ведь при обновлений страницы число постоянно увеличивается

А мне нужно
один просмотр на одного пользователя на одну страницу

[Пычев Анатолий]

zeni1agent, получается нужно еще и сессию идентифицировать
А вообще не совсем понятно: Если пользователь сегодня посмотрел и завтра - это один просмотр или два? Если он выходил из аккаунта и снова зашел, то номер сессии поменяется - это один просмотр или два? Или если пользователь посмотрел, то для него счетчик останавливается на этой странице, тогда значение может быть только 0 или 1, тогда вообще смотри предложение Дмитрий Добрышин и записывай просмотры в метаданные пользователя (например массив - id страниц просмотренных)

Можно еще сюда заглянуть

[zeni1agent]

Пычев Анатолий,

Если в подробностях то
Функция должна срабатывать только один раз
Для одной страницы
Только для одного пользователя
Навсегда
Без ризницы
выходил он из акаунта,
чистил ли комп ,
переходил с компа на телефон,
Только один просмотр на одного пользователя для одной страницы

в качестве основы функция

setPostViews($postID)

Работает не только для подсчета просмотров но и для подсчета рейтинга и количества скачивание в посте

А для этого нужно запретить пользователю повторно вызывать функцию
ведь что при обновлений страницы
что при нажатий на кнопку и отправке с помощью метода post

Будет срабатывать повторно функция тем самым будет идти накрутка поста

[Пычев Анатолий]

тогда только предложение Дмитрий Добрышин

[zeni1agent]

Пычев Анатолий, Я не до конца понимаю что вы подразумеваете под предложением
Но денег у меня.
Неужели нет ни какой простой логики решения этой проблемы?

[Дмитрий Добрышин]

zeni1agent, а чем то, что я написал не подходит? Вы даже можете пойти дальше - составить в БД таблицу с привязкой - пользователь + страница. Нет такой записи - выполняете и дописываете запись, если уже есть - пропускаете. В общем тут огромное поле для творчества ;-)

[Пычев Анатолий]

Дмитрий Добрышин, Поддерживаю.

Под "предложением" я понимаю план который Вам предложили

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

[zeni1agent]

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

либо в новой таблице, либо в метаполе

А какой вариант меньше будет влиять на скорость сайта?

[Пычев Анатолий]

Если правильно напишите, то думаю разница будет не большая, и многое будет зависеть от количества пользователей.

Но проще писать с метаполем.

[zeni1agent]

Пычев Анатолий, Ладно но я не до конца понимаю как выстроить правильную логику
Единственное что я смог сделать это

function Agent($postID) {
   $DDD = get_post_meta($postID, 'TOT', true);
   if ($DDD == ''){	
		$DDD = 1;
		update_post_meta($postID, 'TOT', $DDD);
		echo 'Yes TTTYYYTTT';
	}
}
 Agent(get_the_ID()) ;

Но тогда она считает только первого пользователь посетившего страницу а всем остольны автоматически скроет

[Пычев Анатолий]

надо сохранять в метаполе пользователя а не страницы.

И сохранять надо массив идентификаторов просмотренных страниц
Ну как-то так. Только на ошибки проверьте, я по памяти набросал.

function Agent($postID) {
   $uid = get_current_user_id();
   $DDD = get_user_meta( $uid, 'TOT', true );
   if ( !$DDD || !isset($DDD[$postID]) ){	
    $DDD[$postID] = 1;
    update_user_meta($uid, 'TOT', $DDD);
    echo 'Yes TTTYYYTTT';
  }
}
 Agent(get_the_ID()) ;

[zeni1agent]

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

function create_table(){
	 global $wpdb ; 
	 $mebd ="	CREATE TABLE post_counts
	 (idpost CHAR(25) NOT NULL,
	  iduser CHAR(25) NOT NULL,)";
	 $wpdb ->query($mebd);
	 
 }

[Дмитрий Добрышин]

zeni1agent, таблицу вы создали, теперь только обработку сделать типа этой:

function agent($postid) {
    global $wpdb;
    $uid = get_current_user_id();
    $wpdb->get_results( "SELECT iduser FROM post_counts WHERE idpost='" . $postid . "' AND iduser='" . $uid . "'");
    if ($wpdb->num_rows < 1) {
        ... тря-ля-ля что нужно сделать ...
       $wpdb->query("INSERT INTO post_counts (idpost, iduser) VALUES('" . $uid . "','" . $postid . "')");
    } else {
        ... тра-ля-ля уже тут были ...
    }
}