ソースコードの減らし方 - 基本的な考え方と10個の方法 - クラウドワークス エンジニアブログ

ステップ数で評価が決まる現場では全く役に立たないテクニックではありますが、ソースコードの減らし方について紹介したいと思います。 開発Div. エンジニアのayasudaです。 2014年の夏にジョインし、会社名と同じサービス、クラウドワークス の開発に携わっています。 ご覧の通り、消したソースコードの方が多いので、ステップ数換算だとマイナスの働きしかしてませんね！ 本記事では、特に Ruby on Rails の運用されているプロダクトコードにおける、ソースコードの減らし方について紹介していこうと思います。 基本的な考え方 ソースコードを減らすときの大原則は「ボーイスカウト・ルール - プログラマが知るべき97のこと」です。 普段、ソースコードを触るときに、一つでも良いので簡単な改善を入れる。これを積み重ねるのが大事です。 一度に一気に直そうとするのはあまり良くありません。大抵の場合、デグ

[primedesignworks]

クラウドワークスはソースコードよりも理不尽な案件を言いまくる発注者を減らした方が良いと思う。

[otchy210]

スクリプト言語はこう言うのやっぱり大変そうだなぁ。リファクタに関しては静的言語の方が圧倒的に楽だ。

[hatomugicha]

趣味の範囲でしかないド素人だけど、ただ上達したいのなら、無駄や悪手だと言われる方法でとにかく長いソースコードを書いて、実用的に使い、修正するときの苦労を解くと味わうのが良く効く

[mangakoji]

こういう再設計っぽいことが許される環境羨ましいな。趣味のコード以外では、書き直し怖い怖いヒイ。読めないのは相手のせいでもエンバグは俺のせい。

[estragon]

ボーイスカウト・ルールねぇ。当たり前と思われるような小さなことの積み重ねが、なかなか、多人数チームだとできないんだよねぇ

[lbtmplz]

恐怖との闘いだ...

[sds-page]

反論してやろうかと思ったら割と正論だった

[kiyo_hiko]

"おすすめなのは、「将来削除する旨を記したコメント」と、参照されたらログを吐いておくという手法です。"

[sonots]

仕様を削る、仕様を整理する、仕様をシンプルにする、などの視点も重要

[ledsun]

ソースコードの修正箇所は偏りがある。ボーイスカウト・ルールなら、修正されやすい部分を優先してリファクタリングできる。

[sylvan_l]

ソースコードの減らし方 - 基本的な考え方と10個の方法

[snishimura0926]

「どこかの格言によれば、コメントとコードが矛盾しているときは、たいていの場合両方ともバグっています。」

[iox]

“ボーイスカウトルール”

[masaru_b_cl]

継続が大事

[gabuchan]

よいよい

[coco5959]

ソースコードの減らし方 - 基本的な考え方と10個の方法 - CrowdWorks Engineer Blog: 2016 - 03 - 11 ソースコードの減らし方 -…

[june29]

全面的に同意。og:image が本文中に登場していなくておもしろい。

[michiomochi]

depricated_warning出すの良い。

[koogawa]

メモ

[yoshuki]

“ボーイスカウト・ルール”

[lovelogic2]

これなら減らせる！

[digitalglm]

ソースコードの減らし方 - 基本的な考え方と10個の方法 - CrowdWorks Engineer Blog: 2016 - 03 - 11 ソースコードの減らし方 - 基本的な考え方と10個の方法 プログラミング コーディング…

[serihiro]

廃止予定のメソッドでログ吐くのfluentdとかでログ拾ってなんらかの手段で通知できる前提かな。そうしないと気づかなそう。

[vanbraam]

いつも思うが,テストの共通化は難しい;単純に共通化すると,テストがframework化してテスト自体の正しさが理解しづらくなる;残りは大体普遍的な内容だと思うが,lifactoringだけ少し興味を感じた.レビュワーへの優しさ重要

[surume000]

1,2で既にMECEじゃねえんだけど。

[gam-22]

いい話

[t_trad]

こういうリファクタリングがほとんど実行不可能なレベルの芸術的スパゲティコードが存在する。そしてそういうコードを書く人ほど「仕事が速い」と評価されて新規開発に呼ばれ書き散らして去って行く

[oakbow]

削除行数の方が多いのは、普段から果敢にリファクタリングに挑んでる証拠ですな。周囲の理解があってもやっぱ消すのは怖いしね。aggregate_failures知らんかったから今度試してみよう。

[kyohei_hamada]

alias_method_chainで廃止予定のメソッド呼ばれたらログ吐くようにしておくの使えそう

[eichisanden]

ボーイスカウトルールは実践してる。せざるをえないコードをメンテしてる訳だが。

[hsaitou]

ありがたい

[takuya_1st]

このやり方ではコードは減るが手間は減らない。むしろ直交性が犠牲になって依存関係が深いスパゲティになる

[ryuzee]

いいまとめ！！

[tokida]

ソースコードの減らし方 - 基本的な考え方と10個の方法 - | ステップ数で評価が決まる現場では全く役に立たないテクニックではありますが、ソースコードの減らし方について紹介したいと思います。 開発Div.

[kamei_rio]

コピー＆ペーストと魔法の言葉でハチャメチャ大混乱

[luccafort]

なるほど、動的に呼び出される場合はログを吐くか。コードをダイエットそのものに対してすごく肯定的なわけではないけどリファクタリングなども含めての話ならアリっすな。