タグ

ブックマーク / ayato.hateblo.jp (2)

  • コードレビューについて - (define -ayalog '())

    普段お仕事している中で何故かコードレビューをしている時間がわりとあって、暇さえあれば(暇がなくても)コードレビューしている。 そんな中でどういうところを見たらいいのか、あるいは見るべきなのかというのが自分の中である程度蓄積された気がするので書いてみる。あと最後に普段考えていることを少し書いた。 前提 現在の僕の参加しているプロジェクトはこんな感じ Rails プロジェクト( AngularJS 使ったりしている) Git 使ってる( Pull Request ベースの開発で以下が merge 条件) 2 人以上に approve される テストが通ること(継続的インテグレーションの実施) 静的コード解析は導入している( Rubocop, jshint, pre-commit など ) テストのカバレッジは計測していない(月一くらいで測ってるらしいんだけど、だからどうっていう話はない) プ

    raimon49
    raimon49 2015/01/19
    >あと何度レビューしてもなかなか伝わらない人もいるもんで、毎回同じ指摘せざるを得なくてイジメみたいになってしまう。つらい。指摘する方もつらい。 / 分かる。コードレビュー導入する以前の話のような気もする。
  • コードを改悪されて本気でプログラマ辞めようかって脳裏をよぎった - (define -ayalog '())

    2013-12-04 コードレイプされて気でプログラマ辞めようかって脳裏をよぎった Java 開発 つらい現実。 「だから言ったのに」事案2013-12-04 11:05:15 via web 自分の書いたプログラムが大陸人の手によって改悪されまくってて涙で前が見えない。2013-12-04 11:16:49 via web あのね。。拡張for文で書いてたのにfor(int i=0; i<items.length; i++)って書きなおされてたんだ…。2013-12-04 11:18:41 via web てか、嘘でしょ。なんでSetをtoStringして文字列表現にして「,」でsplitするとか考えるの…。意味分かんないんだけど…。2013-12-04 11:19:51 via web こう、胸の奥がキューッてなってる。苦しい。2013-12-04 11:21:11 via web

    raimon49
    raimon49 2013/12/10
    「レイプなんて強い表現使うなよ」と思って読み始めたら本当にその通りの内容だった。職業プログラマやってるとあり得そうな光景が目に浮かんでしまうところが余計にキツい。
  • 1