Question

Today I implemented a small piece of code that include analytics only in production environment using something like:

<% if Rails.env.production? %>
  analytics here
<% end %>

I don’t see anything wrong about it… however one of my c0-workers told me that it wasn’t a good practice at all, that it would for sure introduce problems in a future maintaining phase and that was much better to set up a constant in the environment and do something like

<% if defined?(INCLUDE_ANALYTICS) && INCLUDE_ANALYTICS %>
  analytics here
<% end %>

I don’t see such a big difference between these 2 approaches, however I would like to see what other developers think about it and how would other solve this in a better way 🙂

Answer 1

1. If the block contains another ruby code inside then Rails.env.production? is bad here as you are losing ability to test that block. Imagine you use helper inside which may explode on some pages. Simple controller tests with views just render pure HTML without loading any external resources and running JS, so it shouldn’t affect your stats. Only browser based integrations tests (Selenium) may affect stats.

2. INCLUDE_ANALYTICS is quite good option, but usually there is too many of such constants in the application scope. The better option is to introduce YAML configuration file. It gives ability to define multiple production configuration files (main production, beta production without analytics etc…). Once used it helps externalize other settings easily.