Webサービスなどでの運用し続けるコードベースでは、「ボーイスカウトルール」を意識してコードを綺麗にしてきましょう。

1. 1Confidential Crowdworks, Inc. All Rights Reserved.
コードを削除し続けるという習慣
“ボーイスカウトルールを覚えて帰って下しあ”
2017年4月26日/第２回 人形町Techで騒がNight♥
株式会社クラウドワークス エンジニア
安田篤史

2. 2Confidential Crowdworks, Inc. All Rights Reserved.
提供

3. 3Confidential Crowdworks, Inc. All Rights Reserved.

4. 4Confidential Crowdworks, Inc. All Rights Reserved.
Add 124,468
Delete 162,683

5. 5Confidential Crowdworks, Inc. All Rights Reserved.
Why?

6. 6Confidential Crowdworks, Inc. All Rights Reserved.
Web サービスは、
続く

7. 7Confidential Crowdworks, Inc. All Rights Reserved.
増え続ける
• 機能
• コード

8. 8Confidential Crowdworks, Inc. All Rights Reserved.

9. 9Confidential Crowdworks, Inc. All Rights Reserved.
断捨離

10. 10Confidential Crowdworks, Inc. All Rights Reserved.
は、割愛

11. 11Confidential Crowdworks, Inc. All Rights Reserved.
どうやって
削除するものを
見つけるのか？

12. 12Confidential Crowdworks, Inc. All Rights Reserved.
わざわざ探さない。

13. 13Confidential Crowdworks, Inc. All Rights Reserved.
ボーイスカウト
ルール

14. 14Confidential Crowdworks, Inc. All Rights Reserved.
[08]ボーイスカウトルール
ボーイスカウトには大切なルー
ルがあります。それは、「来た
時よりも美しく」です。
- プログラマ知るべき97こと

15. 15Confidential Crowdworks, Inc. All Rights Reserved.
恐怖との闘い方

16. 16Confidential Crowdworks, Inc. All Rights Reserved.
def no_called_method
+ log(”このメソッド呼び出しあるらしい”)
do_something
end
そっと、ログを仕込むだけ

17. 17Confidential Crowdworks, Inc. All Rights Reserved.
 削除したいメソッド
である旨
 いつ、削除予定か
 呼び出し元

18. 18Confidential Crowdworks, Inc. All Rights Reserved.
これがベスト！

19. 19Confidential Crowdworks, Inc. All Rights Reserved.
以下、本編

20. 20Confidential Crowdworks, Inc. All Rights Reserved.
やばい、削除しちゃいけないやつを
削除しちゃった・・・
“何をするとやらかすか・３つの事例”
2017年4月26日/第２回 人形町Techで騒がNight♥
株式会社クラウドワークス エンジニア
安田篤史

21. 21Confidential Crowdworks, Inc. All Rights Reserved.
動的呼び出しの罠

22. 22Confidential Crowdworks, Inc. All Rights Reserved.
class Foo
def bar
do_something
end
end
Foo.new.send(“bar”)
Object.send を使うと
grep できなくなるよ！

23. 23Confidential Crowdworks, Inc. All Rights Reserved.
keys = {foo: :bar, buz: :qux }
keys.each do |key, value|
define_method(“#{key}=“) do |arg|
proxy.send(value, arg)
end
end
いろいろ組み合わせると
カオスだよ！

24. 24Confidential Crowdworks, Inc. All Rights Reserved.
動的言語ェ・・・

25. 25Confidential Crowdworks, Inc. All Rights Reserved.
同じ名前のメソッドだらけ
だと、削除していいかわか
んないよ！

26. 26Confidential Crowdworks, Inc. All Rights Reserved.
class Foo
def bump!; do_something; end
end
class Bar
def bump!; do_something; end
end
class Baz
def abadakedabura(param)
param.bump! # <- こいつ誰だよ
end
end

27. 27Confidential Crowdworks, Inc. All Rights Reserved.
コードベース以外からの
お呼び出し

28. 28Confidential Crowdworks, Inc. All Rights Reserved.
メソッド消したら、
バッチが死んだよ！

29. 29Confidential Crowdworks, Inc. All Rights Reserved.
class Foo
def self.piyo; do_something; end
end
05 1 1 * * cd /path/to/rails;
RAILS_ENV=production bundle exec rails run
‘Foo.piyo’

30. 30Confidential Crowdworks, Inc. All Rights Reserved.
地雷はいっぱい

31. 31Confidential Crowdworks, Inc. All Rights Reserved.
ボーイスカウト
ルール