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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

>>28
勘違いしてませんが？
Main関数とロジックを同居させるなの意味がわかりませんか？

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

謙虚になれや
オブジェクト指向とは礼儀作法と心得よ

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

>>30
はいはいもうえーから

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

>>27
Mainクラス云々については単に誤解してると思うけど
このスレの登場人物じゃあ多分おまいが一番マシな感性持ってそうやなｗ

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

>>29
作ってへんやんけ

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

やりもせずに何抜かしとんねん

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

ほんま口だけやな

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

あら。続いていたのか。

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

このスレの主旨的にはprivateメソッドのテストコードを書きたいんだよね？
書く必要がないって主張は違うよね？

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

それはそれでありだと思うよ

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

結局のところ何でもあり
僕は単体テストさえやってないからね

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

スレタイと書いてあることがチグハグで趣旨が解りづらいな。
スレタイに従えば、privateを呼び出したいみたいだが、>>1の発言を見るとprivateの呼び出しは推奨しないように見える。
まぁ、私も推奨しないけど。

という訳で、適当に独り言を語ってみる。

単体テストって、例えば...
Queueという名前のクラスがあって、そのクラスの中に
Enqueue、Dequeue、Peek、Clearメソッドが定義されていたら、それらメソッドを呼び出して、その結果を予想するコード(テストコード)を書いて実行させる方法がオブジェクト指向プログラマーにとって一般的だと思うけど...その際にprivateメソッド(内部実装)をテストコードから呼び出さないといけない理由がわからん。

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

t-wadaさんがどういう人か調べてみたところ、日本が誇るJavascript使いなんだな。
すると、t-wadaさんの立場ではprivateをテストしないというのは全くもって正しいと思う。
privateでやる事はライブラリを呼び出すだけなので、テストするのは無駄。

しかし、他の言語ではアルゴリズムの実装という仕事があり、アルゴリズムの実装をテストしたいという要求は常に存在する。

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

例えば多くのパーサーは公開メンバとしてpush()を持つ。
文字をプッシュする関数だ。
めぼしい公開メンバはこの程度しかない。

しかしその裏に100を超える非公開のメンバがあっても驚かない。
非決定性のパーサはその程度のメンバを持つ。

文字をプッシュする公開メンバ一つで、多くの状態、多くのメンバをテストするのは面倒な話で、何方にせよ非公開のメンバ、非公開の状態変数について知識が無ければテストできない。
だったら内部の状態について一切テストしないか、内部の状態を直接テストするか二択となる。

内部の状態についてはブラックボックスとして扱うべきと述べているのがt-wadaさんだが、それでは実装が非常に困難だ。

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

パーサーの話ついでに、もう一つアイデアを提供しよう。
決定性パーサーでは、還元が行われたとき、外部に影響を及ぼす。
通常この時点で意味動作を行う。
t-wadaさんは、この時に限りテストを行うべきであり、それ以外でテストを行ってはいけないと述べる。

つまり、シフトが行われるときはテストしてはならない。
今どの状態にあるかは外部にかかわりのない事なのでテストとしてはならない。
入力文字として改行を与えるとどの状態に遷移するかテストしてはならない。

これは厳しすぎやしないだろうか？

**デフォルトの名無しさん** · 2020/07/06(月) 00:01:34.67

詳細を知ってはならないということは、どの状態を経てその記号が生み出されたのか知ってはならないということであり、これは非常につらい。

**デフォルトの名無しさん** · 2020/07/06(月) 00:03:58.43

システム利用者がシステム作者に「詳細を検査しちゃだめじゃないか！」と怒っているように見える。

**デフォルトの名無しさん** · 2020/07/06(月) 00:24:27.58

>>42
t-wadaはもともとJavaな人

**デフォルトの名無しさん** · 2020/07/06(月) 00:27:13.73

クリスマスプレゼントが何かを知るために箱を振って音を聞くよりも、箱を開けて直接見るほうが早い。

**デフォルトの名無しさん** · 2020/07/06(月) 00:29:02.17

箱を振って「プレゼントはプリキュア」と観測できたとする。
これは果たしてどの程度もっともらしいだろうか？

**デフォルトの名無しさん** · 2020/07/06(月) 01:18:51.21

privateなメソッドの動きが仕様書で定義されてたらテストしなくちゃいけないし書いてなかったら書く必要無い
そしてprivateなメソッドの仕様まで定めた仕様書は俺は見たことない

**デフォルトの名無しさん** · 2020/07/06(月) 02:20:25.77

>>47
うるせーばか。文句を言うために都合の良いデータを調べてきたに決まってるだろ

**デフォルトの名無しさん** · 2020/07/06(月) 07:14:05.86

一気に胡散臭くなったな

**デフォルトの名無しさん** · 2020/07/06(月) 07:37:12.27

>>50
仕様書には普通アクセス修飾子を何にするかなんて書かないからね
設計書にもそんな細かいこと書かないでしょ

**デフォルトの名無しさん** · 2020/07/06(月) 07:38:08.27

書いてない関数を勝手に作るなんて禁止に決まってるだろ
テストのためだけに関数は作らない

**デフォルトの名無しさん** · 2020/07/06(月) 07:59:18.67

>>54
すごい、そんなガチガチなの銀行系？

**デフォルトの名無しさん** · 2020/07/06(月) 08:00:23.42

関数を作るのに許可がいるなんてすごいことだからとてもすごいと思いました

**デフォルトの名無しさん** · 2020/07/06(月) 09:07:12.86

>>1にはテストとしか書かれていないけど、記事元はユニットテスト(単体テスト)の話だよね？

**デフォルトの名無しさん** · 2020/07/06(月) 10:09:49.30

組み込みでの話らしいよここw

**デフォルトの名無しさん** · 2020/07/06(月) 10:19:56.49

>>57
ユニットテスト以外でメソッドのテストなんかするのか？

**デフォルトの名無しさん** · 2020/07/06(月) 12:11:06.79

>>3
このコード出すやつが混じってるんだろ？
議論無駄やん

**デフォルトの名無しさん** · 2020/07/06(月) 12:27:52.10

>>59
自分であたかも他のテストがあるように言っておいて変な話だがだが、しないね。

単体テストで不具合を見つけた後、デバッガを使って更にどこにバグの原因が潜んでいるのか分析することはあるけど、それはもはやテストではなくデバッグだしね。

**デフォルトの名無しさん** · 2020/07/06(月) 12:34:18.82

てかさ、タイトルがカプセル化なのに、なんで>>3も>>5もテストコード実行場所とテストコード実行場所が同じ階層に記述されているのさ。

**デフォルトの名無しさん** · 2020/07/06(月) 12:35:06.14

ミス。テストコード実行場所とテスト対象が同じ階層に書かれているのはなんで？だ。

**デフォルトの名無しさん** · 2020/07/06(月) 14:43:00.46

和田理論は穴が多くてお話にならないな。

**デフォルトの名無しさん** · 2020/07/06(月) 14:44:50.89

では穴のない理論をおなしゃす

**デフォルトの名無しさん** · 2020/07/06(月) 14:51:45.46

和田理論に騙された人は抗議したほうが良い。

**デフォルトの名無しさん** · 2020/07/06(月) 14:54:43.27

wadaは忘れろ

privateだからテストしなくていいなどという都合のいい法則は存在しないだがprivateだからとテストをしないキチガイは確実に存在するのだ

**デフォルトの名無しさん** · 2020/07/06(月) 15:05:56.82

和田理論被害者の会ひつよう。
とてもひつよう。

**デフォルトの名無しさん** · 2020/07/06(月) 15:07:12.95

一意見を急に”理論”とか”法則”とか言う方が頭どうかしてるよ

t-wadaはTDDをわかりやすく解説することに定評があるだけ
ただいろんな会社からテストのコンサルティングで雇われる程度には有能だから
君たちの意見よりは一般には受け入れられやすい

**デフォルトの名無しさん** · 2020/07/06(月) 15:10:07.70

和田理論被害者の会ニューヨーク支部もひつよう。
とてもひつよう。

**デフォルトの名無しさん** · 2020/07/06(月) 15:12:35.36

このスレ、脱線が好きだなw

**デフォルトの名無しさん** · 2020/07/06(月) 15:14:01.90

前スレで王家秘伝のレシピ教えたのに。
誰も活用しないんだな。

**デフォルトの名無しさん** · 2020/07/06(月) 15:17:52.38

俺も活用していないわそういえば。

**デフォルトの名無しさん** · 2020/07/06(月) 15:18:30.87

>>61
> 単体テストで不具合を見つけた後、デバッガを使って更にどこにバグの原因が潜んでいるのか分析することはあるけど、それはもはやテストではなくデバッグだしね。

単体テストで不具合を見つけた後にするもの＝デバッグ
デバッグの前にする不具合を見るけるもの＝テスト

だろ？

テストの後にするデバッグは、テストではなくデバッグだしねって
あんた何言ってるの？

**デフォルトの名無しさん** · 2020/07/06(月) 16:05:37.75

オブジェクト指向の話をしよう

彡ミ
↓↓↓
彡 ⌒ ミ
(´・ω・`) 頭皮、毛髪に触れるものは全て検査する

**デフォルトの名無しさん** · 2020/07/06(月) 17:22:23.35

>>74
そんな細かいこと気にすんなよ。揚げ足取りめ。
よく単体テストは不具合を見つけるためにやると言われるが、別視点の考え方もあるって話だ。

ここでは些細な話だったな。

**デフォルトの名無しさん** · 2020/07/06(月) 17:32:36.88

>>74
> テストの後にするデバッグは、テストではなくデバッグだしねって
> あんた何言ってるの？

テストの後にするデバッグはデバッグでしかないと思いますけど、どこに突っ込みを入れたいのですか？

**デフォルトの名無しさん** · 2020/07/06(月) 18:58:02.99

>>77
↓ここにツッコミを入れたい

> それはもはやテストではなくデバッグだしね。

**デフォルトの名無しさん** · 2020/07/06(月) 20:17:44.91

>>60
そうそれ
あれ書く奴とは1byteたりとも意見交換したくないわ

**デフォルトの名無しさん** · 2020/07/06(月) 22:24:42.23

まだプログラミング教室やってんのかw

**デフォルトの名無しさん** · 2020/07/07(火) 11:04:13.93

>>78
(テストとデバッグは違う作業だと思うのだが)

**デフォルトの名無しさん** · 2020/07/07(火) 11:33:22.19

もしかすると、バグの原因を調査する作業はテストに含まれるかどうかって話かな？

**デフォルトの名無しさん** · 2020/07/07(火) 11:34:20.85

和田メソッド被害者の会に入れてほしいです。

**デフォルトの名無しさん** · 2020/07/07(火) 11:40:38.80

>>81
当たり前だろ？
テストではなくデバッグだしねっていうのが意味わからんって言ってる
テストはテストだろ

**デフォルトの名無しさん** · 2020/07/07(火) 12:01:27.16

突っ込みどころねーじゃんw
それなのに喧嘩腰で突っ込んで周囲に突っ込まれただけか。

**デフォルトの名無しさん** · 2020/07/07(火) 12:04:20.99

privateメソッドのチェックもできない奴は出荷すんぞ

**デフォルトの名無しさん** · 2020/07/07(火) 12:25:55.22

privateのチェックって具体的にどうやるの？
Queueというクラスをテストするケースを例に教えて。

**デフォルトの名無しさん** · 2020/07/07(火) 12:31:21.40

queue = Queue()
queue.push(17)
asserEqual( queue.length, 1)
a = queue.pop()
assertEqual( a, 17)
assertEqual( quele.length, 0)

だいたいこんなもんだろ。

**デフォルトの名無しさん** · 2020/07/07(火) 12:37:00.92

>>88
おお、俺もそんなのイメージしてた。
そんなコードでpublic経由でprivateを呼び、ついでにカバレッジテストとか済ませる感じかな。

...だったらいいのだが、なーんか、このスレの人達の言動を見ていると怪しいんだよな。
88は別に問題ないけど。

**デフォルトの名無しさん** · 2020/07/07(火) 12:56:17.65

カバレッジテストをするかどうかは、ケースバイケース。どちらでもいいとして、一番気にしているのは、そもそも>>1の記事主に批判的な人はオブジェクト指向を理解しているのか？という点。
怪しいというのは、そこね。

**デフォルトの名無しさん** · 2020/07/07(火) 13:10:52.54

>>85
デバッグはテストじゃなくてデバッグだしねと言われても当たり前としか言えないし、
デバッグの前にやるテストはテストだしねで終わりだろ？

**デフォルトの名無しさん** · 2020/07/07(火) 13:14:04.43

>>91
言いたいことは何となく理解できる。
「私以外はみんな馬鹿」
ってことだろ？

つまりキミの主な言語はJava。

**デフォルトの名無しさん** · 2020/07/07(火) 13:19:42.71

どうやら図星だったようですね。

Javaではありがちなんですよ。

**デフォルトの名無しさん** · 2020/07/07(火) 13:28:08.57

>>91
とまぁ、>>92みたいな低能が蔓延るスレで世間の常識は通用しないってことが証明されちゃったわけだ。
立ち去るがいい。

**デフォルトの名無しさん** · 2020/07/07(火) 13:34:52.03

>>92
何がいいたいの（笑）
俺を馬鹿にしたいのかな？でも文章がそうなってないな。

**デフォルトの名無しさん** · 2020/07/07(火) 14:34:40.27

>>95
Java！Java！

**デフォルトの名無しさん** · 2020/07/07(火) 14:48:31.09

>>95
馬鹿にしてない。
馬鹿に付ける薬はない。

**デフォルトの名無しさん** · 2020/07/07(火) 14:56:56.51

オブジェクト指向はダメなんじゃないか？というのが前スレの趣旨で、非公開のメンバは何のためにあるの？テストどうするの？という話になった。

それに対する回答が「非公開のメンバはブラックボックスとして扱いテストしてはならない」という和田メソッドが示された。

それに対して「入力に使われる一つのメンバしかもたないパーサ」という実例を挙げ、「文字列を入力され構文木を返すような状態機械のテストが非常に困難」という話が出た。
この場合、内部を観測できないのであれば、すべての入力の組み合わせ（受理できない入力もテストするなら、それは無限である）に対して、すべての取りうる構文木のセットを検査しなければならない。
つまりそれは太陽系よりもはるかに広く、銀河の向こうまでテストするということである。

それに対して「オブジェクト指向を知らない」などと抜けたことを言うので、「Java！」という結論が出された。

**デフォルトの名無しさん** · 2020/07/07(火) 15:04:16.80

> それに対する回答が「非公開のメンバはブラックボックスとして扱いテストしてはならない」という和田メソッドが示された。

正確には

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

まとめ
繰り返すと、プライベートなメソッドや関数をテストする必要は無いと考えています。
プライベートなメソッドは、実装の詳細であるからです。

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

実装の詳細ならむしろテストが必要だと思うのだけど
なんで詳細だからテストしなくて良いのだろう
t_wada被害者の会に僕も入会させてください

**デフォルトの名無しさん** · 2020/07/07(火) 15:07:05.23

>>99
公開メンバは文字を入力するpush()のみと示されてるだろ。
複数の文字を入力して構文木を得る、ごくありふれた状態機械の話だよ。

公開メンバは一つしかない。

ほらテストしてみろ。

**デフォルトの名無しさん** · 2020/07/07(火) 15:07:48.37

>>98
> それに対して「入力に使われる一つのメンバしかもたないパーサ」という実例を挙げ、「文字列を入力され構文木を返すような状態機械のテストが非常に困難」という話が出た。
> この場合、内部を観測できないのであれば、すべての入力の組み合わせ（受理できない入力もテストするなら、それは無限である）に対して、すべての取りうる構文木のセットを検査しなければならない。

それはパブリックメソッドでも同じこと
内部を観測して、すべての入力の組み合わせをテストしないといけないから

**デフォルトの名無しさん** · 2020/07/07(火) 15:08:26.27

>>101
> ほらテストしてみろ。

テストしてもいいが、先にパブリックメソッドの場合の例を書いてくれ
それと同じことをやればいいだけなんだが

**デフォルトの名無しさん** · 2020/07/07(火) 15:10:39.95

単体テストっていう言い方がまずいのかも
インターフェーステストとインプリメンテーションテストに分けるのが良い気がします

**デフォルトの名無しさん** · 2020/07/07(火) 15:11:55.03

言い張るんだったら、ほんとに和田メソッド被害者の会作っちゃうよ？
会員200万人目指しちゃうよ？
いいの？

**デフォルトの名無しさん** · 2020/07/07(火) 15:12:43.32

>>105
先にパブリックメソッドの場合に
どうするかを答えてくれないか？

**デフォルトの名無しさん** · 2020/07/07(火) 15:12:43.34

>>105
言いたいことわかってくれるあなたは、PythonとかC#とかC++ですね？

**デフォルトの名無しさん** · 2020/07/07(火) 15:13:07.34

自作自演ｗ

105 名前：デフォルトの名無しさん[] 投稿日：2020/07/07(火) 15:11:55.03 ID:zTLocdwC [8/9]
言い張るんだったら、ほんとに和田メソッド被害者の会作っちゃうよ？
会員200万人目指しちゃうよ？
いいの？

107 名前：デフォルトの名無しさん[] 投稿日：2020/07/07(火) 15:12:43.34 ID:zTLocdwC [9/9]
>>105
言いたいことわかってくれるあなたは、PythonとかC#とかC++ですね？

**デフォルトの名無しさん** · 2020/07/07(火) 15:13:54.75

王家の秘伝メソッド普及委員会を立ち上げよう

**デフォルトの名無しさん** · 2020/07/07(火) 15:14:08.44

>>108
>>104 に言いたかっただけだろ。
そのくらいわかれ。

**デフォルトの名無しさん** · 2020/07/07(火) 15:15:25.23

>>110
そんなどうでもいいものばっかりにレスしてないで、
先にパブリックメソッドの場合に
どうするかを答えてくれないか？

**デフォルトの名無しさん** · 2020/07/07(火) 15:15:52.28

Javaは色々研究して提唱してえらいなあと思う部分もあるんだけど、すぐに宗教化してしまうからな。
そこがJava！なんだよな。

**デフォルトの名無しさん** · 2020/07/07(火) 15:17:07.58

どうも都合が悪かったようだな
俺のレスに答えられずひたすら関係ない話を始めた

**デフォルトの名無しさん** · 2020/07/07(火) 15:18:08.09

>>108
この自作自演は恥ずかしいなww

**デフォルトの名無しさん** · 2020/07/07(火) 15:20:13.83

でも俺が設立するからには、普通じゃダメなんだよ。
俺がやるからには、和田メソッド被害者の会長が和田さん。
ここまでやってこそ本物だと思うんだよね。
まず、和田さんを説き伏せなきゃ。

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

いいだしっぺが説き伏せるそうです。
実際に行動を起こせるか見てみましょう

**デフォルトの名無しさん** · 2020/07/07(火) 15:22:05.58

和田さん頼むわ。
見てるんやろ？

**デフォルトの名無しさん** · 2020/07/07(火) 15:22:43.01

いきなり他人だよりです（笑）
自分から知らせに行く勇気もないようですね

**デフォルトの名無しさん** · 2020/07/07(火) 15:24:58.29

>>104 をベースに考察していけば良いのでは。

**デフォルトの名無しさん** · 2020/07/07(火) 15:26:46.74

ではまずいいだしっぺから
なぜパブリックメソッドにすると
すべての入力の組み合わせをテストしなくてよくなるのか？
C1カバレッジって言葉知ってますかね？

**デフォルトの名無しさん** · 2020/07/07(火) 15:27:35.56

訂正、すべての入力の組み合わせは2カバレッジだった

**デフォルトの名無しさん** · 2020/07/07(火) 15:27:50.82

C2カバレッジだった

**デフォルトの名無しさん** · 2020/07/07(火) 15:39:26.39

>>87
一般的にはQueueはprivateのメソッドも単純なロジックしか持たないから
public経由のテストで十分な場合が多いかもしれないけど
それがあらゆる状況に当てはまるわけではない

例えばgrowableなring bufferでqueueを実装するとして
バッファを拡大させるロジックに独自の最適化をいろいろと施してるような場合、
各種分岐に応じたメモリコピーの方法だったり、それに応じたメモリの初期化状態の確認だったり
privateなhead/tailポインタへのアクセスだったり、
public経由だけでは確認できないテストをしたほうがいい場合がある

**デフォルトの名無しさん** · 2020/07/07(火) 15:42:28.40

>>123
訂正

一般的にはQueueはprivateのメソッドも単純なロジックしか持たないから
public経由のテストで十分な場合が多いかもしれないけど
それがあらゆる状況に当てはまるわけではない

もしpublic経由のテストで不十分な場合、それはメソッドの責務が大きいと考えられる
リファクタリングして複数のクラスに分離するのが正しい対応
そうすれば自然publicメソッドとなりテストが可能になる

**デフォルトの名無しさん** · 2020/07/07(火) 15:53:43.14

キミも和田メソッドの被害者なんだよ。
可哀そうに。
和田さん本人だとしても、被害者であることに変わりはないんだよ。

**デフォルトの名無しさん** · 2020/07/07(火) 15:59:04.01

あれ、説き伏せるんじゃないの？

**デフォルトの名無しさん** · 2020/07/07(火) 16:00:23.93

もう和田さんは改心してこっちの味方だから。