古のJavaを使うということ / JJUC CCC 2016 Spring

#jjug_ccc #ccc_m71

Transcript

1. 古のJavaを使うということ #ccc_m71 JJUC CCC 2016 Spring ぜろゆ
2. #ccc_m71 目次 * 自分について - これは誰ですか？ * 出会い - これはJavaですか？ * コメント - これは消せますか？ * テスト - これは直せますか？ * 周りに広める - これは使えますか？ * 古のJava - これは変えられますか？
3. #ccc_m71 自分について - これは誰ですか？ * ぜろゆ * 文系出身・プログラミング(ほぼ)未経験で就職 * 社会人3年目 * Twitter：@zer0_u
4. #ccc_m71 出会い - これはJavaですか？ * Javaで「Hello,World!」できる程度で入社 　　「Java完全にマスターした」 * 研修で「動くものをつくる」魅力にはまる 　→ 自分でもコードを書くように * よりよいものを作るために勉強&写経 　* JUnit実践入門 　* リーダブルコード　などなど
5. #ccc_m71 出会い - これはJavaですか？ * 配属！ * 知ってるコードを書くとコンパイルエラーが出る 　→ (o･д･)？？？ * どうやらJavaにはバージョンというものがあるらしい * じゃあ、この現場で使ってるJavaは？
6. #ccc_m71 出会い - これはJavaですか？ ＞＞＞ J2SE1.4 の気配を察知 ＜＜＜
7. #ccc_m71 J2SE1.4 - これはバージョンアップできますか？ * 諸般の事情によりバージョンアップ不可 * この現場にいる限り逃げられない ↓ このときのぜろゆ (o･д･) 。◦(とはいえ書けるものが少ないだけでしょ？)
8. #ccc_m71 出会い - これはJavaですか？ ↓ いまのぜろゆ (o･д･) 。◦(そう簡単な話でもなかったんだよなぁ)
9. #ccc_m71 コメント - これは消せますか？ * 業務 - 保守(不具合修正)が9割 　* 自分がゼロからコードを書くことはまずない 　* 誰かの書いたコードを読んで修正・追加がメイン * 状況 　* バージョン管理あり 　* コードレビューあり
10. #ccc_m71
11. #ccc_m71 コメント - これは消せますか？ (o･д･)「うわぁ、コメントアウトめっちゃいっぱいある」 (o･д･)「バージョン管理してるし消しちゃえ」 (o･д･)「小手先で直すとコード汚くなるな...」 ..... (o･д･)「できた！ コードレビューお願いします！」
12. #ccc_m71 コメント - これは消せますか？ (*‘∀‘)「ここ何で消したの？」 (o･д･)「え？ コメントアウト邪魔じゃないですか？」 (*‘∀‘)「修正範囲は最低限だって言ったよね？」
13. #ccc_m71 コメント - これは消せますか？ * 用語「修正範囲」 　* 限りなく狭いほうがよいとされているもの 　* 同じ問題を解決できる複数の手法がある場合、 　　修正されるコードの行数が少ないほうが良い
14. #ccc_m71 コメント - これは消せますか？ * 用語「修正範囲」 　* 限りなく狭いほうがよいとされているもの 　* 同じ問題を解決できる複数の手法がある場合、 　　修正されるコードの行数が少ないほうが良い 　* 特に既存の部分を(コメントも含め)削除することは 　　できる限り避けるべきである　
15. #ccc_m71 コメント - これは消せますか？ (o･д･) (とはいえこれ半端に手を入れても意味なさそうだが) (o･д･)「わかりました、やりなおします」 (o･д･) (どうする？)
16. #ccc_m71 コメント - これは消せますか？ (o･д･) (どうする？) 　1. 先輩のいうことを受け入れる 　2. 強行突破する(同じ主張を繰り返して折れるのを待つ) 　3. 何か素敵な策を思いつく
17. #ccc_m71 コメント - これは消せますか？ (o･д･) (・・・！) (o･д･) (修正範囲を妥当に見せかければあるいは) ・・・ (o･д･)「できました！もう一度レビューお願いします！」
18. #ccc_m71 コメント - これは消せますか？ * ここで取った作戦 　* 「できる限り不具合を酷く見せる」 　　* 史上最悪の不具合に見せかけるためにひたすら 　　　発生条件を増やし続けて1週間 　* 修正しても致し方ないと思える範囲を広げれば 　　 コメントアウトの削除も許されるはず…！ 　* 一部コメントアウトの削除はあきらめる → コメントアウト部分の削減に成功！
19. #ccc_m71 コメント - これは消せますか？ (まとめ) * 最善のコードを書くためには被害状況を盛ることも大事 * コメントアウト一つといえど舐めてかかってはならない * 現状のコードにはそれなりの理由があって成り立っている 　ことがわかった * とはいえJavaDoc形式でコメントアウトするのはやめろ * Javaっぽい話はこの次から →
20. #ccc_m71 テスト - これは直せますか？ * 業務 　* 誰かの書いたコードを読んで修正・追加がメイン * 状況 　* バージョン管理あり 　* コードレビューあり 　* テストコードなし 　* 仕様ほぼなし (都度ベストな対応を考える)
21. #ccc_m71 テスト - これは直せますか？ * 用語「動作確認・保証」 　* 画面上で操作を行い、不具合が修正されているか・ 　　別の不具合が起きていないか・過去に修正した 　　不具合が再発していないかを確認すること * 開発者でも同じように動作確認・保証を行う
22. #ccc_m71 テスト - これは直せますか？ (o･д･)「画面ぽちぽちするの面倒」 (o･д･)「JUnitとかいうのあるんでしょ？」 (o･д･)「テストコードが大事って知ってるもん」 (o･д･)「テストファースト！」
23. #ccc_m71 テスト - これは直せますか？ プライバシーに配慮して一部に目隠しを施しています
24. #ccc_m71 テスト - これは直せますか？ ＿人人人人＿ ＞　ない　＜ ￣Y^Y^Y￣
25. #ccc_m71 テスト - これは直せますか？ (o;･д･)「な、なければ書けばいいんだよ」
26. #ccc_m71 テスト - これは直せますか？ ＿人人人人人人人人＿ ＞　突然のJ2SE1.4　＜ ￣Y^Y^Y^Y^Y^Y^Y￣
27. #ccc_m71 テスト - これは直せますか？ * JUnit (基礎知識) 　* 現在の主流はJUnit4 　* アノテーションを使って簡易に記述できることが特長 * アノテーションが利用できるのはJ2SE1.5から
28. #ccc_m71 テスト - これは直せますか？ ＿人人人人人人人人＿ ＞　しかしJ2SE1.4　＜ ￣Y^Y^Y^Y^Y^Y^Y￣
29. #ccc_m71 テスト - これは直せますか？ (o;д;)「今からJUnit3勉強しても間に合わない...」 (o;д;)「関連オブジェクト多くてテストできない...」
30. #ccc_m71 テスト - これは直せますか？ (o･д･)「...！」
31. #ccc_m71 テスト - これは直せますか？ (o･д･)「テストしたいコード部分は1クラスのみ」 (o･д･)「確かCSVファイルをDBっぽく使えるものがある」 (o･д･)「この部分だけ抽出してテストを書けないか？」
32. #ccc_m71 テスト - これは直せますか？ * 書けた http://bit.ly/1srLofN
33. #ccc_m71 テスト - これは直せますか？ * やったこと(概要) 　* JDK8が有効になっているeclipseを別で用意 　* DBUnit＋MySQLでDB接続を再現 　* テスト対象コードは本番に載せるものをほぼコピー 　　(o･д･)「後方互換バンザイ」
34. #ccc_m71 テスト - これは直せますか？ * 周囲への説明 　(o･д･)「これがあれば開発者の画面テストがほぼ不要」 　(*‘∀‘)「ほんとに？デグレ起こさない？」 　(o･д･)「これが(修正前)→こうだったのが(赤) 　　　　　→こうして(修正後)→こうじゃ(緑)」 　(*‘∀‘)「まぁ最初のテストとしてはよさそうだね...」 　(o･д･) (お、まずまずの感触)
35. #ccc_m71 テスト - これは直せますか？ * 結果 　* 当該不具合についてはテストコードで修正の妥当性を 　　確認できるようになった 　* 周囲にJUnitの存在と有用性を少しアピールできた 　* 継続して実行する基盤を作る前に異動...
36. #ccc_m71 テスト - これは直せますか？(まとめ) * 「テストが大事」という金科玉条にこだわると進めない * できる範囲のことで全力を尽くす 　* JDK8が有効な開発環境を用意する 　* 自分が触るコードだけでもテストを書く * 疑う人には実演が効果的 　* 画面ぽちぽちにかかる時間を計っておくとなおよし
37. #ccc_m71 ここまでのまとめ - これはJavaですか？ (o･д･)「少しずつコードも読みやすくなってきた」 (o･д･)「(自分のところだけとはいえ)テストコードも書いて 　　　　デグレが起きづらくなってきた」 (o･д･)「この取り組みを広めたら製品はよりよくなるはず」
38. #ccc_m71 ここまでのまとめ - これはJavaですか？ ＿人人人人人人人人人人人人人人＿ ＞　そううまくいくわけがない　＜ 　Y^Y^Y^Y^Y^Y^Y^Y^Y^Y^Y^Y
39. #ccc_m71 周りに広める - これは使えますか？ * ここまででわかったこと 　* 製品をよりよくするためにはコードの改善も必要 　　* 不要なコメントアウトの削除 　　* テストコードの追加 など 　* 一部ではなく全体的な改善が製品の品質向上に 　　役立つはず 　* とはいえいちいち前提を説明して回るのは面倒
40. #ccc_m71 周りに広める - これは使えますか？ (o･д･)「これをほかの人にも広めたい」 (o･д･)「ちょうどいい機会(略)あるし伝えてみようか」 ・・・ (o･д･)「ということで、一気にとはいわずとも少しずつ 　　　　改善していくと全体の効率が云々」
41. #ccc_m71 周りに広める - これは使えますか？ (´ー｀)「それ、お客様の業務にとっては 　　　　 どんなメリットがあるの？」 (o･д･)「直接は関係ありませんが、何か不具合が見つかった 　　　　際に修正する効率が上がります」 (´ー｀)「それ別にコードを書かなくても達成できるよね？」
42. #ccc_m71 周りに広める - これは使えますか？ (o･д･) (そうだけどさぁ) (o･д･)「コードを書く手間はありますが 　　　　自動で確認できる以上は活用しない手はないかと」 (´ー｀)「動作確認は最悪外部の人に依頼できるし、 　　　　 開発者は不具合修正に集中してほしい」
43. #ccc_m71 周りに広める - これは使えますか？ (o･д･) (そういう) (o･д･) (話じゃ) (o･д･) (ないんだ) (o･д･) (YO!!!!!!!) (o･д･) (とはいっても通じないんだろうなぁ) (o･д･)「はい、わかりました」
44. #ccc_m71 周りに広める - これは使えますか？ (o･д･)「自分1人だと効果アピールが弱いのかな」 (o･д･)「仲間を増やして全体が効率化していることを 　　　　示せばさすがに認められるはず」
45. #ccc_m71 周りに広める - これは使えますか？ * また別の日 (o･д･)「こうすると開発が早くなるんだよー」 ('ω') 「でもこれ書けるようになるまで時間かかるでしょ」 (o･д･)「そんなにかからんよ、普段書いてるコードと 　　　　ほとんど同じだし」 ('ω') 「であれば目視確認のが早いんじゃない？慣れてるし... 　　　それにテストコードが間違ってたらどうするの？」 (o･д･) (Oh...)
46. #ccc_m71 周りに広める - これは使えますか？ (o･д･)「もういいや...自分のとこだけがんばろ」
47. #ccc_m71 周りに広める - これは使えますか？(まとめ) * コードを書いて品質を向上させるという考え方は 　意外と少数派 * まずは自分の行動で範を示すしかない 　(その前に異動したけど) * 仲間を増やすにはわかりやすい実績が大事かもしれない
48. #ccc_m71 古のJava - これは変えられますか？ * 古いJavaのままでも「製品として成立させる」ことは可能 　* セキュリティの話は省略 * Javaを使っている製品のライフサイクルなども考慮して 　バージョンについて話す必要がある 　* サポートが切れそうな製品のJavaを無理に上げる 　　必要があるか？
49. #ccc_m71 古のJava - これは変えられますか？ * 開発者個人として古のJavaに出会ったときに打てる手(1) 　* 現行のJDKベースの開発環境を確保 　* 製品に必須ではない部分(テストコード等)は 　　現行のJDKベースで書く 　　* 必須でないところは最低限の手間で書く 　* IDEの設定でコメント部はできる限り薄い色にする 　　* 見えなければどうということはない 　　* だからJavaDoc形式でコメントアウトはやめろと
50. #ccc_m71 古のJava - これは変えられますか？ * 開発者個人として古のJavaに出会ったときに打てる手(2) 　* 実績を積む 　　* 自分が手を入れた部分でデグレを起こさない 　　* 修正にかかるコスト(工数)の可視化
51. #ccc_m71 古のJava - これは変えられますか？ * 開発者個人として古のJavaに出会ったときに打てる手(3) 　* 最善を学び続ける 　　* 取るべき手段をできるだけ増やす 　　* そのまま適用できるケースはないが 　　　状況の中で最善を尽くすには最善を知るしかない
52. #ccc_m71 古のJava - これは変えられますか？(まとめ) * 正しい製品を正しく作るために 　* 今まで作り上げたものを直視する 　　* コメントアウトにも相応の事情がある 　* コードを使って品質を向上させる手段があることを示す 　　* まずは自分のところからテストを書く 　　* 仕様はテストコードで示す勢い 　* 最善を学び続ける
53. Thank You! #ccc_m71