「コードを書いたことがない人にコードレビューをしてもらわないといけない辛さをなんとかしたい」

http://qa.atmarkit.co.jp/q/2996
メモ

釣りかもしれないけれど,良くある話でもあるから困る.

コードを書いたことがない人にコードレビューをしてもらわないといけない辛さをなんとかしたい

ITの部署でコードを書いている派遣社員です。
正社員にコードレビューをしてもらいます。
正社員はコードを書いたことがありません。おそらく。

辛さその1
ITのツールも使いこなせないためコードレビューは顔を合わせて行います。
コードレビュー前に大量の資料が必要。
コードレビューでのお願いごとを書く。
誰が誰に対して何をしてほしいか?
大局観で大掛かり。

辛さその2
理解できないコードレビューのコメントに対応しなければいけません。
コードがあっちこっちに飛んで読みづらいから一箇所に書け。メソッドを使うなということ。
メソッドを使うなら処理される順番にコードに番号を振れ。資料にコードをコピーして番号を振る。
コードの行数はこれでいいのか?少ないと仕事をしていないとみなされる。

辛さその3
正社員が何をしているのかわからない。
職場に対する不平不満ばかり。

http://b.hatena.ne.jp/entry/qa.atmarkit.co.jp/q/2996

  • id:theatrical 馬鹿がレビューする -> 馬鹿の言うとおりにしたらコードがゴミ屑になった -> バグが出た -> バグ出したPG作るとかつかえねーな -> お前やめろよ ってなるから早くやめた方がいい。

理不尽っすよね.でもこの理不尽が日本のIT業界の日常なんだからまったく.

  • id:yojik 気持ちはわかるが、ブログとかに書くとよいと思う。

ブログに書くと特定されやすくなるのでお勧めできない.

  • id:eduard 私は逆の立場だ。ろくにかけないのにレビューしないといけない。勉強すればいいんだろうけど、そんな時間がない。わけのわからないものの責任を取らされるのは辛い。すべきじゃないんだろうな、こんな仕事は
  • id:xinn30 正社員のレビュアーです。自分のセンスを押しつけてすみません。ただレビューイも責任を押しつけるだけじゃなくて、頷ける説明をしてほしいです。

レビュアーはプログラミングが上手い人がなるのが大前提.

プログラミングができない人は,スキルが低すぎるからどれだけ懇切丁寧に説明しても理解できないんですよ.

  • id:knjname こんな現場を改善しようとするなんて狂気の沙汰。南アフリカ共和国から犯罪を1人で闘って無くすと言ってるのと変わらない。はやく逃げろ。って言いたいところですけど、逃げられたら逃げてますよねそりゃ。
  • id:tmurakam このはてブのコメント一覧を印刷して上役のほうに見せればいいと思うよ。たぶんクビになるけどね。
  • id:horsetail こういう場合にこういう提案をするとひどく面倒くさがられるのがまた萎えるとこなんだよなあ
  • id:kusigahama "そういう世界があることを知る" のは経験になるけど、あとは居るだけ時間の無駄
  • id:shsh0shsh 結局こういう時の正社員の役割って監視官なんだな。刑務所なら刑務官。

説明すれば理解して変更してもらえるような組織なら,プログラミングのできない人がするレビューなんて最初からやらない.理解できないからこそ,そういう壮絶な無駄をする。

だから説明したり改善しようとしたりすると,上から疎まれて,雇い止めになるだろう.非正規雇用とは正社員の雇用を守るための調整弁に過ぎないから必然的にそうなる.

  • id:golden_eggg あるあるネタ。オイラの経験上になるけどこの手の話で出てくる "正社員" といわれる人は、俺はコードを「書かない」とよく言うけど「書けない」の誤りであることが多数
  • id:hobo_king 実在するぞ。俺がいたことあるから間違いない。
  • id:app2641 そんな環境ならレビューの意味ないんじゃないの? コードが分からない人の行うレビューになんの意味があるのか
  • id:ash1taka 書いたことない人にコードレビューしてもらうとか、英語知らない人に英語の添削してもらうようなもんじゃないの...? 英語でいう単語帳と、文法帳みたいのを1日かけて用意してレビューしてもらうの無駄じゃない...?
  • id:yoshi_kanagawa たぶん会社の開発ルールとして正社員のレビュー必須になってて、儀式としてやらないといけないんだろうなぁ・・。
  • id:Junpecon さすがにそんなレビューはないが、1ファイル1メソッドしか認められない規約ならあったな。ソース管理のためにコメントアウトで残すというのもあったけど、こんなのITあるあるでしょ?
  • id:temtem for文まわして特定のプロパティを固定値に書き換えてるだけのコードを「説明してください」みたいなね。たったの5行を。何を説明すればいいのかわからなかった。個人的な経験だけど。
  • id:pratto 運用だと、コマンドを知らない人がレビューだなwしかも指摘内容がオプションの意味とか。果てには「そのコマンドでないといけないのか」なんて、だったらls以外、ほかに最良のコマンドがあるなら、教えてくれよw
  • id:yukkesan it小町。ステップ数で見積しろって言う会社あるからね、未だに。長い長いmain()をドヤ顔で見せる人とか。
  • id:ikd9684 「コードの行数はこれでいいのか?少ないと仕事をしていないとみなされる」てとこに驚愕。/まぁ釣りなんだろうけど。
  • id:kaz_hiramatsu 大企業だと普通にあった。正社員は主にリリース管理とか仕様策定とかして、設計とコーディング、テストは派遣に投げる。で、レビューは正社員にしてもらう。
  • id:typex2 釣りかもしれないけどこれが日本のITの実情だと思う。。