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...
<?php
require("class.phpmailer.php");
global $szovege;
$time = time();
$cimzett = "hirdetes##kukac##orszagoslovasar.hu";

$sikeres = "index.html";
$sikertelen = "hird-felt.html";

$mail = new PHPMailer();
$mail->SetLanguage("hu", "lang");

move_uploaded_file($_FILES["kep"]["tmp_name"], "hirdetesek/".$_FILES["kep"]["name"]);
$keys = array_keys($_POST);
$count = count($_POST);
for ( $array_index=0; $array_index<=($count-1); $array_index++ )
	{
	if ( $keys[$array_index] == "kep" or $keys[$array_index] == "MAX_FILE_SIZE" )
		{}
	else
		{
		if ($array_index==0)
			{$szovege = "->".$keys[$array_index].": ".$_POST[$keys[$array_index]]."<-\n";}
		else
			{$szovege .= "->".$keys[$array_index].": ".$_POST[$keys[$array_index]]."<-\n";}
		}
	}
$fname = "hirdetesek/szoveges".$time.".txt";
$szoveges = fopen($fname, "a");
fwrite($szoveges, $szovege);
fclose($szoveges);

$mail -> From = "info##kukac##orszagoslovasar.hu";
$mail -> FromName = "Országos Lóvásár.Hu Hirdetésfelvétel";
$mail -> AddAddress($cimzett, "Hirdetésfelvétel");
$mail -> AddAttachment("hirdetesek/".$_FILES["kep"]["name"]);
$mail -> AddAttachment($fname);
$mail -> Subject = "".$time." számú hirdetés";

if (!$mail->Send())
	{
	unlink("hirdetesek/".$_FILES["kep"]["name"]);
	unlink($fname);
	echo "<script>window.location.replace('".$sikertelen."');</script>";
	}
unlink("hirdetesek/".$_FILES["kep"]["name"]);
unlink($fname);
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.