Red Baron: Bitte helfen!

Hallo, kann mir bitte jemand sagen, warum der folgende Code nicht funktioniert? Es wird immer die Farbe "rot" gesetzt.

<html>
<head>
<title>Test</title>

<script type="text/javascript">

function Farbe (R, G, B) {
  this.R = R;
  this.G = G;
  this.B = B;
  this.hex = "#";
}

function HintergrundWechseln() {
  var Hintergrund = new Farbe("document.form.EingabeR.value", "document.form.EingabeG.value", "document.form.EingabeB.value");
  document.bgcolor = Hintergrund.hex + Hintergrund.R + Hintergrund.G + Hintergrund.B;
}
</script>

</head>
<body bgcolor="#000000">

<form name="form" action="">
<input type="text" name="EingabeR" size="2" maxlength="2">
<input type="text" name="EingabeG" size="2" maxlength="2">
<input type="text" name="EingabeB" size="2" maxlength="2">

<input type="button" value="Farbe wechseln" onclick="HintergrundWechseln()">
</form>

</body>
</html>

  1. Hi,

    Bitte wähle künftig ein aussagekräftiges subject.

    var Hintergrund = new Farbe("document.form.EingabeR.value"...

    Du übergibst hier strings anstelle der Formulardaten.

    Gruesse, Joachim

    --
    Am Ende wird alles gut.
    1. Und wie ändere ich das?

      1. Hi,

        Und wie ändere ich das?

        wie ich das sehe solltest Du Dich dringend mal ein bisschen einlesen. Die Doku hier ist prima ;-)

        "String"
          Variable

        Probiers mal ohne " und beachte mein zweites Posting.

        Gruesse, Joachim

        --
        Am Ende wird alles gut.
        1. OK, Danke erstmal. Hab vor 30 Minuten mit Javascript angefangen.

          1. Hallo Red Baron.

            OK, Danke erstmal. Hab vor 30 Minuten mit Javascript angefangen.

            Ungeachtet dessen solltest du beim nächsten Thread einen geeigneteren Titel wählen. „Bitte helfen!“ hat einen Aussagewert gleich Null.

            Einen schönen Dienstag noch.

            Gruß, Mathias

            --
            sh:( fo:} ch:? rl:( br: n4:~ ie:{ mo:| va:) de:> zu:} fl:( ss:) ls:[ js:|
            „It is required that HTML be a common language between all platforms. This implies no device-specific markup, or anything which requires control over fonts or colors, for example. This is in keeping with the SGML ideal.“
            [HTML Design Constraints: Logical Markup]
    2. Hi,

      und Deine Referenz solltest Du ändern:
      document.getElementsByTagName("body")[0].style.backgroundColor = Hintergrund.hex + Hintergrund.R + Hintergrund.G + Hintergrund.B;

      Gruesse, Joachim

      --
      Am Ende wird alles gut.