[php] syntax error - klasa Opcje

[hhg]

napisalem moja pierwsza w zyciu klase i od razu dostaje blad (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Parse error: syntax error, unexpected T_VARIABLE in /home/bart/public_html/mailing/mail.class.php on line 5

gdzie 5 linia wskazuje na pierwszy przypisanie w klasie

nie pisze ze ta klasa jest juz gotowa ale prosze o wskazowki co moze jeszcze nie zadzialac, i czy te 'public' powinienem gdzies powinienem zmienic na inne?

z gory dzieki za pomoc

[PHP] pobierz, plaintext 
<?php
 
class site
{
	public $ip = $_SERVER['REMOTE_ADDR'];
	public $host = $gethostbyaddr($this->ip);
 
	public $subject;
	public $firm;
	public $name;
	public $telephone;
	public $sender;
	public $receiver;
 
	public function validate($attribute)
	{
		if (empty($attribute))
			return false;
		else
			return true;
	}
 
	public $text = nl2br("
		Wiadomosc od: $this->sender
		Firma: $this->firm
		Telefon: $this->telephone
		Imie i nazwisko: $this->name
 
		$this->text;
 
		IP nadawcy: $this->ip
		HOST nadawcy: $this->host
	");
 
	public function $mailto()
	{
 
		if (!(validate($this->subject) && validate($this->firm) && validate($this->name) && validate($this->telephone) && validate($this->sender) && validate($this->receiver)))
		{
			echo "Wszystkie pola sa wymagane. Popaw wprowadzone dane!";
			exit();
		}
 
		$headers = "MIME-Version: 1.0 rn Content-type: text/html; charset=iso-8859-2 rn From: $this->sender rn";
 
		if (mail($this->receiver;$this->subject;$this->text;$headers))
			echo "email zostal pomyslnie wyslany";
 
	}
 
}
 
 
 
 
?>
[PHP] pobierz, plaintext 

Ten post edytował hhg 7.01.2007, 04:17:14

[altruista]

ja mialem tak jak pliki *.php byly parsowane przez stara wersje php.

[koodlaty]

[PHP] pobierz, plaintext 
<?php
public $ip = $_SERVER['REMOTE_ADDR'];
public $host = $gethostbyaddr($this->ip);
?>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
public $text = nl2br("
		Wiadomosc od: $this->sender
		Firma: $this->firm
		Telefon: $this->telephone
		Imie i nazwisko: $this->name
 
		$this->text;
 
		IP nadawcy: $this->ip
		HOST nadawcy: $this->host
	");
?>
[PHP] pobierz, plaintext 

Pola klasy dekleruj na samym poczatku a ich warosci okreslaj w konstruktorze:

[PHP] pobierz, plaintext 
<?php
class site
{
	 public $ip;
	 public $host;
	 public $text;
 
	 function __construct()
	 {
		  $this -> ip = $_SERVER['REMOTE_ADDR'];
		  $this -> $host = $gethostbyaddr($this->ip);
		  $this -> $text = nl2br("
			 Wiadomosc od: $this->sender
			 Firma: $this->firm
			 Telefon: $this->telephone
			 Imie i nazwisko: $this->name
 
			 $this->text;
 
			 IP nadawcy: $this->ip
			 HOST nadawcy: $this->host
		  ");
	 }
 
	 ...
 
}
?>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
public function $mailto()
?>
[PHP] pobierz, plaintext 

Przekombinowałeś. Nazwy metod nie moga rozpoczynać się od - '$'


W treść klasynie nie wnikam... (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Poczytaj jeszcze trochę i kombinuj dalej... (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Ten post edytował koodlaty 7.01.2007, 10:23:59

[dr_bonzo]

Uruchamiasz kod pod php5 czy php4?

[hhg]

wszystko pod php5


dobra dzieki wprowadzilem poprawki i maile sa wysylane (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) (IMG:http://forum.php.pl/style_emoticons/default/exclamation.gif) ! niestety nie widac od kogo sa te maile tylko apache sa wstawiane jakby za zmienna $this->sender, ale to juz wina serwera prawda?

jeszcze mam do Was pytanie ktore zmienna i funkcje bezpieczniej byloby poprzedzic 'protected' lub 'private'

oto obecny kod

[PHP] pobierz, plaintext 
<?php
class mailing
{
 
	public $ip;
	public $host;
	public $text;
 
	public function __construct()
	{
		$this -> ip = $_SERVER['REMOTE_ADDR'];
		$this -> host = gethostbyaddr($this->ip);
		$this -> text = nl2br("
			Wiadomosc od: $this->sender
			Firma: $this->firm
			Telefon: $this->telephone
			Imie i nazwisko: $this->name
 
			$this->text
 
			IP nadawcy: $this->ip
			HOST nadawcy: $this->host
		");
	}
 
	public $subject;
	public $firm;
	public $name;
	public $telephone;
	public $sender;
	public $receiver;
 
	public function validate($attribute)
	{
		if (empty($attribute))
			return false;
		else
			return true;
	}
 
	public 
 
	public function mailto()
	{
 
		if (!($this->validate($this->subject) && $this->validate($this->firm) && $this->validate($this->name) && $this->validate($this->telephone) && $this->validate($this->sender) && $this->validate($this->receiver)))
		{
			echo "Wszystkie pola sa wymagane. Popaw wprowadzone dane!";
			exit();
		}
 
		$headers = "MIME-Version: 1.0 rn Content-type: text/html; charset=iso-8859-2 rn From: $this->sender rn";
 
		if (mail($this->receiver,$this->subject,$this->text,$headers))
			echo "email zostal pomyslnie wyslany";
 
	}
 
}
 
 
$mojmail = new mailing();
$mojmail -> subject = 'temat wiadomosci';
$mojmail -> firm = 'nazwa firmy qwrewrwer';
$mojmail -> name = 'Jan Kowalski';
$mojmail -> telephone = '666';
$mojmail -> sender = 'james@bond.com';
$mojmail -> receiver = 'MOJ@ADRES.PL';
$mojmail -> mailto();
?>
[PHP] pobierz, plaintext 

[Cysiaczek]

Skrypt raczej nie będzie przez to bezpieczniejszy. Zobacz do czego są te słowa kluczowe : http://www.php.net/manual/pl/language.oop5.visibility.php

Pozdrawiam.