Como asignar valores a propiedades dentro de una clase

[juandedios]

Estoy trabajando con PHP 7 y tengo problemas al asignar valores a las propiedades.

Código PHP:

class miClase {
 
   public propiedad1;
   public propiedad2;
 
   public static function miMetodo($arg) {
      $this -> propiedad1 = $valor1;
      $this -> propiedad2 = $valor2;
   }
 
}
 

Y me sale esto:
PHP Fatal error: Using $this when not in object context in "la_ruta_del_archivo"

La línea que me marca es donde dice

Código PHP:

  $this -> propiedad1 = $valor1;  
   

Que estoy haciendo mal?

[prueba230683]

Desde un método estático no puedes hacer referencia a la variable $this. Si necesitas utilizar la variable $this entonces el método no es estático. Luego:

Código PHP:

class miClase {
 
   public $propiedad1;
   public $propiedad2;
 
   public function miMetodo($arg) {
      $this -> propiedad1 = $valor1;
      $this -> propiedad2 = $valor2;
   }
 
}
 

PD: los nombres de las propiedades deben ir con $.

[Triby]

Habría que ver un poco más de tu clase y cómo la manejas, supongo que debes tener alguna instancia de la clase, tal vez usando patrón Singleton. Pero, en general, para usar métodos estáticos, también las propiedades deberían ser estáticas y acceder a ellas con self:: en lugar de $this->

Código PHP:

class miClase {
     
       public static $propiedad1;
       public static $propiedad2;
     
       public static function miMetodo($arg) {
          self::$propiedad1 = $valor1;
          self::$propiedad2 = $valor2;
       }
     
    }
 

Esto debería funcionar, sin embargo, podría no ser lo más adecuado, todo depende de cómo tengas el resto de tu clase.

[juandedios]

Intenté como lo sugeriste y me bota error, la clase va así

Código PHP:

class ControladorCarrito {
 
    public $cart = array();
    public $codigo = "";
    public $nombre = "";
    public $marca = "";
    public $modelo = "";
    public $talla = "";
    public $imagen = "";
    public $categoria = "";
    public $cantidad = 1;
    public $precio = "";
 
    public function __construct() {
        parent::__construct();
 
        if (isset($_SESSION['cart'])) {
            $this -> cart = $_SESSION['cart'];
        }
    }
 
    public static function ctrSearchProduct($valor) {
        $tabla = "productos";
        $item = "codigo";
        $valor = $valor;
 
        $producto = ModelosProductos::mdlMostrarProducto($tabla, $item, $valor);
 
        $status = 0;
 
        if ($producto) {
            $this -> codigo = $producto["codProducto"];
            $this -> nombre = $producto["nomProducto"];
            $this -> marca = $producto["marProducto"];
            $this -> modelo = $producto["modProducto"];
            $this -> talla = $producto["talProducto"];
            $this -> imagen = $producto["imgProducto"];
            $this -> categoria = $producto["catProducto"];
            $this -> cantidad = 1;
            $this -> precio = $producto["pveProducto"];
 
            $status++;
        }
 
        return $status;
        
    }
}
 

Así como está, me bota error con el $this en la parte donde empiezo a asignar los valores en el método ctrSearchProduct

Cuando declaro las propiedades con static y asigno los valores como sugeriste:

Código PHP:

  self::codigo = $producto["codigo"];  
   

Me bota un error diciendo que el "=" no se puede usar

[Triby]

Sí, error mío, se debe agregar $ en las propiedades, por ejemplo:

Código PHP:

self::$codigo = $producto["codigo"];
 

Cuál es la idea de tener un constructor para inicializar la clase y usar métodos estáticos (donde no hay un objeto o instancia de clase de por medio)?

[prueba230683]

Cambia

Código PHP:

public static function ctrSearchProduct($valor)
 

por:

Código PHP:

public function ctrSearchProduct($valor)
 

En tu código, no tiene sentido marcarla como estática, si la estás utilizando para modificar propiedades del objeto. Luego en tu código tendrías que invocarla:

Código PHP:

$carr = new ControladorCarrito();
$carr->ctrSearchProduct(123)