Problema con script

Morta · #1 (**permalink**) 28/07/2003, 03:18

Hice este código con la idea de que la primera vez seleccione una imagen al azar entre varias y luego, cada 5 segundos cambie de imagen tambien de una forma aleatoria entre esas imágenes. La primera carga se hace bien y tambien la segunda, pero ya no selecciona la tercera imagen y no entiendo por qué.
¿Alguien puede ayudarme?
Muchísimas gracias, un saludo

<script language="javascript">

</script>

biblio · #2 (**permalink**) 28/07/2003, 15:13

Fijate en este enlace

http://clickpe.7host.com/greencard/

Saludos

Kaopectate · #3 (**permalink**) 28/07/2003, 15:25

Hola Morta.

Lo que yo veo que esta raro es el uso que haces del document.write().

Cuando la página se carga inicialmente en el navegador, es el momento apropiado para agragar líneas al programa usando el write() de JavaScript, pero una vez cargada, ya es inutil usar el comando a menos que se ejecute una recarga de un frame entero (por ejemplo).

Me pregunto si lo que necesitas es cambiar el contenido de un único objeto IMG cada cinco segundos, en cuyo caso el problema es facilmente resolvible.

Por otro lado veo extraño que en la función repetir, no le asignas el objeto que genera tu constructor a nadie. ¿Es intencional?

Saludos.

Morta · #4 (**permalink**) 28/07/2003, 16:26

Muchas gracias Biblio, un efecto muy interesante pero creo que adolece de lo que yo pretendo y es que la primera imagen sea aleatoria y luego se vayan cargando una a una todas las imagenes de la serie, me da igual si una a continuación de la otra es decir por orden o bien mejor aleatoriamente, de todas formas repito que es muy interesante ese script.
Un saludo
Morta

Morta · #5 (**permalink**) 28/07/2003, 16:30

Muchas gracias Kaopectate, puede que el script tenga algun fallo por mi parte, la verdad es que no controlo demasiado el tema. De todas formas el script en su primera secuencia funciona correctamente es decir, carga la primera imagen aleatoria y luego la segunda, pero ya no carga la tercera. Te agradecería una solución. El uso de document.write() igual no es correcto.
Un saludo
Morta

Kaopectate · #6 (**permalink**) 28/07/2003, 18:13

Prueba esto a ver si te ayuda, tiene un problema con el alt porque creo que ese atributo no es manejable vía JavaScript, pero como aproximación creo que funciona.

Código PHP:

  <html>

 <head>

  <script language="JavaScript">

 
   var Img = new VecImgs(); 

   var Alt = new VecAlts(); 

 
   function VecImgs(){ 

    n=0; 

    this[n++] = "paginas/fotos/imagen1.jpg"; 

    this[n++] = "paginas/fotos/imagen2.jpg"; 

    this[n++] = "paginas/fotos/imagen3.jpg"; 

    this[n++] = "paginas/fotos/imagen4.jpg"; 

    this[n++] = "paginas/fotos/imagen5.jpg"; 

    this[n++] = "paginas/fotos/imagen6.jpg";

    this.N=n; 

   }

 
   function VecAlts(){ 

    n=0; 

    this[n++]="imagen 1 \n\251 autor 1"; 

    this[n++]="imagen 2 \n\251 autor 2"; 

    this[n++]="imagen 3 \n\251 autor 3"; 

    this[n++]="imagen 4 \n\251 autor 4"; 

    this[n++]="imagen 5 \n\251 autor 5"; 

    this[n++]="imagen 6 \n\251 autor 6";

    this.N=n; 

   }

 
   function repetir(){ 

    pos = Math.floor(Math.random() * Img.N);

    sSrc = Img[pos];

    sAlt = Alt[pos];

    with (document.getElementById("aImg")){

     href = sSrc;

     alt = sAlt;

    }

    document.getElementById("iImg").src = sSrc;

   } 

 
  </script>

 </head>

 <body>

  <script language="JavaScript">

   pos = Math.floor(Math.random() * Img.N);

   sSrc = Img[pos];

   sAlt = Alt[pos];

   document.write("<a id='aImg' href='"+sSrc+"' alt='"+sAlt+"' target='_blank'><IMG id='iImg' SRC='"+sSrc+"' width=186 height=279>"); 

   id = setInterval("repetir()",5000);

  </script>

 </body>

</html>

Saludos.

Morta · #7 (**permalink**) 29/07/2003, 01:14

Muchas gracias Kaopectate, es mas o menos lo que necesito. Lo único es que no entiendo porqué dices que el atributo ALT no es manejable via Javascript ya que en el script que hice yo si que funciona. He intentado probar con tu ejemplo pero no soy capaz de hacer que funcione.
Un saludo

Kaopectate · #8 (**permalink**) 29/07/2003, 08:43

Yo tampoco lo he podido hacer funcionar respecto al ALT.

Te explico:

Lo que haces cuando generas código con document.write() es escribir código HTML antes de que el browser interprete la página, por lo tanto, puedes escribir los comandos tal y como desees que funcione.

El problema es que una vez cargada la página, el document.write() no tiene mayor efecto porque el browser no se va a enterar de que tu agregaste o no unos comandos HTML mas. Por eso funcionaba tu ejemplo con las primeras 2 imágenes, la primera era la que tenías cargada y luego el primer document.write(), luego de eso, la función generaba un comando HTML cada 5 segundos que el browser no veía.

Respecto a mi ejemplo, desde JavaScript puedes manipular el objeto "tipo a" que equivale a tu <a>, le puedes cambiar su href y varias cosas mas, pero no veo que le puedas cambiar el atributo ALT.

Si alguien sabe como, espero que nos lo diga.

Saludos.

Morta · #9 (**permalink**) 29/07/2003, 16:03

Muy agradecido Kaopectate, si parece dificil el conseguirlo
Si doy con ello ya te lo comentaré aunque mis conocimientos de Javascript se están agotando.
Un saludo

Kaopectate · #10 (**permalink**) 29/07/2003, 16:08

No digas eso...estan creciendo...

Beakdan · #11 (**permalink**) 30/07/2003, 09:36

Kaopectate:

Cita:

Si alguien sabe como, espero que nos lo diga.

Del modo en que has hecho el script, colocas lo que debería ser el "alt" de la imagen en el anchor, y si recuerdas, los anchor no tienen atributo "alt", sino "title".
Entonces, todo se reduce a que cambies :

Código:

with (document.getElementById("aImg")){
   href = sSrc;
   //esto se cambia de alt a title
   title = sAlt;
}

y también cambias el "alt" por "title" donde usas document.write.

Ahora, con respecto a cambiar el alt de una imagen... Veo que usan metodos DOM par el acceso a los id's de los elementos. Bueno, podemos usar métodos DOM para manipular los atributos de dichos elementos.

En el caso de la imagen, para cambiar el "alt" (o asignarlo) puedes poner:

Código:

document.getElementById("iImg").setAttribute("alt", sAlt);

Saludos.

Kaopectate · #12 (**permalink**) 30/07/2003, 09:56

¡Gracias Beakdan por todas tus observaciones!!!

Morta · #13 (**permalink**) 30/07/2003, 16:19

Muchas gracias Beakdan y Kaopectate. Ahora ya funciona.
Un saludo, da gusto con la qente que se enrolla bien

Este es el script ya modificado:

<script language="JavaScript">

var Img = new VecImgs();
var Alt = new VecAlts();

function VecImgs(){
n=0;
this[n++]="paginas/fotos/imagen1.jpg";
this[n++]="paginas/fotos/imagen2.jpg";
this[n++]="paginas/fotos/imagen3.jpg";
this[n++]="paginas/fotos/imagen4.jpg";
this[n++]="paginas/fotos/imagen5.jpg";
this[n++]="paginas/fotos/imagen6.jpg";
this.N=n;
}

function VecAlts(){
n=0;
this[n++]="imagen 1 \n\251 autor 1";
this[n++]="imagen 2 \n\251 autor 2";
this[n++]="imagen 3 \n\251 autor 3";
this[n++]="imagen 4 \n\251 autor 4";
this[n++]="imagen 5 \n\251 autor 5";
this[n++]="imagen 6 \n\251 autor 6";
this.N=n;
}

function repetir(){
pos = Math.floor(Math.random() * Img.N);
sSrc = Img[pos];
sAlt = Alt[pos];
with (document.getElementById("aImg")){
href = sSrc;
title = sAlt;
}
document.getElementById("iImg").src = sSrc;
}

<script language="JavaScript">
pos = Math.floor(Math.random() * Img.N);
sSrc = Img[pos];
sAlt = Alt[pos];
document.write("<a id='aImg' href='"+sSrc+"' alt='"+sAlt+"' target='_blank'><IMG id='iImg' SRC='"+sSrc+"' width=186 height=279>");
id = setInterval("repetir()",2000);
</script>

Kaopectate · #14 (**permalink**) 30/07/2003, 16:25

Bueno saber que ya funciona...suerte!!!

Morta · #15 (**permalink**) 31/07/2003, 00:19

Muchas gracias amigos.
Podeís ver vuestro trabajo funcionando en
http://www.barranquismo.sitio.net/
Un saludo

biblio · #16 (**permalink**) 31/07/2003, 00:52

Que bien que lo hayas logrado, ya ví la página.

Saludos