2019年10月に行われた大学の情報LT大会にて発表したスライドです。個人的にアンチパターンであると考えているコードの例とその改善案を7人のアンチパターンおじさんと一緒に学ぶことができます。

1. コーディングアンチパターン
〜失われた7人のアンチパターンおじさん〜

2. 自己紹介
• トコロテン(@tokoroten_lab)
• 好きなもの
• 猫
• 男の娘
• 趣味
• 競技プログラミング
• Web開発
• ゲーム開発
• お絵かき

3. アンチパターンおじさん一覧
• 🗽コーディング規約ガバガバおじさん🗽
• 🌳森羅万象for, whileおじさん🌳
• 💥四方八方breakおじさん💥
• 🧙マジカルおじさん🧙
• 🌍ハイパーグローバルおじさん🌍
• 👏メモリガチホールドおじさん👏
• 😏イキリマルチスレッディングおじさん😏

4. 🗽コーディング規約ガバガバおじさん🗽

5. 🗽コーディング規約ガバガバおじさん🗽
nums_gokei heikinprintNumsInfo

6. コーディング規約
• 構成要素
• 命名規則
• コーディングスタイル
• コメント
• etc
• 例
• PEP8
• unity community – Csharp Coding Guidelines
• Google HTML/CSS Style Guide
ここくらいは意識しよう

7. 🗽コーディング規約ガバガバおじさん🗽
nums_gokei
heikin
printNumsInfo print_nums_info
nums_sum
nums_avg
言語の統一(英語)
ケースの統一(スネークケース)
詳細度の統一(‘nums_’をつける)

8. 🌳森羅万象for, whileおじさん🌳

9. 🌳森羅万象for, whileおじさん🌳
新しいリスト生成
全ての要素を反復
条件に合うもの

10. 🌳森羅万象for, whileおじさん🌳
新しいリスト生成
全ての要素を反復
条件に合うもの
forEachを利用

11. 🌳森羅万象for, whileおじさん🌳
新しいリスト生成
全ての要素を反復
条件に合うもの
forEachを利用
filterを利用

12. 🌳森羅万象for, whileおじさん🌳
全ての要素を反復
新しいリスト生成
forEachを利用

13. 🌳森羅万象for, whileおじさん🌳
mapを利用
全ての要素を反復
新しいリスト生成
forEachを利用

14. 🌳森羅万象for, whileおじさん🌳
反復
全ての要素を反復
条件付き
for
forEach
新しいリスト生成
条件なし
mapfilter
処理のスコープを考える

15. 💥四方八方breakおじさん💥

16. 💥四方八方breakおじさん💥
最後尾
ループカウンタでbreak
先頭
入力でbreak

17. 💥四方八方breakおじさん💥
ループの終了条件にする
最後尾でbreak
先頭でbreak

18. 🧙マジカルおじさん🧙

19. 🧙マジカルおじさん🧙
validate1
validate2

20. 🧙マジカルおじさん🧙
validate1
validate2
validateBoundBySumWithMod
validateBoundByProductWithMod

21. 🌍ハイパーグローバルおじさん🌍

22. 🌍ハイパーグローバルおじさん🌍
input_dataがグローバル
変数名が一般的
グローバルの必要性

23. 🌍ハイパーグローバルおじさん🌍
input_dataがグローバル
変数名が一般的
グローバルの必要性
ローカル変数に変更
引数として受け渡し

24. 👏メモリガチホールドおじさん👏
malloc()でメモリ確保
即リターン
ホールド完了

25. 👏メモリガチホールドおじさん👏
malloc()でメモリ確保
free()で解放

26. イ
キ
リ
マ
ル
チ
ス
レ
ッ
デ
ィ
ン
グ
お
じ
さ
ん
😏
😏

27. イ
キ
リ
マ
ル
チ
ス
レ
ッ
デ
ィ
ン
グ
お
じ
さ
ん
😏
😏
countを1000個のスレッド
から更新

28. 😏イキリマルチスレッディングおじさん😏
count
thread1 thread2 thread1000thread3 …

29. 😏イキリマルチスレッディングおじさん😏
count
thread1 thread2 thread1000thread3 …
読み込み時
書き込み時
0 1
1 2
1
2
??? <= 1000
??? + 1

30. イ
キ
リ
マ
ル
チ
ス
レ
ッ
デ
ィ
ン
グ
お
じ
さ
ん
😏
😏
排他制御を行う