[SOLUCIONADO] PHP OO Duda sobre como llamar correctamente a un metodo de una clase desde otra

[matiD]

Como están?

Estoy comenzando con la POO, y me ha surgido una duda, me puse a leer en internet, y encuentro muchas respuestas y varias se contradicen entre sí. Otras explican que no se debe hacer de tal manera porque, por más que funcione correctamente no es lo adecuado en base a las buenas técnicas de programación.

Bien seguramente mi duda es bien simple para los que saben o tienen un poco de práctica en el tema.

Tengo un sitio web en el cual estoy creando un sistema login y registro, el mismo tiene las contraseñas encriptadas con un Salt aleatorio que creo para cada usuario al momento del registro.

Bien en el archivo clases/Usuario.php tengo la siguiente clase

Código PHP:

<?php
class Usuarios
{
    public $idusuario = null;
    public $usuario = null;
    public $pass = null;
    public $nombre = null;
    public $apellido = null;
    public $mail = null;
    public $tipousuario = null;
    public $idlocal = null;
    public $esadmin = null;
    public $puntos = null;
    public $salt = null;
    
    public function __construct($datos = array()) 
    {
        if (isset($datos['usuario'])) 
        {
            $this->usuario = stripslashes(strip_tags($datos['usuario']));
        }
        if (isset($datos['pass']))
        {
            $this->pass = stripslashes(strip_tags($datos['pass']));
        }
    }
    
    
    public function loginUsuario()
    {...63 lines }
 
    //Funcion de registro de usuarios
    public function registraUsuario()
    {
        $success = false;
        try
        {
            //Tomamos los valores del post
            $this->usuario = $_POST['usuario'];
            $this->pass = $_POST['pass'];
            $this->nombre = $_POST['nombre'];
            $this->apellido = $_POST['apellido'];
            $this->mail = $_POST['mail'];
            $this->tipousuario = $_POST['tipousuario'];
            $this->idlocal = $_POST['idlocal'];
            $this->esadmin = $_POST['esadmin'];
 
            //Nos conectamos a la base de datos
            $con = new mysqli(DB_HOST, DB_USUARIO, DB_PASSWORD, DB_DATABASE);
            //Preparamos el query SQL
            $sql = 'INSERT INTO usuarios VALUES (usuario, contrasena, salt, nombre, apellido, mail, tipo_usuario, id_local)'
                    . 'VALUES (?,?,?,?,?,?,?,?)';
            //Creamos la sentencia preparada a utilizar
            $pre = $con->prepare($sql);
            $pre->bind_param('ssssssii', $this->usuario, $this->pass, $this->salt, $this->nombre, $this->apellido,  $this->mail, $this->tipousuario, $this->idlocal);
            $pre->execute();
           
            $con->close();
            
            $success = true;
            
            return $success;
            
        } catch (Exception $ex) {
            return $ex->getMessage();
        }
    
    }
 

Y tengo otra clase que genera contraseñas que se encuentra en el archivo 'clases/generaSalt.php'

Código PHP:

<?php
 
class generaSalt
{
    public $longitud;
    public $cadena;
    public $salt;
    
        public function crearSalt(){
    //Se define una cadena de caractares. Te recomiendo que uses esta.
    $this->cadena = "ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz1234567890";
    //Obtenemos la longitud de la cadena de caracteres
    $longitudCadena=strlen($this->cadena);
 
    //Se define la variable que va a contener la contraseña
    $this->salt = "";
    //Se define la longitud de la contraseña, en mi caso 10, pero puedes poner la longitud que quieras
    $this->longitud=50;
 
    //Creamos la contraseña
    for($i=1 ; $i<=$this->longitud ; $i++){
        //Definimos numero aleatorio entre 0 y la longitud de la cadena de caracteres-1
        $pos=rand(0,$longitudCadena-1);
 
        //Vamos formando la contraseña en cada iteraccion del bucle, añadiendo a la cadena $pass la letra correspondiente a la posicion $pos en la cadena de caracteres definida.
        $this->salt .= substr($this->cadena,$pos,1);
    }
    return $this->salt;
}
}
 

Bien ahora viene mi duda.
Quiero utilizar correctamente el método 'crearSalt' de la clase generarSalt, en el método registraUsuario de la clase Usuarios

Como podría introducirlo correctamente dentro del try de registraUsuarios?

Muchas gracias!

[Heli0s]

No importa que estés dentro de una clase, puedes usar la clase salt igualmente

Código PHP:

  $var = new generaSalt();
$var->crearSalt();  
   

Es un ejemplo, otra forma seria crear la función crearSalt como estática, lo cual haría que la puedas usar sin instanciar un objeto.

Código PHP:

  $var = generaSalt::crearSalt()  
   

Aunque en este caso no podrías usar los atributos de la clase.

Un saludo

[matiD]

Iniciado por Heli0s

Código PHP:

  $var = generaSalt::crearSalt()  
   

Aunque en este caso no podrías usar los atributos de la clase.

Es decir que si lo único que voy a utilizar es el resultado del Salt esta sería la forma más adecuada? no necesito los atributos solo el resultado de la función.

obviamente tengo que incluir el archivo que contiene la clase generarSalt, dentro del de Usuarios.

Una aclaración al margen, No agrego la funcion directamente en la clase usuarios, por motivo de que en otra opción del sitio cuando siga avanzando en su programación voy a necesitar una clave aleatoria de 5 digitos, entonces lo que voy a hacer es modificar la función y agregarle el parametro longitud para que cuando la utilice en registraUsuarios tenga 50 de longitud y cuando la vaya a utilizar en la otra parte del sitio le pondré 5.

Muchas gracias por la respuesta!!!

[Heli0s]

Iniciado por matiD

Es decir que si lo único que voy a utilizar es el resultado del Salt esta sería la forma más adecuada? no necesito los atributos solo el resultado de la función.
Exacto

obviamente tengo que incluir el archivo que contiene la clase generarSalt, dentro del de Usuarios.

Una aclaración al margen, No agrego la funcion directamente en la clase usuarios, por motivo de que en otra opción del sitio cuando siga avanzando en su programación voy a necesitar una clave aleatoria de 5 digitos, entonces lo que voy a hacer es modificar la función y agregarle el parametro longitud para que cuando la utilice en registraUsuarios tenga 50 de longitud y cuando la vaya a utilizar en la otra parte del sitio le pondré 5.
Mira las funciones require_once e include_once, te pueden interesar

Muchas gracias por la respuesta!!!

Un saludo

[matiD]

Heli0s Muchas gracias por todo, si lo tengo como include_once. Y ya está funcionando perfectamente!!!

Realmente tengo mucho lio sobre cuando usar include, include_once, requiere_once... no logro entender bien la diferencia pero bueno por ahora todo va funcionando.

Te hago una consulta, a ver si inicio un nuevo tema por esto o no.

La forma de tomar los valores de POST y asignarselo a los atributos de la clase Usuarios es correcta o hay alguna forma recomendada de hacerlo?

[Heli0s]

Correcto es, lo que le falta es muchisima seguridad, pero te recomiendo que para eso abras otro tema para que esté todo mejor organizado.

Un saludo