|
From: Rod J. <rod...@in...> - 2003-06-23 17:59:37
|
> Regarding the implementation: I've noticed that the Clinic interface duplicates ClinicDAO's methods to a large extent, and that ClinicImpl adds cross-referencing and caching to the entities (data access aspects). This isn't really proper separation between data access code and business logic, but of course the borders are often somewhat blurring. In our case, it's probably more appropriate to merge the two, i.e. offer just a Clinic interface, with the common logic in an AbstractClinic base class, and a ClinicJdbcImpl default implementation. I agree with this. As Juergen says, it's most important to separate web and business code. Thanks very much for all you hard work on this, Ken. I'm unlikely to get a chance to look at the code in detail before Boston, but the first version was looking good and I'm sure you've continued to improve it substantially! Regards, Rod |