Senior Member
Join Date: Jun 2010
Location: Beaverton, OR
Posts: 267
|
LOL ignorance duly shown. Entered the code where you said and it worked. Thank you Mirtos.
|
#11 |
Senior Member
Join Date: Jan 2013
Location: Rochester, MN
Posts: 1,517
|
Quote:
You would want Code:
Math.floor(Math.random() * x) + 1; OK, back to Gen Con for me! :) |
|
#12 |
Senior Member
Join Date: Oct 2011
Posts: 865
|
Bad code on my part. sectionIdentifier not used.
to make it more configurable on your part: make the function definition like this: Code:
function tableResults(argument, rollModifier) { Code:
if (!rollModifier) { rollModifier = 0; } var randomNum = rand(100) + rollModifier; make your HTML like this: Code:
<form> <input type="button" name="Button1" value="Outer Rim" onClick="var x = tableResults('varisia', 0);this.form.name.value=x.nam e;this.form.content.value = x.content;" size="" /> <input name="name" style="font-weight:bold;"></input> <textarea name="content" cols="50" rows="11" wrap="soft"></textarea> </form> <form> <input type="button" name="Button2" value="High Ice" onClick="var x = tableResults('varisia', 25);this.form.name.value=x.nam e;this.form.content.value = x.content;" size="" /> <input name="name" style="font-weight:bold;"></input> <textarea name="content" cols="50" rows="11" wrap="soft"></textarea> </form> <form> <input type="button" name="Button3" value="Boreal Expanse" onClick="var x = tableResults('varisia', 50);this.form.name.value=x.nam e;this.form.content.value = x.content;" size="" /> <input name="name" style="font-weight:bold;"></input> <textarea name="content" cols="50" rows="11" wrap="soft"></textarea> </form> |
#13 |
Senior Member
Join Date: Jun 2010
Location: Beaverton, OR
Posts: 267
|
Although I can only rely on the randomness. It appears to be working. Outer Rim gets results of 1-100, High Ice 26-125, and Boreal Expanse 51-150. I did not change sectionIdentifier to regionNumber.
|
#14 |
Senior Member
Join Date: Oct 2011
Posts: 865
|
fair enough. my previous post right before yours will make it so you can control the number a little bit more as well. up to you what you prefer.
|
#15 |
Senior Member
Join Date: Oct 2011
Posts: 865
|
Quote:
my bad. so the function should be: Code:
function rand(x) { return Math.floor(Math.random() * x) + 1; } Last edited by mirtos; August 14th, 2014 at 02:11 PM. |
|
#16 |
Senior Member
Join Date: Jun 2010
Location: Beaverton, OR
Posts: 267
|
Would I replace all of this
var randomNum = rand(100) + ((sectionIdentifier -1) * 25) var arrayOfChoices = allArrays[argument]; var error = ""; if (!arrayOfChoices) { error = "no such set: " + argument; } with if (!rollModifier) { rollModifier = 0; } var randomNum = rand(100) + rollModifier; |
#17 |
Senior Member
Join Date: Oct 2011
Posts: 865
|
no. You would replace:
var randomNum = rand(100) + ((sectionIdentifier -1) * 25) with if (!rollModifier) { rollModifier = 0; } var randomNum = rand(100) + rollModifier; ----------------------------------------------------------------- leave the arrayOfChoices section alone. |
#18 |
Senior Member
Join Date: Jun 2010
Location: Beaverton, OR
Posts: 267
|
Crap I seem to have broke it. This is what I currently have.
Quote:
|
|
#19 |
Senior Member
Join Date: Oct 2011
Posts: 865
|
I look at it in like 45 minutes.
Edit: off the bat, looks like you have a typo. You have a this.form.name.value = x.nam e It should be x.name (You have a space misspelling the word name) Just a guess, as I'm currently on a bus. Last edited by mirtos; August 14th, 2014 at 03:18 PM. |
#20 |
|
|