プログラマ「ソースコードが汚い？動いてるからええやんｗ」 上司「そうだな。でもお前さぁ・・・」

昔、書いたソースコードが汚いと上司に指摘されて「動いてるからよくないっスか？ｗｗｗ」と言ったら「そうだな。でもお前、ぐちゃぐちゃに混ぜられた牛丼出されて食うか？食えればいいか？」と返された。

当時の上司には非常に感謝してる。

— ツララボ＠福岡エンジニア (@tsuralabo) September 18, 2019

この上司の方はもう定年辺りだと思いますが、自分のエンジニア人生に多大なる影響を与えた方です。

「お前みたいなカスでも簡単にシステム作れる時代が絶対くるから、今のうちに何がクソなのか見極める技量を養え」と言われて育てられました。

CTOになれましたよ、Mさん。

— ツララボ＠福岡エンジニア (@tsuralabo) September 18, 2019

Comments

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

ユーザーはソースコード見ないし牛丼混ぜたら味わい変わるし、汚いかもしれない厨房の店で食事したいかどうかかな

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

>>1
ひろゆき「アジャイル開発やめてもらっていいですか？」

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

実際は動けばいいよって言ってくる上司の方が多い気がする

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

>>2
綺麗かつ正確に書ける奴は居ないって
理解してるんやろな
ただでさえ仕様変更の連続だし

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

アベ政治と同じだな。
図体がでかいだけで、ろくに社会に有意義な政策も機能も果たせていない。ごちゃごちゃ。まるで必要な物事を整理して遮断できない。世界でも日本人だけ、こんなに効率の悪い社会生きてるのは。
中国や韓国ではキャッシュレス化が進んでいるが、日本では今だにごちゃごちゃとした現金主義。さらに、中国や韓国ではIT技術者の待遇はよく、休みもちゃんとあり、高収入だ。みんなこぞって中国や韓国に技術者たちが流出している。日本はIT技術者をないがしろにしており、他国に比べて労働環境が悪く、地獄。故に、だれもこない笑
こんな国はなかなかないよ。でも、こんなめちゃくちゃなことをやって、中国や韓国に労働面でもはるかに差をつけられている日本人の半数近くがアベ政治を支持しているという有様。頭狂ってるわマジで 南海トラフ大地震に期待だわ。日本を荒らしてこんな社会風潮を一変させてほしい
それしかこの腐った国と腐った市民たちの意識を変える手段はない。

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

病人が来た

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

触れるな

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

程度によるだろ
キレイすぎるコートは仕様変更しようとすると全直しになったりするし
発注に合わせたオンリーワンのコード書いたりすると逆にミスを生みやすい
汚いコードは書く方もラクだし、まあずっと同じコードの派生だけじゃいかんけど

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

汚いソース書かれると障害発生時や改修時に書いた奴を殴りたくなる衝動に駆られる

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

>>7
障害発生前にチェックしないのが悪いw

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

綺麗汚いというより、後で見て割と素早く流れがわかる形になってればいいと思う

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

>>8
ゲイツ「俺ならもっと短く書ける」
短いプロシージャは見渡しいいけど
解読可能か自信ないや
古代呪詛みたいなのあるでしょ

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

料理は一度作ったものを汚いからって作り直したりしないんだよなあ…

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

>>9
タモリさんはカレーライスを均一に混ぜる派らしいで

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

業務で絶対にやらなきゃいけないならともかくガチの初心者への指導には綺麗よりも動くこと目的にさせた方がやる気が出やすい
ただそれだけのことなのに、狭い世界でドヤってんだろうなぁ

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

おまじないが常用化されてる会社もあるで
命名規則とかコーディング規約が明確に存在すんのにルールがおかしい会社もあったで　横ｘｘ桁までとか
なお指定された変数名が20文字以上あったりで指定数に収まらなかったからガン無視で作って逃げた。この木なんの木の所　管理者が入院で逃げて納期間に合わなくてデスマ時で誰も仕様理解してなかったから動けばいいから形だけでも作れって感じだった

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

>>11
テンションマックスなやつー
脳汁ドバドバ

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

ソースは牛丼と違って客に見せる最終アウトプットじゃないからこの例えはおかしいよね
厨房が散らかってても綺麗な牛丼が出てくればＯＫじゃん

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

個人的にはコードが汚いってのは非常口に物が積み上がりまくってて通れないみたいな感覚だから単に気持ち悪いみたいな例えはやめてほしい
綺麗好きとかじゃないから
普通に命に関わる事だから

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

自分が食うわけじゃないなら別によくね？w

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

日本のIT業界は3行で終わるコードを元請け！下請け！孫請け！と無駄に100行書いて情弱老害を騙し派遣屋さんだけが儲かるシステムになってる

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

嘘松

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

三項演算子禁止とか言ってそう

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

綺麗に書く時間があるときはそうしてる

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

>>20
自己研鑽と自己鍛錬ね自家発電もお手の物

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

>>20
自己研鑽と自己鍛錬ね
自家発電もお手のモノ

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

ビビンバなど韓国料理ですら、客に出される時点ではまだ混ざってない

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

※9
自炊や家庭料理ならそうだろうがな、
客に商品として提供した物なら作り直しを要求されることもありえるぞ

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

一見きれいなコードのバグは分かりにくい。

あと、さりげなくコメントが間違っている奴。

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

問題はそこじゃなくて期日守れて儲かるかどうかでは？

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

今後引き継いでいく人のことを考えたら綺麗に組まないとなぁ

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

動いてるからいいっしょじゃなくて、バグあっても分からんやん
綺麗にしろで察しないようなのには、バグるまで泳がせて自然とたどり着いて実感させた方がいい
なお、研修期間

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

動いてるからいいっしょじゃなくて、バグあっても分からんやん
綺麗にしろで察しないようなのには、バグるまで泳がせて自然とたどり着いて実感させた方がいい
なお、研修期間

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

ユーザー報告受けてバグ治す仕組みが分からない
バグで開発終了みたいな話が聞きたい

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

この上司の世代ならマシンパワーの縛りがきつかったから
「短く・動作が速く」も至上命題だったよね
少ない部品で高速ピタゴラスイッチを作るために頭を悩ます

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

そもそもレベル低い会話だな　可読性くらいの品質新入社員くらいでも知ってるわ　おそらくニワカ

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

いや意味分からん
牛丼って年月が経ってから肉を引っ剥がして豚肉に置き換えたりするのか？

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

動くのは最低条件。その上で綺麗なソースの方が気持ちいだろってお話。
ただなぁ、気にしだすと際限ないんだよな。

作ってる間は体裁なんか気にしてられないし(まず動くものを作らなきゃ意味が無い)、出来上がった後で体裁を整える時間がとれるかどうか。その作業でエンバグする可能性もあるし。長い行は適当に改行入れてスペースで桁合わせとか(ブロックのインデントとは別)。

まあ今どきはその辺も自動でやってくれるみたいだけど。
現場から離れてずいぶん経つのでよく知らんが。

今はJSでマクロ組む程度だからテキストエディタで十分だし。

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

>>32
体裁整えた時は「美しい」と思えても、時間が経つと「こうした方がもっと良くね？」ってなって整え直したくなったり。さらに時間が経つと「前の方が良くね？」とかなったり。いつまでもソースをいじりまくる羽目になる事も。

C言語のif文とかfor文とかの中身が1行なら「{}」が省略できるけど、その時の気分やコードの詰まり具合で省略したりしなかったり。昔は画面が狭かった(24行)から極力行数を減らしたかったし。で、後から見返すと「見づらいな」ってなって省略しない書き方に統一しなおしたり。「{」の位置もそうだな。右端に書くか単独行に書くかとか。

そんな事を何時までもやってて先輩に怒られた思い出。

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

>>34
チソポジがジャストフィットするまで
妙な動きをするやつな

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

あ、これキレイ汚いの基準が人によってバラバラだから議論が平行線になるヤツだ

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

動けばよいでぐちゃぐちゃは少しでも不都合が出た瞬間に全力で後悔する奴

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

必要な量の5倍くらいの、支離滅裂を極めたコードを書く自称ベテランが、案外多い。

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

上司って大体ソースコードなんて見ないし、
むしろ今上司の人が昔に書いたクソコードを保守させる人が多い中素晴らしい上司だわ

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

キレイなソースの方が気持ちいいとかじゃねーよ
キレイなソースの方が他人が理解しやすいかつ変更しやすいから良いんだよ

明確な利点がなかったら誰もやらんし、大事だとも言わんわ