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

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

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


もうやだこの仕事

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

マクロ脳

128 ：風吹けば名無し：2018/07/08(日) 10:28:55.38 ID:6shcjLBfD.net

>>119
0始まりのほうが使いやすいやん

129 ：風吹けば名無し：2018/07/08(日) 10:28:59.44 ID:pHxqqzmwd.net

数百年数千年単位で変わらないですって断言すればええやん

130 ：風吹けば名無し：2018/07/08(日) 10:29:01.80 ID:+ywvE4350.net

>>111
++i すき
i++ きらい

131 ：風吹けば名無し：2018/07/08(日) 10:29:02.08 ID:lAbWK6+D0.net

>>34
仕事出来無さそうやね…

132 ：風吹けば名無し：2018/07/08(日) 10:29:03.05 ID:fvNA8vDV0.net

この変数名なんて読むんや・・・英和辞書見よ

133 ：風吹けば名無し：2018/07/08(日) 10:29:05.60 ID:TVhJafVP0.net

#define ZERO 0

134 ：風吹けば名無し：2018/07/08(日) 10:29:05.99 ID:V03DIjU40.net

>>124
monthなんてわざわざ共通化させるか？

135 ：風吹けば名無し：2018/07/08(日) 10:29:07.96 ID:cMx6Bu0M0.net

1年が12ヶ月から変わる可能性は新しい暦法の制定やろうけど
13ヶ月に増えたりするのは地球軌道が変わるレベルやからそのときは生きておらんやろ
だいたい外部定義ファイルでコンパイルなしに変更できるのでなけりゃシンボル定義する意味ねーわ

136 ：風吹けば名無し：2018/07/08(日) 10:29:10.51 ID:km8QYPJb0.net

>>88
コメントも1行に含んで金貰えてた時代が懐かしいわ

137 ：風吹けば名無し：2018/07/08(日) 10:29:10.88 ID:OBVgh4NP0.net

>>124
無能PGやぞ
指摘するだけ無駄や

138 ：風吹けば名無し：2018/07/08(日) 10:29:12.37 ID:ByhnLbdlp.net

>>107
その判断基準統一するための基本設計やろ
個人の判断の余地がある時点で設計としては不十分

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

本人しかわからんクソコード書いてんだから残念でもないし当然

140 ：風吹けば名無し：2018/07/08(日) 10:29:23.00 ID:oQ36AWTN0.net

>>119
普通は0基準でカウントしていくもんちゃうの
無駄な文字増えるだけやん

141 ：風吹けば名無し：2018/07/08(日) 10:29:37.95 ID:tpT11+xlr.net

ワイ独学てCやってるけど方程式を解くので詰んでるわ
ガウスの消去法ってなんやねん

142 ：風吹けば名無し：2018/07/08(日) 10:29:42.44 ID:rKdoSrAfp.net

ワイ導入部隊、大して使い回されるものもないのでクソ汚いコーディング

143 ：風吹けば名無し：2018/07/08(日) 10:29:42.49 ID:zilW9cmM0.net

怒りでプルプルタイピングニキ

144 ：風吹けば名無し：2018/07/08(日) 10:29:46.20 ID:q/gz8qOxM.net

実は12よりもiに問題あるけどな
month<=12 みたいに変数に意味がついてれば12の意味も速攻わかる

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

>>97
その考え方が古くない？
googleの開発統計だと製造で一番不具合が少ないのは2名で開発したときって出てる
3名以上でも1名でもダメ

146 ：風吹けば名無し：2018/07/08(日) 10:29:52.63 ID:oQ36AWTN0.net

>>115
3秒考えてるじゃん？

147 ：風吹けば名無し：2018/07/08(日) 10:29:53.03 ID:FrQR5Jbu0.net

SE？

148 ：風吹けば名無し：2018/07/08(日) 10:29:53.80 ID:7eiV2DPJ0.net

>>93
Unix「システム時刻は32ビットあれば足りるやろ…」
なお

149 ：風吹けば名無し：2018/07/08(日) 10:30:07.46 ID:OPAsj2nD0.net

>>141
なんの本に書いてあるんだよ

150 ：風吹けば名無し：2018/07/08(日) 10:30:11.52 ID:TEX2tkQe0.net

コード内にマジックナンバーあるだけで発狂するからやめて

151 ：風吹けば名無し：2018/07/08(日) 10:30:18.13 ID:4XydH8vO0.net

>>10
はい不敬罪