PhpDig.net - View Single Post

ruchit · 09-25-2005, 09:00 PM

brock

looking at your code.. here's a small piece of advice... since you are not making any changes you would be better off putting your headers outside the while loop... so that the processor cycles are not wasted interpreting same lines of code again and again... infact you can also keep your message outside the while loop... just call the mail function (like)...
mail($email, 'Response 1', "Dear $fname, " . $message, $headers);
and remove this line from your code
$message = "Dear $fname, ";

Infact it is much more recommended to create one include file with your own mailing function that has headers parameter with default value defined in it since generally we use the same headers for all mail functions in a single website.

09-25-2005, 09:00 PM	#4
ruchit Green Mole Join Date: Sep 2005 Posts: 3	just an advice brock looking at your code.. here's a small piece of advice... since you are not making any changes you would be better off putting your headers outside the while loop... so that the processor cycles are not wasted interpreting same lines of code again and again... infact you can also keep your message outside the while loop... just call the mail function (like)... mail($email, 'Response 1', "<font face='verdana' size='2'>Dear $fname,<br /><br />" . $message, $headers); and remove this line from your code $message = "<font face='verdana' size=2>Dear $fname,<br /><br />"; Infact it is much more recommended to create one include file with your own mailing function that has headers parameter with default value defined in it since generally we use the same headers for all mail functions in a single website.