Cómo puedo llamar a un método dentro de la misma clase?

[neodani]

Buenas noches,

Estoy atascado en un punto. Tengo mi clase usuario definida, la cual tiene varios métodos, pero me gustaría poder llamar dentro de un método a otro que hay en la misma clase. Con un ejemplo creo que se entenderá mejor.

Código PHP:

class Usuario{
 
    private $_nombre;
    private $_usuario;
    private $_password;
    private $_correo;
    private $_pais;
    private $_birthday;
    private $_sexo;
    private $_userReferido;
    private $_userKey;
    private $_con;
        
    public function __construct($DbConexion){
            $this->_con=$DbConexion;
    }
    
    // Registrar nuevo usuario
    public function addNewUser($nombre,$usuario,$password,$correo,$pais,$birthday,$sexo,$userReferido){
        
        // Cargo las variables
        $this->_nombre=$nombre; 
        $this->_usuario=$usuario;
        $this->_password=$password;
        $this->_correo=$correo;
        $this->_pais=$pais;
        $this->_birthday=$birthday;
        $this->_sexo=$sexo;
        $this->_userReferido=$userReferido;
        
        $this->_userKey = randomKeygen($length=10);
        echo $this->_userKey;
        
        // Inserta datos en la TBL_USERS
        $query = "INSERT INTO tbl_user (`user_login`,`user_pass`,`user_activation_key`) VALUES (:username,:password,:user_activation_key)";
 
        $comando = $this->_con->prepare($query);
        $comando->execute(array(':username'=>$this->_usuario,
                                ':password'=>$this->_password,
                                ':user_activation_key'=>$this->_userKey));
        
        // Inserta datos en la TBL_USERS_PERSONAL
 
        // Inserta datos en la TBL_USERS_PREFERENCIAS
 
    }
    
    // Genera un password aleatorio
    function randomKeygen($length=10){
        $key = '';
        list($usec, $sec) = explode(' ', microtime());
        mt_srand((float) $sec + ((float) $usec * 100000));
        
        $inputs = array_merge(range('z','a'),range(0,9),range('A','Z'));
 
        for($i=0; $i<$length; $i++)
        {
            $key .= $inputs{mt_rand(0,61)};
        }
        return $key;
    }
}
 

Dentro del método addNewUser() me gustaría poder utilizar el método randomKeygen() para generar la clave de activación de la cuenta. Pero cuando lo hago así (linea 31), me dice "Fatal error: Call to undefined function randomKeygen()"

¿Sabéis cómo lo tengo que hacer?

Muchas gracias de antemano!

[neodani]

Podría ser así?

$this->_userKey = self::randomKeygen($length=10);

o así?

$this->_userKey = Usuario::randomKeygen($length=10);

Lo hago bien?

Qué diferencia hay entre usar la primera o la segunda?

Muchas gracias de antemano

[maycolalvarez]

$this->_userKey = $this->randomKeygen(10);

con respecto a self usalmente se usa para hacer referencia a constantes y miembros estáticos

si usas Usuario:: es como si usaras self, a diferencia que afectaría si decides cambiar el nombre de la clase

[neodani]

Iniciado por maycolalvarez

$this->_userKey = $this->randomKeygen(10);

con respecto a self usalmente se usa para hacer referencia a constantes y miembros estáticos

si usas Usuario:: es como si usaras self, a diferencia que afectaría si decides cambiar el nombre de la clase

Con tu respuesta no me quedo seguro, si hago bien usándolo así o mejor la que propones?

$this->_userKey = $this->randomKeygen($length=10);

Gracias

[GatorV]

Es mucho mejor usar $this-> si estas llamando a la propía instancia, ya que si lo haces con self:: o el nombre de la clase no va a funcionar, mira el siguiente ejemplo:

Código PHP:

class Test
{
        private $_foo;
 
        public function __construct() {}
 
        public function viewFoo1()
        {
                   echo $this->getFoo();
        }
 
        public function viewFoo2()
        {
                   echo self::getFoo();
        }
 
        public function setFoo($foo)
        {
                    $this->_foo = $foo;
        }
 
        public function getFoo()
        {
                   return $this->_foo;
        }
}
 
$fooObj = new Test();
$fooObj->setFoo('bar');
$fooObj->viewFoo1();
$fooObj->viewFoo2();
 

Ejecuta el ejemplo y ve el resultado y te podrás dar cuenta el porque es necesario usar $this para llamar a métodos de la misma instancia.

Saludos.

[neodani]

Iniciado por GatorV

Es mucho mejor usar $this-> si estas llamando a la propía instancia, ya que si lo haces con self:: o el nombre de la clase no va a funcionar

Precisamente probé mi código de las 3 formas y funcionó. No dio ningún error y los valores se insertaron correctamente en la BD

Iniciado por GatorV

Ejecuta el ejemplo y ve el resultado y te podrás dar cuenta el porque es necesario usar $this para llamar a métodos de la misma instancia.

Saludos.

Ejecuté el código de ejemplo y me pintó 2 veces seguidas "bar" por lo que creo que también funcionó de las dos maneras.

Aunque usaré la que me recomendáis tu y maycolalvarez $this->

Muchas gracias a los dos!

[GatorV]

Cierto, al parecer dentro de la clase es indiferente ya que trata de resolverla a la misma clase, aunque no es el comportamiento recomendado es mejor usar $this.

El error se presenta si lo llamas desde afuera de la función.

Saludos.