【悲報】プログラマー新人おれ「変数名はaとかbでええやろ」 上司「あのさぁ」

1 ：VIP

上司「わかりにくいからこれ」

おれ「しかし、このほうが簡潔ですしミスも少ないと思います…」

上司「変数名で1文字とかありえないから。まじで気をつけて」

おれ「…はい」


何故なのか
古臭いしきたりかなんかあるのかこの会社

21/04/14 14:00 ID:67UZX8lm0.net

Comments

[1. 自他とも認めるﾊﾟｰﾃｨﾏﾝ]

自分が分かればそれでいいって考えがいかにもアスペ的

[2. 自他とも認めるﾊﾟｰﾃｨﾏﾝ]

許されてもローカル変数までだろ

[31. 自他とも認めるﾊﾟｰﾃｨﾏﾝ]

>>2
使ったとしても、数行のブロック内で、かつ宣言後に1度しか使わない場合に限るようにしているわ。関数やメソッド内で複数箇所で計算なり処理なりをするなら、ちゃんと名前を付ける。というか、そこまで必要とされる変数なら名前があるはず。

逆に名前を思いつかないってことは、その変数は必要ない可能性の方のが高い。設計から見直せって思うわ。

[3. 自他とも認めるﾊﾟｰﾃｨﾏﾝ]

名前を考えるのが一番めんどくさい

[4. 自他とも認めるﾊﾟｰﾃｨﾏﾝ]

Nベーシックだとそもそも一文字しか使えなかったような
というかメモリの節約にもなるし

[43. 自他とも認めるﾊﾟｰﾃｨﾏﾝ]

>>4
今時はコンパイラが頑張ってくれるだろ・・・

[5. 自他とも認めるﾊﾟｰﾃｨﾏﾝ]

IDEが進化してるからこそプログラムを書く量をけちる意味が分からないわ補完できるし
なんか最近の言語も新バージョン出るたびに、僕が考えた短く書けるかっこいい言語仕様！みたいになってるけどあれ読みづらくなってるだけなんじゃないの

[28. 自他とも認めるﾊﾟｰﾃｨﾏﾝ]

>>5
そもそもIDEに頼らないと読めないコードを書くこと自体がアウト

IDEが使えないライブラリやサービスのドキュメント内で、aとかbとか書かれていたらどう思うか

[39. 自他とも認めるﾊﾟｰﾃｨﾏﾝ]

>>28
変数名をちゃんと付けるのは当然だけど今時はIDEに任せられる事は任せるでしょ。

[42. 自他とも認めるﾊﾟｰﾃｨﾏﾝ]

>>28
可読性を上げるべきなのは分かるけどIDE前提でえぇやろ。
ピュアっピュアなエディタ使って書くんか？化石かよ。

[6. 自他とも認めるﾊﾟｰﾃｨﾏﾝ]

変数の命名規約はスコープの長さに比例する
ループインデックスなど局所性の強いものはnでもiでもよい

[7. 自他とも認めるﾊﾟｰﾃｨﾏﾝ]

スコープがかなり限定的ならあり。5行以内で意味を為さなくなる一時的な変数とか。
グローバルでやるやつは大規模なアプリ作ったこと無いんだなって思う。

[9. 自他とも認めるﾊﾟｰﾃｨﾏﾝ]

>>7
これかな
listをlstって書くやつすらありえんと思ってるわ

極論だけど、短すぎるのと長すぎるのは後者の方がまだまし

[8. 自他とも認めるﾊﾟｰﾃｨﾏﾝ]

自分が一生そのプログラムを見るならエエけどそんなん無理だから誰でも解るように書くんだわ。引き継ぎした奴から恨みを買うか恩を貰うかはそういうとこやぞ

[19. 自他とも認めるﾊﾟｰﾃｨﾏﾝ]

>>8
1週間前の自分は他人だから、一生面倒見るのは不可能ってことだな！
aとかbとか使ったコードを1週間後に読んでみて欲しいわ

[10. 自他とも認めるﾊﾟｰﾃｨﾏﾝ]

変数名を辞書で調べて英語にして、後でわかんなくなることある。

[11. 自他とも認めるﾊﾟｰﾃｨﾏﾝ]

時間経ったら自分でも分からんくなるぞ
他人は尚更読めないぞ

[12. 自他とも認めるﾊﾟｰﾃｨﾏﾝ]

可読性考慮しないやつは自己中プログラマーだからな
スパゲティコードか書いて試験でトラブったら他人に丸投げしそうならgauge予備軍

[13. 自他とも認めるﾊﾟｰﾃｨﾏﾝ]

多重ループでIとj使うのは辞めて

[15. 自他とも認めるﾊﾟｰﾃｨﾏﾝ]

基本的に数年後に他人が読む、という前提で書かないといかんからなあ。
自作プログラムなら好きにすればいいけど、業務用でこれやっちゃいかん。
使うとしても本当に局所的なやつだけ。

>>13
これは分かる。
座標関係のループでxとyとか使うんならまだ分かるんだけど、そうではないようなところでiとjで回されると訳分からなくなることが結構ある。

[29. 自他とも認めるﾊﾟｰﾃｨﾏﾝ]

>>13
foreach(data as key01 => val01) {
foreach(val01 as key02 => val02) {
foreach(val02 as key03 => val03) {

意味があるなら意味のある名前を付けて欲しいね

[14. 自他とも認めるﾊﾟｰﾃｨﾏﾝ]

「このblastcorpsとかjetforce_geminiとか分かりにくいから。blastdozerなりstar_twinsに変えといてね」
「...(この上司できるのかダメなのかどっちだ...) 」

[16. 自他とも認めるﾊﾟｰﾃｨﾏﾝ]

反論はしても良いと思うけど、
経験や知識がそれを担保してないとダメ
そんなクソコード書くペーペーがお気持ち表明したって
どうしようもない位自覚した方が良い
他人の言う事聞きたくないなら、自分で勉強しましょう
とりあえずリーダブルコードから読もうか

[17. 自他とも認めるﾊﾟｰﾃｨﾏﾝ]

変な名前つけられるくらいならaやbでいいわ
そのかわり設計書に全部書け

[30. 自他とも認めるﾊﾟｰﾃｨﾏﾝ]

>>17
ザ・エアプ

[18. 自他とも認めるﾊﾟｰﾃｨﾏﾝ]

未来の自分の為に意味のある単語で書く

[20. 自他とも認めるﾊﾟｰﾃｨﾏﾝ]

MSXのベーシックは変数名2文字までだったな

[21. 自他とも認めるﾊﾟｰﾃｨﾏﾝ]

この真逆で、ギリシャ神話の固有名詞で統一してマイクロサービスを組んでる奴は居た
ちゃんと神話の役割とサービスを対応させて独特な世界観を構築してたが、速攻で普通の名前に書き直されてた

[22. 自他とも認めるﾊﾟｰﾃｨﾏﾝ]

リーダブルコード読め

[23. 自他とも認めるﾊﾟｰﾃｨﾏﾝ]

自分の記憶だよりなので時間が経つと自分に帰ってくる。

[24. 自他とも認めるﾊﾟｰﾃｨﾏﾝ]

ミスがすくないってwwwwww
aとかbとかcとか変数つくってミスしないやつのほーがすげーわ。

[25. 自他とも認めるﾊﾟｰﾃｨﾏﾝ]

ローカル変数名はまあ許そう
モジュール名とか関数名はちゃんとつけろまじで殺意覚えるから

[26. 自他とも認めるﾊﾟｰﾃｨﾏﾝ]

自分しか使わないものを自分で作るなら好きにすればいい。
それ以外の場合は、他人が見ても類推できるフレーズにした方がいい。
関数名の文字数すらケチる必要があったはるか昔じゃあるまいし。

[27. 自他とも認めるﾊﾟｰﾃｨﾏﾝ]

一時変数なんざ
あきゅーむ、べーす、かうんと、でーた、えくせとら、ふりー、とかでええねん

[32. 自他とも認めるﾊﾟｰﾃｨﾏﾝ]

やすやすと変数を定義しようって根性が気に食わない

[33. 自他とも認めるﾊﾟｰﾃｨﾏﾝ]

無能の書いたクソコードの変数名やサブルーチン名を意味の通る真面目な名前に置換すると
そこそこマシになったりする
まともなコメントも無いので無能が何をやりたかったのか推察してコメントとして追記する
こんな悲しい作業をやらされる身にもなれ

[34. 自他とも認めるﾊﾟｰﾃｨﾏﾝ]

これを古いしきたりというスレ主にはPGは向いて無いと思う
そこそこ大きなプログラム組んだりしてれば普通は職場ではやらんだろ

[35. 自他とも認めるﾊﾟｰﾃｨﾏﾝ]

新人が何言ってんだろ
学習と現場は別物だし「僕はこう習いましたが！（ｷﾘｯ」とでも思ってんのか
1か月後、このスレ内容を見返してもまだ考えが変わらないなら
スレ主はプログラマーの資質無いよ

[36. 自他とも認めるﾊﾟｰﾃｨﾏﾝ]

老人なもんで1文字でも少ない方が優秀という感覚が抜けない

[37. 自他とも認めるﾊﾟｰﾃｨﾏﾝ]

命名規約ってのがあってな…
例えばローカル変数で文字型なら”hoge_l_c”みたいにする、のが現場によって統一されてる
場合によっては構文チェッカにその辺のルール適用されてるからアラート出るんよ

[38. 自他とも認めるﾊﾟｰﾃｨﾏﾝ]

ほんとレベル低いな。何十年前の議論だよ。

[40. 自他とも認めるﾊﾟｰﾃｨﾏﾝ]

他人が見ること考えないとね
あと自分がそのプロジェクトしばらく離れてまた戻ってきたときとか絶対わからんと思うよ
でも命名規約ぎちぎちなのダルイのもわかる
英語力ないから名前考えるのめんどい
そこで一時手が止まると集中切れる

[41. 自他とも認めるﾊﾟｰﾃｨﾏﾝ]

わかるのなら別に1文字でもいい
max関数の引数とか
グローバル変数で1文字はゴミオブゴミ
けど長すぎても読みづらい