[PHP] Pierwszy skrypt na klasach - dobry ? Opcje

[Dominator]

Powoli przerzucam się z programowania strukturalnego na obiektowe, więc napisałem skrypt.
Dobrze napisałem ? Co zmienić/poprawić ?

[PHP] pobierz, plaintext 
<?php
class User
{
	public $_login;
	public $_password;
	public function Login($_login, $_password)
	{
		$this->login = $_login;
		$this->password = $_password;
		if($this->login == "123")
		{
			if($this->password == "456")
			{
				return "Logged";
			}
			else
			{
				return "Bad password";
			}
		}
		else
		{
			return "Bad username";
		}
	}
}
 
if($_GET['username'] && $_GET['password'])
{
	$user = new User;
	echo $user->Login($_GET['username'], $_GET['password']);
}
[PHP] pobierz, plaintext 

Ten post edytował Dominator 8.09.2012, 22:32:18

[SmokAnalog]

Pomijając wątpliwą logikę tego skryptu, sama struktura obiektowa jest w miarę poprawna poza jednym szczegółem. Mianowicie, deklarujesz w klasie pole (zmienne) $_login i $_password, a wewnątrz metody login() przypisujesz wartości do pól $login i $password. Błędu PHP nie wyrzuci, bo nie wymaga deklaracji pól, ale "niesmak pozostaje" (IMG:style_emoticons/default/smile.gif)

Pomysł ze zwracaniem łańcucha znaków w metodzie login() jest dosyć słaby, ale domyślam się, że dopiero zacząłeś przygodę z obiektowością i - sądząc po jakości kodu - z samym PHP, więc zapewne sam do tego dojdziesz.