Tiered development and OO are 2 different things.
I do not think your code is circular, because circular references usually
bomb out.
Presentation calls the BusinessLayer
BusinessLayer calls the DataLayer
DataLayer returns something (DataSet, IDataReader, Scalar) to the
BusinessLayer.
BusinessLayer uses the return object(DataSet,IDataReader,Scalar) to do
something usually. (Create a collection, check values, sometimes it just
shoots the DataSet up to the Presentation layer)
Presentation Layer "presents" the thing it got from the BusinessLayer.
Sometimes developers use the same names for the method calls.
I wouldn't un-architect this....until you understand what it does..and if
they are perhaps just using a decent 3tier design.
...
"riga" <ri**@discussions.microsoft.com> wrote in message
news:C5**********************************@microsof t.com...
Hi,
I was given an ASP.NET (C#) application to modify.
The code looks really strange.
It's supposed to do simple things but it has
Facade/BusinessRules/Common/DataAccess layers.
For example,
simple method in Navigate.ascx.cs is instantiating a new object
and calls obj1.COBdateInit().
obj1 does exactly the same thing.
It call obj2.COBDateInit()
obj2 does the same thing again, it call obj3.COBdateInit()
and that one retrieves data.
Is it OOP? Why not to get data at the first step?
Isn't it an overkill?
Thanks,
RO