  |
俺（レビュアー）「ここはMath.minを使えば1行で書けるのでそうしてください」
俺「（だから何だよ…と思いつつ）盲目的にAIに従うのではなく、それを受け入れた以上コードの責任はあなたが持ってください」
レビュイー「しかしこの先コードを読むのもAIになってきている以上、AIが生成するコードはAIにとって読みやすい良いコードなんじゃないでしょうか？そもそもですが前から増田先輩は小さいことにこだわってapproveを遅らせすぎです。さっさとリリースしたほうが顧客価値に繋がるのでは？」
30分ぐらい悩んだが反論できず「確かにそういう考えもありますね」って言ってapproveした。
どうするのがよかった？
野太いちんぽぶち込んでわからせる
返り討ちに会いました🥰
3秒で「確かにそういう考えもありますね」と認めるのが最善やろなぁ
うむ。この元スレ主は30分かかっているがそれでも偉い。 土人にならずに進化したパティーンだ。 「俺がカスミンっつったらカスミンなんだよ使え！」ってのが土人だからね。
カスミン』（英: Mistin）は、2001年10月13日から2003年10月1日までNHK教育テレビで放送されたテレビアニメ作品、及びこれを原作とする漫画作品。 概要 小学4年生の少女・春野カスミが、ヘ...
マスミンと間違えてて草
「Math.minを使えば1行で書けるのでそうしてください」 この指摘が的外れだと思うな javascriptかtypescriptだとおもうけど、 三項演算子でも１行で書けるし if 文でも１行で書ける あと Math.mi...
そーいやテトリスも1行のコードのが昔バズったな
ではAIが生成するコードはAIにとって読みやすい良いコードであるということをあなたが証明してください、でなければ受け入れません、で俺ならたぶんはねた 遅い件に関しては自分が時...
レビュー観点が曖昧だからそうなるんだろ 1行で書くことで保守性が上がるとか、性能が大幅に上がるとかじゃなければそこを指摘して修正するコストに見合わない
「じゃあお前はAIが死ねと言ったら死ぬんだな？！」 （小学生並みの指導）
定量的な指標で判断しないからそうなるんだよ Code Complexityとかやってないでしょ
確かにそういう考えもありますね。(秒で返信してapprove) (しかし、自分で責任持てないコードは今後修正する際も自分で責任持てないコードであり続ける可能性が高い。こいつを上層部...
匿名サイトでの同僚悪口は最底辺無能の仕草
AIに聞けば30分も悩まず返信できたのに
これやな もうレビュイーもレビュアーもいらない AIだけがやりとりしてりゃいい
ますみんと言えば林真須美なんだけど、自分は真犯人は長女だったんじゃないかと思ってる…😟 いずれにせよ、林真須美は犯人ではない、犯人は当時子供だった人だと思ってる
保守を自分がやらないならさっさと通したほうがいいな
バカは死ななきゃ治らないとので人事を恨みましょう Math.minはif文で書くより文脈が絞られるので大体の場合了解性が上がりますね AIが書いたコードがAIにとって最も分かりやすいコード...
増田の後輩です。情報漏洩で窓口に通報しました。期待通りに動いてくれてありがとう
ほめてくれて、うれしいきもちでいっぱいですヾ(｡＞﹏＜｡)ﾉﾞ✧*。
そんなクソガキはクビにしろ。そのうち会社に害を与える何かをやらかす。
会社に入社してる奴は大体成人してるっしょ ガキとは言わんよ 大人なら大人としてわからせねば
どの言語のMath.minなんだろう。JavaScriptとか可変長引数を受け取って一番小さなものを返すMath.minならforを使うことにならない? .NETみたいに二つの数値のうち小さいほうを返すMath.minかな? ...
AIにとっての読みやすさを考慮しなくてもAIは瞬時に読めるから、AIにとっての読みやすさは考慮しなくていい。 approveを遅らせているのは増田ではなく、人が書いてしまえばすぐに終わる...
それをapproveするかどうかよりも、 approveした以上はコードの責任を元増田さんも持つことになるし、それをapproveしたなら今後も同じようにapproveする一貫性を持つ必要がある。 この重さ...
レビュイーさんの発言は全然かなあ。元増田さん言いくるめられただけじゃない？ この先コードを読むのもAIになる→疑わしい、レビュイーさんの考えにすぎないのでは？ AIが生成する...
なるほど、
しかしこの先コードを読むのもAIになってきている以上、AIが生成するコードはAIにとって読みやすい良いコードなんじゃないでしょうか？ 「AIにとって読みやすいかどうか」なんてモ...
AIが全部取って代わってくれるのはまだ先じゃないかなぁ。少なくともこの現場では 今はまだ人間が読みやすいコードがいいんじゃないかな AIが読みやすいコードって何って話もあるけ...
コードレビューをAIにやらせればいいのでは。 テストもリファクタリングもやってくれるでしょ。
小さいことにこだわっている、というとこに明確に反論出来てないから負けたんだろうね。 と言うか、AIが書いたんだもんとか頑張っちゃうならお前いらなくねーか？って話になりそう...
チームワークが合わない人と仕事するとモチベ落ちるしストレス溜まるよ 引き続き部下がそんな姿勢でコード書いてくるなら、別の部隊に配置変えできないか上司に相談するわ 小さい会...
もしこれがマルチスレッド処理は可読性落ちるから全部シングルスレッドで処理しろみたいな話だったらもっと燃えて欲しい await/asyncよくわからんから禁止ジジイにみんなでなろう
Approve したあとにしれっとそれを直す PR 自分で作るやで
AIレビューのルールで標準ライブラリ使うようなinstructions入れておけばいいんちゃうの https://docs.github.com/ja/copilot/how-tos/use-copilot-agents/request-a-code-review/use-code-review#%E3%82%AB%E3%82%B9%E3%82%BF%E3%83%...
でたっw ライブラリーw 横文字好きだねえw
ごめんな、instructionsもカタカナにしておけばよかったね
黙って approve したあと、コーディング規約に追記しておく
”AIが生成するコードはAIにとって読みやすい良いコード”か？ をAIに聞いてみれば自ずと答えが出る するとCopilotは「俺はAIが読みやすいコードではなく人間にとって読みやすく、保守し...
> この先コードを読むのもAI 自分で今の話じゃなくて将来の話をしているのに、説得した気になっているのなんなの。
この先、AIがコードを読む事はあるかもしれませんが、人も読みます。 AIの出力したコードは信頼が置けず、将来も人がコードを評価するのだから、Math.min で1行にまとめることには意味...
価値観合わせのために合宿に行け
私だったら 「今自分がコードを書いている以上は人の手から離れない。人が読めるコードを書くべき」 か 「AIも今後発展していく訳で、今AIが書いたコードがクソになる可能性があるこ...
sevenspice ぶっちゃけレビュー文化うんざりしてる。見逃してほしくないやつほど見逃されるし、まあ何かあったところで直すからとっととapproveしてリリースしてしまうのがいいのはその...
会社のレビューポリシーに合致してるか確認して合致してれば指摘はそのままで良いんじゃないの？ レビューポリシー自体がおかしいというなら責任者に掛け合えって投げればいい。
このレビュイーの気に入らないところは、眼の前の楽をするためにたいして調べもせず、考えもせずにAIをダシにしているところだな。 こういった人間は楽をするために周りに負担を与...
odakaho 「修正必須じゃないけどこうするとより良いと思います〜」みたいなノリのコメントすると必ずスルーする子がいて結構扱いに困ったことがあるけど、これはAIによる裏付け？があるのでなおタチが悪い。 
razokulover レビュイーの方はどうかと思うがレビューする側も良くなさそう 
otation 「先輩にこのように指摘されました。直すべきでしょうか？」ってAIに聞いてみなって言うかな。まず間違いなく今のLLMなら誤りを認めるし、AI経由なら後輩を説得する手間もない 
troron 寛容で柔軟な増田先輩 
bellonieta 短く書いた方がLLMへの入力トークン減らせる 
19
