query about StartableFacility extension methods

[Adam Langley]

Hi Team,

I have been using the StartableFacility during the registration of my service.
My service is publicly registered as an interface, but the 'start' method belongs to the concrete type.

When I register like this:

container.Register(
                Component.For<IQueryRemoteHandler>().ImplementedBy<ServerQueryRemotingHandler>()
                .StartUsingMethod(server => (server as ServerQueryRemotingHandler).Start)
                );

You see I have to cast my service before I can get to the 'start' method

This suggestion is really just about usability: Shouldn't the generic type argument for the 'StartUsingMethod' be the 'ImplementedBy' type, not the 'For' type?
The container knows the concrete type, so it should be able to pass that to the StartUsingMethod... not the interface

What do you think?

Adam Langley
Senior Developer

+64 9 486 9010
+64 21 884400
alan...@winscribe.com
www.winscribe.com

 Please consider the environment before printing this email!

[Krzysztof Koźmic]

it could be, but I'm not sure that would be possible without heavily
rebuilding the entire fluent API.

[Adam Langley]

Hey Krzysztof,

Thanks - I thought that might be the case, the nature of fluent and heavily nested generics...
But thought I would ask!

Adam Langley

--
You received this message because you are subscribed to the Google Groups "Castle Project Users" group.
To post to this group, send email to castle-pro...@googlegroups.com.
To unsubscribe from this group, send email to castle-project-u...@googlegroups.com.
For more options, visit this group at http://groups.google.com/group/castle-project-users?hl=en.

[Krzysztof Koźmic]

Also remember that if you want to avoid casting you may specify the name
of the method as string.

Always a trade off

[Adam Langley]

Absolutely - I figured you probably realised I wanted static checking... my main reason from moving away from configuration via Xml.

Thanks again,

Adam Langley

 Please consider the environment before printing this email!

[Jason Meckley]

Why not have your object inherit IStartable? you still get static checking and you don't need to be explicit about this when registering the component.

[Krzysztof Koźmic]

because we don't want to force anyone to inherit startable