privateメソッドについての思いの変遷 — 裏紙

新人の頃の☃「private？メソッド？というのがあるのか。ふむ。ふむ……？」 新人ではなくなったが若手の頃の☃「メソッドが大きくなってきたな。privateメソッドで分割だ！」 若手とは言えなくなった頃の☃「privateメソッドのテストコードってどう書いたら良いんだ？リフレクションか？」 2、3年前の☃「privateメソッドは共通処理を切り出すためのもの。呼び出し元のpublicメソッドのテストコードで担保される」 最近の☃「privateメソッド スベテ コロス！！！」 解説 新人の頃は割愛。 次の若手の頃の話は、これは大きいメソッドを単にぶつ切りにして満足しちゃってた感じ。 臭いものに蓋してるだけで何の解決にもなっていませんでしたね、今から思うと。 それからprivateメソッドのテストコードについて悩みました。 どうすれば良いんだ？と。 悩んだ挙句protectedにしちゃたり

[otihateten3510]

やべー理解できない　例がほしい

[w1234567]

デカ過ぎるクラスが問題なだけでprivateメソッド自体は問題じゃないことが多いよね

[naka-06_18]

クラスをばこばこたくさん作りたくない時に private なメソッドを書きそうな感じなの感。

[gfx]

わからんでもない。

[ssids]

だいぶ前に同様の話題が出てたけど、 reflection 使ってテスト中で動的生成するのがいいんじゃね？という話も出てたかと。ただ設計を先に疑うべきなのはその通り。

[hush_puppy]

クラス分けるコードの匂いにプライベートメソッドか。そういうのもあるのか。

[kknsd]

javaだとpackage privateに逃げられるんやで

[n314]

行ったり来たりしているうちにバランスの問題と思うようになって、それがいわゆるセンスと呼ばれるものなんだろうか。

[tpircs]

全部publicにしちゃったら公開範囲ガバガバになっちゃうような・・・。package privateならまだわかる。でも、個人的には公開範囲で困るようなコードあんまり書いてなかったりするからガバガバでも問題ないことが多い。

[fukken]

ファウラーだったかケントベックだった忘れたが、privateメソッドよりもprivateなフィールドのpublicなメソッドの方がよい、と言っていた記憶。

[lazex]

つまり private なメソッドが存在しない JavaScript が最高ってわけですね

[Nkzn]

privateメソッドをテストしたい欲求が出てきたらそれは別クラスを作れるくらいの責務があるのではという感じかな。

[indication]

visial studio ultimateじゃないと、publicの試験しか簡単にできないから思い悩んでる

[umai_bow]

publicメソッドのテストだけでprivateのバグがすべて見つからなかったら何か間違ってる気がする

[kmaebashi]

カプセル化の目的が、開発者(チーム)以外の単なる利用者に実装の詳細を見せたくない、ということであれば、それがクラスを単位にすることがそもそも変で、package privateにしておけばよいのでは、と思っている。

[yashigani_w]

そんながんばってユニットテストを書こうとしなくてもいいんじゃないだろうか

[berlysia]

privateメソッドが出てくるのは責務の分割が不足しているんじゃないか的な話

[shinp]

引数なし、返り値無しのprivateメソッドは存在価値が無いので作らないように心がけよう。

[akabekobeko]

状態の排除や疎結合を推進してゆくと純粋な関数とか小さな数個の状態だけ持った class へ分散していって private なものは減ってく感じする。単体テストも絡み、私も悩み続けている。

[assaulter]

なるほど...

[amedama41]

クラスの不変条件を壊さないメソッドはどのクラスに所属させても問題ないはずなので、そういうメソッドは別クラスに置いてテストすればよいかと

[krrrr]

あるべき話と実際のコード（設計に悩む時間の都合等）は別になるし、困ったら妥協して@VisibleForTestingつけつつpackage privateにしてテスト書いたりもするな

[poppun1940]

privateメソッドについての思いの変遷 — 裏紙 privateメソッドについての思いの変遷 UnitTestでのprivateメソッドとの向き合い方 - ジムには乗りたい を読んで、自分のprivateメソッドに対する思いを書き殴りたくなったので書いて

[aiko876roll]

なんというか抽象度上げて疎結合にしようよに繋がる話なのかね

[komutan1]

Javaはインターフェースでカプセル化できるから実装クラスは全部publicでOKって話になってなかったっけ？あれ、なっていない？

[marisatokinoko]

わかる、カプセル化は幻想で、テスタブルなのが最優先。適切にサービス化してDIするみたいなのが関数型うんぬんも含めて最近のフレームワーク見ててもトレンドなんかなと思う

[jojo800]

具体例くれ

[oktnzm]

"そんでもって最近は、なるべくprivateメソッドは導入したくないと思っています。" 一周まわってglobalおじさんに進化してて草

[su_zu_ki_1010]

色々なクラスに同じような処理を行うprivateメソッドが出来ちゃうかな？というような時は別途クラスを作ってそこにpublicなメソッドを書くようにしている、って理解であってますでしょうか。

[wordi]

逃がした先もinternalクラスになりそうな

[rtryoda]

“privateメソッドを導入しようと思ったタイミングで設計に問題があるのでは？”

[teracy_junk]

『私はprivateメソッドを導入しようと思ったタイミングで設計に問題があるのでは？と疑ってみるようにしています』