ATTENTION: I've decided to put the upgrade on hold due to a compatibility issue of our server environment with the latest CS installer package. CS 2008 now requires SQL Server 2005 as the backend DB but our database server currenlty has SQL Server 2000 installed on it. I'll resume the upgrade once I figure out when Telligent is releasing a patch to the schema compatibility issue. For now, we will continue to use the old version of CS while waiting for the said patch. If you have any questions about this process, please don't hesitate to post them on our forums and I'll answer them as soon as I can. Thanks for your patience and support guys! I'll let you know as soon as this is resolved. - Keith Rull

Answers to Skills Test Part 1

Nice! I guess we do have architecture guys here!

anyway, here's the answer to Skills Test Part 1

1.) The code closely resembles a factory method pattern... but mixed with the singleton flavor making it really unstructured. (nice catch cruizer!)

2.) the return statements really doesn't do anything at all since the operations are done internally(very good analogy punzie).

3.)this line Me.CreateClient(String.Empty, String.Empty, String.Empty) promotes redundancy... we dont need this anymore since the string are already initialized already as empty.

4.) there are no exposed constructors.(good job jokiz)

5.) the CreateClient could be the initialize method for the class.. but since it is private then there is no use for it.

6.) If its an actual singleton pattern, then.. we should have a static variable that would act as the instantiation object for the class just like what tuldoklambat has said.

any more additional analogy to the code?


Posted Feb 08 2006, 06:49 AM by keithrull

Comments

cruizer wrote re: Answers to Skills Test Part 1
on 02-09-2006 10:14 AM
that's why OOP in the hands of someone who does not understand it yet is a dangerous thing :P

i think the local community can do well to promote/educate devs on good OO design. i mean, i'd love to learn that.

Add a Comment

(required)  
(optional)
(required)  
Remember Me?

Enter the numbers above:

Copyright DevPinoy 2005-2008