2018-04-10

anond:20180409181511

サブルーチンに値を引数で渡してたら「引数禁止。全部グローバル変数で渡せ」とか、「サブルーチンを使うとあちこちに飛んでわかりにくい。同じ処理をするときコピペでそれぞれの場所に同じコードを書きなさい」とか言われたことがあるわ。

そこまでひどくなくてもレヴューを受けると「うわ、こいつレベル低っ」「この人Excelばっかりでコードいたことないんだろうな」みたいな人ばっかり。

レビュー機能してる現場に遭遇したことない。

記事への反応 -
  • 指摘内容にどうしても個人差があるじゃん?

    • サブルーチンに値を引数で渡してたら「引数は禁止。全部グローバル変数で渡せ」とか、「サブルーチンを使うとあちこちに飛んでわかりにくい。同じ処理をするときはコピペでそれぞ...

      • 「引数は禁止。全部グローバル変数で渡せ」 えー、グローバル変数変わるタイミング分かりづれー 「サブルーチンを使うとあちこちに飛んでわかりにくい。同じ処理をするときはコ...

      • 30年以上昔の話だよな?これ

    • あれは組織内の力関係をわからせるためにやるんだよ。 全く同じように書いてもスルーされる人といじめられる人がいると気づく。 本当にレビューしたかったら名前を伏せるはず。

    • 業務内容がわからないと適切な指摘はできない 好みの問題で指摘が右往左往する 複数人でやろうものならAさんがOKと言ってもBさんがNGと言ったら終わらない 単に「やった」感を...

    • 俺はデミグラス派

    • むしろソースレビューが機能してる現場ってのを教えて欲しい。 マウンティングしたり人格否定以外に。

記事への反応(ブックマークコメント)

ログイン ユーザー登録
ようこそ ゲスト さん