Validazione Banner javascript

cla83

Validazione Banner javascript

Salve a tutti,
spero che qualcuno di Voi possa aiutarmi a correggere il codice di questo script che ho trovato in rete, all'inizio aveva più di 20 errori di validazione e sono riuscito ad arrivare a 3 errori. grazie

il codice è il seguente


<script type="text/javascript">
number_of_sponsors=5;
var sctr=0;
var halt=0;
var isn=new Array();
for (i=0;i<number_of_sponsors,i++){
 isn*=new Image();
}
isn[0].src="sfondi/led/1.jpg";
isn[1].src="sfondi/led/2.jpg";
isn[2].src="sfondi/led/3.jpg";
isn[3].src="sfondi/led/4.jpg";
isn[4].src="sfondi/led/5.jpg";
var durl=new Array();
function rotateIt(){
 if (halt!=1){
  sctr++;
  if (sctr>number_of_sponsors-1){
   sctr=0;
   }
   document.sponsor.src=isn[sctr].src;
  setTimeout("rotateIt()",2000);
  }
 }
function doIt(){
 halt=1;
 location.href=durl[sctr];
 }
function dispIt(){
 parent.window.status=durl[sctr];
 }
</script>

gli errori che ottengo sono questi:

Line 18, Column 30: character ";" not allowed in attribute specification list
for (i=0;i<number_of_sponsors;i++){

Line 18, Column 30: element "number_of_sponsors" undefined
for (i=0;i<number_of_sponsors;i++){

You have used the element named above in your document, but the document type you are using does not define an element of that name. This error is often caused by:

incorrect use of the "Strict" document type with a document that uses frames (e.g. you must use the "Frameset" document type to get the "<frameset>" element),
by using vendor proprietary extensions such as "<spacer>" or "<marquee>" (this is usually fixed by using CSS to achieve the desired effect instead).
by using upper-case tags in XHTML (in XHTML attributes and elements must be all lower-case).
Line 44, Column 9: end tag for "number_of_sponsors" omitted, but OMITTAG NO was specified
</script>

You may have neglected to close an element, or perhaps you meant to "self-close" an element, that is, ending it with "/>" instead of ">".

Line 18, Column 11: start tag was here
for (i=0;i<number_of_sponsors;i++){

ayrton2001

Ciao cla83,
adesso "zeru erruri"


<script type="text/javascript">
var number_of_sponsors=5;
var sctr=0;
var halt=0;
var isn=new  Array();
for (i=0; i<number_of_sponsors;i++){
 isn*=new Image();
}
isn[0].src="sfondi/led/1.jpg";
isn[1].src="sfondi/led/2.jpg";
isn[2].src="sfondi/led/3.jpg";
isn[3].src="sfondi/led/4.jpg";
isn[4].src="sfondi/led/5.jpg";
var durl=new Array();
function rotateIt(){
 if (halt!=1){
  sctr++;
  if (sctr>number_of_sponsors-1){
   sctr=0;
   }
   document.sponsor.src=isn[sctr].src;
  setTimeout("rotateIt()",2000);
  }
 }
function doIt(){
 halt=1;
 location.href=durl[sctr];
 }
function dispIt(){
 parent.window.status=durl[sctr];
 }
</script>

cla83

Ciao ayrton2001,
grazieeee mille, sei un grande

cla83

Ciao Ayrton,
ho provato con la tua modifica però gli errori ci sono sempre

sono questi, allego uno screenshot.

cb83.altervista.org/errori.jpg

ayrton2001

Scusami, non avevo capito che ti riferivi alla validazione W3C.
prova cosi:


<script type="text/javascript">
<!--
var number_of_sponsors=5;
var sctr=0;
var halt=0;
var isn=new  Array();
for (i=0; i<number_of_sponsors; i++){
 isn*=new Image();
}
isn[0].src="sfondi/led/1.jpg";
isn[1].src="sfondi/led/2.jpg";
isn[2].src="sfondi/led/3.jpg";
isn[3].src="sfondi/led/4.jpg";
isn[4].src="sfondi/led/5.jpg";
var durl=new Array();
function rotateIt(){
 if (halt!=1){
  sctr++;
  if (sctr>number_of_sponsors-1){
   sctr=0;
   }
   document.sponsor.src=isn[sctr].src;
  setTimeout("rotateIt()",2000);
  }
 }
function doIt(){
 halt=1;
 location.href=durl[sctr];
 }
function dispIt(){
 parent.window.status=durl[sctr];
 }
// -->
</script>

cla83

Scusami tu ayrton, non ero stato chiaro infatti.
Ora funzione alla perfezione, grazie mille

Giusto per capire, ho visto che hai aggiunto "<!--" per commentare quella parte di codice,
se ho capito bene serve per nascondere quella parte di codice al browser e far si che il validatore non lo vede? grazie ancora

ayrton2001

@cla83 said:

Scusami tu ayrton, non ero stato chiaro infatti.
Ora funzione alla perfezione, grazie mille

Giusto per capire, ho visto che hai aggiunto "<!--" per commentare quella parte di codice,
se ho capito bene serve per nascondere quella parte di codice al browser e far si che il validatore non lo vede? grazie ancora

Diciamo di si, la migliore cosa sarebbe mettere il javascript in un file esterno.

cla83

Grazie per il chiarimento e soprattutto per l'aiuto.

ayrton2001

Figurati.