Web Development

function submitForm(form, messageDiv, action)
{
	var targetDiv = document.getElementById(messageDiv);
	var inputs = document.forms[form].getElementsByTagName("input");
	var selects = document.forms[form].getElementsByTagName("select");
	var fieldName = document.getElementsByTagName("label");	
	var length = selects.length + inputs.length;
	
	var data = new Array();
	
	for(i=0; i < length; i++)
	{			
		if (inputs[i]) 
		{
			if (inputs[i].value == "")
			{
				targetDiv.innerHTML = inputs[i].name+" is missing";
				var error = 1;
				return;						
			}
			else
			{
				var error = 0;
				var field = inputs[i].name;
			        var value = inputs[i].value;
			        data[field] = value;				
			}
			
		}	
		
		if (selects[i])
		{
			if (selects[i].value == "")
			{
				targetDiv.innerHTML = selects[i].name+" is missing";
				var error = 1;
				return;
			}
			else
			{
				var error = 0;
				var field = selects[i].name;
				var value = selects[i].value;
                                data[field] = value;
			}	
			
		}
	}	
		if (error == 0)
		{
			targetDiv.innerHTML = "";
			saveForm(form, messageDiv, data);
		}				
	
}

I've managed to half do this by using a numerical value for the key but I really need to have the form field name as the key so that I can associate the form field with the data when I pass it to the php script.  Having said that when I use numeric keys on the second half is completed (the selects) becuause the inputs are all over written because the array key isnt incremented until the next iteration of the for loop.

I would really apprciate some guidance on this, Im kinda new to JavaScript!!

Cheers y'all

Bryant May