[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

[-mas-]

no i przydaloby sie zmienne $_login i $_password ustawic jako private. Nie wiem co to jest $this->login skoro w klasie masz zadeklarowane zmienne $_login i $_password

podając w konstruktorze zmienna $_login musisz ja przypisac do zmiennej w klasie w twoim przypadku o takiej samej nazwie
$this->_login=$_login;