Moin
<head>
<script>
sinnvoll wäre <script type="text/javascript">
var contentArray = new Array("fishfingers","Utah","mercurichrome","vespa","flouresce","spantangle","daedal","hogwarmer","spoot","maneouvre","pneumatic","basejumping","beanthrower","catgut","stormpie");
function randomContent(arr)
das Argument arr nutzt du gar nicht, also weglassen
{
arr = contentArray;
hier definierst du es nämlich, aber als globale Variable, obwohl sie nur lokal gebraucht wird.
Besser:
var arr = contentArray;
dann sollten Änderungen an der lokalen Variablen auch keine Folgeschäden beim globalen Array contentArray hinterlassen
alert(contentArray.length);
// remove random elements
while(arr.length>5) {
randy = Math.floor(Math.random()*arr.length);
arr.splice(randy,1)
}contentOutput = "";
for(j=0;j<arr.length;j++) {
thisContent = arr[j];
contentOutput += thisContent + "\r";
}
document.forms[0].elements[0].value = contentOutput;
}var randy = 0;
</script>
</head>
<body onLoad="randomContent(contentArray);">
lass dann hier auch das Argument weg
<form action="javascript: randomContent(contentArray)">
und hier, ebenso das Leerzeichen nach javascript:
<textarea cols="24" rows="10"></textarea>
<input value="Randomize" type="submit">
</form></body>
Gruß
rfb