-----Original Message----- From: Christopher Lenz [mailto:[EMAIL PROTECTED] Sent: 24 February 2003 12:02 To: Cactus Developers List Subject: Re: [Proposal] Package Refactoring Proposal
Vincent,
I think it'd be interesting to get a JDepend report into our framework build process. That would help us identify problems with the current package structure as well as with any structure we'll be moving to.
It's pretty easy to do (even without using Maven ;-) ), and I'd give it a shot when a find some spare hours. What do you think?
Hum... One of the reason I wanted a cleaner package structure is that I
needed to perform some refactoring to decouple classes.
For example, I would like to be able to use the FormAuthentication class both from a ServletTestCase and from a StaticTestCase (a Static test case simply calls a URL and has no testXXX() methods - only begin and end ones). However, FormAuthentication requires a WebRequest and does not currently work with a BaseWebRequest.
Thus, we should not rely too much on the current dependencies as they may not be correct evyerwhere.
That said, I have never been able to get anything useful from jdepend
(not that it isn't nice, just that I always don't know what to do after
reading the report).
Yeah, it's a bit tricky. Definitely easier to use on projects with smaller codebases, now that I've starting letting it go on Cactus :-P
That said, it's just a metric, and it *may* help us figure out problems with the package structure. For example, I very much like avoiding cyclic dependancies between packages. JDepend says Cactus has quite a few of those, but I haven't nailed them all down yet.
For example, there's a cyclic dependancy between o.a.c and o.a.c.util. That seems to come from the o.a.c.util.AbstractWebConfiguration and o.a.c.util.FilterConfiguration classes. Shouldn't these move into the o.a.c.configuration package like the others?
If you could demonstrate it is useful for us, sure please go ahead! :-)
Not sure yet ;-)
-- Christopher Lenz /=/ cmlenz at gmx.de
--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
