Friday, July 16, 2010

Copy And Edit Revisited

Vadim reminds me that I need to address root causes of Copy-Paste-Edit programming, rather than merely ranting about how bad a practice it is and how it ruins good code. Of course, he is right. That is part of being Vadim.

I've previously ranted about the ill-effects of copy-paste-edit programming, but it would be unfair to say that there is never a need for it, or that people who did it were simply stupid and lazy.  The problem would not be so prevalent if it did not have some reasonable basis for practice. However well-intentioned and useful it is, its net effect on a code base is overwhelming negative.

Here are a few root causes I recognize, and I'm open to hear more.
  • Tedious construction semantics encourage copying. Many APIs are very thin access to bean-like objects, and yet using them correctly can be a trick. You have to know what to set, and in what order, and what to call next.  It is far easier to copy a correctly-set-up object use than to make one from scratch.
  • Copied code is a working initial state from which to make progress. This is useful not only when dealing with complicated code in the current system, but is especially true and especially beneficial when copying code examples from docs, books, or online code repositories.
  • Complex multi-line operations have many ways to break. If the system shows poor cohesion or has multiple ways of doing the same job (and some ways work better than others) then copying a usage that works seems pretty safe.  It is worse when the API has a "seed and harvest" interface, where one must set certain variables (seeding), then call a function, then collect the results from various variables (sometimes the same ones, sometimes not).
  • Copying minimizes the chance of breaking existing code, which is the primary fear in legacy systems. If some (or most) code is ill-tested, then any editing of existing code can fail in unexpected ways. Copying an algorithm and changing the variable names to match the local context preserves the original code. 
  • It is easier to copy than to study.  Copying code that pretty much works now, you can make minimal changes and hopefully it will work without you having to understand the underlying concerns. It allows you to get in and out quickly without doing research. Copy/paste creates a point in time at which it is basically sound, and can be refactored to purpose. 
  • Moving code into functions requires thinking about design.  Is it one method or several? To which class does it belong? The original owner of the code? The new code? A class currently used by both? A new class? An existing library? 
Is copying always wrong? I think it is not.  The act of copying itself should kick in your "spider sense," but it is not necessarily harmful.  Copying examples from outside the system might be useful as a starting point.  Copying code from inside the system (including copying an existing test to help create a new test) can be helpful to the programmer.  However, having duplicated code in the system is always wrong, and having code that is placed badly is wrong as well.

It could be useful to put some rules around your use of copying.
  • Don't leave duplicated code in the system. If you copy, consider your copy to be merely a starting point, and also point of technical debt. Pay it off quickly by refactoring the duplication out of existance as soon as possible.
  • Even when you write unique code, extract methods and move them to their most appropriate class, so methods can be called instead of copied next time. If you leave manipulations in the user of a class, you are encouraging the next user of that functionality to copy it as well. 
  • If the code is ugly enough (complex intialization, multi-step operations), extract the non-unique parts as new methods called from both copies. Interfaces do not have to be ugly and complex.
  • When you find duplication, or create duplication, finish your refactoring step by sweeping the code base for other copies, and correct them likewise.  It doesn't help if two versions out of a dozen are refactored. You want the whole system to improve as you work.
  • Copying might not be so bad if you refactor the area you're copying from and move the extracted methods to appropriate new homes.  Copying one or two function calls might not hurt you like copying blocks of code.
Everything I said about the evils of copying code still stand.  If you copy a block of code, you are probably going to screw over your whole development organization a little (especially when the system changes and your original is no longer correct).  Copy-Edit is still the way to make a bloated mess of your system.

2 comments:

  1. How does your opposition to cut and paste influence your opinion on code reuse in general?
    (I know that the two are only tenuously related.)

    ReplyDelete
  2. I'm big on reuse within a project, but not via copying. I like seeing things refactored so one bit of functionality can be used everywhere.

    I love reusing libraries when possible. If I have a few projects in the same language which can use the same code, I prefer breaking it into a library rather than copying it from one to the other.

    I don't consider duplication to be reuse, in general. It is reimplementation instead. I consider retyping/pasting to be a low-cost version of reinventing. Sometimes, though, you need the code to come into your code base and all you have is source from somewhere else.

    Nobody ever plagiarized by refactoring. Copying can actually get you in a lot of trouble if you copy from external sources without due diligence. You can be banned from projects, fired, sued, etc. So it's dangerous.

    Copying within a project won't get you fired or sued necessarily, but it will make a lot of trouble when the code needs to be changed in all of the places where it occurs.

    Reuse, use of a class or method from many places, is much safer for the project and for your career.

    ReplyDelete