サブルーチンに値を引数で渡してたら「引数は禁止。全部グローバル変数で..

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

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

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

Permalink | 記事への反応(2) | 16:51

記事への反応 -

ソースレビューって意味あるの？
指摘内容にどうしても個人差があるじゃん？
- anond:20180409181511
  サブルーチンに値を引数で渡してたら「引数は禁止。全部グローバル変数で渡せ」とか、「サブルーチンを使うとあちこちに飛んでわかりにくい。同じ処理をするときはコピペでそれぞ...
  - anond:20180410165141
    「引数は禁止。全部グローバル変数で渡せ」えー、グローバル変数変わるタイミング分かりづれー「サブルーチンを使うとあちこちに飛んでわかりにくい。同じ処理をするときはコ...
  - anond:20180410165141
    30年以上昔の話だよな？これ
- anond:20180409181511
  あれは組織内の力関係をわからせるためにやるんだよ。全く同じように書いてもスルーされる人といじめられる人がいると気づく。本当にレビューしたかったら名前を伏せるはず。
  - anond:20180409182020
    ？？ソースコードのレビューの話なのこれ？？
    - anond:20180409182142
      ソースレビューといいながら人のレビューをしている話ですよ
      - https://anond.hatelabo.jp/20180409182410
        実際、ただのマウンティングだろ
    - anond:20180409182142
      ソースレビュー＝自分がいかに有能かをアピールする場だからな
  - anond:20180409182020
    レビュー依頼出す方も匿名で出したいよな
- https://anond.hatelabo.jp/20180409181511
  業務内容がわからないと適切な指摘はできない好みの問題で指摘が右往左往する複数人でやろうものならAさんがOKと言ってもBさんがNGと言ったら終わらない単に「やった」感を...
- anond:20180409181511
  俺はデミグラス派
- anond:20180409181511
  むしろソースレビューが機能してる現場ってのを教えて欲しい。マウンティングしたり人格否定以外に。