[HTML][PHP]Problem z formularzem Opcje

[damian0021]

Witam,

słuchajcie zaczynam dopiero naukę PHP - może dla Was to banalne ale dla mnie trudne - także proszę o pomoc

Zatem mam formularz na swojej stronie i plik PHP wygląda następująco :

[PHP] pobierz, plaintext 
<? 
$headers .= "Content-type: text/html; charset=utf-8\";
$imie = $_POST['imie'];
$email = $_POST['email'];
$telefon = $_POST['telefon'];
$data = $_POST['data'];
$godzina = $_POST['godzina'];
$toaddress = "jakismail";
$subject = "Nowa wiadomość";
$all=
		"<strong>Dostałeś nowe informacje od klienta :</strong></br></br></br>
 
		<strong>Nazwa:</strong> ".$imie."</br>
 
		<strong>E-Mail:</strong> ".$email."</br>
 
		<strong>Telefon:</strong> ".$telefon."</br>
 
		<strong>Data:</strong> ".$data."</br>
 
		<strong>Godzina:</strong> ".$godzina."</br>
";
mail("$toaddress", "$subject", "$all","$headers");
?>
[PHP] pobierz, plaintext 

HTML myślę, że nie muszę pokazywać. Maile się wysyłają wszystko jest pięknie tak jak powinno ale ...

Wysyłam to za pomocą POST'a :

[HTML] pobierz, plaintext 
<form name="myForm" method="POST" action="mail.php">
[HTML] pobierz, plaintext 

I teraz - w momencie gdy użytkownik kliknie w Wyślij - wszytsko ładnie się wysyła - mail dociera. Plik mail.php jest w szablonie strony czyli od góry <? kod?> potem szablon i wklejone w pewnym miejscu podziękowanie za maila zamiast formularza.

Jednak gdy użytkownik wejdzie po prostu z na domena.pl/mail.php - mail jest również wysyłany (pusty) - można to jakoś zabezpieczyć ?

[IAmBoskiM]

isset($_POST) zawsze będzie zwracało true, albowiem zmienna $_POST jest zawsze zdefiniowana, nawet jak jest pusta.

Zamiast tego użyj:

[PHP] pobierz, plaintext 
<?php // lepiej używać pełnych tagów
if (!empty($_POST)) {
   // w klamry weź wszystko
}
?>
[PHP] pobierz, plaintext 

Swoją drogą, zamiast pisać cały czas $zmienna = $_POST['zmienna'] możesz wykorzystać coś takiego (ale tylko jeżeli wszystkie elementy $_POST mają klucze stringowe):

[PHP] pobierz, plaintext 
foreach ($_POST as $key => $value) {
   $$key = $value; // ew. ${$key}, ale oby dwa powinny działać
}
[PHP] pobierz, plaintext 

A poza tym nie wiem czy wiesz, ale jeżeli wstawisz zmienną w cudzysłów (") to zostanie ona zamieniona na jej wartość. Np.:

[PHP] pobierz, plaintext 
$str = 'tekst';
echo '$str'; // wypisze $str
echo "$str"; // wypisze tekst
echo "fajny $str"; // wypisze fajny tekst
[PHP] pobierz, plaintext 

I ogólnie twoje zrobienie tego stringa zrobiłbym inaczej:

[PHP] pobierz, plaintext 
$str = <<< ID
tekst
znowu tekst
itd
itd
itd
ID;
[PHP] pobierz, plaintext