Visa ett inlägg
Oläst 2007-08-28, 23:19 #7
haavi haavi är inte uppkopplad
Nykomling
 
Reg.datum: Aug 2007
Inlägg: 3
haavi haavi är inte uppkopplad
Nykomling
 
Reg.datum: Aug 2007
Inlägg: 3
Hej superhaga,
enligt PHP.net så tar mail-funktionens första parameter en sträng som bland annat kan se ut så här: "[email protected], [email protected]" eller så här: User <[email protected]>, Another User <[email protected]>.

Jag skulle därför ha skrivit din kod så här förmodligen:

Kod:
$email = "";
foreach ($mailadresser as $mailadress) {
	$email .= $mailadress . ", ";
}

$to = $email;

if (@mail($to, $subject, $message, $headers)) {
	$strSuccess = 'Ditt mail har nu skickats!';
	} else {
	$strError = 'Kunde inte skicka mailet';
}
På det här sättet behöver du bara anropa mail-funktionen en gång vilket jag skulle tro är mycket bättre prestandamässigt både mot webbservern och mailservern. Hämtar du e-postadresserna från en databas så se också till att du använder DISTINCT i dina SQL-anrop, exempelvis så här:

Kod:
$sql = "SELECT DISTINCT mail FROM anvandare";
$result = mysql_query($sql) or die(mysql_error());

$email = "";

while($row = mysql_fetch_assoc($result)) { 
	$email .= $row['mail'] . ", ";
}
På så sätt ser du till att du inte skickar ut samma mail flera gånger till någon om det skulle hända att det finns dubletter av samma e-postadresser i din databas (vilket du borde se till att det inte gör förresten)
haavi är inte uppkopplad   Svara med citatSvara med citat