[PHP]Klasy - problem początkującego. Opcje

[Rysh]

Sytuacja wygląda następująco, napisałem wcześniej niedużą stronę żeby podszkolić się w PHP, niestety pisałem wszystko na funkcjach. Po moim ostatnim temacie postanowiłem napisać wszystko od początku, już na obiektach. Jakoś nie mogę się do tego przyłożyć i zrozumieć. Czego nie napiszę wyskakuję błąd:

[PHP] pobierz, plaintext 
<?php
class user {
	private static $settings = array(
		'login_min' => 3,
		'login_max' => 16,
		);
	public static function zarejestruj ($login, $haslo, $rehaslo) {
		try {
			if ($haslo != $rehaslo) {
				throw new Exception("Podane hasła muszą być takie same.");
			}
			if (strlen($login) >= self::$setting['login_min'] && strlen($login <= self::$setting['login_max'])) {
				throw new Exception("Login musi mieć od ". self::$setting['login_min'] ." do ". self::$setting['login_max'] ." znaków.");
			}
		}
		catch(Exception $wyjatek) {
			echo $wyjatek -> getMessage(); 
		}
	}
}
?>
[PHP] pobierz, plaintext 

Fatal error: Access to undeclared static property: user::$setting in /home/rysh/public_html/class/user.class.php on line 13

Chciałbym zacząć od rejestracji, jeśli miałbym przykład dobrze wykonanej działającej funkcji później bym miał z górki, ponieważ mógłbym się wzorować na niej. Niestety już ta pierwsza sprawia mi problemu.

Posiadam również książkę "PHP5, radocha z programowania" z Helionu Steven'a Holzner'a - jednak tam jest opisane jak utworzyć Lwa oraz go wyświetlić. W rejestracji potrzebuje również sprawdzić przecież czy $login oraz reszta spełnia wszystkie wymagania, a tego już tam nie ma. Prawdopodobnie już teraz mam jakieś rażące błędy, proszę o jakieś nakierowanie mnie, abym w końcu mógł zacząć czerpać radość z programowania obiektowego

Rysh.

[marcio]

@up herezje mowisz przy static ma dac $this -> idz poczytaj najpierw manual.

W klasie masz pole statyczne $settings a odwolujesz sie do $setting i tam masz blad.

[PHP] pobierz, plaintext 
if (strlen($login) >= self::$setting['login_min'] && strlen($login <= self::$setting['login_max'])) {
//
 
[PHP] pobierz, plaintext 

Ten post edytował marcio 6.03.2010, 13:22:20

[MateuszS]

Rysh, napisz sobie też prostą klasę do wypluwania komunikatów (taki mini widok), bo wyjątków do wszystkich komunikatów się nie stosuje raczej, jak sama nazwa wskazuje, jest to wyjątek.

A co do mniejszych rzeczy, to może wczytaj np. w konstruktorze ustawienia, które miałbyś w osobnym pliku np. config.php, zamiast tworzyć tablicę statyczną.

Ten post edytował MateuszScirka 6.03.2010, 13:27:25

[Pilsener]

[PHP] pobierz, plaintext 
throw new Exception("Podane hasła muszą być takie same.");
[PHP] pobierz, plaintext 

- nie lepiej zwracać klasę komunikatu oraz jego id? Treść komunikatu to już sprawa bardziej interfejsu, czyli widoku. No i login chyba będzie sprawdzany nie tylko tutaj? Może jakaś klasa do walidacji? Przemyśl to na spokojnie, rozrysuj na papierze, zrób schemat co ma być w czym i co ma robić.

I radość czerpie się z dobrego programowania, każdy kod można spitolić, zamykanie kodu w klasach nie gwarantuje, że ktoś na jego widok nie dostanie palpitacji serca

[Quantum]

Poza tym, powinno się unikać stosowania bloku try-catch wewnątrz klasy, w metodzie rzuć tylko wyjątek, metodę uruchom w sekcji "try", poza klasą. przykład:

[PHP] pobierz, plaintext 
class A
{
 public function b($x = 0)
 {
   if($x < 5) throw new Exception('X nie może być mniejsze niż 5');
 }
}
 
$a = new A;
 
try {
 
 $a->b(1);
 
} catch (Exception $e) {
 
 echo $e->getMessage();
 
}
 
[PHP] pobierz, plaintext