|
SYS-CON.TV Webcasts
Comments
Did you read today's front page stories & breaking news?
SYS-CON.TV
|
Top Links You Must Click On
Tips From The Masters Code Reuse - Pros and Cons
Notes of a soldier from the oh-so-bloody front
Jul. 17, 2006 10:15 AM
Depending on the scope of your project, you may have the opportunity for code reuse. The reasons you might want to do so are two-fold. First, you reduce duplication of efforts. If you have already created a hyperlink enabled CellRenderer for your DataGrid once, why do it again? Second, you create, or build upon, an ever growing utility code base. While it may not be in the "utils" package per se, you'll soon end up with re-usable events, common GUI controls and widgets, and yes even utility classes. Whether by merely being in a different folder means the client doesn't own it is up to you or your sales team.
This also helps ensure I don't code something similar to what has already been done. To the client's point of view, they already paid for a LoginForm...why should they have to pay for a new one when all they want is for it to be green instead of blue? "Dan, the login form's blue by default, but the mockups I have here are green." "That's because the View using it sets it to blue; it has a color property that is an inherited style; you can style it yourself, just do color='#yourcolor'." "Oh... nice!" Suddenly, I spend 10 minutes finding the file and including it, setting its color, registering for its login event, and building a test file to see it in action. This instead of 4 hours doing the same building a green one. Take how much you make per hour, multiply by 4, and then subtract what you make in 15 minutes. That difference is what you just saved the client by "asking a question." I've learned the hard way to ask a lot of questions to the point of being annoying and forcing people to repeat themselves. The same goes for building more complex things that have to be unique to a point. The most common example which I already alluded to is the CellRenderer. This is a class commonly utilized in Flash and Flex development to customize what's showing in a DataGrid column; each row will render the custom class instead of the default and pass it an item to render. Since every DataGrid on a project is unique, you inevitably end up with a multitude of cellrenderer classes. These classes are, at least for me, notoriously hard to share so you attempt to make them as generic as possible so others can either use them as is, or do the most common thing and extend your base one to customize it to their needs. This is an important point. The 30 or so lines of code that are required to setup a cellrenderer are suddenly already written for you, and the other developers on your team. This is a great place where inheritance really works and should be exploited. It doesn't stop there; some of the cellrenderers created could be used elsewhere as well. The only challenge is how they're designed from a visual standpoint. Styles can handle a lot, but most designers I have had the pleasure of working with have a knack of making something unique, and unique isn't always re-usable. You can either find a happy compromise with your designers, or respect their artistic integrity and recognize the fact that that they design is truly made specifically for a certain need and shouldn't be made re-usable.
Con of a Component Buffet The second con which can really come to the forefront in teams is you just don't like the component. Any developer who isn't apprehensive about using a component they didn't write makes me nervous. You trust code you didn't write? Sometimes you don't have a choice or recognize the alternatives are unacceptable. We're all human, and have our coding styles, and even if notation and other rules are enforced on your team, you can still dislike the implementation of something. This should be in the back of your mind when creating code for reuse as well. How will my code be perceived? The base rules such as encapsulation should be followed, but obviously there are other esoteric and styles of implementation that can drastically impact your involvement to ensure others spend very little of theirs getting acclimated to how it works for example.
Sum Greater Than Its Parts Let's turn it up a notch, though. What if you've built a DataGrid that has custom cellrenders, text fields on the bottom to filter it, and is tied to a specific ValueObject it knows how to display, and display well? Can you re-use that? Absolutely! Just because you don't right now doesn't mean you won't later. You should typically design with the intention to do so, but not so much you don't ever actually complete anything beyond a pimped out skeleton. Adamantine ingots may have great potential, but are merely blocks of immobile metal until merged with, say, a regenerative Canadian supersoldier's skeleton. Put what you write to use sooner rather than later. This is where code reuse really shines. You're near mini-application status component which does a lot is now re-used by your team. If the components within it follow normal styling rules and expose their innards to those who wish to extend it, your golden if certain custom styling needs to be applied. What could possibly be a con about this?
Package Structure Some projects do get large enough where you do in fact have more than one project folder. Code re-use is typically thought of through Views of some sort client developers. There is however, no reason you cannot re-use Views across projects. You just reference the package path. The pros are, work effort utilized on a project can be re-used on other projects. Sometimes you can plan for this re-use, and sometimes it's a pleasant surprise.
Licenses
Global Ramifications For example, you build your own List control to render extremely unique rows that animate instead of simply refreshing. In practice, you see that it is extremely slow, especially when multiple instances of it are used. You go in and re-factor it, finding many places in the code you can speed up. Suddenly, the entire app speeds up where those components are used. One of these changes was to expose a method for updating one specific item versus all of them when a piece of data changes. This is accomplished via a new method. That's a bad thing. Suddenly all the classes that use it now have to update themselves just to access one of the new optimizations. Touching a point I brought up earlier, sharing views between projects - the problem with that is that, if you change a view in one project, you've just affected the other one. Code that was seemingly "working" you just broke, and you didn't know it because you were never compiling, nor even involved in the other project. The important point here is: if you are going to identify something as re-usable, make it so, and put it into a global package such as your company's folder, or another aptly named common folder. Developers who go into the code in those packages know full well the ramifications of what their changes could do, for good or ill.
Tool Shed
Summary With reuse comes great responsibility. If you are designating something as reusable, it can be quite frightening to realize that a lot of others are suddenly depending on this pinnacle piece of code. That is a risk worth taking, and a strength you should prey upon. As long as you recognize the dependencies, and reduce coupling, you'll start gaining a lot more efficiency from the time you and your team spend coding. When I first learned about reuse, I could never get it to work in practice. I either ran out of time and copy-pasted or my original design was perceived to me as being "flawed" even though it worked perfectly fine. I was just being unreasonable. Keep it simple, and you'll do fine. Reader Feedback: Page 1 of 1
Your Feedback
Enterprise Open Source Magazine Latest Stories . . .
Subscribe to the World's Most Powerful Newsletters
Subscribe to Our Rss Feeds & Get Your SYS-CON News Live!
|
SYS-CON Featured Whitepapers
Most Read This Week |
|||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||