Funcion generar contraseña

[jeyn]

Hola

Tengo una funcion para generar contraseñas y me funciona bien, el problema es que me la imprime y cuando llamo a la funcion desde el script la variable esta vacia, no se como guardar la contraseña en la variable sin que se queda vacia al llamarla.

No se si me he explicado bien.

Funcion:

Código PHP:

  /*Generar contraseña aleatoria*/
function generar_contrasena($tam=7,$may=FALSE)
{
    
    //El tamaño por defecto es 7 para una longitud de 8,    
    //Tamaño Mínimo
    $min=7;
    //Tamaño Máximo
    $max=14;
        
    if($may === FALSE)
    {
        $cadena='0123456789abcdefghijklmnopqrstuvwxyz';
    }
    else
    {
        $cadena='0123456789ABCDEFGHIJKLMNOPQRSTUVWXYZ';
    }
    if(($tam >= $min)&&($tam <=$max))
    {
        //Generación aleatoria segun sea mayuscula o minuscula.
        for($i=0;$i<=$tam;$i++)
        {
            //Guardamos en un arreglo.
            $pila[]=$cadena{rand(0,25)};
        }
        //Desordenamos el arreglo.
        shuffle($pila);
                
        //Mostramos la contraseña   
        foreach($pila as $letra => $contrasena)
        {
           echo $contrasena;        
        }     
    }
    
    
} 
   

Gracias

Un saludo

[nettolz]

Te dejo mi función que me parece más práctica, yo la uso en todos mis codigos ( cuando lo requiero ).

Código Javascript:

<script>
var caracter="1234567890";
caracter+="QWERTYUIOPASDFGHJKLZXCVBNM";
caracter+="qwertyuioplkjhgfdsazxcvbnm";
var numero_caracteres=10;
var total=caracter.length;
function azar(){
var clave="";
for(a=0;a<numero_caracteres;a++){
clave+=caracter.charAt(parseInt(total*Math.random(1)));
}
document.forms[0].clave.value=clave;
}
</script>
 

[jeyn]

Gracias por contestar, la funcion es en javascript y yo no he trabajado con java, y segun dicen js es mas vulnerable que php, no soy ningun experto, pero la probare a ver que tal.

Gracias

Un saludo

[GrupoC]

Evidentemente, esa función es vulnerable, dado que genera el "hash" de contraseñas de forma rudimentaria (por lo que hay millones de colisiones de contraseñas en un espacio reducido).

No sé porqué reinventar la rueda, cuando existen múltiples algoritmos que han demostrado su eficiencia y la escasas colisiones, como puedan ser: MDX (MD5, la más actual), SHA1, SHA2...

Además, son algoritmos unidireccionales.

Saludos!

[Carlmycol]

Código PHP:

  /*Generar contraseña aleatoria*/
function generar_contrasena($tam=7,$may=FALSE)
{
    
    //El tamaño por defecto es 7 para una longitud de 8,    
    //Tamaño Mínimo
    $min=7;
    //Tamaño Máximo
    $max=14;
        
    if($may === FALSE)
    {
        $cadena='0123456789abcdefghijklmnopqrstuvwxyz';
    }
    else
    {
        $cadena='0123456789ABCDEFGHIJKLMNOPQRSTUVWXYZ';
    }
    if(($tam >= $min)&&($tam <=$max))
    {
        //Generación aleatoria segun sea mayuscula o minuscula.
        for($i=0;$i<=$tam;$i++)
        {
            //Guardamos en un arreglo.
            $pila[]=$cadena{rnd(0, 25)};
        }
        //Desordenamos el arreglo.
        shuffle($pila);
        $retornable='';
        //Mostramos la contraseña   
        foreach($pila as $letra => $contrasena)
        {
           $retornable.=$contrasena;        
        }
        return $retornable;
    }
} 
   

No entendi bien que quieres hacer, supuse que querías retornar la contraseña creada, he modificado un poco el codigo para que lo haga (return $retornable), otra cosa, tienes rand(0, 25), me pareció extraño, pero bueno...

Aún asi, no he entendido bien cual era la duda, cualquier cosa avisa con un post n_n

[jeyn]

Primero muchas gracias

Te explico

La funcion actua correctamente y el echo me da la contraseña generada, pero cuando la llamo desde el script

Código PHP:

  if(generar_contrasena()){
$error.="<b>El Nombre De Usuario No Existe.</b<br>";
}
$contrasenagenerada = md5($contrasena); 
   

La variable $contrasena esta vacia.

cuando ejecuto el scrip lo primero que me sale es el valor de $contrasena

Ejemplo:

45fuatf8 se ha enviad a su correo .........................................

Con lo cual la funcion actua bien y lo primero que escribe es el echo y a continuacon el texto que hay para el usuario, pero cuando envio el correo la variable $contrasena esta vacia.

Y la variable $contrasengenerada tambien, asi que en la BBDD inserta un codigo vacio.

No se si ahora me he explicado algo mejor

Un saludo

[Italico76]

if(generar_contrasena()){

es $generar_contrasena con el signo $

[HackmanC]

Hola,

Seguramente funciona bien tu método, pero, ¿porque no usas el lenguaje PHP?

Código PHP:

  function generar_contrasena($len = 7, $shf = false) {
  $j = array_merge(range('a', 'z'), range('0', '9'));
  if ($shf) {
    $j = array_merge($j, range('A', 'Z'));
  }
  shuffle($j);
  $len = min(max($len, 7), 14);
  return join(array_slice($j, 0, $len), '');
}
 
$contrasena = generar_contrasena();
$contrasenagenerada = md5($contrasena); 
   

Saludos,

[jeyn]

Muchas gracias ya esta solucionado.

Un saludo