Ok, you named at least one of your field wrong.  It is "OtherCharges", not "Othercharges".  JS is case sensitive. ... View more

I have this crazy solution for you In a custom keystroke event, have an "X" value entered by a user changed for 0 (this will render calculation easier) if (event.willCommit == true){ if (event.value == "X") event.value = 0 } then, in a custom format event, have the value you just changed to 0 appear as the original X: if (event.value == 0) event.value = "X" and just to make things even better, have your kestroke event test against a regular expression that permits only numbers and "X".  No risk of error after that. ... View more

First of all, congrats for naming your field the way you did.  It will render the code much simpler.  Instead of hardcoding your arrays the way you did, save time by populating your arrays with JS.  The push() method of the array object will append an element at the end of you array by updating the original array: var aCheck = [] var aKm = [] for (var x=1;x<=20;x++){     aCheck.push("Check."+x)     aKm.push("KMsRow"+x) } notice that I used "a" as a prefix for arrays.  It is easier to remember that this variable contains an array.  Use prefixes like f for fields, v for values, n for numbers and s for strings.  Trust me, as a begginer, this will help you identify bugs much quicker. Unless you absolutly need these arrays somewhere else, you do not have to declare them since you created a loop afterwards that loops 20 times.  So either you create arrays and loop throught THAT, OR, you loop 20 times and do want you need in the loop. to loop inside ALL the elements of an array, use the "in" command like so: for (x in aCheck){    console.println(aCheck )  //will print all elements of the array to the console } to evaluate if a checkbox is checked, do not evaluate it's value as you did, use the isBoxChecked() method instead.  it returns a boolean. Do you need this to be a function?  You created a function that as no argument ***function myKM("Nothing here")*** yet you are trying to pass two arguments to it ***myKM(km, check)***.  That will not work. So unless you need the arrays, I would not use them and go strait with the loop.  Copy this in the custom calculate tab of you "Total" field var sum = 0                                                                            //Will store the sum for (var i = 0; i < 20; i++){ var cCheck = this.getField("Check."+x).isBoxChecked(0)     //stores true or false var vKm = this.getField("KMsRow"+x).value                         //stores the value of the current row if (cCheck) sum += vKm                                                        //If true (checked), adds value to the sum } event.value = sum                                                                  //assign a value equal to the sum to current field ... View more

All in all, "Actual Score" will have this script: var valSum = 0 var qty = 10 for (var x=1;x<=qty;x++){ var myExpVal = this.getField("Dropdown."+x).valueAsString var aExpVal = myExpVal.split("-") valSum += Number(aExpVal[0]) } event.value = valSum and "Bench....."  will have this script: var valSum = 0 var qty = 10 for (var x=1;x<=qty;x++){ var myExpVal = this.getField("Dropdown."+x).valueAsString var aExpVal = myExpVal.split("-") valSum += Number(aExpVal[1]) } valSum /= qty event.value = valSum I highligthed the differences between the two Now the later one doesn't take into account a N/A box.  It will attempt to average all ten fields even if only one is filled.  If you might encounter a situation where the user will not make 10 choices, you need to adjust the second script (the first one only adds 0 to the sum so it is not a problem) ... View more