дайте пару советов по PHP

файл mailer.php
<?
	$to      =  '[email protected]';
	$subject =  'Feedback REcloud';
	$headers  = "Content-type: text/html; charset=utf-8 \r\n";
	$headers .= "From: Realestatecloud.com <[email protected]>\r\n";
	$headers .= "Reply-To: [email protected]";
	
		
	$templ =   file_get_contents ('sm_template.html');  
	
	$macros = array();
	$myvars = array();
	
	for($i=1;$i<=20;$i++){
		$macros[] = '%var'.$i.'%';	
		$myvars[] = $_REQUEST['var'.$i];
	}
	
	$message = str_replace($macros, $myvars, $templ);

	mail($to, $subject, $message, $headers); 


?>

что тут может не так? или что то можно сделать более оптимизированно?
ну и вообще как нынче заголовки к письму прописывают. спасибо

18 комментариев

  • аватар SeL
  • 0
Чесно говоря я не знаток ПХП, да и 100 лет его в глаза не видел, но меня вот что заинтересовало:
1. а что если в глобальном массиве "$_request" будет передано не 20 переменных, вида «var1-20», а меньше?
2. не стоит ли поставить исключение для стороки отпраки мэйла, если в ПХП есть оператор исключения, что-то подзабыл.
ну в том то и дело что брал с запасом переменных, просто это скрипт для универсальной отправки форм с разным количеством полей. тоже конечно смущает такая реализация но другого не придумал кроме как передавать количество полей
возможно ли узнать сколько передается переменных?
var data_post = {
                var1: $("#name").val(),
                var2: $("#email").val(),
                var3: $("#message").val()
            };



            $.ajax({
                type: "POST",
                url: "mailer.php",
                data: data_post
  • аватар SeL
  • 0
Можно использовать foreach, что-то типа:
foreach ($_Request as $key => $value){
   $macros[]=$key;
   $myvars[]=$value;
   }
Размер вроде как count($_Request);

Еще возможно в php етсь ф-ция возвращающая список ключей массива, а также список значений, тогда можно через них еще.
  • аватар mihon
  • +1
Советую прекратить изобретать на пхп, вот почитайте: //swiftmailer.org/
спасибо но я не умею пхп а еще мне говорили что пхп это фикалька
  • аватар mihon
  • 0
Так на кой черт вы просите совета?! Программист который «мне говорили», внатуре классно".
я не программист
  • аватар mihon
  • 0
Ну и за каким дьяволом вы спрашиваете совета в программировании, коли вы не в зуб ногой?
ну почему же я же написал клиентскую и серверную часть. я просто не знаю приемов в пхп и чем сейчас пользуются.
если я не программирую это не значит что я не программист
  • аватар mihon
  • 0
Противоречие самому себе, да и запятых не мешало бы малость добавить.
Эй бро, ты что, впервые видишь чувака который нихуя не программирует и при этом считает себя крутым программистом? ;)
  • аватар z1mmer
  • +1
привет. я нихуя не программирую, но программист пиздатый.
Стас, ты? ))
  • аватар vit
  • +1
bananan 3 декабря 2011, 23:58 #
я не программист

bananan 4 декабря 2011, 00:12 #
если я не программирую это не значит что я не программист


да это пиздец рассуждения
  • аватар ololo
  • 0
вполне логичные рассуждения,
их можно выразить как
Если я программирую, это еще не значит что я программист )

Чтобы оставлять комментарии, нужно или зарегистрироваться.