Betriebsblind...Script läuft nur ein mal

[Struppi]

2. Ein kleines JavaScript (wirklich winzig):

function lichtaus(NAME1)
{
if (document.getElementById(NAME1).style.color == "#CCCCCC")
    {
     document.getElementById(NAME1).style.color = "#FFCC00";
    }
else
    {
    document.getElementById(NAME1).style.color = "#CCCCCC";
    }
}

1. du solltest mit Variabeln arbeiten, damit sparst du dir Tipparbeit und Overhead durch mehrfachen Funktions aufruf.

2. Bei Fehler solltest du Kontrollieren, das das was du erwartest auch passiert, bzw. die Werte überprüfen.

3. nur als Tipp, es gibt beim Programmmieren Konventionen, die es Programmieren leichter machen sollen Programme zu entschlüsseln. z.b. kennzeichnen Variabeln in Großbuchstaben normalerweise Konstanten.

Also hier ein Ansatz der dir helfen hätte können den Fehler zu finden:

var DEBUG = true;

function lichtaus(name)
{

if(typeof document.getElementById == 'undefined')
return alert('dein Browser ist zu alt für meine Seite');

var obj = document.getElementById(name);

if(!obj || !obj.style) return alert('oops');

if(DEBUG) alert(obj.style.color);

obj.style.color = (obj.style.color.toLowerCase() == '#ccc') ? '#FC0' : '#ccc'

}

Struppi.