コード改善 meetup #2 #code_kaizen

びん @honbin

#code_kaizen / 他1コメント b.hatena.ne.jp/entry/qiita.co… “コードを改善する<br>3つの方法 - Qiita” htn.to/33kzf9

ぷーじ @yug1224

「みなさんリファクタリングしてますか？」ｗ #code_kaizen

ずきゅ～んたん @ZuQ9Nn

俺。リファクターなんてコメントつけてない、コードの整理とかにしてるわ #code_kaizen

はむ@フリーランス @secret_hamuhamu

〇〇を共通化したみたいなコミットメッセージだからリファクタ数測れない... #code_kaizen

神速 @sinsoku_listy

リファクタリングのコミットに "refactor" ってそもそも使ってないからな... #code_kaizen

takasek @takasek

git log --no-merges --all | grep Date: | wc -l (全件) に対して、　log --grep='refactor' --no-merges --all | grep Date: | wc -l の件数を見よう👀 #code_kaizen

神速 @sinsoku_listy

"Remove unused method" はよく使う #code_kaizen

ずきゅ～んたん @ZuQ9Nn

やりたい人を巻き込む。ペアプロ、コードレビュー、プルリク #code_kaizen

ずきゅ～んたん @ZuQ9Nn

プルリクは殴りつけるようなもの #code_kaizen

takasek @takasek

#code_kaizen 最近ずっとこれ試してみてます　これだとリファクタコミットの件数は :recycle:でgrepできますね　Emojiで楽しく綺麗なコミットを手に入れる | MEMOPATCH memo.goodpatch.co/2016/07/beauti… #code_kaizen

ぷーじ @yug1224

- リファクタをよくやるする人がいると他の人もリファクタするようになる - 外国籍のエンジニアはリファクタする - 若いエンジニアはリファクタする - ハイスキルでも外部パートナーはリファクタしない 最後ｗ #code_kaizen

しも𝕏 @smdhogehoge

後で面倒見なくていいからだろうな #code_kaizen

takasek @takasek

null安全はいいぞ。だって、型安全はいいぞ。 // Speaker Deck speakerdeck.com/takasek/nullan… 今日の資料です #code_kaizen

ぷーじ @yug1224

「null安全はいいぞ。だって、型安全はいいぞ。」の話 #code_kaizen

神速 @sinsoku_listy

"サバンナでテストを書かれている方" / #code_kaizen

ずきゅ～んたん @ZuQ9Nn

null安全で無いコードはレガシーである。 #code_kaizen

ずきゅ～んたん @ZuQ9Nn

今までnullが当たり前にあったのが急に無くなると、初めは戸惑うんだよな。もはや、ヌルポが出ない、考えなくて良い、いちいち!= nullなんて分岐を書かなくて良いとメリットも多いけど #code_kaizen

ずきゅ～んたん @ZuQ9Nn

Dateは標準時間とか色々ハマる場合あるからな #code_kaizen

しも𝕏 @smdhogehoge

大で小を兼ねるな #code_kaizen

ぷーじ @yug1224

"大は小を兼ねるな" #code_kaizen

ずきゅ～んたん @ZuQ9Nn

null安全、型安全は機械的にやるべき！ #code_kaizen

ずきゅ～んたん @ZuQ9Nn

昔JavaでNullObjectのパターンとかあったような。 #code_kaizen

r.ishibashi @cactaceae

Dateといえば、間違ってibmのDateTimeをimportして、しかもライブラリのバグを踏んで４年放置されたバグチケットが回ってきて解決したのはいい思い出 #code_kaizen

しも𝕏 @smdhogehoge

DBから値引っ張ってくるときに見つからないときはNoResultExceptionを出す奴必ず使うとか #code_kaizen

ずきゅ～んたん @ZuQ9Nn

確かにnullにせず、Errorオブジェクトにすれば使わなくても済むな #code_kaizen