Forum PHP.pl

Witam,
Napisałem funkcje wywylania maili:

[PHP] pobierz, plaintext 
<?php
function kontakt(){
  if ($_GET['id'] == 'administrator') {
	print $kontakt_admin;
	  print '<table>';
	  print '<tr>';
		print '<td>Imię i nazwisko</td>';
		print '<td><input name="imie" type="text" class="pole" /></td>';
	  print '</tr>';
	if($_POST['imie'] == "") {
	  print '<tr>';
		print '<td></td>';
		print '<td><div class="blad">Proszę podać imię, nazwisko, bądź ksywę.</div></td>';
	  print '</tr>';	
	} else {
	}
	  print '<tr>';
		print '<td>Adres e-mail:</td>';
		print '<td><input name="email" type="text" class="pole" /></td>';
	  print '</tr>';
	if($_POST['email'] == "") {
	  print '<tr>';
		print '<td></td>';
		print '<td><div class="blad">Proszę podać poprawny adres e-mail.</div></td>';
	  print '</tr>';	
	} else {
	}
	  print '<tr>';
		print '<td>Temat:</td>';
		print '<td><input name="temat" type="text" class="pole" /></td>';
	  print '</tr>';
	  print '<tr>';
		print '<td>Treść:</td>';
		print '<td><textarea name="tresc" class="textarea"> </textarea></td>';
	  print '</tr>';
	if($_POST['tresc'] == "") {
	  print '<tr>';
		print '<td></td>';
		print '<td><div class="blad">Proszę wpisać treść wiadomości.</div></td>';
	  print '</tr>';	
	} else {
	}
	  print '<tr>';
		print '<td></td>';
		print '<td><input type="submit" value="Wyślij" />&nbsp;&nbsp;&nbsp;<input type="reset" value="Wyczyść" /></td>';
	  print '</tr>';
	  print '</table>';
	 if (($_POST['imie'] != "") AND ($_POST['email'] != "") AND ($_POST['tresc'] != "")) { 
	   $wyslij = mail(".$mail_admin.",$_POST['temat'],$_POST['tresc'],"From: ".$_POST['imie']."<".$_POST['email'].">");
		 if ($wyslij) {
		   print "<script>alert ('Wiadomość została wysłana')</script>";
		   header("Location: kontakt.php?id=wyslane");	
		 } else {
		   print "<script>alert ('UWAGA! Wystąpił błąd przy wysyłaniu.')</script>";
		   header("Location: kontakt.php?id=administrator");		 
		 }
	  } else {
	print 'Wystąpił błąd w otwieraniu strony. Prosimy ponformować <a href="'.$mail_admin.'" class="link">Administratora</a>.';
	  }
}
?>
[PHP] pobierz, plaintext 

Niewiem dlaczego ale na tej lini:

[PHP] pobierz, plaintext 
<?php
if (($_POST['imie'] != "") AND ($_POST['email'] != "") AND ($_POST['tresc'] != "")) {
?>
[PHP] pobierz, plaintext 

Parse error: syntax error, unexpected $end

wywala mi bład. Proszę o pomoc w rozwiazaniu problemu.

Ten post edytował matyskiewicz 29.10.2006, 21:46:44

unexpected $end

widocznie gdzies nie domknales klamry

edit:
sprawdzilem w quancie i brakuje klamry na koncu

Ten post edytował venomz 29.10.2006, 21:49:17

Hmm, no tak, głupi błąd!
Ale przynajmniej mi unaocznił kolejny błąd jakim jest od razu wyświetlenie braku wypełnieniał pola, oraz funkcja nie wysyła maili (IMG:http://forum.php.pl/style_emoticons/default/sad.gif)
Jak można rozwiązać ten problem?
http://www.zw.matyskiewicz.biz/zw/kontakt....d=administrator

quant co to?

Ten post edytował matyskiewicz 29.10.2006, 21:55:40

Sprawdzaj, czy został wysłany submit (submit też wysyła się przez GET i POST). Jeśli tak - to znaczy, że możesz validować. Jeśli nie - to znaczy że jest to pierwsze wejście.

Ten post edytował Master Miko 29.10.2006, 21:59:20

quanta plus - edytor pod linuxa. zainstaluj sobie jakis edytor ktory koloruje skladnie - latwiej Ci bedzie wylapac glupie bledy tak jak ten wyzej (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

a nawet taki mam (IMG:http://forum.php.pl/style_emoticons/default/biggrin.gif)
A rozwiazaniem powyzszego problemu (nie wysylanie) jest brak znaczników <form> (IMG:http://forum.php.pl/style_emoticons/default/sadsmiley02.gif)
Poprawiony kod i dzialajacy wyglada tak:

[PHP] pobierz, plaintext 
<?php
function kontakt(){
  if ($_GET['id'] == 'administrator') {
		print $kontakt_admin;
		print '<form action="" method="post">';
		print '<table>';
		print '<tr>';
		  print '<td>Imię i nazwisko</td>';
		  print '<td><input name="imie" type="text" class="pole" /></td>';
		print '</tr>';
	  if($_POST['imie'] == "") {
		print '<tr>';
		  print '<td></td>';
		  print '<td><div class="blad">Proszę podać imię, nazwisko, bądź ksywę.</div></td>';
		print '</tr>';	
	  } else {
	  }
		print '<tr>';
		  print '<td>Adres e-mail:</td>';
		  print '<td><input name="email" type="text" class="pole" /></td>';
		print '</tr>';
	  if($_POST['email'] == "") {
		print '<tr>';
		  print '<td></td>';
		  print '<td><div class="blad">Proszę podać poprawny adres e-mail.</div></td>';
		print '</tr>';	
	  } else {
	  }
		print '<tr>';
		  print '<td>Temat:</td>';
		  print '<td><input name="temat" type="text" class="pole" /></td>';
		print '</tr>';
		print '<tr>';
		  print '<td>Treść:</td>';
		  print '<td><textarea name="tresc" class="textarea"> </textarea></td>';
		print '</tr>';
	  if($_POST['tresc'] == "") {
		print '<tr>';
		  print '<td></td>';
		  print '<td><div class="blad">Proszę wpisać treść wiadomości.</div></td>';
		print '</tr>';	
	  } else {
	  }
		print '<tr>';
		  print '<td></td>';
		  print '<td><input type="submit" name="administrator" value="Wyślij" />&nbsp;&nbsp;&nbsp;<input type="reset" value="Wyczyść" /></td>';
		print '</tr>';
		print '</table>';
		print '</form>';
	   if (($_POST['imie'] != "") AND ($_POST['email'] != "") AND ($_POST['tresc'] != "")) { 
		 $wyslij = mail(".$mail_admin.",$_POST['temat'],$_POST['tresc'],"From: ".$_POST['imie']."<".$_POST['email'].">");
		   if ($wyslij) {
			 print "<script>alert ('Wiadomość została wysłana')</script>";
			 header("Location: kontakt.php?id=wyslane");	
		   } else {
			 print "<script>alert ('UWAGA! Wystąpił błąd przy wysyłaniu.')</script>";
			 header("Location: kontakt.php?id=administrator");		 
		   }
	  }
	} else {
	  print 'Wystąpił błąd w otwieraniu strony. Prosimy ponformować <a href="'.$mail_admin.'" class="link">Administratora</a>.';
	}
 
}
?>
[PHP] pobierz, plaintext 

Dzialajacy ale nie do konca: informacja o niewypelnieniu pola powinna sie pojawic po przycisnieciu submita a nie przed... jak z tego problemu wyjść?
http://www.zw.matyskiewicz.biz/zw/kontakt....d=administrator

Ten post edytował matyskiewicz 29.10.2006, 22:37:38

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

Sprawdzaj, czy został wysłany submit (submit też wysyła się przez GET i POST). Jeśli tak - to znaczy, że możesz validować. Jeśli nie - to znaczy że jest to pierwsze wejście.

[PHP] pobierz, plaintext 
<?php
if(!empty($_POST['submit'])) // walidacja...
?>
[PHP] pobierz, plaintext 

Masz to w ogóle źle napisane...
Zamiast:

[HTML] pobierz, plaintext 
<form action="" method="post">
<input type="submit" name="administrator" value="Wyślij" />
[HTML] pobierz, plaintext 

[HTML] pobierz, plaintext 
<form action="skrypt.php?id=administrator" method="post">
<input type="submit" name="submit" value="Wyślij" />
[HTML] pobierz, plaintext 

Ten post edytował Master Miko 29.10.2006, 22:41:05

a moge prosic latwiejszym jezykiem?

I walidacje dac tutaj:

[PHP] pobierz, plaintext 
<?php
if ($_GET['id'] == 'administrator') {
...
		print $kontakt_admin;
?>
[PHP] pobierz, plaintext 

Ten post edytował matyskiewicz 29.10.2006, 22:40:31

Masz źle napisany skrypt - W formularzu który GENERUJE HTML (czyli cały <form> itp) masz zapisaną walidacje ZAWSZE - tzn bez względu czy został wysłany, czy nie zawsze waliduje:

[PHP] pobierz, plaintext 
<?php
if($_POST['imie'] == "") {
print '<tr>';
 print '<td></td>';
  print '<td><div class="blad">Proszę podać imię, nazwisko, bądź ksywę.</div></td>';	
print '</tr>';
 } else {
 }
?>
[PHP] pobierz, plaintext 

Cóż mogę radzić? Zmień koncepcję.

Ten post edytował Master Miko 29.10.2006, 22:44:27

No to sie teraz załamałem.
W takim razie jaka bedzie najlepsza koncepcja (proszę o przyklad), postaram sie sam stworzyc.

No niestety... kiedyś też robilłem formularze i mi nie wychodziły (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)

Tutaj poszukaj (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Wybierz taką, która najbardziej Ci odpowiada.

Ten post edytował Master Miko 29.10.2006, 23:01:22

dzieki znalazłem cos idealnego:) Pozdrawiam dziekuje za okazana pomoc.