1風吹けば名無し2022/06/06(月) 12:13:37ID:dBfYE3Vh00606.net
上司「おい！聞いてんの？」
ワイ「あっ…あっ…」
上司「もういいわ！もっとコード綺麗にしろ！見にくい」
2風吹けば名無し2022/06/06(月) 12:13:59ID:dBfYE3Vh00606.net
コミュ障でもいけるんやなかったのか…
3風吹けば名無し2022/06/06(月) 12:14:12ID:M/oEaRw9d0606.net
根拠述べたら述べたで上司の二言目になるしそれでええんちゃう
4風吹けば名無し2022/06/06(月) 12:14:18ID:Z75L4FvJ00606.net
せやな
5風吹けば名無し2022/06/06(月) 12:14:19ID:pRmvLIG400606.net
なんかこうやったら動いたんで…
6風吹けば名無し2022/06/06(月) 12:14:47ID:kan53Xm1d0606.net
ワイ「理由はわからないけど動くんすよ。消すと止まります」で通せ
7風吹けば名無し2022/06/06(月) 12:14:54ID:XPhidSEPM0606.net
ワイ「コピペしたんでわかりません！」
8風吹けば名無し2022/06/06(月) 12:14:57ID:dBfYE3Vh00606.net
なんで一行一行レビューされないといけないねん…
11風吹けば名無し2022/06/06(月) 12:15:35ID:pRmvLIG400606.net
そのためのコメントなんやけどな
12風吹けば名無し2022/06/06(月) 12:15:42ID:hp3dvEPPa0606.net
ワイ「ここってどういう意味のコード？」
PG「そこは他の場所をコピーして書いたので意味はわかりません」
26風吹けば名無し2022/06/06(月) 12:18:11ID:pRmvLIG400606.net
>>12
しゃーない
締め切りもあるしな
69風吹けば名無し2022/06/06(月) 12:33:16ID:Qc9yff91d0606.net
>>12
下請け「ここどこ参考にしてますか？」
プロパー「ここコピレば良いよ」
まずこの前身がある
13風吹けば名無し2022/06/06(月) 12:15:48ID:x8BimX6+a0606.net
コメントつけてないんで忘れました！！
14風吹けば名無し2022/06/06(月) 12:15:50ID:9Pv7a5SAM0606.net
こう書けば10ミリ秒早いんですよって適当に言えばいい
15風吹けば名無し2022/06/06(月) 12:15:51ID:oZr6UuE2d0606.net
まあなにやってんのかよくわからんコードはメンテが大変やからせめてコメントくらいは入れてほしくはある
22風吹けば名無し2022/06/06(月) 12:17:41ID:hp3dvEPPa0606.net
>>15
// nullチェックをする
こういうクソコメやめろ
nullチェックしてるのは見ればわかんねん
どういう場合にnullになる可能性があってなぜnullチェックせなあかんのか書けよと
28風吹けば名無し2022/06/06(月) 12:19:25ID:S6REyKX/00606.net
>>22
そのレベルのコメントが各行に書くやつがいて笑ったわ
103風吹けば名無し2022/06/06(月) 12:46:03ID:Cr5HqbjZr0606.net
>>22
仕様書ないの？
105風吹けば名無し2022/06/06(月) 12:47:31ID:5yCvQp8f00606.net
>>103
仕様書とコードの内容が合っていると良いな…
17風吹けば名無し2022/06/06(月) 12:16:16ID:1eY57JYy00606.net
ワイはレビューとかでめちゃくちゃ喋らせられるで
23風吹けば名無し2022/06/06(月) 12:17:45ID:dBfYE3Vh00606.net
>>17
本当それ嫌いやわ
粗探しされてるみたいで怖い
37風吹けば名無し2022/06/06(月) 12:23:51ID:JoI9pBMUd0606.net
>>23
実際粗探ししとるんやで
小姑が人差し指でほこりツツツってやるようなもんやし
コーディングが終わった後に無闇矢鱈に直す方がおかしいから、レビュー時は粗探しになるのもしゃーない
そもそもコミュ障にできるとか思ってる方が間違いやしな
20風吹けば名無し2022/06/06(月) 12:17:21ID:g7svqEZ5d0606.net
リンターでも導入すれば
それが意味ないほど訳わからんコードしか書けへんのなら知らん
21風吹けば名無し2022/06/06(月) 12:17:39ID:/TVi40RN00606.net
そらゲームのバグとかほったらかしになるよね
27風吹けば名無し2022/06/06(月) 12:18:38ID:88FYBo/Y00606.net
アホで草
29風吹けば名無し2022/06/06(月) 12:19:29ID:EF9VUpAU00606.net
下手に直さないのは正しいけどな
32風吹けば名無し2022/06/06(月) 12:22:39ID:fr5ro31fa0606.net
if(true == true)
48風吹けば名無し2022/06/06(月) 12:26:59ID:EoMJC9Fga0606.net
>>32
//とりあえずルート固定
if(true)
ってコード見たときは戦慄した
もちろんelseにも処理は書いてある
50風吹けば名無し2022/06/06(月) 12:27:52ID:VF8NgNkCM0606.net
>>48
意味なさすぎてワロタ
33風吹けば名無し2022/06/06(月) 12:22:57ID:WOjQv91U00606.net
ワイ有能「こことりあえず動いてるんですけどよくわからないんですよね 誰か教えてください」
35風吹けば名無し2022/06/06(月) 12:23:44ID:LmJGmJaua0606.net
>>33
みんな「・・・・・」
39風吹けば名無し2022/06/06(月) 12:24:30ID:WOjQv91U00606.net
>>35
じゃあとりあえず残しときますね～で余裕や みんなわからんもんはしゃーない
36風吹けば名無し2022/06/06(月) 12:23:49ID:B2Ro17GL00606.net
実際分からなかったら分からないって言って納得してくれるんか
45風吹けば名無し2022/06/06(月) 12:26:06ID:WOjQv91U00606.net
>>36
わからんなら教えたらええだけやし
38風吹けば名無し2022/06/06(月) 12:24:30ID:F2RWcw29d0606.net
普通に無能やん
足りないのはコミュ力ではなく説明能力
43風吹けば名無し2022/06/06(月) 12:25:47ID:XDguWCfl00606.net
>>38
それをコミュ力言うねんアホ
54風吹けば名無し2022/06/06(月) 12:28:43ID:5edYn+3Vd0606.net
理由説明できないのはコミュ障とは違うやろ？
41風吹けば名無し2022/06/06(月) 12:25:14ID:WOjQv91U00606.net
わからんくせにわかってるふりするのが一番悪手やろ なんで理系のやつってみんなわからんって言えへんのや
76風吹けば名無し2022/06/06(月) 12:35:15ID:LmJGmJaua0606.net
>>41
わからんって言うとマウント取ってくる奴が多いからな
なんというか精神的に学生のままなんよな
42風吹けば名無し2022/06/06(月) 12:25:21ID:XUwQU+nFp0606.net
よくわからんけどこれコメントアウトするとエラー吐くんすよｗ
46風吹けば名無し2022/06/06(月) 12:26:39ID:iiHbsVfm00606.net
ぼく「消したらかわいそうなのでのこしました」
47風吹けば名無し2022/06/06(月) 12:26:53ID:5QMMmPaKd0606.net
よくわからないですがこれ消すと動かなくなります
51風吹けば名無し2022/06/06(月) 12:27:56ID:lI0bJJaRM0606.net
動けば同じなのにな
理系の奴等は形にばかりこだわって本質を理解してない
53風吹けば名無し2022/06/06(月) 12:28:21ID:hp3dvEPPa0606.net
>>51
なお動かない模様
60風吹けば名無し2022/06/06(月) 12:30:06ID:nPNYcRTG00606.net
//なぜか消すと動かない
ってしとくよね
64風吹けば名無し2022/06/06(月) 12:30:54ID:pRmvLIG400606.net
>>60
でも優秀なプログラマでも書いてる時あるよなこれ
65風吹けば名無し2022/06/06(月) 12:32:37ID:hp3dvEPPa0606.net
>>64
本当に深淵にまで辿り着くと言語やOS自体のバグって結構多いんよな
85風吹けば名無し2022/06/06(月) 12:37:14ID:RdVJR+qk00606.net
>>65
oracle提供のライブラリがバグっててメモリオーバーフローしたことあったわw
70風吹けば名無し2022/06/06(月) 12:33:45ID:EGh10Xdhd0606.net
前の現場ではこうやることになってました
これが最強カードや
72風吹けば名無し2022/06/06(月) 12:34:18ID:5Nj6hg18r0606.net
>>70
たまにそれ言う派遣さんいて困るわ
81風吹けば名無し2022/06/06(月) 12:36:49ID:xf+gAGsQp0606.net
コメントは事実じゃなく理由を書けってのは新人の時に教わるよな
あとコードはコメントから書けってのも教わる
92風吹けば名無し2022/06/06(月) 12:39:35ID:hp3dvEPPa0606.net
>>81
＞コードはコメントから書け
めっちゃええ文化やな
先輩優秀なんやろな
108風吹けば名無し2022/06/06(月) 12:51:53ID:SBXi83eDM0606.net
>>81
コメントに理由を書いて、コメントからコードを書けば○○が必要だからこう書くって徹底できるな
96風吹けば名無し2022/06/06(月) 12:41:00ID:RdVJR+qk00606.net
変数名とメソッド名を適切に付けられないやつが多過ぎる
97風吹けば名無し2022/06/06(月) 12:42:05ID:hp3dvEPPa0606.net
>>96
まあその辺はリーダブルコード読めとしか
100風吹けば名無し2022/06/06(月) 12:43:14ID:ZrMN8gNfM0606.net
>>96
それは
クライアントにもよる
98風吹けば名無し2022/06/06(月) 12:42:34ID:Cr5HqbjZr0606.net
コードとしては通るのかもしれんが業務処理としてそれがなんの意味を持つのか知って書いてないとレビュアーが責任持つことになるんやぞ
責任皆無なら何も言わんしそもそもレビューなんか必要ない
86風吹けば名無し2022/06/06(月) 12:37:21ID:KntpCxtGM0606.net
ワイは段落ごとに解説入れとるで
引用元上司「このコードなんの意味あるの？」ワイプログラマー「…」
コメント