Problema con bucles

_cronos2 · #1 (**permalink**) 04/08/2010, 17:40

Hola, estoy trabajando en un script por puro entretenimiento. Este script sirve, en teoría, para que el usuario introduzca un texto, de más o menos carácteres, en un input, y de este texto se seleccionen x letras para ponerlas en mayúsculas. El problema está en que uso varios bucles para recorrer arrays y carácteres, aquí el código:

Código Javascript:

Ver original<script type="text/javascript">
function alternar(t1, t2){
 var txt = t1.value;
 var aceptados = /[A-z]/;
 var letras = new Array();
 for (a = 0; a<txt.length; a++){
  if(aceptados.test(txt.charAt(a))){
   letras[letras.length] = a;
  }
 }
 var cant = Math.round(Math.random()*letras.length);
 var aleat = new Array();
 for (b = 0; b<cant; b++){
  aleat[b] = Math.round(Math.random()*letras.length);
 }
 for(i = 0; i<cant; i++){
  for(j = 0; j<cant; j++){
   if(aleat[i]==aleat[j] && i!=j){
    if(aleat[j]<letras.length){
     aleat[j] += 1;
    }else{
     aleat[j] -= 1;
    }
   }
  }
 }
 var result = '';
 for(t = 0; t<txt.length; t++){
  if(!(aceptados.test(txt.charAt(t)))){
   result += txt.charAt(t);
  }else{
   for(l = 0; l<letras.length; l++){
    for(r = 0; r<aleat.length; r++){
     if(letras[l] == aleat[r]){
      result += txt.charAt(letras[l]).toUpperCase();
     }else{
      result += txt.charAt(letras[l]);
     }
    }
   }
  }
 }
 t2.value = result;
}
</script>

La cuestión es que si yo por ejempplo pongo 'abc123' (con 3 mayúsculas y posiciones 2, 2 y 3) sale 'aaabbbccCaaabbbccCaaabbbccC123'. Sé que tiene que ser por los bucles porque se repite parte del string pero lo he revisado 40 veces y todavía no logro ver dónde está el error. A ver si me podéis ayudar.
Saludos y gracias :D

Adler · #2 (**permalink**) 05/08/2010, 06:17

Hola

No entiendo que es lo que pretendes hacer. ¿Poner en mayúscula la primera letra de cada palabra? ¿poner en mayúscula la primera letra de cada palabra si tiene mas de 2 caracteres? ¿.....?

Suerte

_cronos2 · #3 (**permalink**) 05/08/2010, 10:13

No, lo que quiero hacer es imitar esas páginas donde entras y te salen varios inputs para que introduzcas un texto tipo nick del messenger o lo que sea, y te lo convierte; y por ejemplo en un input te sale con letras con acentos circunflejos y cosas por el estilo, y hay otro input en el que sale el texto que pusiste pero ciertas letras al azar están en mayúsculas. Es una horterada pero el otro día estaba pensando en ello y pensé que el código sería más fácil de lo que en realidad es o me parece.
Ejemplo: al principio de la página sale 'niCks paRa mSn y LeTraS gRanDeS y raRas paRa meSSegeR síMboLos raRos', eso es a lo que me refiero con letras, al azar, mayúsculas.

Saludos (:

Adler · #4 (**permalink**) 05/08/2010, 15:21

Hola

Algunas modificaciones

Código Javascript:

Ver original<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html xmlns="http://www.w3.org/1999/xhtml"lang="es" xml:lang="es">
<head>
<script type="text/javascript">
function alternar(t1, t2){
 var txt = t1.value;
 var aceptados = /[a-z\s]/;
 var letras = new Array();
 for (a = 0; a<txt.length; a++){
  if(aceptados.test(txt.charAt(a))){
   letras[letras.length] = a;
  }
 }
 
var cant = Math.round(Math.random()*letras.length);
 
 
 
var aleat = new Array();
for (b = 0; b<cant-1; b++){
 aleat[b] = Math.round(Math.random()*letras.length);
 }
 
 
nuevoArray = aleat.sort();
 
var mayuscula = '';
var result = '';
for(t = 0; t<txt.length; t++){
 
    for(z = 0; z < nuevoArray.length; z++) {
     
        if(t == nuevoArray[z]){
            mayuscula = txt.charAt(letras[t])
                result += txt.charAt(letras[t]).toUpperCase();
            break;
            }
 
    }
 
    if(txt.charAt(letras[t]) != mayuscula) {
         result += txt.charAt(letras[t]);
        mayuscula = '';
    }
 
}
 
 
 t2.innerHTML = result;
}
</script>
 
</head>
<body>
<form>
<input type="text" value="un texto cualquiera no importa el largo" id="texto" />
<textarea id="resultado" /></textarea>
<input type="button" value="Traer Formulario" onclick="alternar(document.getElementById('texto'), document.getElementById('resultado'))" />
</form>
</body>
</html>

Suerte

_cronos2 · #5 (**permalink**) 05/08/2010, 17:17

Gracias, el código funciona perfecto con letras, pero tengo problemas con números. Cuando pongo alguno, siempre sale al final, y si pongo más de uno sólo sale el primero. Por lo demás, no entiendo:

En el segundo bucle, por qué pones cant-1
Por qué ordenas el array de números aleatorios

Saludos (:

Adler · #6 (**permalink**) 06/08/2010, 07:10

Hola

Respecto a tus preguntas

Cita:

1ª Por que sino el último elemento del array sería undefined
2ª Lo puedes ordenar o no ordenarlo, no influye en nada, puesto que se ha de recorrer toda la matriz aleat. Con respecto a esto también he hecho alguna modificación para no evaluar indices que ya están evaluados

Fíjate en los cambios con respecto al script anterior

Código Javascript:

Ver original<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html xmlns="http://www.w3.org/1999/xhtml"lang="es" xml:lang="es">
<head>
<script type="text/javascript">
function alternar(t1, t2){
var letras = t1.value.toLowerCase().split('');
 
var cant = Math.round(Math.random()*letras.length);
 
var aleat = new Array();
for (b = 0; b<cant; b++){
 aleat[b] = Math.round(Math.random()*letras.length);
 }
 
ordenaArray = aleat.sort();
 
var aceptados = /[a-z\s]/;
var mayuscula = '';
var result = '';
for(t = 0; t<letras.length; t++){
 
    for(z = 0; z < ordenaArray.length; z++) {
     
        if(t == ordenaArray[z]){
ordenaArray.splice(z,1) // eliminamos este elemento
if (aceptados.test(letras[t])) {
 
 
            mayuscula = letras[t];
                  result += letras[t].toUpperCase();
            break;
} else {
 result += letras[t];
mayuscula = letras[t];
break;
}
 
             }
 
    }
 
 
    if(letras[t] != mayuscula) {
ordenaArray.splice(t,1) // eliminamos este elemento
         result += letras[t];
        mayuscula = '';
    }
 
}
 
 
 t2.innerHTML = result;
}
</script>
 
</head>
<body>
<form>
<input type="text" value="un texto cualquiera con números 12 3 e incluso con caracteres especiales ?¿ no @importa" id="texto" />
<textarea id="resultado" /></textarea>
<input type="button" value="Traer Formulario" onclick="alternar(document.getElementById('texto'), document.getElementById('resultado'))" />
</form>
</body>
</html>

Suerte

_cronos2 · #7 (**permalink**) 06/08/2010, 11:51

Ok, muchas gracias, ahora sí funciona perfecto. Lo único que me queda es entenderlo xD Por cierto, lo del for todavía no me quedó claro. Si por ejemplo cant==5, habría aleat[0], aleat[1], aleat[2], aleat[3] y aleat[4]... ¿por qúe iba a ser undefined aleat[4]? Espero que me puedas responder.
Gracias otra vez :D

Adler · #8 (**permalink**) 06/08/2010, 12:48

Ha tenido que ser que en alguno de los ciclos use "menor o igual", pero en lo que lo he imprimido no lo encuentro, así que nada, lo edito