カプセル化■プライベートメソッドをテストする方法

**デフォルトの名無しさん** · 2020/07/05(日) 20:47:46.60

■短い回答
プライベートをテストしたい場合は設計に問題があるので、パブリックに変更してテストしましょう

■これに対する（変な人の）驚いた反論

プライベートを一時的にパブリックにして、テストが終わったら
プライベートに戻すなんてやるわけないだろｗ
↑誰もそんなコトしろなんて言ってない

■テスト専門家による回答

t-wadaのブログ
https://t-wada.hatenablog.jp/entry/should-we-test-private-methods

短くまとめると、プライベートなメソッドのテストを書く必要は無いと考えています。

ほとんどのプライベートメソッドはパブリックメソッド経由でテストできるからです。
プライベートメソッドは実装の詳細であり、自動テストのターゲットとなる「外部から見た振る舞い」ではありません。
プライベートなメソッドのテストに関しては、4つの考え方があります。

・パブリックメソッド経由でテストする
・別クラスのパブリックメソッドとする
・テスト対象の可視性を（やや）上げる
・プライベートのまま、リフレクションでアクセスしてテストを書く

パブリックメソッド経由でテストする
多くの場合、そのクラスのパブリックメソッド経由でプライベートメソッドのテストも同時に行えます。テストできているか不安があるならテストカバレッジを確認しましょう。

別クラスのパブリックメソッドとする
プライベートなメソッドのテストを書きたいということは、実はテスト対象の責務が多すぎることを示唆している場合があります。
テストがどうしても書きたい場合は、その責務はテスト対象のプライベートな振る舞いではなく、他の誰かのパブリックな振る舞いなのでしょう。

**デフォルトの名無しさん** · 2020/07/05(日) 20:48:27.27

前スレ

カプセル化の有害性、オブジェクト指向は愚かな考え
https://mevius.5ch.net/test/read.cgi/tech/1592491656/

**デフォルトの名無しさん** · 2020/07/05(日) 20:51:53.15

クソコード例。こんなコード書いてるやつが、privateのテストで
パブリックに変更してテストするのはおかしいとか言ってる（笑）
lenが配列（笑）理由 int型にはnullが入れられないから（笑）

https://mevius.5ch.net/test/read.cgi/tech/1592491656/805
ずれるかもしれないが下のような場合、privateにnullを突っ込んだらヌルポだが
privateをわざわざコード弄ってまで別にテストするようなことは少なくとも
ビジネスソフトでは知ってる限り無い。組み込みとかは知らんし必要ならやれば良いけど。

class ChinTester {
public void testChin(int[] len) {
if (len==null){System.out.println("You are a woman");
return;}
if (len.length<11){ uncS(len);}
else{funcB(len);}
return;}

private void funcS(int[] len){
if (len.length<9){System.out.println("Smallest");
}else{System.out.println("Smaller");}
return;}

private void funcB(int[] len){
if (len.length<14){System.out.println("Medium");}
else if (len.length<16){System.out.println("Bigger");
}else{ System.out.println("Wow!");}
return;}
}

**デフォルトの名無しさん** · 2020/07/05(日) 20:53:33.00

983 名前：デフォルトの名無しさん[] 投稿日：2020/07/05(日) 14:18:27.81 ID:9F15TCk0 [68/74]
正直あの短さでOOかどうかと（スタティックでインスタンス化もないコードだが）言うのは
不毛だけどID:JiRnWiGCの組み込みおじさんのがOO感はあるよ。

で、staticで出されてもprivateのテストがどうかと言う話には全く寄与しないわけだが、
じゃあ逆に、>>805のチンコテストのfuncSとfuncBはどうやってテストするの？

パブリック経由で全パターンと言うことならこれでこの話はおしまい。
パブリック経由でやりましょう。

違うと言うなら具体的にコードでおながいします。

**デフォルトの名無しさん** · 2020/07/05(日) 20:54:53.36

>>4への回答
設計に問題があるので、コードを修正しましょう。
修正すれば自然とpublicになります↓

926 名前：デフォルトの名無しさん[] 投稿日：2020/07/05(日) 12:20:23.64 ID:MQ9nuMmc [21/53]
>>805
こう書いた方が良いと思うの
https://paiza.io/projects/mPhqBnYZnQukkW6HY9LmOQ

**デフォルトの名無しさん** · 2020/07/05(日) 20:57:49.33

>>5の補足、これはまだプライベートですが
普通はMainクラスにコードをごちゃごちゃ書かないので
judgeLengthを含むクラスを作成します。

そしてmainから呼び出します。
必然的にjudgeLengthメソッドはパブリックになります。

**デフォルトの名無しさん** · 2020/07/05(日) 21:03:41.24

>>6
judgeLengthをpublicにしてよいかどうか、オブジェクトをわけて良いかどうかは微妙なところで
publicにした場合って他のオブジェクトからも参照可能になってしまうので
そこの依存があとあとjudgeLengthの修正をできなくする可能性があるので
privateにするっていうのは他のオブジェクトから依存させないようにして独立させる意味もあるので
テストするためにpublicにしますっていうのは僕はやっぱり反対ですね

**デフォルトの名無しさん** · 2020/07/05(日) 21:04:50.23

僕が書いたように同じオブジェクトにテストコードも書いちゃえば良いと思います

**デフォルトの名無しさん** · 2020/07/05(日) 21:07:40.54

>>7
微妙でもなんでもねーよ
Mainクラスに関係ない処理を入れるな

これは小さいサンプルだが、大きくなった時
Mainクラスに、そんな処理はいってたらおかしいだろ
なんで（大きくなったコードの）その他の部分は別のクラスにあるのに
この処理だけMainクラスにあるんだ？って

**デフォルトの名無しさん** · 2020/07/05(日) 21:09:37.34

>>9
Mainクラスと関係ある処理だからMainクラスにあるんじゃよ
Mainクラスからしかアクセスしないからprivateなんじゃよ
テストするためにpublicにするのはおかしいのじゃよ

**デフォルトの名無しさん** · 2020/07/05(日) 21:11:26.55

>>10
Mainクラスと関係ある処理だから？

じゃあ名前が悪いよね。
誰が「Mainクラス」と聞いて
その中にある処理を想像できるんだ？ｗ

**デフォルトの名無しさん** · 2020/07/05(日) 21:13:00.77

>>11
えっと、前にも言ったけどpaizaはMainという名前じゃないと動かないんだよ
Mainとわけてクラスを作ることもできるけど面倒だからやらなかっただけ
普通のプログラマならpaizaの制約ぐらい知ってるだろうし説明する必要もないと思ってた
君の無知さを想像できなかった僕のミスだ

**デフォルトの名無しさん** · 2020/07/05(日) 21:13:24.90

君が無知すぎてごめん

**デフォルトの名無しさん** · 2020/07/05(日) 21:14:24.05

誠にごめんなさい

**デフォルトの名無しさん** · 2020/07/05(日) 21:14:43.14

>>12
だから分けろって。設計がおかしいだろ。

Mainクラスにメインの処理を入れるな
設計が悪いって言ってるのになんで理解できない？

**デフォルトの名無しさん** · 2020/07/05(日) 21:15:37.89

プライベートをテストしたいっていうのは
本質的に設計が悪いってことが理解できないやつ
もしくは設計なんてしたことがないやつなんだろう

**デフォルトの名無しさん** · 2020/07/05(日) 21:15:45.13

>>15
てめえでやれやハゲ

**デフォルトの名無しさん** · 2020/07/05(日) 21:16:23.44

>>17
だからやるのが正解だろｗ

**デフォルトの名無しさん** · 2020/07/05(日) 21:17:00.58

>>16
君は簡単なロジックしか組む機会がなかった幸せな人だと思うよ
僕は君が羨ましい、幸せな人生を歩んでいるね

**デフォルトの名無しさん** · 2020/07/05(日) 21:17:17.30

>>18
やってない君が不正解だね

**デフォルトの名無しさん** · 2020/07/05(日) 21:19:16.27

僕は下痢便コードをこうするべきって修正したよ
paizaで動くようにMainというクラス名に変えたけどね
paizaだからそうなるよねってみんな理解してくれるものと思ってた
paizaを知らない木偶の坊がクラス名に文句つけてきたとき僕は絶望した

**デフォルトの名無しさん** · 2020/07/05(日) 21:20:28.56

まだやってたのかｗ
>>5みたいな下痢便コードが出てきた時点で
「あっ・・・（察し）」でスレ終了でしょ

**デフォルトの名無しさん** · 2020/07/05(日) 21:21:29.14

こうするべきだと思うならやれば良いがな
自分でやりもせず他人に文句いうだけの人間は木偶の坊のそしりを免れないよ

**デフォルトの名無しさん** · 2020/07/05(日) 21:22:53.14

Mainに他のクラスの処理を全て同居させる

全部Mainにあるから呼び出せるよね？と
publicメソッドをprivateメソッドに変更

publicメソッドはMainのみ！
と言い

あぁ、プライベートメソッドのテストができない～と嘆く（笑）

アホなのか？自分でテストできないようにクソ設計に変更して
自業自得じゃんｗ

22 · 2020/07/05(日) 21:23:27.95

×>>5みたいな下痢便コードが出てきた時点で
○>>3みたいな下痢便コードが出てきた時点で

すまん訂正

**デフォルトの名無しさん** · 2020/07/05(日) 21:25:43.01

>>24
君はまだMainという名前のレトリックから抜け出せてないように見える
paizaで動かすためにMainという名前にしたってだけだから
実際にはそれなりの名前になるでしょう
privateメソッドは当然クラスと関係あるものになるでしょうということ

Mainという名前に囚われ過ぎておられるように見受けられる
自分だったらこう書くのにって言うのがあるならそれを実践してみるべきかと思われます

**デフォルトの名無しさん** · 2020/07/05(日) 21:25:44.19

おそらく設計とは何かを知らずに、
ただ動けばいいと思ってるんだろう

テストしやすく設計するのも
設計の一つ

**デフォルトの名無しさん** · 2020/07/05(日) 21:26:22.28

>>27
Mainを勘違いしてたアホが何抜かしとんねん

**デフォルトの名無しさん** · 2020/07/05(日) 21:26:36.76

>>26
だからテストしやすようにjudgeLengthメソッドを含んだ
クラスを作るだけ