3 ms·
Well written article, very clear. Nice job. The only thing that I do not like is that you would have to put some database field name in the controller to set d
by daok 17y ago
Well written article, very clear. Nice job.
The only thing that I do not like is that you would have to put some database field name in the controller to set data to the persistence instead of having a DataAccessLayer to handle it. The main problem is if in the future you refactor the persistence of change it, you will have to go to multiple place to be able to make everything work. I might be wrong... but at the first look, this is what I see. Oh by the way, do you always but your email address on each method of all your code :P ?
- leftnode 17y agoThanks for the comments. Most of that code was copied from my framework, which is full of Doxygen comments, and I use my username in our subversion repository and my email address for the author of each method since several methods can have multiple authors.
- jrockway 17y agoWhat value do you derive from documenting the author of each method? That seems like a lot of overhead for very little benefit, especially when your VCS can do an annotation and give you more detail than one comment can.