funkcja mail() + wysylanie maili, dlaczego wysylka kilka e-maili a nie 1? Opcje

[gladiror]

Witam!
Mam problem - mam skrypt:

[PHP] pobierz, plaintext 
<?php
 
mail("ga@wp.pl","aa","asas");
 
?>
[PHP] pobierz, plaintext 

Chodzi o to, że skrypt jest uruchomiony np. 31 lipca, a wiadomosci przychodza sobie pare dni poxniej i jest ich wiecej niz 1... Da sie to jakos ominac??

[TomASS]

Proszę o większą ilość kodu. Generalnie, jeśli na serwerze ustawione jest wszysto oki doki, to powinno wysłać tylko jednego maila.

[gladiror]

[PHP] pobierz, plaintext 
<?
 
/* USUWA Z BAZY DANYCH OSOBY KTÓRE NEI DOKONAŁY AKTYWACJI KONTA W CIˇGU 1 DNIA */
 
$dbh=mysql_connect ("localhost", "aa", "aa") or die ('I cannot connect to the database because: ' . mysql_error());
mysql_select_db ("aa"); 
$dzisiaj = date("Y-m-d");
$zapytanie = "SELECT * FROM koszyk WHERE potwierdzone = '' or potwierdzone = 'NIE'";
$wykonaj = mysql_query($zapytanie);
while($wiersz=mysql_fetch_array($wykonaj))
{
$data_z_bazy = "".$wiersz['data_zamowienia']."";
$dzisiaj = (strtotime($dzisiaj) - strtotime($data_z_bazy)) / 86400;
$nr_rekordu = "".$wiersz['id_koszyk']."";
	if($dzisiaj>1)
	{
	$zapytanie_usun = "DELETE FROM koszyk WHERE id_koszyk = '$nr_rekordu'";
	$wykonaj2 = mysql_query($zapytanie_usun);
	}
	else
	{
	print("");
	}
 
}
/* USUWA Z BAZY DANYCH ZAMÓWIENIA KTÓRE NIE ZOSTAŁY AKTYWOWANE W CIˇGU 1 DNIA */
 
?>
 
<h1>Twój koszyk:</h1>
 
 
 
<?
$zapytanie_koszyk = "SELECT * FROM `koszyk` WHERE klient = '$login' and kodzik='' order by cena";
$wykonaj_koszyk = mysql_query($zapytanie_koszyk);
$czy_koszyk_jest_pusty = mysql_num_rows($wykonaj_koszyk); 
 
 
 
if($czy_koszyk_jest_pusty==0)
{
print("<h2>Twój kosztyk jest pusty!</h2>");
}
else
{
 
	while($wiersz=mysql_fetch_array($wykonaj_koszyk))
		{
		$numerek = "".$wiersz['id_koszyk'].""; /* nr produktu w koszyku w bazie danych */
		}
 
 
 
 
	if ((isset($_GET['action'])) && (is_numeric($_GET['id'])))  /* usuwanie produktów z koszyka */
	{
		  if ($_GET['action'] == "usun") 
		   {
			 $zapytanie_usun = "DELETE FROM koszyk WHERE id_koszyk = '$numerek' and kodzik=''";
		  $wykonaj_usun = mysql_query($zapytanie_usun);
		  }
	}
 
 
	$zapytanie_koszyk2 = "SELECT * FROM `koszyk` WHERE klient = '$login' and kodzik='' order by cena";
	$wykonaj_koszyk2 = mysql_query($zapytanie_koszyk2);
	$look = mysql_num_rows($wykonaj_koszyk2); 
	$id="0";
 
 
		if($look<>0)
		{
 
		print("<table>");
		print("<tr><td><B>Lp</B></td>
		<td align="center"><B>Nazwa</B></td>
		<td align="center"><B>Rozmiar</B></td>
		<td align="center"><B>Kolor</B></td>
	<td align="center"><B>Cena</B></td>
	<td align="center"><B>Ilo&para;ć</B></td>
	<td align="center"><B>Suma</B></td>
	<td align="center"><B>Modyfikacja</B></td>
	</tr>");
 
		while($wiersz=mysql_fetch_array($wykonaj_koszyk2))
		{
 
		$id = $id+1;
 
		print("<tr>");
		print("<td align="center">$id</td>");
		print("<td align="center">".$wiersz['towar']."</td>");
		print("<td align="center">".$wiersz['rozmiar']."</td>");
		print("<td align="center">".$wiersz['kolor']."</td>");
		$cenka = "".$wiersz['cena']."";
		$ile = "".$wiersz['ilosc']."";
		print("<td align="center">".$wiersz['cena']." zł</td>");
		print("<td align="center">".$wiersz['ilosc']."</td>");
		$suma = $cenka * $ile;
		print("<td align="center">$suma zł.</td>");
		$numerek = "".$wiersz['id_koszyk']."";
		print("<td align="center"><a href="koszyk.php?action=usun&id=$numerek">Usuń</a></td>");		
		print("</tr>");
		$razem = $razem + $suma;
		}
		print("</table>");
 
 
 
		print("<form action="koszyk.php" method="post">");
 
 
 
 
		print("<br>Koszt przesyłki: ");
		print("<A HREF="javascript:displayWindow('../../data/data/wysylka2.php',400,250)">=> Zobacz <=</a><br><br>");
 
 
 
 
		$zapytanie = "SELECT * FROM `koszyk` WHERE klient = '$login' and data_wyslania <> '0000-00-00'";
		$wykonaj = mysql_query($zapytanie);
		$ile_odp = mysql_num_rows($wykonaj); 
 
 
 
 
 
 
 
 
 
 
 
		if(!$platnosc):
		{
		print("Wybierz rodzaj płatno&para;ci: ");
		print("<select name="platnosc">");
		print("<option></option>");
		print("<option>Przelew</option>");
		print("<option>Wpłata w UP</option>");
		print("<option>Odbiór osobisty</option>");
 
			if($ile_odp>0):
			{
			print("<option>Za pobraniem</option>");
			}
			else:
			{
			print("");
			}
			endif;
		print("</select>");
		print("<input type="submit" name="submit" value="wybierz">");
 
		}
		else:
		{
 
			if($platnosc=='Przelew'):
			{
			$razem = $razem + 7;
			}
			else:
			{
			print("");
			}
			endif;
 
			if($platnosc=='Odbiór osobisty'):
			{
			print("");
			}
			else:
			{
			print("");
			}
			endif;
 
			if($platnosc=='Za pobraniem'):
			{
			$razem = $razem + 15;
			}
			else:
			{
			print("");
			}
			endif;
 
			if($platnosc=='Wpłata w UP'):
			{
			$razem = $razem + 7;
			}
			else:
			{
			print("");
			}
			endif;
 
 
 
 
		print("<br>Wybrałe&para; następuj&plusmn;cy rodzaj płatno&para;ci: $platnosc");
		print("<br>Razem z przesyłk&plusmn;: $razem zł <br><br>");
 
 
 
 
		print("<div align=center"><h2>Dodaj uwagi/ pro&para;by do zamówienia</h2>Jeżeli chcesz komu&para; wysłać paczkę jako prezent to napisz tutaj to.<br><br><textarea cols="20" rows="10" name="uwagi"></textarea><br><input type="submit" name="wyslij_zamowienie" value="Wy&para;lij moje zamówienie do realizacji"></div>");
 
		}
		endif;
 
 
 
	}
	else
	{
	print("<h2>Opróżniłe&para; koszyk!</h2>");
	}
}
 
print("<input type="hidden" name="aa" value="$platnosc">");
 
 
 
		if(isset($wyslij_zamowienie))
		{
		$zapytanie9 = "SELECT * FROM `klienci` WHERE wlaczone = 'TAK' and uzytkownik = '$login'";
		$wykonaj9 = mysql_query($zapytanie9);
 
 
 
		while($wiersz=mysql_fetch_array($wykonaj9))
		{	
		$adres_email = "".$wiersz['mail']."";
		}
		$kod = rand(1000, 10000000);
 
 
 
 
		$tresc = "
Witamy!<br><br>
 
Wła&para;nie dokonałe&para;/a&para; zakupów w sklepie<br><br>
 
Twój koszyk wygl&plusmn;da następuj&plusmn;co:<br><br>
 
 
 
<table border="1">
<tr><td><B>Lp</B></td>
<td align="center"><B>Nazwa</B></td>
<td align="center"><B>Rozmiar</B></td>
<td align="center"><B>Kolor</B></td>
<td align="center"><B>Cena</B></td>
<td align="center"><B>Ilo&para;ć</B></td>
<td align="center"><B>Suma</B></td>
</tr>
 
";
		$zapytanie_koszyk2 = "SELECT * FROM `koszyk` WHERE klient = '$login' and kodzik='' order by cena";
		$wykonaj_koszyk2 = mysql_query($zapytanie_koszyk2);
		$look = mysql_num_rows($wykonaj_koszyk2); 
		$id="0";
		$razem = '0';
 
		while($wiersz=mysql_fetch_array($wykonaj_koszyk2))
		{
 
		$id = $id+1;
 
		$tresc .= "<tr>";
		$tresc .= "<td align="center">$id</td>";
		$tresc .= "<td align="center">".$wiersz['towar']."</td>";
		$tresc .= "<td align="center">".$wiersz['rozmiar']."</td>";
		$tresc .= "<td align="center">".$wiersz['kolor']."</td>";
		$cenka = "".$wiersz['cena']."";
		$ile = "".$wiersz['ilosc']."";
		$suma = $cenka * $ile;
		$tresc .= "<td align="center">".$wiersz['cena']." zł</td>";
		$tresc .= "<td align="center">".$wiersz['ilosc']."</td>";
		$tresc .= "<td align="center">$suma zł</td>";
		$numerek = "".$wiersz['id_koszyk']."";
		$tresc .= "</tr>";
		$razem = $razem + $suma;
		}
 
 
 
 
 
 
 
 
 
			if($aa=='Przelew'):
			{
			$razem = $razem + 7;
			$oplata = 7;
			}
			else:
			{
			print("");
			}
			endif;
 
			if($aa=='Odbiór osobisty'):
			{
			print("");
			$oplata = 0;
			}
			else:
			{
			print("");
			}
			endif;
 
			if($aa=='Za pobraniem'):
			{
			$razem = $razem + 15;
			$oplata = 15;
			}
			else:
			{
			print("");
			}
			endif;
 
			if($aa=='Wpłata w UP'):
			{
			$razem = $razem + 7;
			$oplata = 7;
			}
			else:
			{
			print("");
			}
			endif;
 
 
 
 
 
 
 
 
 
 
 
 
 
		$zapytanie10 = "UPDATE koszyk SET kodzik='$kod', przesylka='$aa', uwagi='$uwagi' WHERE klient = '$login' and kodzik=''";
		$wykonaj10 = mysql_query($zapytanie10);
 
		$tresc .= "</table>
 
 
 
<br>
Twój login to: $login<br>
Twój nr zamówienia to: $kod<br>
Rodzaj płatno&para;ci: $aa<br>
Opłata za przesyłkę: $oplata zł<br>
Razem do zapłaty (z przesyłk&plusmn;): $razem zł";
 
 
if(($aa=="Przelew") or ($aa=="Wpłata w UP")):
{
$tresc .= "asdsa asdas asd asd ad asd asd ";
}
else:
{
print("");
}
endif;
 
 
 
$tresc .= "
<br><br>
 
Jeżeli potwierdzasz tranzakcję proszę nacisn&plusmn;ć poniższy link lub skopiować go do przegl&plusmn;darki internetowej:<br><br>
 
 
 
Pozdrawiamy i życzmy udanych zakupów<br><br>
 
 
";
		/* Aby wysłać e-mail w formacie HTML, należy ustawić nagłówek Content-type. */
		$naglowki  = "MIME-Version: 1.0rn";
		$naglowki .= "Content-type: text/html; charset=iso-8859-2rn";
		/* dodatkowe nagłówki */
		$naglowki .= "From: Potwierdzenie zakupów <klimpol@klimpol.pl>rn";
 
 
 
		mail($adres_email,"Potwierdzenie zakupów", $tresc, "$naglowki");
 
		print("<h2>WejdĽ na swoj&plusmn; pocztę i potwierdĽ zakupy!</h2>");
 
		}
		else
		{
		print("");
		}
 
 
print("</form>");
 
 
?>
[PHP] pobierz, plaintext 

Ten post edytował gladiror 9.09.2005, 08:54:01

[revyag]

Uwagi: (IMG:http://forum.php.pl/style_emoticons/default/snitch.gif)
- wszechobecne if i else, aż się prosi żeby dać tam switch'a
- składnia alternatywna ma to do siebie że nawias otwierający zmieniasz na : a zamykający na endif; u ciebie jest to i to
- w miejscu if(isset($wyslij_zamowienie)) robisz petle po zapytaniu do bazy, po składni zapytania wynika, że ma zostać pobrany email jednego usera, więc po co ta pętla ?
- włącz error_reporting na E_ALL i zobacz czy nie ma warningów.

[TomASS]

Do bardzo konstuktywnych uwaga Revyaga dorzuce:

• 
  

  [PHP] pobierz, plaintext 
  <?php
   
  $zapytanie = "SELECT * FROM koszyk WHERE potwierdzone = '' or potwierdzone = 'NIE'";
   
  ?>
  [PHP] pobierz, plaintext 

  
  
  A dlaczego status niepotwierdzonego koszyka jest pusty lub napis 'NIE'? Nie lepiej byłoby ustawić pole potwirdzone aby wartość domyślna była 'NIE'?
  
• Czemu służy kod w linii 25:
  

  [PHP] pobierz, plaintext 
  <?php
   
  	else
  	{
  	print("");
  	}
   
  ?>
  [PHP] pobierz, plaintext 

  
  Jeśli jest potrzebny to możesz śmiało napisać:
  

  [PHP] pobierz, plaintext 
  <?php
   
   else print("");
   
  ?>
  [PHP] pobierz, plaintext 

  
  
• Dlaczego dołączasz spacje przed danymi z bazy danych, np:
  

  [PHP] pobierz, plaintext 
  <?php
   
  $numerek = "".$wiersz['id_koszyk']."";
   
  ?>
  [PHP] pobierz, plaintext 

  
  
• Do zapytania dorzuć jeszcze ID klienta (chyba zmienna $login) po klient będzie mógł skasować nie tylko swoje koszyki (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) (swoją drogą chyba lepiej jest używać POST niż GET):
  

  [PHP] pobierz, plaintext 
  <?php
   
  	if ((isset($_GET['action'])) && (is_numeric($_GET['id'])))  /* usuwanie produktów z koszyka */
  	{
  		  if ($_GET['action'] == "usun") 
  		  {
  			 $zapytanie_usun = "DELETE FROM koszyk WHERE klient = '$login' AND id_koszyk = '$numerek' and kodzik=''";
  		  $wykonaj_usun = mysql_query($zapytanie_usun);
  		  }
  	}
   
  ?>
  [PHP] pobierz, plaintext 

  
  
• W linii:
  

  [PHP] pobierz, plaintext 
  <?php
   
  if($czy_koszyk_jest_pusty==0)
   
  ?>
  [PHP] pobierz, plaintext 

  
  Możesz napisac:
  

  [PHP] pobierz, plaintext 
  <?php
   
  if(!$czy_koszyk_jest_pusty)
   
  ?>
  [PHP] pobierz, plaintext 

  
  
• 
  

  [PHP] pobierz, plaintext 
  <?php
   
  $id = $id+1;
   
  ?>
  [PHP] pobierz, plaintext 

  
  Na
  

  [PHP] pobierz, plaintext 
  <?php
   
  $id++;
   
  ?>
  [PHP] pobierz, plaintext 

[gladiror]

Więc może zacznę od 1 wypowiedzi (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) ALe THX, że napisaliscie co jest źle, to będe uczył się na błędach (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) Nie uczyłem sie z książek, także błędy mogły wystąpić...

- wszechobecne if i else, aż się prosi żeby dać tam switch'a

Jakoś tak przyzwyczaiłem sie do tej podstawowej funkcji... (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

- składnia alternatywna ma to do siebie że nawias otwierający zmieniasz na : a zamykający na endif; u ciebie jest to i to

Czyli chodzi, że trzeba pisać w taki sposób:

[PHP] pobierz, plaintext 
<?
if(a==1):
print("1")
else:
print("inne");
endif;
?>
[PHP] pobierz, plaintext 

(IMG:http://forum.php.pl/style_emoticons/default/questionmark.gif) (IMG:http://forum.php.pl/style_emoticons/default/questionmark.gif) ?

- w miejscu if(isset($wyslij_zamowienie)) robisz petle po zapytaniu do bazy, po składni zapytania wynika, że ma zostać pobrany email jednego usera, więc po co ta pętla ?

A jak można wycinągnąc tylko jeden rekord bez uzywania pętli??

- włącz error_reporting na E_ALL i zobacz czy nie ma warningów.

w php.ini - tak??


Jeżeli chodzi o status koszyka - to kwestia sporna... Moim zdaniem nie ma to dużego znaczenia... Czy coś będzie wpisane czy nie... Mozna wpisać NIE, ale równie dobrze można nic nie wpisywać... I tak na jedno wyjdzie.

Jeżeli chodzi o kod z linii 25 to on w sumie nieczemu nie służy... Poprostu jak pisze składanie if to zapisuje sobie całość i później zostają takie wybryki natury (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) Czyli chodzi o to, żeby nie używać nawiasów klamrowych?? Jest w ogóle jakaś roznica czy użyje sie tych nawiasów czy nie ma? Czy wystarczy wybrać sobie którąś z opcji?

Dlaczego dołączasz spacje:

[PHP] pobierz, plaintext 
<?php
$numerek = "".$wiersz['id_koszyk']."";
?>
[PHP] pobierz, plaintext 

A nie można dołączać spacji??

[PHP] pobierz, plaintext 
<?php
$numerek="".$wiersz['id_koszyk']."";
?>
[PHP] pobierz, plaintext 

Tak jest dobrze??

Jeżeli chodzi o koszyk to masz rację z tym dodaniem loginu... Rzeczywiście może wyniknać taka sytuacja... A z tym postem to też poprawie (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

A dalej to już alternatywne sposoby (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) a

Z góry dzięki za pomoc i uwagi! Proszę o odpowiedzi...

Ten post edytował gladiror 9.09.2005, 22:43:43

[TomASS]

1. Ale ze swithem jest przejżyściej
2. Tak
3. $zapytanie9 = "SELECT * FROM `klienci` WHERE wlaczone = 'TAK' and uzytkownik = '$login' LIMIT 1";
4. Tak
5. Co do koszyka, to ja wolę mieć jeden z góry ustalony status mojego rekordu. A nie że zamówienie jest nie potwierdzone, albo i nie...zrób jak chcesz, my tylko radzimy.
6. Nie o te spacje mi chodzi - chodzi mi właśnie o te " ".$wiersz['id_koszyk']." ";
7. Jak chcesz aby ktoś podglądał jakie zmienne przesyłasz, to uzywaj GETa (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Ten post edytował TomASS 9.09.2005, 22:51:18

[gladiror]

Odnośnie stosowania metody POST i GET... W tym skrypcie co mam metodę GET - zamieniłem nazwę na POST i już nie działa... W jaki sposób dzięki metodzie get ktoś będzie mógł zobaczyć moje zmienne? Bo zmienne z formularza to łatwo zobaczyć prawym i pokaż źródło... A za pomocą GET = w jaki sposó?

[PHP] pobierz, plaintext 
<?php
 
$zapytanie9 = "SELECT * FROM `klienci` WHERE wlaczone = 'TAK' and uzytkownik = '$login' LIMIT 1";
 
?>
[PHP] pobierz, plaintext 

Ok, ale teraz w jaki sposób wyciągnąć np. imie klienta? przeciez nie wpisze jak wczesniej .$wiersz[imie]....

Wracając do tych "" => jak ich nie używam, tzn. jak wpiszę samo print("<td>.$wiersz[id_koszyk].</td">); to wyskakuje błąd...