ugrás a tartalomhoz

Jó ez a levélküldő kód?

a.d.a.m · 2006. Feb. 22. (Sze), 13.58
Sziasztok!
Megtennétek, hogy átnézitek a következő programrészletet, elvégre több szem...
  1. <?php  
  2. require("class.phpmailer.php");  
  3. global $szovege;  
  4. $time = time();  
  5. $cimzett = "hirdetes##kukac##orszagoslovasar.hu";  
  6.   
  7. $sikeres = "index.html";  
  8. $sikertelen = "hird-felt.html";  
  9.   
  10. $mail = new PHPMailer();  
  11. $mail->SetLanguage("hu""lang");  
  12.   
  13. move_uploaded_file($_FILES["kep"]["tmp_name"], "hirdetesek/".$_FILES["kep"]["name"]);  
  14. $keys = array_keys($_POST);  
  15. $count = count($_POST);  
  16. for ( $array_index=0; $array_index<=($count-1); $array_index++ )  
  17.     {  
  18.     if ( $keys[$array_index] == "kep" or $keys[$array_index] == "MAX_FILE_SIZE" )  
  19.         {}  
  20.     else  
  21.         {  
  22.         if ($array_index==0)  
  23.             {$szovege = "->".$keys[$array_index].": ".$_POST[$keys[$array_index]]."<-\n";}  
  24.         else  
  25.             {$szovege .= "->".$keys[$array_index].": ".$_POST[$keys[$array_index]]."<-\n";}  
  26.         }  
  27.     }  
  28. $fname = "hirdetesek/szoveges".$time.".txt";  
  29. $szoveges = fopen($fname"a");  
  30. fwrite($szoveges$szovege);  
  31. fclose($szoveges);  
  32.   
  33. $mail -> From = "info##kukac##orszagoslovasar.hu";  
  34. $mail -> FromName = "Országos Lóvásár.Hu Hirdetésfelvétel";  
  35. $mail -> AddAddress($cimzett"Hirdetésfelvétel");  
  36. $mail -> AddAttachment("hirdetesek/".$_FILES["kep"]["name"]);  
  37. $mail -> AddAttachment($fname);  
  38. $mail -> Subject = "".$time." számú hirdetés";  
  39.   
  40. if (!$mail->Send())  
  41.     {  
  42.     unlink("hirdetesek/".$_FILES["kep"]["name"]);  
  43.     unlink($fname);  
  44.     echo "<script>window.location.replace('".$sikertelen."');</script>";  
  45.     }  
  46. unlink("hirdetesek/".$_FILES["kep"]["name"]);  
  47. unlink($fname);  
  48. echo "<script>window.location.replace('".$sikeres."');window.alert('A hirdetés feladási sikeresen befejeződött!');</script>";  
 
1

megnéztem

Hojtsy Gábor · 2006. Feb. 22. (Sze), 15.11
Mire gyanakszol?
2

hát...

a.d.a.m · 2006. Feb. 22. (Sze), 16.11
nincs semmi konkrét gyanúm, de úgy érzem nem ártana, ha valaki nálam hozzáértőbb átnézné a kódot és elmondaná a véleményét.
3

Esetleg csak annyi...

-zsolti- · 2006. Feb. 22. (Sze), 21.10
...hogy a javascriptes elirányítás helyett jobb lenne header-ből, mert így szerintem (nem próbáltam) frissítéssel újra és újra lefut a script, és kapod sorban a leveleket.
4

az nem gond

a.d.a.m · 2006. Feb. 23. (Cs), 07.35
mert azt az oldalt, ami ezt a scriptet tartalmazza csak akkor hívom meg ha előtte az alany kitölt egy űrlapot. Esetleg láttál benne valami más hibát is? Mert előfordult párszor, hogy egyik másik sor nem kerüt bele a txt-be az űrlapból.