22/12/10(土)16:24:40 コード... のスレッド詳細
削除依頼やバグ報告は メールフォーム にお願いします。個人情報、名誉毀損、侵害等については積極的に削除しますので、 メールフォーム より該当URLをご連絡いただけると助かります。
画像ファイル名:1670657080687.jpg 22/12/10(土)16:24:40 No.1002555095
コードレビューが存在しない会社に入社したんだけどよくあることなの？ あひるちゃんでデバッグするのも限界を迎えたんだけど
1 <a href="mailto:s">22/12/10(土)16:26:16</a> [s] No.1002555538
ここでこんな質問するあたり今の私は冷静さを欠いています
2 <a href="mailto:あひるちゃん">22/12/10(土)16:26:43</a> [あひるちゃん] No.1002555665
あるよ
3 22/12/10(土)16:27:20 No.1002555867
知らない文化だねーアヒルちゃん！
4 22/12/10(土)16:29:08 No.1002556402
アヒル「転職したら？」
5 22/12/10(土)16:32:45 No.1002557425
うｎうｎ
6 22/12/10(土)16:33:28 No.1002557610
アヒルちゃんがしゃべった！！！
7 22/12/10(土)16:33:39 No.1002557677
あひるちゃんはどう思う？
8 22/12/10(土)16:34:18 No.1002557880
PGってそんなにココロが壊れるもんなの…
9 22/12/10(土)16:34:38 No.1002557972
ｸﾞﾜｯｸﾞﾜｯ
10 22/12/10(土)16:35:04 No.1002558111
グループで仕事回してなあなあになってるようなとこだとあるよね
11 22/12/10(土)16:35:23 No.1002558215
ChatGTPでやってくれるらしいな
12 <a href="mailto:s">22/12/10(土)16:35:30</a> [s] No.1002558253
怒りでコンプラ違反しそうでヤバい 入社一年も迎えてないやつに認証系投げするなどうなっても知らんぞ どうなっても知らんぞって話はちゃんと上司にしたぞ
13 22/12/10(土)16:35:38 No.1002558302
レビューしない会社ってやべーところじゃないの
14 22/12/10(土)16:36:50 No.1002558646
レビューしてもいつもなし崩しで承認される会社もあるぞ！
15 22/12/10(土)16:37:13 No.1002558748
自分の成長が見込めないような会社はとっとと切った方がいいぞ
16 22/12/10(土)16:37:34 No.1002558840
レビューが形骸化してる会社ならあるぞ弊社だ
17 22/12/10(土)16:37:57 No.1002558947
コードレビューって何かと思ってググったけどこれプログラマーに限らずあらゆる業務でやることでは？
18 22/12/10(土)16:38:00 No.1002558963
仁義なき戦い見るといいよ
19 22/12/10(土)16:38:05 No.1002558994
自分で作って自分で使うようなところだとあんまやらない
20 22/12/10(土)16:38:45 No.1002559189
自分でも重々正気じゃない事は自覚してるんだからさっさと辞めろ 碌なことにならんぞ
21 22/12/10(土)16:39:09 No.1002559291
管理職でもないのに真面目に考えすぎだろ でっかい会社に派遣で行ったとき無かったことあったからそんなもんだよ
22 22/12/10(土)16:39:20 No.1002559336
要は何か起きたときの責任を末端の個人に押し付けるってことだぞ やめとけやめとけ
23 22/12/10(土)16:39:24 No.1002559357
>どうなっても知らんぞって話はちゃんと上司にしたぞ ちゃんと録音した？
24 22/12/10(土)16:39:32 No.1002559386
まあテストはやってるんでしょ 良い書き方は自分で勉強しようね… グワッグワッグワッ
25 22/12/10(土)16:39:36 No.1002559401
コードレビュー無いのもヤバいけどテストケースレビューが無かったら一番ヤバイ 品質の担保が全く出来て無いってことだから
26 22/12/10(土)16:40:43 No.1002559731
マトモなレビュワーが多い会社だと新人の成長速度がすごいらしいね その分離脱率も高い
27 22/12/10(土)16:40:50 No.1002559770
上司がアヒルちゃんになって1年が経つが業務に支障が無い って状況になるかもしれない
28 22/12/10(土)16:41:22 No.1002559946
上司に相談した日時と内容メモしとけよ 損害賠償請求されたときの証拠に使えるから
29 22/12/10(土)16:42:31 No.1002560274
リリースする前にセキュリティチェックとか脆弱性診断みたいなやつやってればいいんじゃね？
30 22/12/10(土)16:43:28 No.1002560540
適当なレビューしてごめん…
31 22/12/10(土)16:44:58 No.1002560920
>どうなっても知らんぞって話はちゃんと上司にしたぞ えらい！
32 <a href="mailto:s">22/12/10(土)16:45:13</a> [s] No.1002560978
>コードレビュー無いのもヤバいけどテストケースレビューが無かったら一番ヤバイ ないよ いやレビューしてるらしいけどフィードバックがないよ この内容で進めますけどいいんですか？とは確認とったよ 転職かなこれ
33 22/12/10(土)16:46:41 No.1002561371
忙しいからコード後で見とくって言われたけど見てなかった挙句不具合発生したのでボーナスちょっと削られました
34 <a href="mailto:s">22/12/10(土)16:48:17</a> [s] No.1002561804
駄目だなんかまたイライラしてきた 酒飲んで寝るね… 話聞いてくれてありがとう…
35 22/12/10(土)16:49:29 No.1002562149
レビューは形式化してるからやめようってのはまともな会社とかスキルあるエンジニアでも言ってる人はちょくちょくいる
36 22/12/10(土)16:50:44 No.1002562516
小規模とは言え一つのシステムの改修丸投げされて一切のレビューなくもうすぐ入れ替え日でこの調子でいいのか…と思いながらずっとやってる
37 22/12/10(土)16:51:04 No.1002562611
まともなレビューはやるべきだけどまともなレビューが出来る人材がいないならやる意味はほぼないって意味ならそう
38 22/12/10(土)16:51:50 No.1002562815
みんなのレビューを統括する役職つくってなればいいグワ
39 22/12/10(土)16:51:53 No.1002562838
ベンチャーなんてそんなもんでは
40 22/12/10(土)16:52:01 No.1002562878
中途に対して社内教育がないというか 上司が上司の仕事をしない会社はある あるのだ 新卒？ …ヨシ！
41 22/12/10(土)16:52:09 No.1002562920
レビューでやたらめったらコーディングルール的なツッコミ入るのに頑なにlint導入しない現場にあたったことあるけど最悪だった
42 22/12/10(土)16:52:34 No.1002563041
どうせレビューしても素通しならアヒルちゃんに話した方が工数取らない分マシ
43 22/12/10(土)16:52:57 No.1002563171
(まあこっちの処理がバグってても相手行側のチェックで弾かれるから大丈夫だろ…) ヨシ！
44 画像ファイル名:1670658817767.png 22/12/10(土)16:53:37 No.1002563383
ｷﾀ━━━━━━(ﾟ∀ﾟ)━━━━━━ !!!!!
45 22/12/10(土)16:54:44 No.1002563733
うちはレビューせずにほぼお客さんにテストさせてるわ 保守料が割に合わんしワガママ放題言うからええやろ…
46 22/12/10(土)16:54:47 No.1002563742
プロジェクトが基本的に自分一人しかいないからレビューするやつ自分！レビューされるやつ自分！ってなってる やる意味あるのかなこれ
47 22/12/10(土)16:55:18 No.1002563901
認証系認証系アミノ式
48 22/12/10(土)16:55:20 No.1002563911
ChatGPTに聞くグワッ
49 22/12/10(土)16:55:39 No.1002564008
>1670658817767.png 糞みたいな揚げ足取りばっかりでコードの品質に貢献しそうにないアヒルちゃんだ…
50 22/12/10(土)16:56:19 No.1002564212
>1670658817767.png みにくいアヒルの子「」…
51 22/12/10(土)16:56:28 No.1002564264
>やる意味あるのかなこれ やらないよりはマシ やらないよりは
52 22/12/10(土)16:56:58 No.1002564399
アヒルちゃんいいよね たまに話しかけてきてくれるから癒される
53 22/12/10(土)16:57:12 No.1002564464
>アヒルちゃんいいよね >たまに話しかけてきてくれるから癒される 病院行け 心のだぞ
54 22/12/10(土)16:57:16 No.1002564482
動けばいいクワ
55 22/12/10(土)16:57:32 No.1002564557
レビューはバグより社員全体の勉強にちょうど良いので能力うんぬんは置いておいてやったほうがいいですよ
56 22/12/10(土)16:58:04 No.1002564714
うちはレビュー以前にコーディングルールもないぜ！
57 22/12/10(土)16:58:28 No.1002564841
>うちはレビュー以前にコーディングルールもないぜ！ メンテや機能追加は…どうなさるんで？
58 22/12/10(土)17:00:14 No.1002565334
「」は病院行けって言ってくれるけど病院に行ったところで仕事量が減ったり改善したりするわけじゃないからなぁ
59 22/12/10(土)17:01:17 No.1002565650
>>うちはレビュー以前にコーディングルールもないぜ！ >メンテや機能追加は…どうなさるんで？ たんとうしゃががんばる
60 22/12/10(土)17:01:18 No.1002565664
>No.1002563383 ナトリウムで作った水に入れると爆発的反応するアヒル思い出す
61 22/12/10(土)17:01:20 No.1002565672
>「」は病院行けって言ってくれるけど病院に行ったところで仕事量が減ったり改善したりするわけじゃないからなぁ メンタルやられて病院行ってるんですよーって上長なりに言えば仕事量減るぞ
62 22/12/10(土)17:01:23 No.1002565685
>「」は病院行けって言ってくれるけど病院に行ったところで仕事量が減ったり改善したりするわけじゃないからなぁ 診断書書いてもらって休むか転職すんだよゥッ！
63 22/12/10(土)17:02:13 No.1002565956
スレ「」が責任を問われる頃には転職してますように
64 22/12/10(土)17:02:14 No.1002565967
少人数かつ全員がわかりやすいコードを書こうという気持ちがあればルールなくてもまあ…
65 22/12/10(土)17:02:31 No.1002566055
バグってもQAチームが忙しそうになるだけなので適当に作る
66 22/12/10(土)17:03:22 No.1002566336
>「」は病院行けって言ってくれるけど病院に行ったところで仕事量が減ったり改善したりするわけじゃないからなぁ 精神科にかかることのデメリットもあるので本当にキツくなった時に行くんだぞ
67 22/12/10(土)17:03:33 No.1002566396
フォーマッターやリンターで検知できないことをやるのがレビューだが それを短時間でやるのって難しいので形骸化しがちだと思う
68 22/12/10(土)17:04:09 No.1002566554
会話だけじゃなくてメールで証跡取っておけよ
69 22/12/10(土)17:04:37 No.1002566708
>それを短時間でやるのって難しいので形骸化しがちだと思う ちゃんと作ったものの全てを説明しようとすると無限に工数必要だし 説明したところでレビューイがちゃんと指摘できるかが怪しいんだよな
70 22/12/10(土)17:05:09 No.1002566874
>会話だけじゃなくてメールで証跡取っておけよ 言った言わないの水掛け論になるよな絶対
71 22/12/10(土)17:05:40 No.1002567023
>>会話だけじゃなくてメールで証跡取っておけよ >言った言わないの水掛け論になるよな絶対 議事録禁止の会議いいよね…
72 22/12/10(土)17:05:43 No.1002567038
バグの原因が社内誰もわかんないんだけど どうすればいいかな……
73 22/12/10(土)17:06:06 No.1002567137
>>>会話だけじゃなくてメールで証跡取っておけよ >>言った言わないの水掛け論になるよな絶対 >議事録禁止の会議いいよね… そんな会社逃げるわ
74 22/12/10(土)17:06:12 No.1002567181
何かあったらおあしすする雰囲気バリバリしてるからな…
75 22/12/10(土)17:06:18 No.1002567212
なので自動テストきっちり書いてレビューはなしって運用はありだと思う 大きな設計上の決断はドキュメントに残すとかかなあ
76 22/12/10(土)17:06:37 No.1002567304
>批判 >コードレビューよりもコーディングにあたっての規則や方法論を整備することのほうが重要であるとの見方もある。エクストリーム・プログラミング (XP) という技法では、ペアプログラミングというプラクティスを推奨しており、コーディングの最中に同時にコードレビューを行うようになっている。XP の信奉者は、リファクタリングやコードの前にテストを書くといったXPのプラクティスによってレビュー/書き直しが不要なコードを作成することでソフトウェア開発がスピードアップすると主張する。 >DOD-STD-2167A[4] では、コードレビューは「労多く益少なし」としている。Lausen と Younessi（IEEE Software, July/Aug 1998, pg 69-73）では、行単位のコードレビューはほとんど価値がないと結論付けている。問題点を除去するという意味では、プログラマに行単位のコードレビューをさせることは、他の手法よりも生産性が低い。 なるほど
77 22/12/10(土)17:07:37 No.1002567573
聞いてない言ってないで責任逃れするカスは普通に居るから証拠は残そう
78 22/12/10(土)17:07:40 No.1002567589
処理として安全なコードならおっけー！してる 個人差がある崩した書き方や圧縮表記については突っ込まないようにしている
79 22/12/10(土)17:08:16 No.1002567765
インターフェースさえあってればいいみたいなところはある
80 22/12/10(土)17:08:21 No.1002567780
エクストリーム・プログラミングはよく言われるけど実際適用出来る組織は多分少ないんじゃねえかなあ…
81 22/12/10(土)17:09:01 No.1002567982
Gitlab使うとかカンバン管理真面目にやるとかツール方面の外堀から埋めとかないとレビューだけしても意味ない
82 22/12/10(土)17:09:27 No.1002568108
でも認証系って言ってるからレビューはした方がいいよな…
83 22/12/10(土)17:09:41 No.1002568193
ペアプロの方が高効率なのは実際そうなのだが コードレビューだって全然無駄ってわけじゃない そしてコードレビューすらする気のない会社にペアプロ導入する能力があるはずもない 転職だな
84 22/12/10(土)17:09:57 No.1002568277
見積もり書見たら？ あとWBSとかみたらかいてあるでしょ？
85 22/12/10(土)17:10:12 No.1002568351
>でも認証系って言ってるからレビューはした方がいいよな… こけるならマシで入れちゃダメなやつ通すのがヤバい
86 22/12/10(土)17:10:40 No.1002568472
>バグの原因が社内誰もわかんないんだけど ミドルウェアやハードウェアの都合でわかんないことはよくある 最も賢い解決方法はその機能を使わないことだ
87 22/12/10(土)17:11:03 No.1002568563
ペアプロはやだなぁ…
88 22/12/10(土)17:11:05 No.1002568570
>見積もり書見たら？ 新卒に閲覧権限あるような会社には思えないな…
89 22/12/10(土)17:11:05 No.1002568573
ペアプロって難しい 話すとコード書く並行思考ができない
90 22/12/10(土)17:11:06 No.1002568579
うちもないよ 上司の書いたクソみたいなコードを直すかかりでブチ切れそう
91 22/12/10(土)17:11:10 No.1002568593
>処理として安全なコードならおっけー！してる >個人差がある崩した書き方や圧縮表記については突っ込まないようにしている 共通のコーディング規約とかないの…？
92 22/12/10(土)17:11:22 No.1002568652
そもそもレビューってプルリクエストとかマージリクエストの仕組み使ってたら自然に起こらない…？ 何送っても無言でOK出すならそれは取り込む人が悪いのでは…
93 22/12/10(土)17:11:31 No.1002568689
コードレビューって知識はもちろん1時間で400行程度読める能力も必要なんだよな
94 22/12/10(土)17:11:44 No.1002568749
酷いのだとDBぐちゃぐちゃにされるから認証系ならちゃんとしたほうがいい
95 22/12/10(土)17:11:53 No.1002568791
真面目にコードレビューしてる現場なんて実際ごく少数では？
96 22/12/10(土)17:12:13 No.1002568885
>真面目にコードレビューしてる現場なんて実際ごく少数では？ 弊社は雰囲気でレビューをしている…
97 22/12/10(土)17:12:51 No.1002569079
コードレビューもいいけどちゃんと設計してくれ
98 22/12/10(土)17:13:22 No.1002569225
弊社はディスクリプション変更だけでも当然二者確認必須だし客のレビューも必須だよ
99 22/12/10(土)17:13:30 No.1002569264
形だけでもやってくれると悪いのは俺じゃないって開き直れる
100 22/12/10(土)17:13:39 No.1002569305
>そもそもレビューってプルリクエストとかマージリクエストの仕組み使ってたら自然に起こらない…？ スレ「」のとこは昨日丸ごと任されててインターフェイスはまだしも中身触るの自分だけとかなのかもしれない
101 22/12/10(土)17:14:02 No.1002569419
レビューイ不足しすぎ問題
102 22/12/10(土)17:15:02 No.1002569723
>レビューイ不足しすぎ問題 レビューイできる人なら当然忙しいからな…
103 22/12/10(土)17:15:28 No.1002569873
>共通のコーディング規約とかないの…？ 厳格なものは無いよ 細かいところまで言及しだすとタウンページみたいになるし言語側も変わるしで難しい
104 22/12/10(土)17:15:41 No.1002569938
年収600とか出てるまともなところってやっぱりまともな人しかない？ 転職サイトに書いてるような土日も勉強してる技術者集団なの？
105 22/12/10(土)17:15:57 No.1002570017
俺は俺の能力に信用が置けないのでレビューされないと怖くて仕方ない
106 22/12/10(土)17:16:27 No.1002570175
プロジェクト炎上して地獄見る前に逃げろ
107 22/12/10(土)17:16:32 No.1002570198
>そもそもレビューってプルリクエストとかマージリクエストの仕組み使ってたら自然に起こらない…？ >何送っても無言でOK出すならそれは取り込む人が悪いのでは… チーム体制によるんじゃないかな 最低限は変なファイル更新しようとしてないかと悪さするコード仕込んで無いかぐらいはチェックするだろうけど スレ「」が求めてるのは対等以上の丁寧な指南役みたいだし
108 22/12/10(土)17:16:39 No.1002570239
土日imgして平日もサボりサボりしてるのはそりゃ伸びないよ 俺が保証する
109 22/12/10(土)17:16:58 No.1002570336
左から右へ受け流されるようなレビューしか受けたことないからレビューでガチ喧嘩が起こる空気を味わったことがない
110 22/12/10(土)17:17:10 No.1002570402
レビューとテストをまじめにやるかどうかはプロジェクトの規模と目的にもよる気がする
111 22/12/10(土)17:17:42 No.1002570557
テストはするでしょ…？
112 22/12/10(土)17:17:55 No.1002570614
>プロジェクト炎上して地獄見る前に逃げろ 逃げられないと心身壊してゾンビみたいになって生き続ける事になるぞ
113 22/12/10(土)17:17:59 No.1002570634
今は車載組込系でレビューしまくりだけど前にいた客先情シス常駐分室ではエンドユーザのテストさえ通ればどうでもよかった…
114 22/12/10(土)17:18:17 No.1002570733
Web/Android/iOSの混成サービスでそれぞれ専門エンジニアが１～２名程度のチームもままあるよね
115 22/12/10(土)17:18:24 No.1002570774
レビューイがレビュー参加多すぎて自分の仕事に手が回らないのがうちの職場だよ そのうちやめていくからレビューイがどんどん減るよ
116 22/12/10(土)17:18:32 No.1002570818
>テストはするでしょ…？ テストコード書かない 書くような場所行きたい…
117 22/12/10(土)17:18:40 No.1002570852
>今は車載組込系でレビューしまくりだけど前にいた客先情シス常駐分室ではエンドユーザのテストさえ通ればどうでもよかった… 人の命がかかってる車と情シスじゃそりゃ違う
118 22/12/10(土)17:18:45 No.1002570878
GitLabはレビューとテストに特化してるから自然と出来るとは思う まあプロジェクトのオーナー次第だけど
119 22/12/10(土)17:18:46 No.1002570893
N○Cとかそんな感じなんじゃないかと思う コードの比較すらしてないようなデグレードバグが現場で起きて怒りを通り越して呆れてしまった
120 22/12/10(土)17:18:59 No.1002570957
サニティチェック程度のやつから要件満たすか設計おかしくないか確認とかいろいろある 会社の文化によるんだろうな
121 22/12/10(土)17:19:25 No.1002571075
web系は速度重視でほとんどやらない印象がある
122 22/12/10(土)17:19:31 No.1002571105
サブバージョン使ってるけど上司がバグチェックもしないコメントも書かないコミットしてくるよ！ 楽しいね
123 22/12/10(土)17:19:43 No.1002571161
>会社の文化によるんだろうな どこにどれだけのコストをかけるかってのは会社の文化と案件次第よね
124 22/12/10(土)17:19:47 No.1002571185
>テストはするでしょ…？ 正常系動作ヨシ！ 証拠にスクショ撮ってエクセル貼り付け証憑ヨシ！ テストヨシ！
125 22/12/10(土)17:19:54 No.1002571207
>web系は速度重視でほとんどやらない印象がある 多分動くから実装してみようぜは結構ある ﾋもそうだったしね
126 22/12/10(土)17:20:12 No.1002571285
こういうスレ見かけるたび思うけどimgってプログラマ多いのかな ちなみに俺は1/4くらい何言ってるかわからん
127 22/12/10(土)17:20:28 No.1002571372
自動テストとか静的解析で担保出来てるならいいけど そういう対策もしてないならヤバい
128 22/12/10(土)17:20:33 No.1002571399
>>テストはするでしょ…？ >正常系動作ヨシ！ テストパターンは網羅されてますよね？
129 22/12/10(土)17:20:54 No.1002571521
>>テストはするでしょ…？ >正常系動作ヨシ！ >証拠にスクショ撮ってエクセル貼り付け証憑ヨシ！ >テストヨシ！ 異常系やれや！
130 22/12/10(土)17:21:11 No.1002571614
>こういうスレ見かけるたび思うけどimgってプログラマ多いのかな プログラマというかSEとSE経由してもっと上の立場になった人とか
131 22/12/10(土)17:21:12 No.1002571622
>テストパターンは網羅されてますよね？ 多分ヨシ！
132 22/12/10(土)17:21:16 No.1002571655
>テストパターンは網羅されてますよね？ jacocoで100%ヨシ！
133 22/12/10(土)17:21:31 No.1002571750
ふふふ…ライセンス回りがばがば過ぎてオレがこれ漏らしたらこの会社終わるな…ってまだ数ヵ月なのにもう悟ってるオレもいるぞ
134 22/12/10(土)17:22:04 No.1002571941
>web系は速度重視でほとんどやらない印象がある 週に何回も本番リリース繰り返すのを最上とする文化圏もあるしねえ
135 22/12/10(土)17:22:30 No.1002572053
>>テストパターンは網羅されてますよね？ >多分ヨシ！ 多分じゃだめだろうが！ >jacocoで100%ヨシ！ それは製造の時にやるもんであって仕様にあったテストデータでテストしないとダメだろ
136 22/12/10(土)17:22:39 No.1002572089
うちも年度末に納めるやつ奴はレビュー無しだな 人乗るシステムだからヤバイと思う でも時間も金もないんだ...許してくれ...
137 22/12/10(土)17:23:08 No.1002572248
>こういうスレ見かけるたび思うけどimgってプログラマ多いのかな >ちなみに俺は1/4くらい何言ってるかわからん スレの伸びだと現場が一番勢いありそう
138 22/12/10(土)17:23:12 No.1002572271
問題起きてもすぐ再リリースできる仕組みが出来てればいいんだけどね… リリースまでに時間掛かる上にテストもしないならウンコ
139 22/12/10(土)17:23:26 No.1002572349
外注コードは割とやばい お前どうしてこんなネストになるんだってほど深く深くネストして 読めなくした上でバグが含まれてる
140 22/12/10(土)17:23:26 No.1002572355
実装やテストの下請けが日本で一番人口の多い職種だから自然とネットで目にする機会が多くなるだけだと思うよ
141 22/12/10(土)17:23:54 No.1002572497
>>web系は速度重視でほとんどやらない印象がある >週に何回も本番リリース繰り返すのを最上とする文化圏もあるしねえ 手続きと手順が面倒すぎて死にそう 本番リリースなんてリリース手順書とリリース用データと戻し手順書全部レビューしてOKでないとできなかった… なおリリース日はレビュー結果に影響されないものとする
142 22/12/10(土)17:23:57 No.1002572510
世の中に出たのはちゃんと動いてるからすごいよな すごいよなあ！
143 22/12/10(土)17:24:54 No.1002572749
>世の中に出たのはちゃんと動いてるからすごいよな >ちゃんと動いてる …
144 22/12/10(土)17:24:55 No.1002572756
ゆるゆるどころかえっあれ検討前の調査どころかもう作れって話だったんですか！？ってこと多すぎて俺もうヤバいかもしれない
145 22/12/10(土)17:25:00 No.1002572782
>世の中に出たのはちゃんと動いてるからすごいよな >すごいよなあ！ 現場に出るとわかる事 世の中とかシステムってかなり適当でも動く
146 22/12/10(土)17:25:14 No.1002572848
はやく退職しちまえそこにしがみついて出世したら今度は自分が責任とる側だぞ
147 22/12/10(土)17:25:14 No.1002572854
神様だって六日で世界作って動かしてるんだ 人間だって似たようなものでいいさ
148 22/12/10(土)17:25:35 No.1002572960
みずほだって９９．９％はちゃんと安定稼働してるんだ ならもうそれでいいじゃあないかい
149 22/12/10(土)17:25:47 No.1002573022
>世の中に出たのはちゃんと動いてるからすごいよな >すごいよなあ！ ～♪
150 22/12/10(土)17:25:48 No.1002573024
まああれだ うちの製品は使いたくねえな
151 22/12/10(土)17:25:53 No.1002573044
>神様だって六日で世界作って動かしてるんだ >人間だって似たようなものでいいさ 俺たちは神様じゃねえんだ
152 22/12/10(土)17:25:59 No.1002573082
>外注コードは割とやばい >お前どうしてこんなネストになるんだってほど深く深くネストして >読めなくした上でバグが含まれてる マジックナンバー駄目！ってコード規約があるのか知らんが数値が全部 U8_VALUE_1A みたいな定数になってて頭抱えたことはあった
153 22/12/10(土)17:26:01 No.1002573089
>世の中に出たのはちゃんと動いてるからすごいよな >すごいよなあ！ しらない すんだこと
154 22/12/10(土)17:26:21 No.1002573177
他人の描いたコードを真面目に読める人はそんな多く無い…
155 22/12/10(土)17:26:36 No.1002573253
転職したら現状の商品概要どころか中身わかんねぇのに改修しろって話来たけど書いてるの一人だけだしコメントロクにねぇし しかもその人は社員じゃねーぜ！！！なんだこれ！！！！
156 22/12/10(土)17:27:14 No.1002573445
計画書やスケジュール上はレビュー工程があるしそれで承認は出てるよ
157 22/12/10(土)17:27:23 No.1002573492
お前に改修させて作った人切りたいんだろ
158 22/12/10(土)17:27:24 No.1002573496
世の中の話聞くとそりゃコンサル入って理想論かざして適当にお高いお賃金もらえるわ って思う
159 22/12/10(土)17:27:28 No.1002573509
コードの質は指摘できるけど機能仕様を正しく実装できてるかなんてレビューしようがねえんだよなあ だからそこを頑張って説明しようとするのやめろ
160 22/12/10(土)17:27:37 No.1002573555
>>外注コードは割とやばい >>お前どうしてこんなネストになるんだってほど深く深くネストして >>読めなくした上でバグが含まれてる >マジックナンバー駄目！ってコード規約があるのか知らんが数値が全部 U8_VALUE_1A みたいな定数になってて頭抱えたことはあった 数字弄ると再リリースか…
161 22/12/10(土)17:28:12 No.1002573708
>計画書やスケジュール上はレビュー工程があるしそれで承認は出てるよ 末端を殺すムーブやめろ
162 22/12/10(土)17:28:21 No.1002573750
>他人の描いたコードを真面目に読める人はそんな多く無い… 舐めんな 一週間前に書いた自分のコードすら読めねえぞ
163 22/12/10(土)17:28:57 No.1002573925
これが俺のかいたコード…？っていつもなってるぞ
164 22/12/10(土)17:29:14 No.1002574007
>舐めんな >一週間前に書いた自分のコードすら読めねえぞ コメントぐらい残しなされ
165 22/12/10(土)17:29:17 No.1002574020
だからコメントを書くのを癖どころかフォーマットにする必要があったんですね おい…どうして主要処理の部分にコメントが1行すら存在しないコードが出て来てる…？
166 22/12/10(土)17:29:23 No.1002574057
コメント残したら多少は読めるでしょ！
167 22/12/10(土)17:30:05 No.1002574227
なぜ俺はこんなコメントを残したんだ？
168 22/12/10(土)17:30:12 No.1002574267
// これ消すとおかしくなる
169 22/12/10(土)17:30:25 No.1002574338
いいですよね残したコメントの意図が毛先もわからないの
170 22/12/10(土)17:31:07 No.1002574520
// 念のため配列hageをすべて0で初期化する
171 22/12/10(土)17:31:08 No.1002574523
たまにコメントだけ直し忘れて整合性が取れなくなってるものがある
172 22/12/10(土)17:31:23 No.1002574578
たまに？
173 22/12/10(土)17:31:28 No.1002574605
特殊処理が必要ってことである関数の100行のうちの10行程度変えるだけなのに 丸々倍にして同じ関数で10行変えただけのコード作る人とかいる それを連続で繰り返した800行のコードを見てくらってくる…
174 22/12/10(土)17:31:32 No.1002574631
>外注コードは割とやばい うちのコードにも外注部分があるけどまじで酷い こんなものかぁ……自社でやるしか無いなってなった
175 22/12/10(土)17:31:41 No.1002574673
>いいですよね残したコメントの内容が古くて信用できないの
176 22/12/10(土)17:31:47 No.1002574696
コメントは書かれてるけどコメントにない処理も挟まってるのいいよね
177 22/12/10(土)17:31:53 No.1002574726
変数で意味明示したらレビューでおこられた無駄だって
178 22/12/10(土)17:31:54 No.1002574731
// A に B を掛けて C に代入する C = A * B;
179 22/12/10(土)17:32:06 No.1002574784
コミットコメントを適当に書くな馬鹿！
180 22/12/10(土)17:32:06 No.1002574788
処理を追加修正した日と名前だけ書いたコード良いよね… 書いた本人は既に退職済みまでがセットだけど
181 22/12/10(土)17:32:29 No.1002574904
>// A に B を掛けて C に代入する >C = A * B; 何のために…？ってなるクソコメント
182 22/12/10(土)17:32:31 No.1002574916
人のところのコード適当に変える上にコメント書く気ないからコメントと実装ずれるみたいな経験もした 年下だったら文句言えるんだけど…
183 22/12/10(土)17:32:54 No.1002575041
保守時にコメントまで整合性取られることは稀なので俺はコメントアンチ派です コメントなくても読めるコードを書け
184 22/12/10(土)17:32:58 No.1002575061
本番とテストで差分がですね
185 22/12/10(土)17:33:05 No.1002575105
ツイッターもコードレビュー全然してないって話だったね まともなレビュー一回も受けたこと無いエンジニアがレビュー品質の悪さを理由にどんどん首切られてる
186 22/12/10(土)17:33:07 No.1002575122
>>他人の描いたコードを真面目に読める人はそんな多く無い… >舐めんな >3日前に書いた自分のコードすら読めねえぞ
187 22/12/10(土)17:33:15 No.1002575165
>処理を追加修正した日と名前だけ書いたコード良いよね… あの無駄に名前残す文化なんなんだろう 画家がお前は
188 22/12/10(土)17:33:17 No.1002575179
>>// A に B を掛けて C に代入する >>C = A * B; >何のために…？ってなるクソコメント だってコメント書けっていうから…
189 22/12/10(土)17:33:43 No.1002575308
コードの説明じゃなくてなんでこういう実装にしたかをコメントに残そうね
190 22/12/10(土)17:34:00 No.1002575378
>>処理を追加修正した日と名前だけ書いたコード良いよね… >あの無駄に名前残す文化なんなんだろう >画家がお前は 犯人捜ししないため 分かってたら切りやすいでしょ
191 22/12/10(土)17:34:08 No.1002575425
>>処理を追加修正した日と名前だけ書いたコード良いよね… >あの無駄に名前残す文化なんなんだろう >画家がお前は さすがにもうないでしょ？
192 22/12/10(土)17:34:11 No.1002575441
なんならコメント書いた翌日に見ても 分からんときがある
193 22/12/10(土)17:34:28 No.1002575536
C == A * B と誤解されないようにしたいのかもしれない
194 22/12/10(土)17:34:31 No.1002575557
>あの無駄に名前残す文化なんなんだろう >画家がお前は 修正前の処理をコメント化する文化もあったんだぞ 今は大分減ったけど
195 22/12/10(土)17:34:48 No.1002575655
昔はコメントにCOPYRIGHT勝手に書いてた人居たなあ
196 22/12/10(土)17:34:53 No.1002575679
git覚えるのって簡単？
197 22/12/10(土)17:35:00 No.1002575712
前のコードコメントアウトして残す文化は都市伝説と思ってるけど本当にあったの？
198 22/12/10(土)17:35:02 No.1002575721
今10年以上前に書かれたコードの改修してるけど 1行80文字に収める為の工夫なのか1文字変数ばっかだしfor文の宣言が改行しまくって8行ぐらいになってて頭おかしくなりそう
199 22/12/10(土)17:35:02 No.1002575723
// 後で書き直す
200 22/12/10(土)17:35:10 No.1002575760
待って！アヒルちゃんレビューって一般的なの！？
201 22/12/10(土)17:35:11 No.1002575771
>何のために…？ってなるクソコメント 行列の掛け算の順序のことをすっかり忘れていてバグらせた俺には必要なコメントだ
202 22/12/10(土)17:35:12 No.1002575775
レビューする側だけど丁寧かつフレンドリーな言葉遣いにして指摘しすぎもあれだから褒めるところは褒めてって考えてやってたらちんちん亭みたいな感じのレビューコメントになってた
203 22/12/10(土)17:35:13 No.1002575782
>git覚えるのって簡単？ はいっ！簡単ですよ
204 22/12/10(土)17:35:23 No.1002575829
>前のコードコメントアウトして残す文化は都市伝説と思ってるけど本当にあったの？ サブバージョンで管理してるけどあったよ
205 22/12/10(土)17:35:35 No.1002575891
>待って！アヒルちゃんレビューって一般的なの！？ ラバーダッキングは一般的なレビュー手法だぞ
206 22/12/10(土)17:35:42 No.1002575919
>git覚えるのって簡単？ 基本的な使い方は簡単 複数人で同じファイル同時に弄って差分がグチャグチャになったときのリカバリー操作とかは勉強しないと難しい
207 22/12/10(土)17:35:49 No.1002575956
ドキュメント作成やレビューなんて片手間にできるだろで工数ないのがウチ
208 22/12/10(土)17:35:56 No.1002575988
>前のコードコメントアウトして残す文化は都市伝説と思ってるけど本当にあったの？ 今もやってるよ
209 22/12/10(土)17:35:57 No.1002575997
うちの会社製造業でDX促進部署みたいなのが新設されて自分修士のときの研究でPython使ってたって理由だけで先週からそこに配置替えになったの でも上長含め所属してる人ら誰もCSの学位ないどころかそこに入るまでプログラムを業務で書いたことなんてないの コードレビューなんて多分誰もやったことないから将来的に多分えらいことになるの
210 22/12/10(土)17:36:03 No.1002576025
>>git覚えるのって簡単？ >はいっ！簡単ですよ わかりました 入門します
211 22/12/10(土)17:36:21 No.1002576127
>待って！アヒルちゃんレビューって一般的なの！？ セルフレビューを言い換えしてるだけだよ
212 22/12/10(土)17:36:24 No.1002576138
>待って！アヒルちゃんレビューって一般的なの！？ 手法としてはよく知られてるよ 身近にやってる人がいるかは別
213 22/12/10(土)17:36:24 No.1002576140
VSCodeはいい… gitにとても優しくしてくれる…
214 22/12/10(土)17:36:25 No.1002576147
>本番とテストで差分がですね 試験環境が本番環境と違う設定で動いていて 試験時は問題なかったけど本番リリース時に問題発生した時はキレた 試験環境の意味ないじゃん！
215 22/12/10(土)17:36:26 No.1002576154
>保守時にコメントまで整合性取られることは稀なので俺はコメントアンチ派です >コメントなくても読めるコードを書け それはコードの表現力と人間の汲み取る力を過信しすぎている なにを思ってこんな構造にしたのかは面倒くさくても残さないと駄目だよ
216 22/12/10(土)17:36:35 No.1002576195
>待って！アヒルちゃんレビューって一般的なの！？ やっぱくまちゃんだよなぁ！？
217 22/12/10(土)17:36:50 No.1002576267
>ラバーダッキングは一般的なレビュー手法だぞ ググったらほんとに出てきてびっくりしてる…
218 22/12/10(土)17:36:59 No.1002576307
>処理を追加修正した日と名前だけ書いたコード良いよね… バージョン管理使えなかった時代の苦肉のお作法だから…… それを今の時代に！？
219 22/12/10(土)17:37:03 No.1002576330
最終的には仕様通りに動くかなんだ綺麗さなんて二の次だ なんで本番で環境が違うんだ終わりだ
220 22/12/10(土)17:37:09 No.1002576366
>コードレビューなんて多分誰もやったことないから将来的に多分えらいことになるの 早く 逃げて
221 22/12/10(土)17:37:15 No.1002576390
>>本番とテストで差分がですね >試験環境が本番環境と違う設定で動いていて >試験時は問題なかったけど本番リリース時に問題発生した時はキレた >試験環境の意味ないじゃん！ 差分はご了承いただけたはずですよね？
222 22/12/10(土)17:37:23 No.1002576438
作者と日付見るだけならgitlensでいいからな…
223 22/12/10(土)17:37:25 No.1002576445
声に出して誰かに説明すると 説明してる途中で アレ…？俺は何を馬鹿なことを言ってるんだ…？ ってなる感覚いいよね
224 22/12/10(土)17:37:35 No.1002576479
git管理してるのにファイル名に日付つけるアホは何なんだろうな
225 22/12/10(土)17:37:37 No.1002576485
ループの括弧とかをコメントで飾りつけするコード規約はやめてくれないかな…ってなる
226 22/12/10(土)17:37:47 No.1002576536
バージョン管理で競合したらマージしないで自分の更新で強制上書きする奴死滅しねーかな～
227 22/12/10(土)17:37:49 No.1002576544
>待って！アヒルちゃんレビューって一般的なの！？ 相談持ち掛けられたけど相手が勝手に自己解決して去っていくことってあるじゃん？
228 22/12/10(土)17:37:54 No.1002576569
俺はsvgを放り込んでコメント代わりにするマン！
229 22/12/10(土)17:38:09 No.1002576641
>ループの括弧とかをコメントで飾りつけするコード規約はやめてくれないかな…ってなる 知らない文化だ…
230 22/12/10(土)17:38:09 No.1002576646
俺のとこは鮭を捕まえる木彫りの熊だよ こいつ見た目と違って賢いんだ
231 22/12/10(土)17:38:22 No.1002576724
>俺はsvgを放り込んでコメント代わりにするマン！ 図形でも描くの！？
232 22/12/10(土)17:38:27 No.1002576739
>うちの会社製造業でDX促進部署みたいなのが新設されて自分修士のときの研究でPython使ってたって理由だけで先週からそこに配置替えになったの >でも上長含め所属してる人ら誰もCSの学位ないどころかそこに入るまでプログラムを業務で書いたことなんてないの >コードレビューなんて多分誰もやったことないから将来的に多分えらいことになるの 適当な保守開発系が手ぐすね引いて待ってるぞ 基本情報だけ取った社員を責任者にするとかザラだし絶対に泣きついてくる
233 22/12/10(土)17:38:40 No.1002576797
正直自分の書いたコードでシステム動いてんのが信じられない
234 22/12/10(土)17:38:49 No.1002576842
>>待って！アヒルちゃんレビューって一般的なの！？ >相談持ち掛けられたけど相手が勝手に自己解決して去っていくことってあるじゃん？ 説明されてるときに一時停止して 解決しましたって去られるのいいよね よくねえよ俺の時間返せ
235 22/12/10(土)17:38:54 No.1002576859
当人は口に出して整理付けてるんだけど周りはブツブツうっせえなにしかならない
236 22/12/10(土)17:38:56 No.1002576877
コメントをサイン代わりにするなって！
237 22/12/10(土)17:39:04 No.1002576910
>サブバージョンで管理してるけどあったよ diffが混ざってえらいことになるんだよね… + // 「」 start - 古いソース + // 古いソース + 新しいソース + // 「」 end
238 22/12/10(土)17:39:34 No.1002577081
>>>git覚えるのって簡単？ >>はいっ！簡単ですよ >わかりました 入門します masterにgit push -fもしねえで使えるようになりてえだと？使えるわけねえだろうが！
239 22/12/10(土)17:40:05 No.1002577238
>当人は口に出して整理付けてるんだけど周りはブツブツうっせえなにしかならない やらない人や開発職じゃない人からはヤベー奴にしか見られないんだよね ちょっとつれえわ…
240 22/12/10(土)17:40:05 No.1002577239
>>うちの会社製造業でDX促進部署みたいなのが新設されて自分修士のときの研究でPython使ってたって理由だけで先週からそこに配置替えになったの >>でも上長含め所属してる人ら誰もCSの学位ないどころかそこに入るまでプログラムを業務で書いたことなんてないの なあに今どきはノーコードツールさえ有れば素人でもお気軽簡単にDXよ
241 22/12/10(土)17:40:13 No.1002577282
「」は{ }と { } のどっち派？前者じゃないと通らない言語つらい
242 22/12/10(土)17:40:36 No.1002577400
>diffが混ざってえらいことになるんだよね… ちゃんとコンフリクトするだけいいぞ #if～#endifはマジでやめろ
243 22/12/10(土)17:40:36 No.1002577404
>コードレビューなんて多分誰もやったことないから将来的に多分えらいことになるの ある程度はツールで大体わかるしなんなら社外サービスに頼めばいいよ そこにお金かけたくないとか言い出したら逃げるしかない
244 22/12/10(土)17:40:38 No.1002577412
共通関数で何してるか知らないし調べる気も無いけど入れる値と出てくる値が想定通りならヨシ！ PGなんてそれで良いんだよ
245 22/12/10(土)17:40:39 No.1002577417
>git管理してるのにファイル名に日付つけるアホは何なんだろうな 焦ってるときフォルダごとバックアップしちゃうときある
246 22/12/10(土)17:40:43 No.1002577443
居るんだよなぁ中途で入社した自分の価値観押し付けてくる発達障害みたいなの 社会は結構なあなあで回ってるんだよ
247 22/12/10(土)17:40:50 No.1002577485
めっっっっちゃ最高の出来栄えだぜ！美しすぎる！コメント残しとこ！ ～翌日～
248 22/12/10(土)17:40:59 No.1002577522
>なあに今どきはノーコードツールさえ有れば素人でもお気軽簡単にDXよ こいつコンサルの回し者だぞ！ 吊るせ！
249 22/12/10(土)17:41:03 No.1002577539
品質保証工程のない製造業と考えるとヤバさがすごい
250 22/12/10(土)17:41:04 No.1002577543
>「」は{ >}と >{ >} >のどっち派？前者じゃないと通らない言語つらい 言語によるなあどっちでもいいやもはや Pythonは嫌い
251 22/12/10(土)17:41:29 No.1002577654
>>ループの括弧とかをコメントで飾りつけするコード規約はやめてくれないかな…ってなる >知らない文化だ… for ( int i = 0; i < MAX_COUNT; i++ ) { ～ } /* for: end */ みたいなのをやるところがたまにある ネストの仕方分かりやすくしたいんだろうがそれはIDEとかエディタの仕事だろうし そもそもそんなにネスト深くするな
252 22/12/10(土)17:41:53 No.1002577756
>>>ループの括弧とかをコメントで飾りつけするコード規約はやめてくれないかな…ってなる >>知らない文化だ… >for ( int i = 0; i < MAX_COUNT; i++ ) >{ > ～ >} /* for: end */ >みたいなのをやるところがたまにある >ネストの仕方分かりやすくしたいんだろうがそれはIDEとかエディタの仕事だろうし >そもそもそんなにネスト深くするな ひどすぎる…
253 22/12/10(土)17:41:53 No.1002577757
yyyymmdd チケット番号 みたいなコメント付きのソースコードはいくらでもある
254 22/12/10(土)17:42:04 No.1002577798
>言語によるなあどっちでもいいやもはや というか手でやらん
255 22/12/10(土)17:42:09 No.1002577822
インデントを理由にPythonを嫌う人多いけどインデントの何が気に入らないんだ
256 22/12/10(土)17:42:17 No.1002577875
>そもそもそんなにネスト深くするな プログラム設計がイケてない可能性が高いよね
257 22/12/10(土)17:42:21 No.1002577900
ノーコードだろうがド素人が組めるわけないだろ！
258 22/12/10(土)17:42:25 No.1002577918
>yyyymmdd チケット番号 >みたいなコメント付きのソースコードはいくらでもある あーbranch名は日付かチケット番号無いと辛いかも
259 22/12/10(土)17:42:25 No.1002577922
>居るんだよなぁ中途で入社した自分の価値観押し付けてくる発達障害みたいなの コード規約が前と違うから嫌だ！前のやり方で自分はやります！ってのはいた
260 22/12/10(土)17:42:29 No.1002577938
異常な値を出力かエラーで落ちてから修正すれば良いや理論 フレンドリーな取引先なら割と許してくれる
261 22/12/10(土)17:42:41 No.1002578000
>インデントを理由にPythonを嫌う人多いけどインデントの何が気に入らないんだ 慣れてないから嫌い
262 22/12/10(土)17:43:02 No.1002578095
>>居るんだよなぁ中途で入社した自分の価値観押し付けてくる発達障害みたいなの >コード規約が前と違うから嫌だ！前のやり方で自分はやります！ってのはいた これ仕事っていうか業務指示放棄じゃない？
263 22/12/10(土)17:43:30 No.1002578249
>品質保証工程のない製造業と考えるとヤバさがすごい まあしくじっても人命には基本関わらないから…
264 22/12/10(土)17:43:37 No.1002578280
発達障害で思い出したけどレビュー基準が気分次第なのいいよね
265 22/12/10(土)17:44:17 No.1002578512
アヒルちゃんって何？
266 22/12/10(土)17:44:33 No.1002578587
おじいちゃん無駄に変数のスコープ広げるのやめてくれ
267 22/12/10(土)17:44:38 No.1002578611
>アヒルちゃんって何？ スレ画が見えんか
268 22/12/10(土)17:44:45 No.1002578647
>>>ループの括弧とかをコメントで飾りつけするコード規約はやめてくれないかな…ってなる >>知らない文化だ… >for ( int i = 0; i < MAX_COUNT; i++ ) >{ > ～ >} /* for: end */ >みたいなのをやるところがたまにある >ネストの仕方分かりやすくしたいんだろうがそれはIDEとかエディタの仕事だろうし >そもそもそんなにネスト深くするな 多分VBで育ったのかな…
269 22/12/10(土)17:44:53 No.1002578695
納期に間に合わなくてリリース後にコードレビューするようになった
270 22/12/10(土)17:45:03 No.1002578751
赤べこの方がいいよね
271 22/12/10(土)17:45:04 No.1002578768
むしろ中途で入ったから文化を教えてくれ…ってなってる お前が思うようにお前がいいと思うコードを書けとは俺を信頼しすぎじゃないか…
272 22/12/10(土)17:45:04 No.1002578770
Pythonは{}とかで閉じなくていいの不安になってくる
273 22/12/10(土)17:45:16 No.1002578826
>for ( int i = 0; i < MAX_COUNT; i++ ) >{ > ～ >} /* for: end */ >みたいなのをやるところがたまにある 波ダッシュまたは全角チルダ使うプログラマーか…
274 22/12/10(土)17:45:17 No.1002578828
>おじいちゃん無駄に変数のスコープ広げるのやめてくれ いいよね自由きままなグローバル変数
275 22/12/10(土)17:45:39 No.1002578944
転職したいよおお！
276 22/12/10(土)17:45:52 No.1002579023
>赤べこの方がいいよね 首振ってくれるからね
277 22/12/10(土)17:45:56 No.1002579044
印刷して一つ一つロジック説明とか未だにやってるところある？
278 22/12/10(土)17:46:01 No.1002579081
>発達障害で思い出したけどレビュー基準が気分次第なのいいよね テストデータ通してもちゃんと合ってるし特に間違いは無いけどこの辺のロジックが気に食わないとか言われた時はキレそうだった
279 22/12/10(土)17:46:04 No.1002579097
ルールがあるからそれに乗っとるんだぞ ルールが無い無法地帯とクソルールの地獄はまあ何とかするか逃げろ
280 22/12/10(土)17:46:09 No.1002579134
俺はデスクに置いてある南ことりちゃんに相談してる
281 22/12/10(土)17:46:30 No.1002579257
レビューに美しくないが出てきた時は俺10秒ぐらい思考が停止したよ
282 22/12/10(土)17:47:01 No.1002579414
むかしむかしjavaコードレビューしてもらって レビュアーにそもそもjavaのインタプリタって信用できなくない？とか言われて 俺に…どうしろというのだ…ってなった いやそんなん言われても俺のコードで何とかできる範囲じゃなくない？
283 22/12/10(土)17:47:07 No.1002579443
一旦声に出して説明するってのは実際大事 でもアヒルちゃんか喋りだしたらメンタルへ！
284 22/12/10(土)17:47:12 No.1002579467
コードスタイルはIDEの自動補完に全て任せる こんなところ社内で決めてネチネチ指摘するのもほぼ意味ないよ
285 22/12/10(土)17:47:16 No.1002579484
ロジックと設計微妙じゃない？はコードレビューで必要な範囲だろう…
286 22/12/10(土)17:47:32 No.1002579582
趣味の範囲になるコメントばっかりしてくる人いるよね…
287 22/12/10(土)17:47:35 No.1002579602
>レビューに美しくないが出てきた時は俺10秒ぐらい思考が停止したよ いやこの感覚は一番大事だぞ
288 22/12/10(土)17:47:43 No.1002579646
厳しい所はスペースじゃなくてできるだけTab使えとか言われる…
289 22/12/10(土)17:47:50 No.1002579680
>>発達障害で思い出したけどレビュー基準が気分次第なのいいよね >テストデータ通してもちゃんと合ってるし特に間違いは無いけどこの辺のロジックが気に食わないとか言われた時はキレそうだった 可読性とか効率が悪いとかではなく？
290 22/12/10(土)17:48:17 No.1002579824
美しくないって言葉は悪いけど 保守性が悪いとか単純に読みづらいとかなら普通に指摘の範囲内だけど
291 22/12/10(土)17:48:18 No.1002579831
>ロジックと設計微妙じゃない？はコードレビューで必要な範囲だろう… パフォーマンスやメンテナンス性に影響出そうなら指摘するな
292 22/12/10(土)17:48:19 No.1002579837
スタイルについては指摘するべきだけど直さなくていいってのがうちのルールだ
293 22/12/10(土)17:48:19 No.1002579839
>厳しい所はスペースじゃなくてできるだけTab使えとか言われる… IDEのフォーマッタかけて終わりじゃねえの！？
294 22/12/10(土)17:48:53 No.1002580024
>厳しい所はスペースじゃなくてできるだけTab使えとか言われる… いやむしろインデント統一するのって第一にやることだろ！？ タブにせよスペースにせよできるだけも何もなくない？
295 22/12/10(土)17:48:56 No.1002580037
だったら最初からそう指摘しろや！！！
296 22/12/10(土)17:49:32 No.1002580255
>可読性とか効率が悪いとかではなく？ Goto文使った俺も悪いんだけど大した処理じゃないからヨシって感じで出した
297 22/12/10(土)17:49:58 No.1002580375
動いてるなら良しで終わりならコードレビューいらないよ
298 22/12/10(土)17:50:10 No.1002580452
引っ掛かったのはたぶんそういう理由なんだろうなってわからんでもないけど抽象的な個人の趣味みたいな文じゃなくて理由言えや！！ どう直せってんだよ！！！
299 22/12/10(土)17:50:37 No.1002580601
>Goto文使った俺も悪いんだけど大した処理じゃないからヨシって感じで出した Goto文は何言われても仕方ないだろ！
300 22/12/10(土)17:50:38 No.1002580608
単体テストをコードで書いたことないんだよな…
301 22/12/10(土)17:50:46 No.1002580645
たぶんどう美しくないか聞くべきで コミュニケーションが大事よねって思う
302 22/12/10(土)17:50:51 No.1002580676
必然性のないgotoは美しくないとか以前に汚れたコードだろ
303 22/12/10(土)17:50:53 No.1002580688
gotoはまあうん…
304 22/12/10(土)17:51:00 No.1002580721
前にあった現場がメモ帳で直接書く化石みてえなところだったからTabとスペース混在すごかった
305 22/12/10(土)17:51:08 No.1002580773
その場で聞けばいいだろ!
306 22/12/10(土)17:51:14 No.1002580801
レビュアーとして美しくないって思ったら何が美しくないのか考えるな 思いつかなかったら忘れて！って言う
307 22/12/10(土)17:51:20 No.1002580840
認証難しすぎる 俺の頭の良さを超えている フレームワークに丸投げさせてくれ
308 22/12/10(土)17:51:39 No.1002580952
>たぶんどう美しくないか聞くべきで >コミュニケーションが大事よねって思う プログラマやSEってサービス業なのをみんな知らないよな
309 22/12/10(土)17:52:01 No.1002581052
VBAの話か？
310 22/12/10(土)17:52:02 No.1002581054
レビューは感想文投げるとこじゃないと思うんだけどそれどうなの！？
311 22/12/10(土)17:52:24 No.1002581191
コーディング規約にgotoはダメって書いてないの？
312 22/12/10(土)17:52:30 No.1002581218
うちのはAA入ってることあるけどよくあるもんなの？ 外したら動かなくなるからみんなほっといてるけど
313 22/12/10(土)17:52:33 No.1002581237
>引っ掛かったのはたぶんそういう理由なんだろうなってわからんでもないけど抽象的な個人の趣味みたいな文じゃなくて理由言えや！！ >どう直せってんだよ！！！ それ相手に言ってみたら？いや別に皮肉で言ってんじゃなくて 指摘スタイルのレビューも大事なんじゃないかなあと
314 22/12/10(土)17:52:51 No.1002581341
>前にあった現場がメモ帳で直接書く化石みてえなところだったからTabとスペース混在すごかった そういうコードVSCodeで開くとレインボーアドオンのおかげですごい気持ち悪くなれる
315 22/12/10(土)17:52:52 No.1002581347
>ここでこんな質問するあたり今の私は冷静さを欠いています これとか >どうなっても知らんぞって話はちゃんと上司にしたぞ これでだいぶまともな「」に見えるな
316 22/12/10(土)17:52:56 No.1002581373
>レビューは感想文投げるとこじゃないと思うんだけどそれどうなの！？ 指摘するところがなくて最後にイチャモンつけるのがそういうところ 会社の文化によってはレビューで指摘ナシが許されないところもある…あるのだ…
317 22/12/10(土)17:52:57 No.1002581378
Goto文は一つの処理に一個ぐらいならセーフって割と思ってるんだけどなぁ… SQLでエラー発生したらGotoで即飛ぶようにしてるよ俺は
318 22/12/10(土)17:53:01 No.1002581398
>レビューは感想文投げるとこじゃないと思うんだけどそれどうなの！？ 一人で書いて終わるものじゃないので 設計思想は共有すべきだと思うの
319 22/12/10(土)17:53:11 No.1002581450
どう美しくないのか後学のために教えてくださいって言えばいいよ
320 22/12/10(土)17:53:35 No.1002581574
>うちのはAA入ってることあるけどよくあるもんなの？ >外したら動かなくなるからみんなほっといてるけど だいぶおかしいところだぞそれ
321 22/12/10(土)17:54:07 No.1002581728
GOSUB使いまくるマン
322 22/12/10(土)17:54:08 No.1002581735
理屈っぽい人多いから大抵理由持ってて聞いたら俺はこう書いた方がいいと思うけどな～って言ってくれる そうじゃないならキレていい
323 22/12/10(土)17:54:18 No.1002581787
>うちのはAA入ってることあるけどよくあるもんなの？ >外したら動かなくなるからみんなほっといてるけど 変なプリプロセッサ使ってらっしゃる？
324 22/12/10(土)17:54:24 No.1002581824
でもCでネストしたループから脱出しつつ最後に特定の処理はしたいみたいな場合はgoto使いたいしなぁ…
325 22/12/10(土)17:54:31 No.1002581865
>どう美しくないのか後学のために教えてくださいって言えばいいよ インデントとか…
326 22/12/10(土)17:54:52 No.1002581967
異様すぎてAAって何の略かなと考え込んでしまった
327 22/12/10(土)17:54:55 No.1002581984
コードレビューも形だけになってぽろぽろコード規約違反を通すようになる場合があるからなんとも言い難い
328 22/12/10(土)17:55:17 No.1002582087
はい…自社のIDEは互換性のせいでわざわざ最新ライセンス取得してから10年前のVer使ってるので自動インデントがすべからくゴミです…
329 22/12/10(土)17:55:28 No.1002582145
全体に対して30%はコメントであることが望ましいって事になってるけど いままで15年仕事しててそんな細かくコメント入れてるソース見たことない
330 22/12/10(土)17:55:35 No.1002582178
>うちのはAA入ってることあるけどよくあるもんなの？ >外したら動かなくなるからみんなほっといてるけど AA…？アスキーアート…？
331 22/12/10(土)17:55:37 No.1002582182
>レビューは感想文投げるとこじゃないと思うんだけどそれどうなの！？ 読みやすい読みにくいみたいな個人の感想は保守性に関わるから感想文投げてもいいよ でも本当に対応すべきなのか次から改善すればいいのかはちゃんと費用対効果考えて指示しようね
332 22/12/10(土)17:55:45 No.1002582213
>>どう美しくないのか後学のために教えてくださいって言えばいいよ >インデントとか… 変数名とか
333 22/12/10(土)17:55:53 No.1002582268
空白を開ける開けないも含めて個人の好みがレビューで幅を利かせて誰がレビューをするかでコロコロ変わることもある
334 22/12/10(土)17:55:58 No.1002582298
コメント絶対にほしいというわけでもないけどあったら楽だなって部分は大体ないし何でそこにあんのって部分は大体ある
335 22/12/10(土)17:56:22 No.1002582400
>AA…？アスキーアート…？ そうｺﾞﾗｧのアスキーアート 全部コメントで入ってる
336 22/12/10(土)17:56:23 No.1002582405
コードレビューって最鑑みたいなもの？
337 22/12/10(土)17:56:33 No.1002582457
>どう美しくないのか後学のために教えてくださいって言えばいいよ 処理長すぎとかネスト深すぎとか多重ループとか名前おかしいとかは美しくないって思うようにしてるな
338 22/12/10(土)17:56:38 No.1002582481
え！？！？！コメントに記号を！？
339 22/12/10(土)17:56:39 No.1002582489
コメントにAAは初めて聞いたし今後も聴きたくねえよ！
340 22/12/10(土)17:56:56 No.1002582569
いいコード悪いコードで学ぶって本あったけど あれも真面目にやるとがんじがらめすぎて無理そう
341 22/12/10(土)17:56:59 No.1002582585
AA無いと困るとか呪文で画像生成するAIみたいな奴だな
342 22/12/10(土)17:57:14 No.1002582656
コメントにAA入っててしかも消したら動かなくなるとかイかれてるにも程がある
343 22/12/10(土)17:57:22 No.1002582695
冗談半分で言ってたここのコメント消すと止まるがホントな奴初めて見た
344 22/12/10(土)17:57:37 No.1002582760
>>AA…？アスキーアート…？ >そうｺﾞﾗｧのアスキーアート >全部コメントで入ってる やっぱり壺はダメだな…
345 22/12/10(土)17:57:48 No.1002582811
流石に作り話だろ…
346 22/12/10(土)17:57:53 No.1002582836
C/C++は言語的な弱点を補う手法としてgotoは仕方ないところある それ以外はダメだ！
347 22/12/10(土)17:57:54 No.1002582844
>いいコード悪いコードで学ぶって本あったけど >あれも真面目にやるとがんじがらめすぎて無理そう あれQiitaとかZennでこれいいコードとして紹介されてるのも悪いコードじゃね？って指摘多すぎて笑っちゃった
348 22/12/10(土)17:58:03 No.1002582883
本年度のクソコード大賞
349 22/12/10(土)17:58:38 No.1002583059
>ここのコメント消すと止まる だいたいSJISでダメ文字が入ってるやつ
350 22/12/10(土)17:58:48 No.1002583120
赤字だから白状するけど機能追加でどうにもできなくてサンプルからわかる範囲でいじって何とかなれー！ってレビューに投げたら素通りして大変なことになりました
351 22/12/10(土)17:58:51 No.1002583131
新入社員になんか聞かれたらまず最初にいいからリーダブルコード読めって言ってるけど読んでくれた子は一人もいない かなしい
352 22/12/10(土)17:59:06 No.1002583215
SJISでダメ文字がたまたま末尾に来てて…ってのはまれに見る 最近UTF-8が多くなって見かけなくなったが
353 22/12/10(土)17:59:20 No.1002583269
>異様すぎてAAって何の略かなと考え込んでしまった 知らない単語出てきて俺も勉強不足だな…って思ってた
354 22/12/10(土)17:59:23 No.1002583279
最終的にプログラミングは宗教と考えないと人や現場によって暗黙のルールがコロコロ変わることに耐えられなくなる だからプログラミングは宗教
355 22/12/10(土)17:59:31 No.1002583313
>だいたいSJISでダメ文字が入ってるやつ やっぱりUTF-8教は最高だな！
356 22/12/10(土)17:59:45 No.1002583390
>新入社員になんか聞かれたらまず最初にいいからリーダブルコード読めって言ってるけど読んでくれた子は一人もいない (めんどくせえ先輩だな…)