- わかるやつおらん？
- レビューって何？
- レビュアー？
- 商品紹介する人
- ああレビューする側になるとバカが食い下がる光景を見るハメになるから疲れるよな
- コードレビューできる大きさのコードでいいプロジェクトが羨ましい
- >>6
これは恥ずかしい - >>9
ゲーム作ったことある？ - >>13
まず君の識見の狭さどうにかしたほうがいいよ - >>15
君は私より識見が広いようで何より - >>13
コードレビュー全くしないってどういうことなの…
1人が全部作るの…？ - >>16
全くしなくは無いよ
ただ組み込みとか家電みたく一行ずつレビューしたりはしなかった。医療機器と家電作ったときは みんなで一行ずつレビューしたわ
- >>17
ここで言ってるのはPRレビューとかのことかと - >>21
うん、途中から「コードレビューの事じゃないな～」と思った。
スレタイがレビューとしかなかったから、てっきり - >>25
コードレビューって一般的にはPRでのレビューのことじゃねえかな
全員で1行ずつ読むってかなり特殊な業界の話じゃ - >>28
車を制御するコード書いてるけど基本会議体だな
他の大多数が違うことを知って衝撃受けた - >>38
これからも絶対に漏れの無いように宜しくお願いシャッス！ - 情報部署のガバガバSEなんてどうとでもなるやろ
- する側もダルいけど受ける側もダルい
- 20ファイルくらいあると「大体ヨシ！」ってなるけど5ファイルくらいだと無駄に細かいところにコメントしだすよね
- [imo]とか[nits]とか使ってる？
- バグ見つけるのが目的だろ
どうでもいい細かい指摘されると
ぶん殴りたくなるそれお前の好みじゃんっていうね
- 動けばいいってもんじゃないし
- 同レベルのエンジニア同士だと割と気楽
ロクにリファクタしてこない奴がブーたれ始めると面倒 - 今時コードレビューないって珍しいな
- よくわからんがまぁ動いているからヨシ！
- ゲームの設計は知らんが今どきgit使ってなかったりモジュール分割してない所とかあるんだ
コード大きいからレビューしないってことはモノリスなの？ - （やべー、黙ってて良かった・・・）
- return文途中で入れるなとか言われたイラッとくる
ガード節で関数の先頭でreturn入れちゃった方が
読みやすいんだが - >>26
コードが読み易いことは同意するが、バグを発見し難くなるのでﾓﾚは指摘する。 - >>26
returnは最後に1つだけって大抵のところでルールになってると思うが - >>31
そうなの？ - >>33
goto論争を知らない世代か・・・
同じ理由で「returnは関数の最後以外には書かない」というルールがあったので、
社内ルールを設けているところでは（余程の新参でもない限り）規約に入れていると思うぞ。 - >>26
マジかよ逆にガード節にしろってコメントしちゃうわ - なんかスレチぽいので去ります
色々ごめんなさい - PRでのレビューって何？？
- >>29
githubのpull request - go toダメなのはわかるが
returnはいいじゃん・・・ - >>36
例外使え。assert() 使え。ガード節とか半世紀前の技術だぞ。マジ半世紀前で自分で言ってビビった・・・
- >>36
コーディング規約どうなってるの？
そういうのだいたい規約に書いてあるでしょ - 「いいんだけどなぁ、いやコード上は問題ないと思うよ？
でもなんかなぁ。もうちょっとスマートに組める気がしなくもないんだ。
ちゃんと検討してこうしたの？」 - 中小メーカー勤務でほぼ誰もまともにプログラムなんて触れない環境で1人でVBAやら組込やらプログラム書いてるけど自分のレベルは分からない
量産用というより社内で使う設計ツールがメインだからまあまあ雑に作ってる自覚はある
最近はPythonでkeras使った機械学習なんかも独学していて「これはもしかしてすごいものを作っているのでは？」と思いつつ「本業ならこれくらい簡単にこなすのだろうな」とも思う
給料は安い…。 - >>41
ＡＩかぁ・・・興味はあるがいまいち食指が動かないんだよな。
宣伝見ても猫の写真を自動判定するとかしか無いし。
本業と言っても、それぞれ得意分野があるからねぇ・・・
[プログラマー] レビューってストレス溜まるよな？
コメント