[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ć ?

[kamilo818]

[PHP] pobierz, plaintext 
if (!isset ($_POST)){
     header ('Location: formularz.php');
     die ();
}
[PHP] pobierz, plaintext 

Ten post edytował kamilo818 9.07.2016, 17:25:14

[damian0021]

Rozumiem, że mam to wrzucić dp mail.php na samym początku formularza?

[kamilo818]

Tak.

[damian0021]

[PHP] pobierz, plaintext 
<? 
if (!isset ($_POST)){
     header ('Location: mail.php');
     die ();
} 
 
 
$headers .= "Content-type: text/html; charset=utf-8\
";
 
 
$imie = trim($_POST['imie']);
$email = trim($_POST['email']);
$telefon = trim($_POST['telefon']);
$data = trim($_POST['data']);
$godzina = trim($_POST['godzina']);
$toaddress = "jakismail";
$subject = "Klient";
 
 
 
 
$all=
		"<strong>Dostałeś nowe informacje o kliencie :</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 

Niestety maila dalej wysyła.

Ten post edytował damian0021 9.07.2016, 17:43:31

[Star]

skoro masz wszystko w tym samym pliku to albo napiszesz jakas funkcje od wysylania i potem cos w stylu

[PHP] pobierz, plaintext 
if( $_POST ) 
{
mojaFunkcjaOdWysylania( $jakiesparametry[] );
exit;
}
 
//formularz
[PHP] pobierz, plaintext 

albo tym ifem obejmiesz caly kod od wysylania czyl

[PHP] pobierz, plaintext 
if( $_POST ) {
$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");
 
header ('Location: mail.php');
}
 
//formularz
 
[PHP] pobierz, plaintext 

[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