2

Estoy haciendo unas practicas ya que estoy aprendiendo POO, pero me quedo atorado en una clase simulando un carrito, el codigo de la clase es:

class ShopingCart
{   
    private $contenidoCarrito;

    function __construct(){
        $this->contenidoCarrito = array();
    }

    public function expressAdd($sku){
        $addItem = array('sku' => $sku, 'qty' => '1');
        $this->contenidoCarrito = array_push($this->contenidoCarrito, $addItem);
    }

    public function showCart(){
        print_r($this->contenidoCarrito);
    }
}

La intención es generar un array donde colocaría los sku y cantidad de los productos, al instanciarlos puedo ver que el array addItem se creo correctamente, pero tras hacer el array_push no hace nada, la funcion showCart siempre me muestra un array vacio.

La estructura de archivos es: config -> Esta disponible en todas las páginas y lo incluyo mediante require, en este archivo genero el objeto $carrito

require_once __DIR__.'/modules/ShopingCart.php';
$carrito = new ShopingCart;

catalogo -> Muestra los productos de la DB y tiene un botón que permite agregar un solo item del producto

class ListarProductos {

    public function listado(){
        $database = new ConecctionDataBase;
        $conexion = $database->conectar();

        $sql = "SELECT * FROM careyou_catalogo JOIN catalogo_inventario ON careyou_catalogo.id = catalogo_inventario.productoId AND disponible=1";
        $consulta = $conexion->query($sql);

        while ($data = $consulta->fetch_assoc()) {
            printf("
                        <div class='card'>
                            <img src='assets/catalogo/{$data['imagen']}' class='card-img-top img-fluid' alt=''>
                            <div class='card-body'>
                                <h5 class='card-title'>{$data['nombre']}</h5>
                                <p class='card-text'>{$data['descripcion']}</p>
                            </div>
                            <div class='card-footer'>
                                <span id='evaluate'>
                                    <button type='button' id='{$data['sku']}' class='btn btn-purple' value='{$data['sku']}'>Ver Detalles</button>
                                </span>
                                <span id='addToCart'>
                                    <button type='button' id='{$data['sku']}' class='btn btn-info' value='{$data['sku']}'><i class='fas fa-cart-plus'></i> Agregar al Carrito</button>
                                </span>
                            </div>
                        </div>
            ");
        }

        $consulta->free();
        $conexion->close();
    }
}

additem.js -> Pasa el valor del botón Añadir a un php donde llamo al método expressAdd y le paso el valor del sku. Ej. 0001000002

$("#addToCart button").click(function(){
        var addSku = $(this).val();

        $.ajax({
            url: '../cart.php',
            type: 'POST',
            dataType: 'html',
            data: {skuCode: addSku, type: 'express'},
            success: function(itemAdded){
                console.log(itemAdded);
            }
        })

    })

cart -> Recibe el valor de AJAX y evalua que metodo usar mediante switch

require_once __DIR__.'/config.php';

/**
 * Controlador del carrito de compras
 */
$sku = limpiar($_POST['skuCode']);
$typ = limpiar($_POST['type']);

switch ($typ) {
    case 'express':
        var_dump($sku);
        $carrito->expressAdd($sku);
        break;

    default:
        # code...
        break;
}

debug -> es un archivo que su única función es la de llamar al método showCart

require_once __DIR__.'/../config.php';
require_once __DIR__.'/../modules/ShopingCart.php';

var_dump($carrito->showCart());
2
  • Agrega como haces la instancia
    – user128299
    el 18 may. 2020 a las 23:04
  • Creo primero el objeto $carrito = new ShopingCart; y posteriormente hago la instancia de la función con $carrito->expressAdd($sku);
    – Nat Najera
    el 18 may. 2020 a las 23:59

1 respuesta 1

0

Considero que tu ejercicio puedes hacerlo así:

  • Declaras como propiedades tipadas (desde PHP 7.4) de la clase a:
    • contenidoCarrito
    • sku
    • addItem
  • Posterior como el método expressAdd() solo se encargará de tomar los valores de un array y con ellos llenar otro y dar el valor de $sku a la clave $sky entonces que sea void
  • Nuestro siguiente método que se llama showCart() devolverá un valor de tipo array y solo haremos un retorno de la propiedad $contenidoCarrito
  • No tiene caso un método constructor pues no es la clase en si la que espera un valor que asignar a $skusino un método interno de está que es la que lo va a usar
  • Continuando con lo anterior, lo único que deberás hacer es a dicha propiedad igualarla a un array vacío
  • A pesar de que en mi anterior propuesta lo había hecho, no veo sentido de que la propiedad $contenidoCarrito sea igualada a la acción que ejecuta array_push; es más sencillo y funcional solo usar este método dentro de expressAdd()

Código versión PHP 7.4:

<?php
declare(strict_types = 1);

class ShopingCart
{
    public array $contenidoCarrito = [];
    public int   $sku;
    public array $addItem;

    public function expressAdd(int $sku): void
    {
        $this->addItem = array("sku" => $sku, "qty" => 1);
        array_push($this->contenidoCarrito, $this->addItem);
    }

    public function showCart():array
    {
        return $this->contenidoCarrito;
    }
}

$carrito = new ShopingCart;
$carrito->expressAdd(12);

var_dump($carrito->showCart());

Con esta salida:

array(1) {
  [0]=>
  array(2) {
    ["sku"]=>
    int(12)
    ["qty"]=>
    int(1)
  }
}

Código versión PHP 7.3.5

<?php 

class ShopingCart
{
    public $contenidoCarrito = [];
    public $sku;
    public $addItem;

    public function expressAdd(int $sku)
    {
        $this->addItem = array("sku" => $sku, "qty" => 1);
        array_push($this->contenidoCarrito, $this->addItem);
    }

    public function showCart()
    {
        return $this->contenidoCarrito;
    }
}

$carrito = new ShopingCart;
$carrito->expressAdd(12);

var_dump($carrito->showCart());

La salida seguirá siendo la misma

8
  • Implementé el código que mencionas, pero me ocasiona un error, si le quito el tipo a los argumentos trabaja, pero al mostrar el carrito lo muestra como vacío como si no se hiciera el array_push.
    – Nat Najera
    el 19 may. 2020 a las 14:19
  • @NatNajera que versión de PHP usas?, usando el código tal cual lo puse te da error?
    – user128299
    el 19 may. 2020 a las 14:19
  • Si, para ser mas específico me genera: Parse error: syntax error, unexpected 'array' (T_ARRAY), expecting function (T_FUNCTION) or const (T_CONST) in . La versión de PHP es 7.3.11
    – Nat Najera
    el 19 may. 2020 a las 14:21
  • Es por tu versión de PHP @NatNajera deja edito
    – user128299
    el 19 may. 2020 a las 15:15
  • Listo @NatNajera ya edité y dejé tanto la primer respuesta como una que te funcione en tu versión de PHP
    – user128299
    el 19 may. 2020 a las 15:22

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.