Article instance no longer synchronized with the corresponding row in a DB table

carewithl · October 13th, 2009, 02:22 PM

hi
Page 233 â Instance method Article.Approve also sets Articles.Approved property to true:

Code:

        public bool Approve()
        {
            bool success = Article.ApproveArticle( this.ID );
            if (success)
               this.Approved = true;
            return success;
        }

So why donât instance methods Article.IncrementViewCount and Article.Rating also set View, TotalRating and Votes properties accordingly? Since they donât set these properties, whenever we rate an article or increment its view count, this Article instance is no longer synchronized with the corresponding row in a DB table

Cheers

Lee Dumond · October 13th, 2009, 06:07 PM

I suspect that this is because if an article is approved, you want that to be reflected in the object right away. However, that behavior may not be desirable for other properties such as ViewCount or AverageRating.

For example, let's say an article has been viewed 57 times. When a user pulls it up, the ViewCount will be incremented to 58 in the database, however, the page will still show 57. In other words, it is showing what the ViewCount was before the current user viewed it, which is probably the behavior the author intended.

Whether or not you agree with this behavior is a matter of preference. Either way is valid.

By the way, I wouldn't sweat the instance properties in the BLL too much, as most of them are never actually used anyway.

carewithl · October 14th, 2009, 01:23 PM

thanx mate

cheers