springframework-developer

Fw: [Springframework-developer] NEW Petclinic version

[<jue...@we...>]

IA0KDQoJLS0tLS1VcnNwcsO8bmdsaWNoZSBOYWNocmljaHQtLS0tLSANCglWb246IGrDvHJnZW4g
aMO2bGxlciBbd2VyazNBVF0gDQoJR2VzZW5kZXQ6IFNvIDIyLjA2LjIwMDMgMjM6MjIgDQoJQW46
IEtlbiBLcmVicyANCglDYzogDQoJQmV0cmVmZjogUmU6IFtTcHJpbmdmcmFtZXdvcmstZGV2ZWxv
cGVyXSBORVcgUGV0Y2xpbmljIHZlcnNpb24NCgkNCgkNCglLZW4sDQoJIA0KCUkndmUganVzdCBh
ZGRlZCB5b3UgYXMgZGV2ZWxvcGVyLCBhbmQgY29tbWl0dGVkIHRoZSBkZW1vIHRvIHNhbXBsZXMv
cGV0Y2xpbmljLg0KCSANCglJJ3ZlIHJlc3RydWN0dXJlZCB0aGUgZGlyZWN0b3JpZXMgYSBiaXQs
IGludHJvZHVjaW5nIGEgZGlyZWN0b3J5IGZvciBidWlsZC10aW1lIGxpYnMgKHJlcGxhY2luZyB0
aGUgSjJFRSBSSSB3aXRoIGEgc2VydmxldC5qYXIpLCB3b3JraW5nIHdpdGggdGhlIHdhciBkaXJl
Y3RvcnkgZGlyZWN0bHkgKGZvciBlYXNpZXIgZGV2ZWxvcG1lbnQpLCBhbmQgdXNpbmcgdGhlIEpT
VEwgc3RhbmRhcmQgVVJJcyAod2l0aG91dCBleHBsaWNpdCBUTEQgcmVnaXN0cmF0aW9uIGluIHdl
Yi54bWwpLiBJdCdzIHZlcnkgZWFzeSB0byBkZXZlbG9wIHdpdGggdGhpcyBraW5kIG9mIHNldHVw
IGUuZy4gaW4gSW50ZWxsaUogSURFQSwgZXZlbiBlYXNpZXIgd2hlbiB1c2luZyBhIERyaXZlck1h
bmFnZXJEYXRhU291cmNlIGluc3RlYWQgb2YgYSBKTkRJIG9uZSAoSSd2ZSBpbmNsdWRlZCByZXNw
ZWN0aXZlIGRlZmluaXRpb25zIGluIGFwcGxpY2F0aW9uQ29udGV4dC54bWwsIGluIGNvbW1lbnRz
KS4NCgkgDQoJVGhlIHVzZXIgaW50ZXJmYWNlIGlzIHN0aWxsLCBlaG0sIHZlcnkgbWluaW1hbGlz
dGljLCBidXQgSSBndWVzcyB5b3UncmUgYXdhcmUgb2YgdGhhdCA7LSkgV2Ugc2hvdWxkIGF0IGxl
YXN0IGFkZCBsaXN0IG5hdmlnYXRpb24gdGhyb3VnaCBvd25lcnMsIHBldHMsIGFuZCB2aXNpdHMu
IEl0J3MgYSBiaXQgY3VtYmVyc29tZSB0byB1c2UgY3VycmVudGx5LCBidXQgb24gdGhlIG90aGVy
IGhhbmQgaXQncyBzdXBwb3NlZCB0byBiZSBhIGNvZGUgc3RydWN0dXJlIHNob3djYXNlIHJhdGhl
ciB0aGFuIHNvbWUgcHJldHR5IGV5ZWNhdGNoZXIuDQoJIA0KCVJlZ2FyZGluZyB0aGUgaW1wbGVt
ZW50YXRpb246IEkndmUgbm90aWNlZCB0aGF0IHRoZSBDbGluaWMgaW50ZXJmYWNlIGR1cGxpY2F0
ZXMgQ2xpbmljREFPJ3MgbWV0aG9kcyB0byBhIGxhcmdlIGV4dGVudCwgYW5kIHRoYXQgQ2xpbmlj
SW1wbCBhZGRzIGNyb3NzLXJlZmVyZW5jaW5nIGFuZCBjYWNoaW5nIHRvIHRoZSBlbnRpdGllcyAo
ZGF0YSBhY2Nlc3MgYXNwZWN0cykuIFRoaXMgaXNuJ3QgcmVhbGx5IHByb3BlciBzZXBhcmF0aW9u
IGJldHdlZW4gZGF0YSBhY2Nlc3MgY29kZSBhbmQgYnVzaW5lc3MgbG9naWMsIGJ1dCBvZiBjb3Vy
c2UgdGhlIGJvcmRlcnMgYXJlIG9mdGVuIHNvbWV3aGF0IGJsdXJyaW5nLiBJbiBvdXIgY2FzZSwg
aXQncyBwcm9iYWJseSBtb3JlIGFwcHJvcHJpYXRlIHRvIG1lcmdlIHRoZSB0d28sIGkuZS4gb2Zm
ZXIganVzdCBhIENsaW5pYyBpbnRlcmZhY2UsIHdpdGggdGhlIGNvbW1vbiBsb2dpYyBpbiBhbiBB
YnN0cmFjdENsaW5pYyBiYXNlIGNsYXNzLCBhbmQgYSBDbGluaWNKZGJjSW1wbCBkZWZhdWx0IGlt
cGxlbWVudGF0aW9uLg0KCSANCglJTU8sIHRoZSBhcHAgc2ltcGx5IGlzbid0IGNvbXBsZXggZW5v
dWdoIGZvciBzZXBhcmF0ZWQgYnVzaW5lc3MgYW5kIGRhdGEgYWNjZXNzIGxheWVycy4gVGhlIG1v
cmUgaW1wb3J0YW50IHRoaW5nIGlzIGNsZWFyIHNlcGFyYXRpb24gYmV0d2VlbiB0aGUgYnVzaW5l
c3MgYW5kIHRoZSB3ZWIgbGF5ZXI6IFRob3NlIHR3byBzaG91bGQgbmV2ZXIgYmUgbWVyZ2VkLCBu
b3QgZXZlbiBpbiB2ZXJ5IHNpbXBsZSBhcHBzLg0KCSANCglCVFcsIEkndmUgbWFkZSB5b3VyIExv
ZzRKIGNvbmZpZ3VyYXRpb24gbGV2ZXJhZ2Ugb3VyICJ3ZWJBcHBSb290S2V5IiBzdXBwb3J0LCBp
LmUuIGEgTG9nNGpDb25maWdMaXN0ZW5lciB0aGF0IGV4cG9ydHMgYSAicGV0Y2xpbmljLnJvb3Qi
IHN5c3RlbSBwcm9wZXJ0eSBzcGVjaWZ5aW5nIHRoZSB3ZWIgYXBwIHJvb3QgZGlyZWN0b3J5LCB1
c2luZyAiJHtwZXRjbGluaWMucm9vdH0vV0VCLUlORi9wZXRjbGluaWMubG9nIiBhcyBsb2cgZmls
ZSBwYXRoIGluIGxvZzRqLnByb3BlcnRpZXMuDQoJIA0KCVJlZ2FyZHMsDQoJSnVlcmdlbg0KCSAN
CgkgDQoNCgkJLS0tLS1VcnNwcsO8bmdsaWNoZSBOYWNocmljaHQtLS0tLSANCgkJVm9uOiBLZW4g
S3JlYnMgW21haWx0bzpra0Bra3RlYy5jb21dIA0KCQlHZXNlbmRldDogU28gMjIuMDYuMjAwMyAx
NDo0MyANCgkJQW46IGrDvHJnZW4gaMO2bGxlciBbd2VyazNBVF0gDQoJCUNjOiANCgkJQmV0cmVm
ZjogUmU6IFtTcHJpbmdmcmFtZXdvcmstZGV2ZWxvcGVyXSBORVcgUGV0Y2xpbmljIHZlcnNpb24N
CgkJDQoJCQ0KDQoJCUp1ZXJnZW4sDQoJCQ0KCQlNeSBuZXcgc291cmNlZm9yZ2UgYWNjb3VudCBu
YW1lIGlzIGtrdGVjLg0KCQkNCgkJSSd2ZSBhbHJlYWR5IGRpc2NvdmVyZWQgYSBjb3VwbGUgb2Yg
cHJvYmxlbXMgaW4gbXkgcG9zdGluZywgOj0oDQoJCQ0KCQkxLiBJbiBteSBGaW5kT3duZXJzQnlO
YW1lIGZ1bmN0aW9uYWxpdHksIEkgZm91bmQgdGhhdCBNeXNxbCBzZWFyY2hlcyBhcmUNCgkJbm90
IGNhc2Utc2Vuc2l0aXZlIGJ1dCBIc3FsIGFyZS4gSSBleHRlbmRlZCBteSBzZWFyY2ggYWxnb3Jp
dGhtIGJ1dCBwdXQNCgkJdGhlIGxvZ2ljIGluIEZpbmRPd25lcnNGb3JtIHdoZW4gaXQgc2hvdWxk
IHByb2JhYmx5IGdvIGluIGEgYnVzaW5lc3MNCgkJb2JqZWN0LiBBbHNvLCB0aGUgbG9naWMgZG9l
c24ndCB3b3JrIHByb3Blcmx5LiBJIGhhdmUgMiBzdHJhdGVnaWVzIEknbQ0KCQljb25zaWRlcmlu
ZyBmb3IgcmVzb2x2aW5nIHRoaXMuIFRoZSBmaXJzdCBpcyB0byBjb2VyY2UgbmFtZXMgc3RvcmVk
IGluDQoJCXRoZSBkYXRhYmFzZSB0byBhIGNhcGl0YWxpemVkIGZvcm0uIFRoZSBzZWNvbmQgaXMg
dG8gdXNlIEhTUUwncw0KCQlTZXRJZ25vcmVDYXNlIGZ1bmN0aW9uYWxpdHkgYmVmb3JlIGNyZWF0
aW5nIHRoZSB0YWJsZXMuIEkgYW0gbGVhbmluZyB0bw0KCQl0aGUgMm5kIGZvciBzaW1wbGljaXR5
LiBBcyBJIGRvbid0IGhhdmUgbXVjaCByZWFsIHdvcmxkIGRhdGFiYXNlDQoJCWV4cGVyaWVuY2Us
IGFueSBhZHZpY2UgeW91IGhhdmUgb24gaG93IHlvdSBoYXZlIGRlYWx0IHdpdGggdGhpcyB0eXBl
IG9mDQoJCXByb2JsZW0gd291bGQgYmUgYXBwcmVjaWF0ZWQuDQoJCQ0KCQkyLiBJbiBteSBDbGlu
aWNKZGJjREFPLCBJIGhhdmUgYSBudW1iZXIgb2YgaW5uZXIgY2xhc3NlcyB0aGF0IHByb3ZpZGUN
CgkJdGhlIGRhdGFiYXNlIHF1ZXJpZXMsdXBkYXRlcywgYW5kIGluc2VydHMuIENsaW5pY0pkYmNE
QU8gY3JlYXRlcw0KCQlpbnN0YW5jZXMgb2YgdGhlc2UgY2xhc3Nlcywgc3RvcmluZyByZWZlcmVu
Y2VzIHRvIHRoZW0gYXMNCgkJUmRibXNPcGVyYXRpb25zIHNvIHRoYXQgc3ViY2xhc3NlcyBtYXkg
c3Vic3RpdHV0ZSB0aGVpciBvd24gc3BlY2lhbGl6ZWQNCgkJdmVyc2lvbnMsIGluY2x1ZGluZyBT
dG9yZWRQcm9jZWR1cmVzLiBBIHN1YmNsYXNzIG11c3QgYWxzbyBvdmVycmlkZSB0aGUNCgkJcmVs
YXRlZCBDbGluaWNEQU8gQVBJIG1ldGhvZC4gSSBkaWRuJ3QgYWRkIHRoZSBzZXR0ZXJzL2dldHRl
cnMgbmVlZGVkDQoJCWZvciBzdWJjbGFzc2VzIHRvIHVzZSB0aGVtLiBBbHNvLCB0aGUgYWZ0ZXJQ
cm9wZXJ0aWVzU2V0IG1ldGhvZCB3aWxsDQoJCW5lZWQgdG8gYmUgY2hhbmdlZCB0byBvbmx5IGFk
ZCB0aGUgZGVmYXVsdCB2ZXJzaW9ucyBmb3IgdGhlc2UgcHJvcGVydGllcw0KCQl0aGF0IGhhdmVu
J3QgYmVlbiBzZXQuIEkgd2lsbCBhbHNvIG1vdmUgdGhlIGlubmVyIGNsYXNzZXMgb3V0c2lkZSB0
bw0KCQlwYWNrYWdlIGxldmVsLiBUaGlzIGV4dHJhIGxldmVsIG9mIGNvbXBsZXhpdHkgaXNuJ3Qg
cmVhbGx5IG5lZWRlZCBmb3INCgkJdGhlIGRlbW8gYnV0IEkgdGhpbmsgaXQgc2hvd3Mgc29tZSBv
ZiBTcHJpbmcncyBzdHJlbmd0aHMuIEkgYWxzbyB3YW50ZWQNCgkJdG8gc2hvdyB0aGUgcG9pbnQg
dGhhdCB0aGUgdGhlc2Ugb2JqZWN0cyBhcmUgdGhyZWFkc2FmZSwgcmV1c2FibGUgaXRlbXMNCgkJ
dGhhdCBzaG91bGQgYmUgY3JlYXRlZCB1cGZyb250IHdpdGggdGhlIFNRTCBjb21wbGllZCBvbmx5
IG9uY2UuDQoJCQ0KCQlSZWdhcmRzLA0KCQkNCgkJS2VuDQoJCQ0KCQkNCgkJasODwrxyZ2VuIGjD
g8K2bGxlciBbd2VyazNBVF0gd3JvdGU6DQoJCQ0KCQk+SGkgS2VuLA0KCQk+DQoJCT5JJ2xsIHJl
dmlldyB0aGUgZGVtbyBhcHAgdG9kYXkuIEV4cGVjdCB0byBnZXQgc29tZSBwcm9tcHQgZmVlZGJh
Y2sgOi0pDQoJCT4NCgkJPkluIHRoZSBjb3Vyc2Ugb2YgdGhpcywgSSdsbCBhZGQgUGV0Y2xpbmlj
IHRvIENWUywgaW4gc2FtcGxlcy9wZXRjbGluaWMgKGFuYWxvZ291cyB0byBKUCdzIHNhbXBsZXMv
cGFnZWRsaXN0KS4gSSBzdWdnZXN0IHRvIGFwcGx5IGFueSBtb2RpZmljYXRpb25zIHRoZXJlLCBp
bnN0ZWFkIG9mIHNoYXJpbmcgZXZlcnl0aGluZyB2aWEgbWFpbCBhdHRhY2htZW50cy4NCgkJPg0K
CQk+SSd2ZSBub3RpY2VkIHRoYXQgeW91IHN0aWxsIGRvbid0IGhhdmUgQ1ZTIGFjY2Vzcy4gRG8g
eW91IGFscmVhZHkgaGF2ZSBhIFNvdXJjZUZvcmdlIGFjY291bnQ/IElmIG5vdCwgcGxlYXNlIGNy
ZWF0ZSBvbmUuIEp1c3QgdGVsbCBtZSB5b3VyIHVzZXJuYW1lLCBhbmQgSSdsbCBhZGQgeW91IGFz
IGRldmVsb3BlciENCgkJPg0KCQk+RmluYWxseSwgdGhhbmtzIGZvciB5b3UgYWxsIHlvdXIgZWZm
b3J0cyEgQSBwcm9wZXIgZGVtbyBhcHAgaXMgdmVyeSB2YWx1YWJsZSBmb3IgdXMsIGV2ZW4gbW9y
ZSBmb3IgaW5jbHVzaW9uIGluIGEgZGlzdHJpYnV0aW9uIHRoYW4gZm9yIHB1YmxpYyBhdmFpbGFi
aWxpdHkgb24gdGhlIHdlYnNpdGUuDQoJCT4NCgkJPlJlZ2FyZHMsDQoJCT5KdWVyZ2VuDQoJCT4N
CgkJPg0KCQk+DQoJCT4gICAgICAgLS0tLS1VcnNwcsODwrxuZ2xpY2hlIE5hY2hyaWNodC0tLS0t
DQoJCT4gICAgICAgVm9uOiBLZW4gS3JlYnMgW21haWx0bzpra0Bra3RlYy5jb21dDQoJCT4gICAg
ICAgR2VzZW5kZXQ6IFNhIDIxLjA2LjIwMDMgMjE6MTANCgkJPiAgICAgICBBbjogJ3NwcmluZy1k
ZXYtbGlzdCc7IFRob21hcyBSaXNiZXJnDQoJCT4gICAgICAgQ2M6DQoJCT4gICAgICAgQmV0cmVm
ZjogW1NwcmluZ2ZyYW1ld29yay1kZXZlbG9wZXJdIE5FVyBQZXRjbGluaWMgdmVyc2lvbg0KCQk+
ICAgICAgDQoJCT4gICAgICANCgkJPiAgICAgICBUbyBUaG9tYXMsDQoJCT4gICAgICANCgkJPiAg
ICAgICBDYW4geW91IGluc3RhbGwgdGhlIGRlbW8gZG93bmxvYWQgZmlsZSBhbG9uZyB3aXRoIGEg
cnVubmluZyB2ZXJzaW9uIG9uIHRoZSBTcHJpbmcgd2Vic2l0ZSwgbWFya2luZyBpdCBhcyBwcmVs
aW1pbmFyeSA/DQoJCT4gICAgICANCgkJPiAgICAgIA0KCQk+ICAgICAgIFRvIEFsbCwNCgkJPiAg
ICAgIA0KCQk+ICAgICAgIEkgYW0gcGxlYXNlZCB0byBhbm5vdW5jZSB0aGF0IEkgbm93IGhhdmUg
cmVhZHkgZm9yIGRvd25sb2FkLCBhIG5ldyB2ZXJzaW9uIG9mIHRoZSBQZXRjbGluaWMgZGVtbyBh
cHAgPGh0dHA6Ly93d3cuZ2xvYmFsZGlhbG9nLmNvbS8lN0Vra3JlYnMvZmlsZXMvcGV0Y2xpbmlj
MS0xLjAuemlwPiAsIGFwcHJveC4gMS45TUIuDQoJCT4gICAgICAgQmUgc3VyZSB0byByZWFkIHRo
ZSBmaWxlLCBwZXRjbGluaWMudHh0IHdoaWNoIGlzIGF0dGFjaGVkIGFuZCBhbHNvIGlzIGluIHRo
ZSBwcm9qZWN0J3Mgd2Vic3JjIGRpcmVjdG9yeS4gVGhpcyBmaWxlIGNvbnRhaW5zIG5vdGVzIHJl
Z2FyZGluZyB0aGUgZGVzaWduLCBpbXBsZW1lbnRhdGlvbiwgYW5kIHVzYWdlLg0KCQk+ICAgICAg
DQoJCT4gICAgICAgSSAgYW0ganVzdCBnZXR0aW5nIHN0YXJ0ZWQgb24gdGhlIGFjY29tcGFueWlu
ZyB0dXRvcmlhbC4gSSB3aWxsIHBvc3QgYml0cyBhbmQgcGllY2VzIG9mIGl0IGFzIHRoZXkgYmVj
b21lIGF2YWlsYWJsZSBmb3IgcmV2aWV3LiBDYW4gSSBnZXQgYW55IHZvbHVudGVlcnMgdG8gaGVs
cCB3aXRoIHRoZSByZXZpZXdpbmcgPz8/DQoJCT4gICAgICANCgkJPiAgICAgICBJIGFtIGFsc28g
aGFwcHkgdG8gc2F5IHRoYXQgU291cmNlZm9yZ2Ugc2VlbXMgdG8gaGF2ZSB3b3JrZWQgb3V0IHRo
ZWlyIGFub255bW91cyBhY2Nlc3MgcHJvYmxlbXMgc28gaXQgaXMgdXNpbmcgYSBwcmV0dHkgdXAt
dG8tZGF0ZSB2ZXJzaW9uIG9mIFNwcmluZy4NCgkJPiAgICAgIA0KCQk+ICAgICAgIEkgaGF2ZSBp
dCBydW5uaW5nIHRlbXBvcmFyaWx5IG9uIGEgc21hbGwsIHByaXZhdGUgc2l0ZS4gSWYgeW91IHdp
c2ggdG8gZ2V0IHRoZSBVUkwsIHNlbmQgbWUgYW4gZW1haWwuDQoJCT4gICAgICANCgkJPiAgICAg
IA0KCQk+ICAgICAgIFRvIE15c2VsZiwNCgkJPiAgICAgIA0KCQk+ICAgICAgIFRpbWUgaXMgcnVu
bmluZyBvdXQgYmVmb3JlIHRoZSBzeW1wb3NpdW0gKGFuZCBJIGxlYXZlIGZvciBhIHdlZWsncyBj
YW1waW5nIGhvbGlkYXkgaW4gdGhlIGdyZWF0IG5vcnRoIHdvb2RzIG9mIFdpc2NvbnNpbiksDQoJ
CT4gICAgICAgc28gSSBoYXZlIHRvIGZvY3VzIG9uIHRoZSB0dXRvcmlhbC4NCgkJPiAgICAgIA0K
CQk+ICAgICAgIFJlcGVhdCBhZnRlciBtZToNCgkJPiAgICAgICBJIG11c3Qgc3RvcCBkZXNpZ25p
bmcsIGNvZGluZywgdHdlYWtpbmcuDQoJCT4gICAgICAgSSBtdXN0IHN0b3AgZGVzaWduaW5nLCBj
b2RpbmcsIHR3ZWFraW5nLg0KCQk+ICAgICAgIEkgbXVzdCBzdG9wIGRlc2lnbmluZywgY29kaW5n
LCB0d2Vha2luZy4NCgkJPiAgICAgICAuLi4gOi0pDQoJCT4gICAgICANCgkJPiAgICAgIA0KCQk+
ICAgICAgDQoJCT4gICAgICAgSSB2ZXJ5IG11Y2ggd2VsY29tZSBhbGwgY29tbWVudHMsIHN1Z2dl
c3Rpb25zLCBpZGVhcywgY3JpdGljaXNtcywgYW5kIGV0Yy4NCgkJPiAgICAgIA0KCQk+ICAgICAg
IEtlbg0KCQk+DQoJCT4/Pz8/Pz8/Pz8/Pz8/Pz8/Pz8/Pz8/Pz8/Pz8/Pz8/Pz8/Pz8/Pz8/P8OT
woYrEhc/Xj/DqcKawopYPz8/Jz8/P3U/PyDDl8KtUCBtentiP3U/bDc/elopej8/Pz9bWj8/PwU/
KD8/JyDDl8KtUHosPyk/PT8/Pz9FP8Oewqw3Pz/DhsKtecOUwp4/Pz8xP8OawoE/P3pvw5jCosOh
wp7Ct10/Mj8/XD8/aT8/w4jCqD8/ej8DecOYwpxqw5fCnTE/w5rCgT9HPz9iPwhtPz8/PyAwPynD
nsK2Pz96Pz9qP2d6Pz/Dl8KxPz9pPz8/Pz8/Pz8/Pz8/Pz8/Pz8/Pz8/Pz8/Pz8/Pz8/Pz8/Pz/D
ksKmPz8/P8Oawpk/KD9PP3rDr8K/wr0/Pz8/P3glPz9SPz8/Pz/DmsKZPyg/Tz96w6/Cv8K9Pz9Y
Pz8/Pz8/eT8rPz8/ej9tPz8/Pz9iPz8/Pz8/cT8/Bz8/P8OBwr8/Ky0/P2I/w5jCp34/w6zCpsK4
Pz8/w5rCmT8oP08/esOvwr/CvT8NCgkJPg0KCQk+DQoJCT4gDQoJCT4NCgkJDQoJCQ0KCQkNCg0K

Re: [Springframework-developer] NEW Petclinic version

[Rod J. <rod...@in...>]

> 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

[Springframework-developer] NEW Petclinic version

[Ken K. <kk...@kk...>]

I will make these changes.

Ken

Rod Johnson wrote:

>>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
>
>
>
>
>-------------------------------------------------------
>This SF.Net email is sponsored by: INetU
>Attention Web Developers & Consultants: Become An INetU Hosting Partner.
>Refer Dedicated Servers. We Manage Them. You Get 10% Monthly Commission!
>INetU Dedicated Managed Hosting http://www.inetu.net/partner/index.php
>_______________________________________________
>Springframework-developer mailing list
>Spr...@li...
>https://lists.sourceforge.net/lists/listinfo/springframework-developer
>
>
>  
>