リファクタリング - 西尾泰和のScrapbox

「このコードは既に十分に綺麗だから、（コメントは追加してもいいけど）リファクタリングする必要はない」" 一生リファクタリングしている自分とは仲良くなってくれなさそう。

[augsUK]

元の人は上司との信頼構築や適切なビジネスコミュニケーションレベルのことも「社内政治」と呼んで避けてきたのかなあと。初手がSlackに他人のソース晒して意見求めるとか地雷感がすごかったけど。

[shimokiyo]

企業として大事なのは「売上を増やす」「コストを減らす」の2つだけなのでリファクタリングがその2つにどう影響するか明確に言語化や数値化しないのはエゴ。会社の利益にならない仕事勝手に始める人っているけどね。

[morimarii]

ソースコードの可読性は組織文化によるところが多いのでいきなりリファクタリングは危険。というか何かをおかしいと思ったとき自分が組織文化の理解が足りないのか組織文化がおかしいのかは見極めないとね

[turanukimaru]

ソースコードに問題があるなら、テストとコメントを書くのをお勧めする。テストはできるだけバグをつつく。例えばパラメータに null を渡してぬるぽを出させてコメントに残す。すぐ直せなくても問題が起きたとき役立つ

[repon]

VCからの資金調達条件が、雇用人数だけだった可能性。資金調達も売り上げよ。

[namisk]

“滅多にいじらないであろうコードのリファクタしたところで得られる効用が低いわけでな” これは気を付けたい。

[nghrk]

テスト書かないのにリファクタリングしようとする人いるよね。あれは「作り直し」または「破壊」

[houyhnhm]

データサイエンスな領域のテストはかなり困難で、エラー吐かずに静かに間違ってたり、結果間違ってても簡単には気付かなかったりする。

[Sixeight]

一方の意見しか読んでないので評価出来ない

[daira4000]

自分ならローカルだけでリファクタリングして挙動を理解したら元に戻すみたいな事やるかなー

[zyzy]

あー……まぁ論文から書き下ろすタイプは確かに……

[saikyo_tongaricorn]

クソコードのままでいいんだけどクソコードの解析にお手間取らせていただきますが許容されてほしい

[door-s-dev]

Cでした。採用云々は流石に元の話題からずれてるかなぁと感じた

[cl-gaku]

納得行くまで机上リファクタリングしてとりあえず引き出しにしまったコードいっぱいあるな

[diveintounlimit]

結構組織の状況によるよなーと思って投票してみたらわりと外れな場所にマップされた。

[nishi1231chang]

過去に、読めないからリファクタしたいという人がいて、してもらったらmapや三項演算子のネスト繰り返す激重なコード書いてた人いたわ。程度によるが、読めないからという理由は能力不足の場合もあるので注意。

[aike]

初期から関わってると増築後もそこそこわかりやすいコードが、増築後に入った人には難解ということはよくある。あとアルゴリズムを知っていると関数にその名前がついてるだけでわかる人にはわかるとか。

[w1234567]

クソコードとクソ担当者のセットにぶち当たった場合はローカルのリポジトリにソースコピーしてリファクタリングしたコメント追加済みの検証用のきれいなコードをくみ上げてメンタル保ったほうがいいぞ

[strawberryhunter]

動くコードが良いコード主義は好きじゃないけど、Slackで意見を求めるのは普通に危なっかしい。業務システムの尺度だけで考えていたから「こいつは一体何がしたいんだ？」と思われていたのかもね。

[programmablekinoko]

どんなに控えめに言っても人の書いたコードの否定になるのでなあ。

[watapoco]

面白いコメントばかりだな。hrjnさんの言う通り採用側は採用側で大いに反省しないといかんよね

[atsushifx]

結局は政治というか人間関係の話。ワインバーグ本やピープルウェア系の本が再流行して、プロジェクトが健康になってほしい

[zentarou]

リファクタリングは福利厚生なのよね

[localhost0]

良いとされるコードって会社ごとに変わるよなぁ

[twotiger]

リファクタリングするより新規に作り直した方が効率的とか、ソシャゲとかでいずれ数年で売上なくなりサ終するからリファクタリングする必要ないとか、あると思います

[zknf]

自分で主体的に動いて仕事を作ってく部署で先輩コードのリファクタ提案は悪手だと思ったな。読めないなら何やってんのかだけ聞いて、それこそ新しいモデルを読みやすく書いて提案するかな、自分なら

[shingo-sasaki-0529]

僕も基本的にリファクタリングを常にし続けたいけど、それによるプロダクト価値の向上を根拠を持って説明できるかと言うと怪しいのでわかりみの深い意見が多かった。

[mobits]

「可読性が文化による」はさすがに草

[rgfx]

これだけでなく、ML/DS勢の「は？テスタブルな機械学習のコード？？？」という話がなあ。元論文や書籍(にある数式)がそのままドキュメントだったりしそうで https://bit.ly/3MnDgHl / はい https://bit.ly/3Ot2QgQ

[outalaw]

異業種だが組織・人材の課題は共通で参考になる。