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

**デフォルトの名無しさん** · 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

誠にごめんなさい