Lass dich hier nicht anquatschen, so schlecht ist der Code nicht, dass kommt mit der Zeit dann wird der Code immer schlanker! Es gibt immer ein paar die meinen sie sind was besserers! Jeder Programmierer ist ein Künstler und jeder schreibt seinen Code so wie er meint!
Klar, mit überflüssigen Befehelen (bzw. Fehlern) und dafür wird nicht getetstet ob das was man will auch funktioiert. Das hat nichts mit 'etwas besseres' zu tun, sondern mit Fehlersuche und Fehlervermeidung.
Es hat aber nichts damit zu tun, ob der code schlank ist oder nicht.
<SCRIPT LANGUAGE="JavaScript">
das ist veraltet es heißt:
<SCRIPT type="text/javaScript">
function doPic(imgName) {
document.mainpic.src = imgName;
und wenn man hier testet ob alles funktioniert ist man nie auf der falschen Seite:
if(!document.images) return;
Das ist der Test auf alte Browser, der in ihrem Code auch vorhanden war, aber lediglich geprüft hat ob es ein Netscape >=3 oder ein IE >4 ist, was definitiv Müll ist, weil Opera 3, iCab, IE 4, Netscape 2(?) usw. diesen Code auch ausführen können.
if(!document.mainpic) return;
und dann testet man Sicherheitshalber ob es dann gewünschte Element gibt.
Das hat nichts mit Besserwisserei oder so zu tun, sondern ist enfach sauberer Programmierstil.
Aber letztlich war ja mein Code nicht falsch.
Struppi.