workplace-1245776_1920

1: 以下、5ちゃんねるからVIPがお送りします 2020/11/30(月) 22:34:30.691 ID:MqKmmX7M0
細えこというやつ
てめえがなおせよって思う

それを直す手間と効果比較して考えて指摘しろや氏ね


2: 以下、5ちゃんねるからVIPがお送りします 2020/11/30(月) 22:37:47.323 ID:Pq3H8H2b0
ワロタ

4: 以下、5ちゃんねるからVIPがお送りします 2020/11/30(月) 22:38:54.969 ID:MqKmmX7M0
経験者求む

5: 以下、5ちゃんねるからVIPがお送りします 2020/11/30(月) 22:39:33.516 ID:agHoZFtP0
バグ報告はもっとして♥

7: 以下、5ちゃんねるからVIPがお送りします 2020/11/30(月) 22:40:43.275 ID:MqKmmX7M0
注釈は表の右側にかけって指摘

下でも右でも距離変わらないし
どっちでもいいだろ
めんどくさ過ぎ

8: 以下、5ちゃんねるからVIPがお送りします 2020/11/30(月) 22:41:11.028 ID:YIFNjGZfM
マウント取るの大好きだからな

10: 以下、5ちゃんねるからVIPがお送りします 2020/11/30(月) 22:43:33.428 ID:MqKmmX7M0
レビューって意味あんなのか?

ストレスしかたまらん

バグとか見つけるのが目的じゃないの

11: 以下、5ちゃんねるからVIPがお送りします 2020/11/30(月) 22:43:59.329 ID:sEFYxRVu0
本人に指摘して直させないと何度も同じ事するだろ

9: 以下、5ちゃんねるからVIPがお送りします 2020/11/30(月) 22:41:21.482 ID:OFJ/Rmtk0
それ直していくら儲かるんだよみたいなのは思うことがある

12: 以下、5ちゃんねるからVIPがお送りします 2020/11/30(月) 22:44:23.193 ID:Xl10MpyI0
いくら儲かるか損するかは経営側が判断するところで、同じ開発目線だと指摘するのが気持ちいいから指摘する

13: 以下、5ちゃんねるからVIPがお送りします 2020/11/30(月) 22:46:04.130 ID:VmBKi3w30
マシンの処理の仕方考えてこういう書き方の方がいいって言われたりするやつ勉強になったけどね
変数とか配列とかDO回しとかクエリの処理とかどこでルーチンを分けてサブルーチンにするかとか

28: 以下、5ちゃんねるからVIPがお送りします 2020/11/30(月) 23:09:17.294 ID:Glpt1t5z0
>>13
それが仕様と反しているかどうかが重要
仕様通りなのに修正するなら金よこせ

14: 以下、5ちゃんねるからVIPがお送りします 2020/11/30(月) 22:46:25.410 ID:MqKmmX7M0
そもそもどっちでもいいこと指摘するなよって思うんだが


注釈(>>0�の位置が表の下でもいいだろ
カス氏ね

16: 以下、5ちゃんねるからVIPがお送りします 2020/11/30(月) 22:48:44.202 ID:VmBKi3w30
いやいやコメントや注釈だってその会社のルールで書き方あるなら学ぶべきだろ
それで金もらうならそこのルールや仕様にならうしかない

18: 以下、5ちゃんねるからVIPがお送りします 2020/11/30(月) 22:49:35.775 ID:MqKmmX7M0
>>16

ルールなんてない。

どこでもいい。
そいつは表の右側が良かったんだろうな


まあやりとりめんどくさいから直したけど

20: 以下、5ちゃんねるからVIPがお送りします 2020/11/30(月) 22:52:41.103 ID:VmBKi3w30
>>18
今の時代は印刷しないんだから画面で表示して位置的にも見やすいか
とかもあるだろうし自分のコードの手直しを他人がやったり足していくもんだろ?
書き方は個人個人だけど見やすさの協調性は周りにあわせないとじゃないの?
ただいじわるでいう奴もいるだろうけどさ・・・

15: 以下、5ちゃんねるからVIPがお送りします 2020/11/30(月) 22:48:06.623 ID:DF38DSku0
評価項目あってのレビューなのに無しでやろうとする奴は地雷

17: 以下、5ちゃんねるからVIPがお送りします 2020/11/30(月) 22:49:03.811 ID:MqKmmX7M0
評価項目なんて設定してないなー

その都度思ったこと言われるだけ

19: 以下、5ちゃんねるからVIPがお送りします 2020/11/30(月) 22:50:15.688 ID:DF38DSku0
だからレビュアーは自分語りでマウントとる奴ばかりになるんだよ
レビューの目的が品質向上から逸れてる可能性ない?

21: 以下、5ちゃんねるからVIPがお送りします 2020/11/30(月) 22:52:49.067 ID:Xl10MpyI0
そんなのいちいち言わせんなっていうのもよくあるからな
forループとif分岐で深いネストになってて、それが数百行ある関数とか、それを他人がメンテナンスするときのコスト考えろやとか

29: 以下、5ちゃんねるからVIPがお送りします 2020/11/30(月) 23:09:46.045 ID:gridEacP0
>>21
メソッド内に普通にSwitch文書くだけで
インデントの量多くてやきもきした気持ちになるのに
世の中そんなヤツがいるんだなあ

33: 以下、5ちゃんねるからVIPがお送りします 2020/11/30(月) 23:16:50.852 ID:Glpt1t5z0
>>21
設計書に流れでフローが書いてあるならその通りにしとくかな?
エラー処理含めて20行ぐらいのブロックだと
フローが30個ぐらい処理があるとネストが3つぐらいでも
5~600行行くじゃない?

23: 以下、5ちゃんねるからVIPがお送りします 2020/11/30(月) 22:55:05.709 ID:VmBKi3w30
PGは書き方とか命令文の使い方とか要望通りにできれば自由だから
変わった人間が多いし変にプライド高い年いっても現場主義の人が多いのも事実だけどね

24: 以下、5ちゃんねるからVIPがお送りします 2020/11/30(月) 22:59:38.239 ID:M5YRmBurp
一度目は注意して直してもらう
同じミス繰り返してたら勝手に直す
評価は下げる
スペルミスとかインデントとかツール入れるだけで起こらなくなるミスを一度言われた以降も繰り返す奴は氏ねばいい

25: 以下、5ちゃんねるからVIPがお送りします 2020/11/30(月) 23:04:19.972 ID:KXQK4VWI0
いま設計書書いてるけど、いまのPJは細かい体裁についてはあまり突っ込まれないからやりやすいわ

26: 以下、5ちゃんねるからVIPがお送りします 2020/11/30(月) 23:06:18.383 ID:Z30CF4Qs0
こういうヤツが好き勝手やって保守性下げていくんだよな

27: 以下、5ちゃんねるからVIPがお送りします 2020/11/30(月) 23:06:40.877 ID:Glpt1t5z0
っていうかレビューのチェックリストがないレビューはやってはならない

30: 以下、5ちゃんねるからVIPがお送りします 2020/11/30(月) 23:12:11.673 ID:Glpt1t5z0
まず、契約は?

請負→はじめの仕様からそれてなければ直す必要なし
派遣→好きなだけ付き合ってやれどうせ向こうの金だ

31: 以下、5ちゃんねるからVIPがお送りします 2020/11/30(月) 23:14:47.718 ID:MqKmmX7M0
請負

そもそも仕様なんてない
ただのレビュー用資料、発表用資料について
そういう指摘されただけ

34: 以下、5ちゃんねるからVIPがお送りします 2020/11/30(月) 23:17:44.536 ID:Glpt1t5z0
>>31
それ、納期までにやること決まってるの?

32: 以下、5ちゃんねるからVIPがお送りします 2020/11/30(月) 23:15:52.821 ID:uJN7le/z0
好みレベルなら指摘しないわ
あと理由が明確に言えないなら指摘しない

35: 以下、5ちゃんねるからVIPがお送りします 2020/11/30(月) 23:19:11.935 ID:aZJaO2760
会社のルールとかっていってもこの業界現役世代は入れ替わりすごいからな
2年立ったらルールが通用しなくなったり
言語も仕様も方法もゴッソリ変わったりする

36: 以下、5ちゃんねるからVIPがお送りします 2020/11/30(月) 23:41:26.494 ID:OFJ/Rmtk0
実際の開発やとコードの書き方とかごちゃごちゃ考えてる暇ない
組み込みとかでない限りはばーーっと書いてばばっとテストして終わり
優秀な人材が書けば違うだろうけど実際にコード書くのは外注の2次とかだからな

ソフトウェアデザイン 2021年1月号
Software Design編集部(編集)
(2020-12-18T00:00:01Z)