[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

[Szymciosek]

Jaki jest sens powiadamiania użytkownika, że tylko hasło jest złe ?

Będzie w bazie użytkownik XXX, ja się o tym dowiem, bo podając dobry login, a złe hasło dostanę taką informację i później tylko zgadywać hasło...