"Długi" 'if', czy wyjątki? Opcje

[404]

[PHP] pobierz, plaintext 
if
		(
			isset($_GET['a']) && validate($_GET['a'], '/^[a-zA-Z0-9\-]+$/') &&
			isset($_GET['b']) && validate($_GET['b'], '/^[a-zA-Z0-9\-]+$/') &&
			isset($_GET['v']) && validate($_GET['v'], '/^[0-9]{2}-[0-9]{2}-[0-9]{4}$/') &&
			isset($_GET['x']) && validate($_GET['x'], '/^([a-z0-9\!\&\-\ ]+)$/') &&
			$event = $this->event->get_details($_GET['a'], $_GET['b'], $asd)
		)
{
//...
}
[PHP] pobierz, plaintext 

No niestety, ale jestem zmuszony do sprawdzenia tylu rzeczy. Pytanie tylko, czy zostawić to tak jak jest, czy skorzystać z wyjątków... Jak Wy radzicie sobie z tak masakrycznym kodem? (IMG:style_emoticons/default/tongue.gif)

[viking]

Za pomocą klas. Załóżmy abstrakcyjnie coś w stylu:

[PHP] pobierz, plaintext 
/model/forms/xxx
$form = new Validator();
$form->addValidator('pole1', array(new Validator_Regex)->addValidator('pole2', array(new Validator_Regex)
[PHP] pobierz, plaintext 

W kontrolerze:

[PHP] pobierz, plaintext 
$f = new Model_Forms_Xxx();
if ($f->dataValid()}...
[PHP] pobierz, plaintext 

[d3ut3r]

[PHP] pobierz, plaintext 
isset($_GET['a']) && validate($_GET['a'], '/^[a-zA-Z0-9\-]+$/')
[PHP] pobierz, plaintext 

Dlaczego funkcja validate nie sprawdzi czy zmienna istnieje ? wtedy odchodzi Ci isset i już nie będzie tak skomplikowane (IMG:style_emoticons/default/smile.gif)

[no_face]

Lol, moze dlatego, ze zabezpiecza sie przed dostepem do nieistniejacego geta?

Chyba, ze miales na mysli takie wywolanie:

[PHP] pobierz, plaintext 
validate('a', '/^[a-zA-Z0-9\-]+$/')
[PHP] pobierz, plaintext 

[d3ut3r]

Lol, moze dlatego, ze zabezpiecza sie przed dostepem do nieistniejacego geta?
Chyba, ze miales na mysli takie wywolanie:

nie miałem (IMG:style_emoticons/default/biggrin.gif) , ehh no tak notice .... w takim razie sorry za wprowadzenie w błąd (IMG:style_emoticons/default/wstydnis.gif)

[404]

@viking: Twoje rozwiązanie wygląda na bardzo dobre. Będę musiał to przemyśleć i sprawdzić jak będzie się to sprawdzać w moim przypadku. Dzięki (IMG:style_emoticons/default/smile.gif)
@d3ut3r: nie wiem czemu nie przyszło mi to do głowy (IMG:style_emoticons/default/smile.gif) Dzięki za podsunięcie pomysłu (IMG:style_emoticons/default/smile.gif)

[Niktoś]

Wydaje mi się, że w tym przypadku lepiej by było skorzystać z bloku try catch.