To build a bit on kglad's suggestion.
First, you should rethink your naming conventions. Since everything in programming is number/integer based - you will be much better off if you replace input names from spelled out numbers to integers.
For example, fourteenthinput should be input_14; fifteenthinput - input_15, etc.
This way you can link different data containing object by integers - especially it is convenient with arrays.
If you do that - you base code can be something like example below. As you see - it takes less than 40 lines of code instead of hundreds.
Also, everything is dynamic and the only thing you will need to do if you add or remove input fields - rewrite correct answers array and adjust couple of initial parameters like numInputs. The rest of the code will take care of answers verification functionality automatically.
Read comments.
Also, never use nested functions - they are evil.
import flash.events.FocusEvent;
import flash.filters.GlowFilter;
import flash.text.TextField;
// grab input texts into sigle array
var answerInputs:Array = [];
// number of input fields
var numInputs:int = 24;
// this variable will be reused in the while loop
var input:TextField;
// loop assumes that text fields are name input_<int> from 0 on. For example input_0, input_1, input_15, etc.
while (numInputs--)
{
input = this["input_" + numInputs];
// add focus event listener
input.addEventListener(FocusEvent.FOCUS_IN, onFocus);
// add field to array
answerInputs[numInputs] = input;
}
// array of correct answers - array elements are positioned in the same index as answerInputs elements
var solutions:Array = [0, 40.25, 0, -52.5, 26.25, -78.75, 0, 0, 0, 40.25, 0, 0, -385, -8.75, 227.5, -175, 7, 140, 0, 0, 40.25, 268800, 2450, 97300];
// assuming that button that invokes checking for answers is named checkForAnswers instead of fourteenthreset_btn
checkForAnswers.addEventListener(MouseEvent.CLICK, checkAnswers);
// glow filters
var greenGF:GlowFilter = new GlowFilter(0x00ff00);
var redGF:GlowFilter = new GlowFilter(0xff0000);
function checkAnswers(e:MouseEvent):void
{
// reusable text field instance
var inputField:TextField;
for (var i:int = 0; i < answerInputs.length; i++)
{
inputField = answerInputs as TextField;
/**
* set filter depending on whether answer is right or wrong -
* compare to solutions array element in the same position
*/
inputField.filters = [Number(inputField.text) == solutions ? greenGF : redGF];
}
}
// single focus listener takes care of all input fields
function onFocus(e:FocusEvent):void
{
TextField(e.currentTarget).text = "";
}
7
Replies
AdChoices