Kelly Mears

Chesterton's Fence

Do not remove something whose purpose you cannot explain.

Method2 min read268 words14 out · 21 in
also calledDon't remove the fence

Chesterton's fence is G. K. Chesterton's parable about reform: coming upon a fence across a road, the sensible reply to "I see no use for this, let us clear it away" is "go away and think. When you can come back and tell me that you do see the use of it, I may allow you to destroy it."

Applied to code, it argues against deleting a guard, a workaround, or an odd-looking line simply because its purpose is not evident. Much production code encodes constraints that are invisible from the code itself — a browser quirk, an ordering requirement, an upstream bug. Removing one produces a fault that looks unrelated and reappears months later. At an interface the fence may not be in the code at all: Hyrum's Law holds that with enough users every observable behavior — error wording, iteration order, timing, even a bug — is depended on by somebody, so a property with no visible purpose can still be load-bearing for a caller nobody can enumerate.

The productive form of the principle is not conservatism but investigation: find the reason, then decide. If the fence turns out to be a real constraint, that is exactly the fact worth recording — see Code Comment, whose useful form answers "what breaks if you rewrite this?".

The principle has a mirror, and both are needed. A fence whose reason has expired should come down; keeping it forever is Technical Debt with a good story. And a fence nobody can explain because nobody wrote it down is why Documentation Rot is a correctness problem, not a tidiness one.

See also10

Unreachable Code

Code no input can execute — usually a guard against a state the surrounding invariants forbid.

Testing & Verification11 connections

Defensive Default

A substituted value that papers over a missing input and hides the misconfiguration that produced it.

Method16 connections

Root Cause Analysis

Escalating past the visible symptom until you find the layer that actually produced it.

Method49 connections

Naming

Choosing identifiers so the code states its own meaning without commentary.

Method59 connections

Code Review

A second person reading a change before it lands, and the practices that make it worth the time.

Version Control & Delivery38 connections

Plausible Mechanism

A causal explanation that was inferred rather than tested, and reads as more rigorous for being specific.

Method63 connections

Silent Failure

A failure that produces no signal, so the absence of an error reads as success.

Method73 connections

Commit Message

The note attached to each change in version control, conventionally a short summary line and a body explaining why, which becomes the main record of intent for anyone who later wonders why the code is the way it is.

Version Control & Delivery13 connections

Architecture Decision Record

A short document recording one significant technical decision, its context, and its consequences, kept alongside the code so later readers know why the system is shaped the way it is.

Version Control & Delivery9 connections

Git Blame

Showing, for every line of a file, the commit and author that last changed it, the starting point for finding out why code is the way it is.

Version Control & Delivery12 connections

Linked from21