Ok...
Ik heb wat in te halen...
Het leuke is dat Spinazie dit keer onderuit gehaald wordt...

$db moet private zijn omdat het geen parent heeft? huh?
Stel: je wilt Register extenden...
Je wilt een nieuwe method toevoegen en iets met de database doen...
Dan wil je dus iets met $this->db doen...
Maar dat kan niet als Register::$db private is, wel als het protected is...
Private is dus nog een extra 'bescherming', niet alleen tegen manipulatie van buiten de class, maar ook tegen manipulatie van child classes...
Verder snap ik de "hij heeft een parent, dus protected" niet...
Wat heeft de parent er aan?
Voor de parent bestaat die variabele dan helemaal niet, en als die em dan zou gebruiken zouden er fouten komen...
Ik gebruik protected alleen als ik wil dat children die waarde niet kunnen veranderen, of alleen via setters...

Hij moet singleton gebruken?
Elke keer?
$this->db is korter dan telkens Database::Connect()...
Database::Connect() lijkt me redelijk singleton, al kan de naam van de method iets beter...

Citaat

Het is trouwens niet nodig in een niet-statische methode een statische variabele te gebruiken, tenzij je van plan bent hem ooit nog zonder instantie aan te roepen.

Je grootste onzin tot nu toe! En ik heb heel wat onzin van je gezien...

Een class is geen object...
Een class is een blauwdruk voor een object...
Je hebt pas een object wanneer je het keyword "new" hebt gebruikt:
$boe = new Boe;

Een class mag best zelf dingen controleren...
Een validatie class kan handig zijn, maar hoeft niet...

Verder zou ik zelf geen Register class maken (wat Spinazie ook duidelijk probeert te maken), ik zou het eerder een User::register() method maken...
De controles voor de waardes tijdens het registreren zijn tijdens het updaten (User::update()) van een user ook nodig...

public function action_register()
	{
		Security::strangerAccess();


		TPL::assign('page', TPL::parse('categories/user/register'));


		if($this->request->param('id') == 'doregister')
		{




			try
			{
				if(!isset($_POST['username']) || !Verification::validUserName($_POST['username']))
					throw new Exception(TPL::cfg_get('lang_invalid_username'));


				if(!isset($_POST['password']) || !Verification::validPassword($_POST['password']))
					throw new Exception(TPL::cfg_get('lang_invalid_password'));


				if(!isset($_POST['password_repeat']) || $_POST['password_repeat'] != $_POST['password'])
					throw new Exception(TPL::cfg_get('lang_password_nomatch'));


				if(!isset($_POST['gender']) || !Verification::trueOrFalse($_POST['gender']))
					throw new Exception(TPL::cfg_get('lang_invalid_gender'));


				if(!isset($_POST['email']) || !Verification::validEmail($_POST['email']))
					throw new Exception(TPL::cfg_get('lang_invalid_email'));


				if(Model_User::exists('username', Security::protect($_POST['username'])))
					throw new Exception(TPL::cfg_get('lang_user_exists'));


				if(Model_User::exists('email', Security::protect($_POST['email'])))
					throw new Exception(TPL::cfg_get('lang_email_exists'));


				if(empty($_POST['code']) || !Verification::verifyCaptcha(strtoupper($_POST['code'])))
					throw new Exception(TPL::cfg_get('lang_invalid_captcha'));


				Model_Verification::setSuccess(TPL::cfg_get('lang_register_success'));
				$code = Model_User::create($_POST['username'], Security::encryptPass($_POST['password']), $_POST['email'], ($_POST['gender'] ? 'Female' : 'Male'), true);
				Model_Verification::sendActivationMail($_POST['email'], $_POST['username'], $code);
			}
			catch(Exception $e)
			{
				Verification::setError($e->getmessage());
			}
	
		}
	}

[offtopic]Bovenstaande code is auteursrechtelijk beschermd![/offtopic]

Dit heb ik erin staan. Waarschijnlijk is het niet perfect, aangezien ik ook nog lerende ben, maar toch iets beter dan wat jij neigt te doen.

[offtopic]Dos, kritiek via MSN! :p[/offtopic]

Nieuwe reactie samengevoegd met originele reactie op 26.03.10 19:14:50:
Hoewel je me op bepaalde punten uitgeluld hebt, wil ik nog wel iets toevoegen.

Citaat

Hij moet singleton gebruken?
Elke keer?
$this->db is korter dan telkens Database::Connect()...
Database::Connect() lijkt me redelijk singleton, al kan de naam van de method iets beter...

Ik bedoel niet dat het op dit moment niet voldoende was, maar meteen je query kunnen opbouwen is iets korter dan eerst Database::Connect() te moeten doen (zoiets zou ik in de header zetten).