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回しとかクエリの処理とかどこでルーチンを分けてサブルーチンにするかとか
変数とか配列とか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�の位置が表の下でもいいだろ
カス氏ね
注釈(>>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分岐で深いネストになってて、それが数百行ある関数とか、それを他人がメンテナンスするときのコスト考えろやとか
forループとif分岐で深いネストになってて、それが数百行ある関数とか、それを他人がメンテナンスするときのコスト考えろやとか
29: 以下、5ちゃんねるからVIPがお送りします 2020/11/30(月) 23:09:46.045 ID:gridEacP0
>>21
メソッド内に普通にSwitch文書くだけで
インデントの量多くてやきもきした気持ちになるのに
世の中そんなヤツがいるんだなあ
メソッド内に普通にSwitch文書くだけで
インデントの量多くてやきもきした気持ちになるのに
世の中そんなヤツがいるんだなあ
33: 以下、5ちゃんねるからVIPがお送りします 2020/11/30(月) 23:16:50.852 ID:Glpt1t5z0
>>21
設計書に流れでフローが書いてあるならその通りにしとくかな?
エラー処理含めて20行ぐらいのブロックだと
フローが30個ぐらい処理があるとネストが3つぐらいでも
5~600行行くじゃない?
設計書に流れでフローが書いてあるならその通りにしとくかな?
エラー処理含めて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年立ったらルールが通用しなくなったり
言語も仕様も方法もゴッソリ変わったりする
2年立ったらルールが通用しなくなったり
言語も仕様も方法もゴッソリ変わったりする
36: 以下、5ちゃんねるからVIPがお送りします 2020/11/30(月) 23:41:26.494 ID:OFJ/Rmtk0
実際の開発やとコードの書き方とかごちゃごちゃ考えてる暇ない
組み込みとかでない限りはばーーっと書いてばばっとテストして終わり
優秀な人材が書けば違うだろうけど実際にコード書くのは外注の2次とかだからな
組み込みとかでない限りはばーーっと書いてばばっとテストして終わり
優秀な人材が書けば違うだろうけど実際にコード書くのは外注の2次とかだからな
ソフトウェアデザイン 2021年1月号
posted with AmaQuick at 2020.12.02
コメント
コメント一覧 (14)
実際にその通りに直すと、それじゃおかしい!ってまた指摘入れるヤツ
氏ねっていつも思う
要求仕様の段階で「ソースコードと違う」って言ってもな・・・
コイツも氏ねっていつも思う
結論 レビューとは修羅場
wavefanc
がしました
レビューする余裕があることに感謝すべき
wavefanc
がしました
するのが面倒
wavefanc
がしました
wavefanc
がしました
技術ないから体裁チェックに躍起になるレビュアいるよな
wavefanc
がしました
wavefanc
がしました
wavefanc
がしました
ねちねち陰湿にやるだけの会社はクソ
wavefanc
がしました
wavefanc
がしました
それが出来ずに組織の中の一部になってる癖にえらそうだな
wavefanc
がしました
下に書く奴はダメだ許せん
wavefanc
がしました
こうゆうコミュニケーション出来なくて内なる不満を貯めてくやつ厄介だわ
wavefanc
がしました
コメントする