Don P: Kniffel mit Javascript

Beitrag lesen

Hallo,

Da lässt sich noch vieles vereinfachen, z.B. hier:

WuerfelWert = new Object; // 5 Wuerfel: WuerfelWert[0]..WuerfelWert[4]

Warum nimmst du ein Objekt für die Würfel 0 bis 4?
Ein Array wäre besser, weil dann auch das Sortieren einfacher wird. Arrays bringen nämlich eine Sortierfunktion mit => kein Bubblesort nötig:

WuerfelWert = new Array(5); // 5 Wuerfel: WuerfelWert[0]..WuerfelWert[4]  

Auch die Würfelbilder wären besser in einem Array aufgehoben statt in einzelnen Variablen. Dann könntest du nämlich einfacher auf sie zugreifen und müsstest nicht viele Fallunterscheidungen machen wie

switch(WuerfelWert[i])
          {
            case 1:
            {
              document.images[i+1].src = wuerfel1.src;
              break;
            }

Wenn ein Würfel nicht berücksichtigt wird, wie du schreibst, dann liegt das vermutlich an einem falschen Index, z.B. 1..4 statt 0..4 oder so ähnlich.

Gruß, Don P