[Php-objetos] Gerardo - prácticos aprobados
Leonardo Tadei - Pegasus Tech Supply
leonardot en pegasusnet.com.ar
Mie Feb 18 00:04:02 ARST 2009
Hola Gerardo!
Tenés ambos trabajos prácticos aprobados.
Felicitaciones!
Te cerró mejor ahora el uso de excepciones? Espero que sí.
Como detalle, noté que en Combustible hacés:
public function getTipo(){
if(!empty($this->tipo)){return $this->tipo;}
else{return false;}
}
Lo cual es correcto. Sin embargo, es siempre más útil en estos casos
lanzar una excepción que devolver "false", al viejo estilo, ya que
controlar errores de este modo significa poner cada getter en un IF, en
vez de encerrar todo un bloque semántico con try/catch y listo.
En otras palabras, tomar una política de gestión de errores y usar
siempre la misma.
Por otra parte, para tu código en particular, tal vez devolver vacío es
un default razonable para este getter, sin IF ni excepción ni nada.
Seguimos!
--
Leonardo Tadei
leonardot en pegasusnet.com.ar
http://blog.pegasusnet.com.ar
Firma pública: http://www.pegasusnet.com.ar/LeonardoTadei-public.key
Más información sobre la lista de distribución Php-objetos