上司「このi<=12って何？」ワイPG「12ヶ月の12です」上司「1年13ヶ月になったらどうすんの？」

1 ：風吹けば名無し：2018/07/08(日) 10:15:30.16 ID:V03DIjU40.net

上司「今後変わらないって言い切れるの？いいの？こんなハードコーディングしちゃって」


もうやだこの仕事

27 ：風吹けば名無し：2018/07/08(日) 10:20:10.46 ID:iIhCbAZq0.net

>>17
これメンス

28 ：風吹けば名無し：2018/07/08(日) 10:20:17.32 ID:P6CP4gEi0.net

第三者が一目でわからなかったってことだから
コメントか何らかの修正が必要

29 ：風吹けば名無し：2018/07/08(日) 10:20:19.95 ID:v6JXUm1nr.net

定数は全ソースコードで共有せな

30 ：風吹けば名無し：2018/07/08(日) 10:20:30.23 ID:V03DIjU40.net

定数ガイジが湧いてきたな

31 ：風吹けば名無し：2018/07/08(日) 10:20:33.74 ID:iFDcBWi4r.net

>>10
平成30年は12ヶ月あるだろ

32 ：風吹けば名無し：2018/07/08(日) 10:20:38.17 ID:FC2r/J3H0.net

「空間が 4 次元になったらどうすんの？」

33 ：風吹けば名無し：2018/07/08(日) 10:20:48.38 ID:3N6PspWj0.net

>>26
これはある

34 ：風吹けば名無し：2018/07/08(日) 10:21:04.55 ID:PnJtgaeXd.net

>>17
仕事するとよく聞くけど、3秒で分かることも考えたく無いなら肉体労働者になればいいのになって思う

35 ：風吹けば名無し：2018/07/08(日) 10:21:11.99 ID:wKg8hfW20.net

普通定数は共通変数化するよね

36 ：風吹けば名無し：2018/07/08(日) 10:21:21.34 ID:Rpk3UJ4Y0.net

平成31年は4ヶ月で
新元号1年は8ヶ月やな

37 ：風吹けば名無し：2018/07/08(日) 10:21:25.50 ID:hV/bD78Wp.net

ポイントガードならちゃんとせーや

38 ：風吹けば名無し：2018/07/08(日) 10:21:31.53 ID:849vQcq30.net

>>17
これ

39 ：風吹けば名無し：2018/07/08(日) 10:21:31.89 ID:nWDI9MmrK.net

その頃にはこのプログラム動いてませんよ

40 ：風吹けば名無し：2018/07/08(日) 10:21:32.77 ID:V03DIjU40.net

動きゃいいだろ👊

41 ：風吹けば名無し：2018/07/08(日) 10:21:36.70 ID:221x+YIrM.net

ワイずっと自営でコード書いてきたからこういう事言われんけど、他人から見たら意味わからん書き方なんやろなぁ

42 ：風吹けば名無し：2018/07/08(日) 10:21:39.42 ID:+tpP8PXOM.net

そんな会社辞めなよ

43 ：風吹けば名無し：2018/07/08(日) 10:21:52.32 ID:Z12nqy+O0.net

イッチが上司の指摘した意味を理解できずに
的外れなとこで騒いでる低脳ってバレちゃったね

44 ：風吹けば名無し：2018/07/08(日) 10:22:00.98 ID:rKdoSrAfp.net

別にお前の言ってることも正しいけどベタ書きしないクセつけとかないといかんでしょ

45 ：風吹けば名無し：2018/07/08(日) 10:22:02.80 ID:OPAsj2nD0.net

>>41
自営って何？

46 ：風吹けば名無し：2018/07/08(日) 10:22:04.34 ID:Om/Q9WGxa.net

イッチがプログラマーに向いてないのは分かった

47 ：風吹けば名無し：2018/07/08(日) 10:22:15.54 ID:t56TUHjk0.net

>>34
もっと効率良くなる方法があるのに目先の手間を惜しむのはバカのやることやぞ

48 ：風吹けば名無し：2018/07/08(日) 10:22:36.44 ID:spd2p153M.net

13に帰るだけやろ

49 ：風吹けば名無し：2018/07/08(日) 10:22:43.38 ID:4P+nbFdnr.net

どの会社いってもmonthで書けって言われるで
1年が13ヶ月になったらどうするの？とか言うのはガイジやけど

50 ：風吹けば名無し：2018/07/08(日) 10:22:44.11 ID:+ywvE4350.net

さすがに12の意味が分からないなんてことはないがハードコードは絶対にやめろ

51 ：風吹けば名無し：2018/07/08(日) 10:22:55.58 ID:FC2r/J3H0.net

>>26
言うほど 12 書いとるとこあるか？
多少ハッシュが余分に引っかかるくらいやろ

52 ：風吹けば名無し：2018/07/08(日) 10:22:56.53 ID:OsH7+38q0.net

ガイジ多くて草
複数箇所で使う可能性のあるマジックナンバーは定数化しろよ
ってか言語によっては月の定数とか標準であるやろ

53 ：風吹けば名無し：2018/07/08(日) 10:22:59.00 ID:oTehxqyy0.net

.iniで設定できるようにするんや

54 ：風吹けば名無し：2018/07/08(日) 10:23:05.15 ID:oQ36AWTN0.net

>>34
そういうことじゃないんだよなあ
考える人が苦にならなくてもそれに時間を取られてる時点で無駄だよね

55 ：風吹けば名無し：2018/07/08(日) 10:23:06.62 ID:yl/UovzrM.net

>>45
個人事業主やろ

56 ：風吹けば名無し：2018/07/08(日) 10:23:08.82 ID:/uUjx/ar0.net

>>10
平成31年だろしくじるなよ

57 ：風吹けば名無し：2018/07/08(日) 10:23:11.25 ID:V03DIjU40.net

エラーメッセージも全部ソースに組み込んでええか？😀

58 ：風吹けば名無し：2018/07/08(日) 10:23:11.54 ID:849vQcq30.net

>>34
MONTHにする手間すら惜しんで12って書くやつこそなんの脳みそも使ってないんだよなぁ

59 ：風吹けば名無し：2018/07/08(日) 10:23:11.65 ID:7eiV2DPJ0.net

イッチの無能が晒されただけやったな

60 ：風吹けば名無し：2018/07/08(日) 10:23:19.77 ID:r/qnLN0h0.net

どっちもガイジ

61 ：風吹けば名無し：2018/07/08(日) 10:23:32.43 ID:75Huqb9C0.net

>>34
あとで見返すと本人が書いたものすら分からなくなることは多々ある
わかりやすいものでもちゃんとやる癖つけないといざというとき死ぬ

62 ：風吹けば名無し：2018/07/08(日) 10:23:37.59 ID:JdhQvTzt0.net

monthにした方が意味が分かりやすいって言うのは分かるけど、イッチの上司が言ってることとはちょっと違くね？
1年が13ヶ月なったらってどういう意味やねん

63 ：風吹けば名無し：2018/07/08(日) 10:23:43.36 ID:r+BoB20/0.net

やっぱmonthが正しいんやな
とんだ恥を晒してしまったわ

64 ：風吹けば名無し：2018/07/08(日) 10:23:44.11 ID:MHn5hruo0.net

太陰暦かな？

65 ：風吹けば名無し：2018/07/08(日) 10:23:48.80 ID:TVhJafVP0.net

ハードコーディング絶対許さないマン

66 ：風吹けば名無し：2018/07/08(日) 10:23:51.11 ID:Z1sY/CPc0.net

>>49
これ
上司の指摘の仕方がガイジなだけであって

67 ：風吹けば名無し：2018/07/08(日) 10:23:58.48 ID:TsOy6qrjd.net

上司も注意の仕方おかしいやろ

68 ：風吹けば名無し：2018/07/08(日) 10:24:00.48 ID:OPAsj2nD0.net

>>55
フリーランスなら他所で仕事するからわけわからんコード書かなくない？

69 ：風吹けば名無し：2018/07/08(日) 10:24:18.02 ID:uHTSvKM70.net

マジックナンバー避けるのは当たり前やろ

70 ：風吹けば名無し：2018/07/08(日) 10:24:20.66 ID:UBRvQVnS0.net

修正でお金取れるしいいじゃん

71 ：風吹けば名無し：2018/07/08(日) 10:24:28.07 ID:ByhnLbdlp.net

>>34
一瞬でも止まるっていうのがポイントやで
ていうか仕事したことないような発言やな

72 ：風吹けば名無し：2018/07/08(日) 10:24:29.73 ID:V03DIjU40.net

12がすぐに分からないって他の惑星の方々？

73 ：風吹けば名無し：2018/07/08(日) 10:24:33.90 ID:TCP+wsXU0.net

まあコメントつけときゃええやろ

74 ：風吹けば名無し：2018/07/08(日) 10:24:34.60 ID:Dj2Xim3E0.net

どうせ見直すような大した仕事じゃないから適当でいいんだよ

75 ：風吹けば名無し：2018/07/08(日) 10:24:44.72 ID:t56TUHjk0.net

>>62
いきなり12って書いてしまうような馬鹿にそんな説明しても理解できないのにな

76 ：風吹けば名無し：2018/07/08(日) 10:24:45.03 ID:rKdoSrAfp.net

>>34
そのみんなの3秒はお前の5秒の考えで消せるやろガイジ

77 ：風吹けば名無し：2018/07/08(日) 10:24:58.07 ID:5m3FSVmC0.net

コーディング規約とかないんか？

78 ：風吹けば名無し：2018/07/08(日) 10:24:58.40 ID:75Huqb9C0.net

ちなみにマジックナンバーは小文字で書くこともあんの？
そっちの方が気になった

79 ：風吹けば名無し：2018/07/08(日) 10:24:59.03 ID:TCP+wsXU0.net

>>72
12には月以外の意味もあるやろ

80 ：風吹けば名無し：2018/07/08(日) 10:25:09.70 ID:0SUwxZC40.net

>>63
というより定数化してメンテしやすくしとくのは基本やぞ

81 ：風吹けば名無し：2018/07/08(日) 10:25:09.76 ID:OsH7+38q0.net

>>70
これ現実にあるから酷いよなぁ

82 ：風吹けば名無し：2018/07/08(日) 10:25:13.12 ID:A2C0Qlrhd.net

ワイ新卒「コメントでmonthって書いとくンゴ」

83 ：風吹けば名無し：2018/07/08(日) 10:25:13.24 ID:TMS3Kbeh0.net

>>34
仕事してないかくっそ独りよがりの作業してそう

84 ：風吹けば名無し：2018/07/08(日) 10:25:14.81 ID:ahEESFgi0.net

アスペにありがちな途中の理論をぶっとばした発言

85 ：風吹けば名無し：2018/07/08(日) 10:25:17.29 ID:jrVH8Fc10.net

>>62
「いつか直さないといけなくなった時に誰が見ても直ぐに分かるようにしとけや」って意味やろ

86 ：風吹けば名無し：2018/07/08(日) 10:25:29.70 ID:m/XpWwhj0.net

これは上司がただしい
12なんて固定の数値コード内に書き込むな

87 ：風吹けば名無し：2018/07/08(日) 10:25:30.67 ID:fQzRDcdw0.net

頑なに西暦を使わないのほんとおかしい

88 ：風吹けば名無し：2018/07/08(日) 10:25:39.85 ID:t56TUHjk0.net

>>73
いちいち12が出てくる行にコメントつける方がmonthって打つより面倒くさいやろ

89 ：風吹けば名無し：2018/07/08(日) 10:25:42.06 ID:PnJtgaeXd.net

>>47
効率良くなる慣習とか見やすさが増すとかならええねん。1を漢字で書いたら伸ばし棒かと思ったとか、そういう次元のアホに会ったことあるわ

90 ：風吹けば名無し：2018/07/08(日) 10:25:50.02 ID:OBVgh4NP0.net

>>17
これ
こういう他人のコードメンテしたことない無能PGが数年後に惨劇を引き起こす

91 ：風吹けば名無し：2018/07/08(日) 10:26:00.38 ID:27yZSRhC0.net

https://i.imgur.com/czYpflY.jpg

92 ：風吹けば名無し：2018/07/08(日) 10:26:03.34 ID:/uQa434j0.net

全部グローバル変数やぞ

93 ：風吹けば名無し：2018/07/08(日) 10:26:07.86 ID:FmX4zzoF0.net

「monthの整数型をintにしとるけど、1年が2147483647ヶ月超えたらどうするんや？」

94 ：風吹けば名無し：2018/07/08(日) 10:26:13.14 ID:TCP+wsXU0.net

>>88
せやね

95 ：風吹けば名無し：2018/07/08(日) 10:26:17.25 ID:Kb2Gwd5Pa.net

イッチ予想外に叩かれて始めてきたから絶許路線に変更してきてて草

96 ：風吹けば名無し：2018/07/08(日) 10:26:22.98 ID:849vQcq30.net

>>72
コードの中に突然12っていう数字があったら「もしかしたら月なんかな？」って思うだけで確信はできん
結局読み解いて行く必要あるからクッソ無駄

97 ：風吹けば名無し：2018/07/08(日) 10:26:27.63 ID:V03DIjU40.net

そもそもワイの解釈ではオブジェクト指向プログラミングにおいて
いちいち人のソースコードにケチをつけるのはナンセンス
コードレビューはCとかの古い文化を引きずっているだけ

98 ：風吹けば名無し：2018/07/08(日) 10:26:28.56 ID:ByhnLbdlp.net

>>62
そういう思考回路が大事って事やろ

99 ：風吹けば名無し：2018/07/08(日) 10:26:30.00 ID:OsH7+38q0.net

>>78
規約次第だけど大文字でスネークが普通じゃない？

100 ：風吹けば名無し：2018/07/08(日) 10:26:30.93 ID:0SUwxZC40.net

>>91
ゆるさん

101 ：風吹けば名無し：2018/07/08(日) 10:26:49.31 ID:v6JXUm1nr.net

>>67
万が一ってこともあるからな
月は12のままだろうけど心構えは正しい
プログラムに限らず過去の人の考えが浅く今苦労する機会なんて度々ある

102 ：風吹けば名無し：2018/07/08(日) 10:26:53.76 ID:Fb88Y/CmM.net

はえー
ワイデザイナーだけど１つ賢くなったわ
サンキューなんJ

103 ：風吹けば名無し：2018/07/08(日) 10:26:57.19 ID:GwXBnJzva.net

自転周期がそんな簡単に変わるなら人類とか生まれてないわ

104 ：風吹けば名無し：2018/07/08(日) 10:26:58.14 ID:rKdoSrAfp.net

ただ今絶賛休日出勤中やけどうちの開発サーバのDB遅すぎてほんまイラつくわ

スキーマ消すだけで何十分かかんねん死ね

105 ：風吹けば名無し：2018/07/08(日) 10:27:01.06 ID:ETnkNV730.net

計算対象期間を12ヶ月から6ヶ月にするとかはあり得るやろうしDIできるようにするべきや

106 ：風吹けば名無し：2018/07/08(日) 10:27:02.37 ID:nMxn7NKqd.net

>>93
草

107 ：風吹けば名無し：2018/07/08(日) 10:27:03.92 ID:1YC3QpKz0.net

分かる分からないの判断基準が最終的に個人の裁定になるし
最終的に「このシステムでは○○って数値は誰でも知ってるだろうから定数化しなくていいか」
ってなったら収拾つかなくなるから何も考えずに全部定数化した方がいい

108 ：風吹けば名無し：2018/07/08(日) 10:27:05.44 ID:M/5e8kQX0.net

>>34
その言い分ならむしろ上司の説教が正しいな

109 ：風吹けば名無し：2018/07/08(日) 10:27:06.86 ID:849vQcq30.net

月は12だけど12は月とは限らん
まぁ上司はガイジだけどこのイッチも十分ガイジ

110 ：風吹けば名無し：2018/07/08(日) 10:27:11.85 ID:7eiV2DPJ0.net

>>97
お前の脳みそはオブジェクト指向以前の問題だぞ

111 ：風吹けば名無し：2018/07/08(日) 10:27:13.35 ID:6shcjLBfD.net

i=0; i<12; i++ すき
i=1; i<=12; i++ きらい

112 ：風吹けば名無し：2018/07/08(日) 10:27:18.86 ID:m/XpWwhj0.net

>>97
草ァ！コピペガイジここに極まれりって感じやねw

113 ：風吹けば名無し：2018/07/08(日) 10:27:18.87 ID:V03DIjU40.net

>>95
してないぞ
今も怒りで指ぷるぷるしながらタイピングしてるわ

114 ：風吹けば名無し：2018/07/08(日) 10:27:22.85 ID:c7puwEsId.net

>>68
保守も全部請け負ってるんやろ(適当)

115 ：風吹けば名無し：2018/07/08(日) 10:27:35.08 ID:+ywvE4350.net

>>71
一瞬でも止まらんやろ

116 ：風吹けば名無し：2018/07/08(日) 10:27:41.03 ID:UBRvQVnS0.net

コーディングなんて会社ごとの謎風習あるからそれに従っとけ

117 ：風吹けば名無し：2018/07/08(日) 10:27:46.46 ID:2sKGDUDcp.net

>>93
ulongで…

118 ：風吹けば名無し：2018/07/08(日) 10:27:46.74 ID:ByhnLbdlp.net

>>72
時間も12*2やん

119 ：風吹けば名無し：2018/07/08(日) 10:27:54.16 ID:r+BoB20/0.net

>>111
はえ〜？
上は混乱したりしないの

120 ：風吹けば名無し：2018/07/08(日) 10:28:04.21 ID:OPAsj2nD0.net

>>111
ワイも嫌い

121 ：風吹けば名無し：2018/07/08(日) 10:28:06.53 ID:FC2r/J3H0.net

有名整数なんて好きにしとけ

小数をハードコーディングするアホはしね

122 ：風吹けば名無し：2018/07/08(日) 10:28:08.18 ID:iIhCbAZq0.net

ONE=1

123 ：風吹けば名無し：2018/07/08(日) 10:28:21.40 ID:vDNsYfmgp.net

ハードコーディング絶対許さないマンけっこうおるけど実際何でもかんでも設定ファイルに出したりされると相当読みづらい
定数切るにしても1ファイルに全部まとめたりする奴ウザすぎる

124 ：風吹けば名無し：2018/07/08(日) 10:28:34.17 ID:wKg8hfW20.net

オブジェクト指向なら尚更共通部分は共通化せなあかんやろ…

125 ：風吹けば名無し：2018/07/08(日) 10:28:43.37 ID:bIG87/48a.net

for ( int nIndex=1; nIndex <= nMonth; nIndex++ )
こう？

126 ：風吹けば名無し：2018/07/08(日) 10:28:44.40 ID:uHTSvKM70.net

>>91
なら納期過ぎても許せや

127 ：風吹けば名無し：2018/07/08(日) 10:28:49.21 ID:wo1mk6p7a.net

マクロ脳