Как более лучше оптимизировать код для добавления товаров в избранное?

Question

[Genri_Rus]

Вообщем нужно как-то сделать так, чтобы избранные работали по такой схеме:

При клике по иконке добавляется товар в избранное, далее при повторном клике - товар удаляется из избранного
Также выводится счетчик (counter), если массив $data пустой, то $counter = 1;
Как вообще оптимизировать вывод counter ? Или если работает, то оставить так ?
Вот пример:

function add_to_id() {
  if ( isset( $_POST['variation_id'] ) && ! empty( $_POST['variation_id'] ) ) {
    $value = intval( $_POST['variation_id'] );
  } else if ( isset( $_POST['product_id'] ) && ! empty( $_POST['product_id'] ) ) {
    $value = intval( $_POST['product_id'] );
  } else {
    wp_send_json_error();
  }
  
  $counter = 0;

  if ( isset( $_COOKIE['wp_id'] ) ) {
    $data = [];
    $data = explode( ',', $_COOKIE['wp_id'] );

    if ( in_array( $value, $data ) ) {
      unset( $data[ array_search( $value, $data ) ] );

      $value = implode( ',', $data );

      setcookie( "wp_id", $value, time()+86400 * 30, COOKIEPATH, COOKIE_DOMAIN );
      wp_send_json_success( count( $data ) );
      return;
    }

    array_push( $data, $value );
    $value = implode( ',', $data );
  }

  setcookie( "wp_id", $value, time()+86400 * 30, COOKIEPATH, COOKIE_DOMAIN );
  
  if ( !empty( $data ) ) {
    wp_send_json_success( count( $data ) );
  } else {
    wp_send_json_success( $counter + 1 );
  }
}

Answer 1

[nokimaro]

Не понятно требование для пустой $data отправлять 1, а не 0
То есть если было одно значение в избранном и стало ноль, то посылаем 1 и при этом если было ноль и добавили первое значение то всё-равно посылаем 1?

<?php
function add_to_id()
{
    if(isset($_POST['variation_id']) && !empty($_POST['variation_id']))
    {
        $value = intval($_POST['variation_id']);
    }
    elseif(isset($_POST['product_id']) && !empty($_POST['product_id']))
    {
        $value = intval($_POST['product_id']);
    }
    else
    {
        wp_send_json_error();
    }

    $data = [];
    if(isset($_COOKIE['wp_id']))
    {
        foreach(explode(',', $_COOKIE['wp_id']) as $val)
        {
            if(!empty($val))
            {
                $data[$val] = $val;
            }
        }
    }
    
    //удаляем value если оно было среди значение в cookie
    if(isset($data[$value]))
    {
        unset($data[$value]);
    }
    else
    {
        $data[$value] = $value;
    }

    setcookie("wp_id", implode(',', $data), time() + 86400 * 30, COOKIEPATH, COOKIE_DOMAIN);

    //если человек кликнул - $data: 1, а при повторном клике (по той же иконке) - $data: 0
    wp_send_json_success(count($data));
}

Comments

[Genri_Rus]

Спасибо за ответ

Правда в вашем случае почему-то кука перманентно удалена:

[nokimaro]

Оптимизация в том что так как функция в результате работы вызывает setcookie и wp_send_json_success, то можно убрать повторы вызовов и сделать логику более простой и прямой
1. получить $value
2. считать wp_id из cookie если они заданы
3. добавить $value в список или удалить из списка
4. setcookie + wp_send_json_success

[nokimaro]

Genri_Rus, я отредактировал ответ. Добавление $value вынес из условия if(isset($_COOKIE['wp_id']))

[Genri_Rus]

nokimaro, да сейчас заработало

Но как быть со счетчиком ?

Т.е. человек кликнул - данные обновились, а второй раз счетчик все время не менятся:



data: 1, а при повторном клике - data: 0

[nokimaro]

Genri_Rus, я написал про этот не понятный мне момент в ответе.
В вопросе указано что если $data пустой, то всё-равно надо послать "1", а выходит для пустой $data надо послать 0?

посылать реальное кол-во данных в $data (от 0 до ...)
wp_send_json_success(count($data));

посылать кол-во данных в $data + 1 (от 1 до ...)
wp_send_json_success(count($data)+1);

[nokimaro]

В вашем случае всегда 1 так как
1. добавляем значение в $data и выводится count($data) = 1
2. повторный клик - удаляет значение из $data. Получается $data пустой но в описании вопроса сказано

$data пустой, то $counter = 1;

то есть опять будет 1

[Genri_Rus]

nokimaro, да, если человек кликнул - $data: 1, а при повторном клике (по той же иконке) - $data: 0

[Genri_Rus]

nokimaro, я ошибся, нужно именно $data = 0 при повторном клике

[nokimaro]

Genri_Rus, это значит всегда выводить реальное кол-во элементов в $data, если пустой то 0

wp_send_json_success(count($data));

[Genri_Rus]

nokimaro, да спасибо, вот так все отлично работает

function add_to_id()
{
    if(isset($_POST['variation_id']) && !empty($_POST['variation_id']))
    {
        $value = intval($_POST['variation_id']);
    }
    elseif(isset($_POST['product_id']) && !empty($_POST['product_id']))
    {
        $value = intval($_POST['product_id']);
    }
    else
    {
        wp_send_json_error();
    }

    $data = [];
    if(isset($_COOKIE['wp_id']))
    {
        foreach(explode(',', $_COOKIE['wp_id']) as $val)
        {
            if(!empty($val))
            {
                $data[$val] = $val;
            }
        }
    }
    
    //удаляем value если оно было среди значение в cookie
    if(isset($data[$value]))
    {
        unset($data[$value]);
    }
    else
    {
        $data[$value] = $value;
    }

    setcookie("wp_id", implode(',', $data), time() + 86400 * 30, COOKIEPATH, COOKIE_DOMAIN);
   wp_send_json_success(count($data));
}

[Genri_Rus]

nokimaro, а вот добавить класс иконке сердца лучше во фронтенде ?

Допустим если человек повторно кликнул и удалил из куки избранный товар, то тогда убирать класс ?