PDA

Archiv verlassen und diese Seite im Standarddesign anzeigen : Javascript Problem



Yakuza112
21.12.2010, 17:14
Hey, ich habe eine Frage zu dem Code..
sry für die Noob Frage^^

aber wie mache ich das, dass anstatt

"Keep us strongDonate"

da steht; sich diese Wörter immer abwechseln :

Keep us strong
Support us
Donate

wie auf dem Bild zu sehen klappt das nicht ganz^^




------------INDEX---------------------------------------------------


<a indepth="true" href="support_001.html" class="donate"></a>
<div class="donate"><script src="random.js" type="text/javascript">
</script><a indepth="true" href="support_001.html" class="donate">Donate</a>
</div>

------------random.js---------------------------------------------------

function randomFromTo(from, to){
return Math.floor(Math.random() * (to - from + 1) + from);
}
var x;
x = randomFromTo(1,3);
if(x==1){document.write("Keep us strong");}
if(x==2){document.write("Support us");}
if(x==3){document.write("Donate");}

------------ENDE---------------------------------------------------

http://img2.imagebanana.com/img/deo3mqgr/thumb/21.12.2010_76486.png (http://www.imagebanana.com/view/deo3mqgr/21.12.2010_76486.png)

nelson1109
21.12.2010, 17:36
Ich weiss jetzt nicht warum du für die zufällige zahl ne extra funktion benutzt (geh mal nicht davon aus das du nochmal ne zufällige zahl brauchst...)

aber probier mal das das solte gehen:

var x = Math.round(Math.random()/(1/3));
if(x == 1) {
document.write("Keep us strong");
}
if(x == 2) {
document.write("Support us");
}
if(x == 3) {
document.write("Donate");
}

Yakuza112
21.12.2010, 17:46
hmm das hat den gleichen Effekt^^

wenn "Donate" alleine da steht funktionert alles wie es soll (manchmal aber auch nicht), aber sobald

Keep us strong
oder
Support us
dort steht wird automatisch das "Donate" unmittelbar angefügt ^^

edit... okay klappt oberes war zwar nich der auschlaggebende grund^^

<a indepth="true" href="support_001.html" class="donate"></a><div class="donate"><script src="random.js" type="text/javascript"></script><a indepth="true" href="support_001.html" class="donate">Donate</a>

war das Problem^^ trzdm danke :D

SFX
21.12.2010, 17:58
sowas habe ich gestern erst in php gehabt

es wäre auf jedenfall deutlich sauberer und wartbarer, wenn man die 3 Phrasen in ein Array packt, und dann anhand der Zahl der Einträge in diesem array einen zufälligen index berechnen lässt

Yakuza112
21.12.2010, 18:01
wenn man die 3 Phrasen in ein Array packt

yo ich werde es mir für die zukunft anschauen -> googlen^^
ja ich hab mir die komplette seite geladen, normal das da nicht direkt alles funzt :P^^

nelson1109
21.12.2010, 18:40
var arr = new Array("Keep us strong", "Support us", "Donte");
document.write(arr(Math.round(Math.random()/(0/2))));
Mal schnell zusammenkopiert... sollte gehen...

Yakuza112
21.12.2010, 18:44
danke nochmal^^ aber es geht bereits.. es lag nicht an dem javascript ansich, wie ich dachte..
es lag an dem code in der "index.html" ^^