CHAT
Search
 
 

Display results as :
 


Rechercher Advanced Search

Latest topics

Detalles

View previous topic View next topic Go down

Detalles

Post  Carlos Mora on Tue Jul 27, 2010 3:31 am

Andrés,

recibe mis más sinceras felicitaciones, la clase está muy bien organizada y codificada, el código es elegante y fácil de leer.
A modo de sugerencia, te comento 2 detalles mínimas que tal vez puedas considerar interesantes:
1) Reemplazar el uso de ClassName por IsKindOf: es un verdadero problema cuando haces adaptaciones al código existente usando clases propias, porque te obliga a tocar el código original, lo que hace que si hay una nueva versión tengas que modificar nuevamente el código.
2) En el mismo sentido, usar 2 niveles de clases por cada clase, tal como lo hace Xailer y FreeWin. Las ventajas con las mismas que en el caso anterior: se pueden aplicar modificaciones a través del subclaseado sin perder compatibilidad con versiones futuras. Esto es un patrón de software que se llama Variaciones Protegidas, seguro lo encuentras en la wikipedia.
Estos cambios se los comenté a Antonio Linares, y al menos el IsKindOf está reemplazando lentamente a ClassName en las nuevas versiones.
Voy a ver si hago algunas pruebas y lo que sea te comento.

Un saludo

Carlos Mora

Posts : 7
Reputation : 0
Join date : 2010-07-27

View user profile

Back to top Go down

Re: Detalles

Post  Andres Reyes on Tue Jul 27, 2010 10:06 am

Carlos,

Gracias, tomare muy en cuenta tus comentarios,

Saludos,

Andres Reyes
avatar
Andres Reyes
Admin

Posts : 38
Reputation : 3
Join date : 2010-06-23
Age : 37
Location : Mazatlan, Sinaloa, Mexico

View user profile http://codigo-base.blogspot.com

Back to top Go down

View previous topic View next topic Back to top


 
Permissions in this forum:
You cannot reply to topics in this forum