Can this even be done?

[killswytch]

Can this even be done? Or I am just doing this wrong? Not quite sure. What is happening, is that I am passing the number of email addresses to add and check. The other function I have writes the input boxes to enter the email addresses based on the number passed between the pages. This works fine. each name of the input type is correct and correllates with the number that was passd. This function queries the db to see if they already exist. A little help would be great!!! Thanks in advance.

function verifyAddress()
{
Global $dbName, $dbHost, $dbUser, $dbPass, $num_email;
// open connection to the database
$conn = mysql_connect($dbHost, $dbUser, $dbPass)
or die("Could Not Connect To Database\n");

// select the database
$db = mysql_select_db($dbName)
	or die("Could not select database(".$dbName.")<br>\n");

$newMail = $num_email;
for ($i=1;$i<$newMail;$i++)
{
	$emailCheck[$i] = "select * from accounts where email_address".$i."='$email_address".$i."'";
}
for ($j=1;$j<$newMail;$j++)
{	
	$row[$j] = mysql_fetch_array($result[$j]);
	If ($row["email_address".$j.""] == $email_address[$j])
	{
		$ErrorString .= "Email Address #".$j." already exists.  Please choose another.";
		return $ErrorString;
	}
	Else
	{
		return true;
	}

}

}

[sepodati]

You need to take the returns out of the loop. The loop will only be evaluated once, then it\'ll either return the error message or true. Try this:

for ($j=1;$j<$newMail;$j++)
{
$row[$j] = mysql_fetch_array($result[$j]);
If ($row[\"email_address\".$j.\"\"] == $email_address[$j])
{
$ErrorString .= \"Email Address #\".$j.\" already exists. Please choose another.<BR>\n\";
$errormsg .= $ErrorString;
}
}

if(strlen($errormsg) > 0)
{
return $errormsg;
}
else
{
return true;
}

You could make it even better by defining $errormsg as true in the beginning of your script. Then you could remove the check to see if there is anything in it and just return it, whatever it is.

function verifyAddress()
{

 $errormsg = TRUE;

 //rest of script...
 //include my changes
 for ($j=1;$j<$newMail;$j++)
 {
  $row[$j] = mysql_fetch_array($result[$j]);
  If ($row[\"email_address\".$j.\"\"] == $email_address[$j])
  {
   $ErrorString .= \"Email Address #\".$j.\" already exists. Please choose another.<BR>\n\";
   $errormsg .= $ErrorString;
  }
 }

 return $errormsg;

}

Hope that helps...

---John Holmes...