更新: | 2022-03-18 |
---|---|
作者: | @voluntas |
バージョン: | 2022.1 |
URL: | http://voluntas.github.io/ |
- 新しいチームに入って最初に見たコードなんでわからないことしか無いので、「なんでこうしてるんだ?」っておもったらまず聞くことを心がけた
- 聞かないことには始まらないし、深い理由を追えない
- 理由を聞くと思想がわかってくるから、他のコード見た時「あ、なるほどな」ってなるようになるのでいい
- コードで大切にしている方針を見ながらレビューするようにした
- ポケットだと「良いコード = みんなが理解しやすいコード」みたいなのがざっくりあったからそれを意識してレビューしてた
- ex) mapとgrep使いまくって一行で納めてるコードより、methodに細分化して名前をつけたコードのほうがみんな分かりやすい的な
- チームで大切にしている考えをもとにレビューしていけば、いいプロダクトになる
- LGTMつかってこ
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
package MyApp::DB; | |
use 5.016; | |
use warnings; | |
use utf8; | |
use Time::Piece::Plus; | |
use Class::Method::Modifiers; | |
use Teng::Schema::Loader; | |
use MyApp::DB::ResultSet; | |
use MyApp::Exception; |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
落書きの報告書 Twitterばかりしてる俺 | |
超高速でビルドするけど 動かないVirtual Machine | |
やさしくないJavaの本ページ破りたい | |
Chromeの裏 2ちゃんを覗いて見つかれば逃げ場もない | |
座って固まり 目を伏せながら | |
心のひとつも解りあえないプログラムをにらむ | |
そして仲間たちは今夜辞職の計画をたてる | |
とにかくもう 会社や家には帰りたくない | |
自分の業務が何なのかさえ 解らず震えている | |
25の夜 |