|
jschell wrote: Perhaps you misunderstood what I was saying.
Perhaps so. Typically when I hear people refer to "in my (your) experience" they are referring broadly to the sum total of all experiences, not narrowly to the one under discussion. I apologize.
As to the rest of our conversation, we're going meta into the importance/relevance of the question relative to other things and away from my original question. I think we view things through different lenses and just disagree, which is cool. Different perspectives provide food for thought.
Thanks for the feedback.
|
|
|
|
|
According to Robert C. Martin's Clean Code, use different functions: the function names give you additional information on the purpose, thus making the code better readable.
|
|
|
|
|
According to Eddy, it is not DRY.
Bastard Programmer from Hell 
If you can't read my code, try converting it here[^]
|
|
|
|
|
Interesting. How do you see multiple methods violating DRY?
|
|
|
|
|
Harley L. Pebley wrote: How do you see multiple methods violating DRY?
Multiple methods do not violate the principle. Repeating a specific value in a methodname, while you have declared an enum that holds the same information would; aight, granted, DRY does not directly apply to method-names.
It is as effective as copying sprocs to a slightly different name and changing a single integer that's updated. Even if you do not copy the sproc-body, you'd still end up with more securables and more failure-points.
The compiler will also not warn about missing methods when someone extends the enum. The biggest difference is that it makes it impossible to use the method in a loop, passing all the enum-values (and that might be a good thing)
From another perspective;
A boolean can be viewed as an enum with two values. I can see the value of methodnames like "Hide" and "Show", even with the existence of the Visible-property. That's propably because their semantics add value; it wouldn't if those methods were called "VisibleTrue" and "VisibleFalse" - that's just a crappy way of saying "Visible = false".
..then again, I upvoted jschell; the compiler will accept both, and there is no "one true way" of doing things. Anything that helps one to maintain the code is welcome.
Bastard Programmer from Hell
If you can't read my code, try converting it here[^]
|
|
|
|
|
Eddy Vluggen wrote: Repeating a specific value in a method name, while you have declared an enum that holds the same information would;
Ah, right. I agree, it would be pretty confusing to have the name and an enum both. Sorry, I guess in my original post I didn't make it clear that I intended those to be mutually exclusive options.
Thanks.
|
|
|
|
|
Well, the title kinda does that 
Bastard Programmer from Hell
If you can't read my code, try converting it here[^]
|
|
|
|
|
Thanks for the pointer to Clean Code. I obviously need to go reread that as I'd quite forgotten RCM had discussed this topic.
|
|
|
|
|
In Java I use the log.Log(Level.type, message) and
In C# I use the log.LogWarning(message) or another.
I find it pretty annoying in Java. They clearly made this better choice - better choice in relation to design e.g extensibility and maintainability. However, I need to work more with the Editor when writing this (more choices and Level import).
In this case I would do as Java does, but also add most common usages e.g. LogDebug, LogWarning, LogError, etc.
I guess it also is worth to consider how many options the enum will represent. The answer to this question might not be the same for "small" enums and "large" enums.
I like this question btw.
Another question, is whether to use enums or classes as arguments, but ... that is another question 
|
|
|
|
|
Thanks for sharing your experience using both approaches in two different environments.
Keld Ølykke wrote: I find it pretty annoying in Java. They clearly made this better choice...
These two statements seem to conflict with each other. If the one approach is "pretty annoying" is it really "better?" And this is reason I raise the question: I have the same conflict internally. Which approach is "better" seems to come down to where you approach the problem from. One day I look at the problem from one perspective and one way seems "better." The next day I look at the code and say "How crappy! This way is better!" What I optimize for changes the context enough to change the answer.
Keld Ølykke wrote: Another question, is whether to use enums or classes as arguments, but ... that is another question
Yes. Yes it is. 
|
|
|
|
|
I'm looking for a CMS (Umbraco) doc type etc. structure for agile (mostly) project collaboration and eventually management. My initial idea is for a content type structure as follows:
Project Home
Project Area
Feature
Documents like overview and description, as well as miscellaneous artefacts.
User stories, Use Cases, Narative specs, etc.
Unclassified Project Documents, with attachements etc.
Then, for the User Stories, I integrate with YouTrack for fine grained, micro-tracking and developer assignment. Time tracking as well, that aggregates up to the Feature (or Story) on Umbraco.
That's my basic prototype. Any suggestions, warnings, anecdotes, or any other feedback would be great. This is a first iteration before demo'ing to the boss and team, and Umbraco is fairly flexible if not too customised.
|
|
|
|
|
|
The fourth result in that list is my duplicate of this question on StackOverflow. The range of results is simply too broad. I am looking for feedback from people who have implemented such a structure, particularly on Umbraco.
|
|
|
|
|
I am editing this post to remove the sample code as I no longer think it would be a good design.
After more research, it appears that the Data Access Object Design Pattern is a good solution to separate a Data Access API from the business objects.
Example: http://www.tutorialspoint.com/design_pattern/data_access_object_pattern.htm
Comments are welcome.
Thanks!
Carlos.
Original Question: What is the best Design Pattern to separate data access from the business layer?
modified 14-Feb-14 12:03pm.
|
|
|
|
|
Why is it necessary to separate the data layer from the business layer? Are you thinking of changing the data repository? Is there another reason?
|
|
|
|
|
Just to counter the other inference in the other response there are good reasons for having a data layer.
It helps to start with a data model. A data model represents data in the application and although it will often be one to one with database tables/objects that isn't always the case.
Once you have that then your data base layer is created to expose that data model.
This is common enough that there are already generic frameworks for it. NHibernate is one and there are others.
In my experience it is a bad idea to use inheritance as you have suggested. Data models seldom represent inheritance and implementing it like that even when it seems like such models are there is often a problem. This of course is compounded by a poor design where inheritance is used rather than a better choice of attributes and/or composition.
You data layer should not expose anything about the database itself. Often it will be better to have helper classes that do the real work and your exposed classes use those to do the real work.
|
|
|
|
|
Carlos Merighe wrote: ANY COMMENTS OR SUGGESTIONS?
STOP SHOUTING!
Carlos Merighe wrote: PLEASE BE SPECIFIC.
STOP POSTING TEXT IN ALL-CAPS!
You weren't specific with wich type of comment you expected, so my apologies, I'm grumpy, ignore it if you're not in a good mood.
What's the difference between Oracle and SQL? Does Oracle not use SQL? Did you mean "SQL Server" or "MySQL"?
Second, you don't want two different implementations to load data from a DataProvider that shares an interface; one can handle both scenario's using a db-agnostic interface (the IDbCommands) - there may be variations in the queries, but the infrastructure would be the same. Any ORM worth it's money would provide the same functionality. ORM's are sweet, they keep code nicely readable.
Alternatively, if you want to dive into some examples created from your dataset - take a look at some code-generators like MyGeneration[^]; there's usually multiple templates to create 3-tier applications (and more) - some templates generating complete websites. A benefit is that the code is generated from data that you're (often) familiar with, making it easier to recognize some of the parts (compared to those abstract examples in the books).
Bastard Programmer from Hell 
If you can't read my code, try converting it here[^]
|
|
|
|
|
I agree with the comment regarding a shared interface.
But inserting large number of records into Oracle works very differently from SQL/Informix. SQL's bulk loader does not share a common interface with Oracle. It is not important to discuss their interfaces here. Besides, that is a design test at this point.
Guys, but the business layer is not inheriting from the data implementation, such as the Oracle-specific implementation class. The business object Bonus does not know where the data is coming from. Removing the inheritance would mean that a client would be able to access the Name property for the business object, which makes no sense to me. Example:
Bonus obj = new Bonus( new LevelBonus_SQL() );
obj.Name = "some name";
obj.Load(); //It will load from SQL, but the business Bonus object does not know
PROBLEM: Remove the inheritance and I can no longer access obj.Name.
modified 13-Feb-14 17:28pm.
|
|
|
|
|
Carlos A Merighe wrote: PROBLEM: Remove the inheritance and I can no longer access obj.Name.
I use the following model when I create my own layers. Following is pseudo code.
class CustomerPrimitive
{
String name
String address
String telephone.
}
class CustomerDb
{
create(CustomerPrimitive)
update(CustomerPrimitive)
delete(Id id)
List<CustomerPrimitive> query(CustomerPrimitiveQuery)
}
Keep in mind that the 'Primitive' is a Data Model Object and not necessarily a direct table representation. The 'Primitive' must NOT have any significant functionality and must NOT have and database specific code in it.
The database specific code goes in the 'Db' classes. You can add abstractions on top of the above but I haven't found them useful (and I have done it.)
The 'Primitive' types should be in their own space (package, namespace, whatever) which is entirely separate from the 'Db' types.
|
|
|
|
|
Ditch the term "Load"; how about "LoadByName" or "LoadById"? Throw an InvalidOperation if a property is set in a non-loaded (but created) object. Personally, I'd have that argument in the constructor, removing the possibility to create an empty class.
Bastard Programmer from Hell
If you can't read my code, try converting it here[^]
|
|
|
|
|
Have a look at my article[^]
Carlos Merighe wrote: it appears that the Data Access Object Design Pattern is a good solution to separate a Data Access API from the business objects.
I would even say it is mandatory. It makes you think in smaller problems, which in turn makes you build simple and easy solutions to those problems. If you pile up everything into one problem/solution you'll end up with a huge mess.
Also it enforces re-usability, which leads to robust and stable code.
Carlos Merighe wrote: What is the best Design Pattern to separate data access from the business layer?
The one that fits you best. I'm not saying that so I can give you an easy answer. There is no "best" solution. Try out different solutions and see what best fits your needs and coding styles. Depending on the technology you can go old school as in my article or go the new way with LINQ or other technologies. Note that if you're in a company you probably want to use their way of working.
hope this helps.
|
|
|
|
|
You could also use the Unit of Work / Repository pattern for your purposes.
Thanks
JD
http://www.seitmc.com/seitmcWP
|
|
|
|
|
i need design pattern which will do continuous monitoring and evaluating and then further evaluating after certain time period. Working on Windows platform. Need help how or which design pattern to use or what approach to use .
Thanks,
|
|
|
|
|
|
i want which particular is more suitable to scenario which i described.
-- modified 11-Feb-14 3:22am.
|
|
|
|
General 
News 
Suggestion 
Question 
Bug 
Answer 
Joke 
Praise 
Rant 
Admin 
Submit an article or tip
