Upgrade to Pro — share decks privately, control downloads, hide ads and more …

Escaping The Tar Pit at Philly.rb

Escaping The Tar Pit at Philly.rb

Presentation about avoiding and escaping the tar pit.

Introduction to the `skunk` gem. Talks a little bit about `flog`; `rubycritic`; `simplecov`; `reek`; and other tools you can use to get metrics related to the codebase.

Ernesto Tagwerker

November 05, 2019
Tweet

More Decks by Ernesto Tagwerker

Other Decks in Technology

Transcript

  1. “No scene from prehistory is quite so vivid as that

    of the mortal struggles of great beasts in the tar pits. […] The fiercer the struggle, the more entangling the tar, and no beast is so strong or so skillful but that they ultimately sink.” Fred Brooks
  2. “Large-system programming has over the past decade been such a

    tar pit, and many great and powerful beasts have thrashed violently in it.” Fred Brooks
  3. Tar Pit Symptoms > Projects running over budget > Taking

    forever to ship small changes > Sacrificing quality, increasing tech debt
  4. “THAT’D BE GREAT” BOSS: “IF YOU COULD COME IN ON

    THIS GREAT, LEGACY PROJECT AND MAINTAIN IT FROM NOW ON…”
  5. “THAT’D BE GREAT” CLIENT: “IF YOU COULD COME IN ON

    THIS GREAT, LEGACY PROJECT AND MAINTAIN IT FROM NOW ON…”
  6. “BECAUSE I NEED IT BY END OF DAY TODAY” CLIENT:

    “HOW LONG IS IT GOING TO TAKE?”
  7. “The degree to which a system, component, or process meets

    implicit and explicit requirements.” IEEE
  8. ISO 9126-1 Software Quality Model ✅ Reliability ✅ Usability ✅

    Efficiency ✅ Maintainability ✅ Portability
  9. ISO 9126-1 Software Quality Model ✅ Reliability ✅ Usability ✅

    Efficiency ➡ Maintainability ✅ Portability
  10. Code Coverage # spec/spec_helper.rb if ENV["COVERAGE"] == "true" require 'simplecov'

    SimpleCov.start do add_group "Models", "models" add_filter "/spec/" track_files "**/*.rb" end end
  11. Code Quality n options (flog; flay; reek; churn; RubyCritic; MetricFu;

    attractor; rubocop; fukuzatsu; turbulence; …)
  12. Complexity class Foo def yay # 11.2 = a =

    eval "1+1" # 1.2 + 6.0 + if a == 2 # 1.2 + 1.2 + puts "yay" # 1.2 end end end
  13. 100 0 1 100_000 “No one understands these files but

    they work. So don’t change them.” Complexity Churn
  14. 100 0 1 100_000 “[…], if the code never changes,

    it's not costing us money.” Sandi Metz Complexity Churn
  15. 100 0 1 100_000 “Everybody understands these files but you

    need to change them often” Complexity Churn
  16. 100 0 1 100_000 Complexity Churn “Sometimes a class becomes

    so complex that refactoring seems too difficult.” Michael Feathers
  17. Are you getting into a tar pit, is it a

    dumpster fire, or have you found a project which is easy to maintain?
  18. foo.rb churn: 10 complexity: 10 smells: 10 smell: 1,000 (10*10*10)

    bar.rb churn: 10 complexity: 10 smells: 10 smell: 1,000 (10*10*10)
  19. foo.rb churn: 10 complexity: 10 smells: 10 code_coverage: 0 stink_score:

    100,000 (10*10*10)*100 bar.rb churn: 10 complexity: 10 smells: 10 code_coverage: 100 stink_score: 1,000 (10*10*10)*1
  20. Skunk $ skunk ... StinkScore Total: 13220.859999999997 Modules Analysed: 71

    StinkScore Average: 0.1862092957746 Worst StinkScore: 2401.75 (lib/rubycritic/source_control_systems/git.rb)
  21. RubyCritic module RubyCritic class AnalysedModule def cost smells.map(&:cost).inject(0.0, :+) +

    # From Reek (complexity / COMPLEXITY_FACTOR) # From Flog end end end
  22. Skunk module RubyCritic class AnalysedModule def stink_score return churn_times_cost if

    perfect_coverage? (churn_times_cost * (PERFECT_COVERAGE - coverage.to_i)) end def churn_times_cost safe_churn = churn > 0 ? churn : 1 safe_churn * cost end end end
  23. Skunk module RubyCritic class AnalysedModule def stink_score return churn_times_cost if

    perfect_coverage? (churn_times_cost * (PERFECT_COVERAGE - coverage.to_i)) end def churn_times_cost safe_churn = churn > 0 ? churn : 1 safe_churn * cost end end end
  24. 100 0 1 100_000 Complexity Churn “Here be dragons. These

    modules are complex and change a lot…” '