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