変数や関数の名前がいつの間にか分かりにくくなる問題 - Qiita

TL;DR: 変数や関数を追加するときは、周りにある他の変数や関数の名前を修正すべきでないか検討せよ いきなりですが問題です。あるソフトウェアモジュールに以下の三つの関数があります。 show showWithSlideAnimation showWithoutAnimation 画面をスライドさせながら出現させるにはどの関数を使用すれば良いでせうか? 関数の名前だけを見て答へてください。 はい、その通り。showWithSlideAnimation が正解です。 では、画面をアニメーションなしで出現させたい場合はどの関数が良いでせうか? はい、showWithoutAnimation が正解ですね。 今度は、画面を回転させながら出現させたいとします。適する関数はあるでせうか? ブーーーッ! 残念、正解は「どの関数も適さないので新しく関数を実装する必要がある」でした。 これで最後です。画面

[gabill]

MacBook (MacBookAirより薄い)

[peccu]

機能は追加だけじゃない } リファクタリング - 変数や関数の名前がいつの間にか分かりにくくなる問題 - Qiita

[nishitki]

リファクタリング - 変数や関数の名前がいつの間にか分かりにくくなる問題 - Qiita

[shiba_yu36]

おもしろい

[C_L]

showがない

[s025236]

例に突っ込むのも野暮だけど…そもそもの誤りは、改名しなかった事じゃなくてアニメーションを関数名に付けてしまった事では？

[masayoshinym]

すごい良いこと書いてるとは思うんだけど旧仮名遣いのせいで読む気がしない。残念。

[odaichan]

難しいですね‼

[clavier]

リファクタリング - 変数や関数の名前がいつの間にか分かりにくくなる問題 - Qiita

[dev0000_1]

既存コードの検討は同意。ただ例について、「通常」「例外」で要件レベルが明らかに分かれている場合、メソッド名をあえて非対称にしておくケースもあるかと思った。

[umai_bow]

ユースケースがリアリティありすぎる

[snowlong]

あるあるだ

[ad2217]

ウケ狙い？

[onefootinthegrave]

回帰テストの手段がちゃんとメンテされてて理解のあるプロジェクトならshowも直せるかもね。

[Sampo]

素晴らしい考察。 コードに改修を入れるときどうしても出てくる「最小限の変更で」実現したいという意識をコントロールしなくちゃいけないのだ。