inline

2010/3/10 Krzysztof Koźmic <[email protected]>

> you put interceptors in whatever order you please.
>

how?


> between selectors order of selectors transfers to order of
> interceptors. Plus I think you should not have multiple selectors for
> one model so it's a non-issue anyway.
>

why? I'm using multiple selectors to apply different and unrelated concerns
to models. Do you think I should centralize a bunch of unrelated stuff into
the same class?


> in addition I'm against passing interceptors selected by one selector,
> to subsequent selectors.
>

this is the current code. how do you apply order? how do you remove
interceptors?

  foreach(IModelInterceptorsSelector selector in selectors)
  {
  InterceptorReference[] interceptors = selector.SelectInterceptors(model);

+ if (interceptors == null)
+ {
+ continue;
+ }
+
+ foreach (InterceptorReference interceptor in interceptors)
+ yield return interceptor;
  }
+
+ foreach (InterceptorReference interceptor in model.Interceptors)
+ yield return interceptor;
  }


>
> 2010/3/10 Simone Busoli <[email protected]>:
> > Except that I don't agree with this principle, the return value doesn't
> let
> > you specify where exactly to put the interceptor. So the return value
> > provides a subset of the functionality provided by the input collection.
> >
> > 2010/3/10 Krzysztof Koźmic <[email protected]>
> >>
> >> void methods should not modify their arguments
> >
> > --
> > You received this message because you are subscribed to the Google Groups
> > "Castle Project Users" group.
> > To post to this group, send email to
> [email protected].
> > To unsubscribe from this group, send email to
> > [email protected]<castle-project-users%[email protected]>
> .
> > For more options, visit this group at
> > http://groups.google.com/group/castle-project-users?hl=en.
> >
>
> --
> You received this message because you are subscribed to the Google Groups
> "Castle Project Users" group.
> To post to this group, send email to [email protected]
> .
> To unsubscribe from this group, send email to
> [email protected]<castle-project-users%[email protected]>
> .
> For more options, visit this group at
> http://groups.google.com/group/castle-project-users?hl=en.
>
>

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

Reply via email to