Kom væk fra document.write()

getElementsByClassName virker vist bare ikke i IE8 det lidt noget øv.

Så brug getElementsByTagName('a') og check om tagget har css-klassen

Er det ikke vildt ueffektivt at skulle hente alle links ?

Kommer an på hvor mange du har. Men getElementsByClass henter jo i princippet alle tags (ved godt at de nye browsere selv sørger for at hente dem hurtigt). Men hvis du skulle lave din egen getElementsByClass funktion til de gamle browsere, skulle du jo alligevel hente ALLE elementer.
Så der har du jo "sparet" lidt, ved kun, at vælge alle links
Håber det giver mening

 var elements;
 if (document.querySelectorAll) {
   elements = document.querySelectorAll('.insertEmail');
 } else if (document.getElementsByTagName) {
   elements = document.getElementsByTagName('insertEmail');
 } else {
   var spans = document.getElementsByTagName('span'), i, length;
   elements = [];
   for (i = 0, length = spans.length; i < length; i++) {
     if (spans[i].className.indexOf('insertEmail') >= 0) {
       elements.push(spans[i]);
     }
   }
 }

Så er du også helgarderet.

edit:
document.querySelectorAll er understøttet i IE8:
http://caniuse.com/queryselector

Nu har jeg lavet det sådan her.

 <span class="foratundgaspam"></span>

 function visemailadresse2() {
 
    if(document.getElementsByClassName){
 
       var elementer = document.getElementsByClassName('foratundgaspam');
 
       var tekst = 'em@il';
    
       for (var i = 0, length = elementer.length; i < length; i++) {
    
          elementer[i].innerHTML = tekst;
    
       }
 
    } else {
 
       var tekst = 'em@il';
 
       document.write(tekst);
 
    }
 
 }
 
 if (window.addEventListener) {
 
    window.addEventListener('load', visemailadresse2, false);
 
 } else if (window.attachEvent) {
 
    window.attachEvent('onload', visemailadresse2);
 
 }

Har i nogen forbedringer til den måde at gøre det på ?

Hvad forskel er der ved at bruge querySelectorAll og getElementsByClassName ? Andet end lidt forskellig understøttelse.

Nu hopper teskten på hjemmesiden bare fordi emailadressen først tilføjes ved 'load'. Før med document.write der stod den der hele tiden uden at teksten hopper.

Du ved reelt ikke om du får lidt spam fordi dit nuværende javascript virker eller om de er fordi dit spamfilter er effektivt.

Jeg tror jeg bare havde skrevet det i klar tekst og stolet på spamfilteret eller slettet de mails og så brugt tiden du nu bruger på at omgå det (uden at vide om det er nødvendigt) på noget mere konstuktivt.

Alternativt kan du jo bare indsætte et billede af din email.

scootergrisen: Det giver ingen mening, for de fleste mail-fetchers benytter en (halv-avanceret) regular expression direkte på kildekoden. Så hvis du skriver din email i klartekst, er det ligemeget hvor på siden den kommer til at være: Om det er i et script tag eller et span tag er ligegyldigt.

Som Brian siger, kunne du evt. sætte et billede ind, men det hjælper vidst kun mod de mest rudimentære, da de fleste bots har indbygget OCR og for at undgå det skal du lige pludseligt i gang med CAPTCHAs og så ryger brugervenligheden for alvor!

Spam-filtre er effektive - Og de fleste benytter en Machine Learning algoritme, der lærer at genkende spam, så når du flagger de første 10 spam mails, så har den allerede lært at ignorere dem.

Ang. querySelectorAll / getElementsByClassName, er querySelectorAll meget mere divers. Den tillader dig at søge i DOM træet ved hjælp af CSS selectors, som .classname og #id. getElementsByClassName har kun een funktion og det er at finde alle de DOM elementer med det givne className.

Emailen skal ikke være et billede fordi det skal ikke være besværligt for alimindelige folk at skrive til mig direkte.

Jeg skriver ikke min email i direkte i koden, jeg skriver ligesom sådan her : mailt... osv
Også fordelt på flere linier.
Også kunne jeg også fordele det på flere variabler for at gøre det lidt mere svært.

Ja jeg gider ikke CAPTCHA på grund af det afholder folk fra at skrive.

Jeg har gmail og modtager lidt i spam mappen men der er måske også mere end det som gmail sorter fra som jeg aldrig ser, der ved jeg ikke.