Ver Mensaje Individual
  #12 (permalink)  
Antiguo 16/04/2002, 11:47
Avatar de caricatos
caricatos
Moderador
 
Fecha de Ingreso: abril-2002
Ubicación: Torremolinos (Málaga)
Mensajes: 19.607
Antigüedad: 22 años, 10 meses
Puntos: 1284
Re: Compartiendo script (para mejorarlo)

Hoal Aleph:
Probé el código y encontre un fallito pequeño:
for (x = 0; x < texto.length; x++)
{
letras[x] = texto.charAt(x);
}

// crea array con cada letra de la clave
var pass = new Array();
for (x = 0; x < clave.length; x++)
{
pass[x] = clave.charAt(x);
}

Te falta el "this.texto.charAt(x)"
y "this.clave.charAt(x)".

Me extrañó que se comiera las mayúsculas en lugar de convertirla, y es que referenciabas al texto y clave que pasabas como parámatro.

Tal vez sea mejor que no repitas el nombre de las variables ¿?

Otra cosilla, he visto que algunas cosas se repiten en dos de las funciones, supongo que podrías poner una función aparte, por ejemplo:
function chequeo (cadena) {
if (typeof(cadena) != "string"){ alert ("El texto debe ser una cadena de caracteres"); return false;

}

Pero eso no tiene importancia, lo que pasa es que cuando estudié informática, casi todos los profes nos repetían eso de:

Divide y vencerás

Un saludo