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

■ このスレッドは過去ログ倉庫に格納されています
2020/07/05(日) 20:47:46.60ID:M+BkbwUs
■短い回答
プライベートをテストしたい場合は設計に問題があるので、パブリックに変更してテストしましょう

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

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

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

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

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

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

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

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

別クラスのパブリックメソッドとする
プライベートなメソッドのテストを書きたいということは、実はテスト対象の責務が多すぎることを示唆している場合があります。
テストがどうしても書きたい場合は、その責務はテスト対象のプライベートな振る舞いではなく、他の誰かのパブリックな振る舞いなのでしょう。
2020/07/06(月) 02:20:25.77ID:T074ZQpk
>>47
うるせーばか。文句を言うために都合の良いデータを調べてきたに決まってるだろ
2020/07/06(月) 07:14:05.86ID:EsDpeV3E
一気に胡散臭くなったな
2020/07/06(月) 07:37:12.27ID:ODctk1XU
>>50
仕様書には普通アクセス修飾子を何にするかなんて書かないからね
設計書にもそんな細かいこと書かないでしょ
2020/07/06(月) 07:38:08.27ID:cDMo9043
書いてない関数を勝手に作るなんて禁止に決まってるだろ
テストのためだけに関数は作らない
2020/07/06(月) 07:59:18.67ID:ODctk1XU
>>54
すごい、そんなガチガチなの銀行系？
2020/07/06(月) 08:00:23.42ID:ODctk1XU
関数を作るのに許可がいるなんてすごいことだからとてもすごいと思いました
2020/07/06(月) 09:07:12.86ID:aZiq8mUC
>>1にはテストとしか書かれていないけど、記事元はユニットテスト(単体テスト)の話だよね？
2020/07/06(月) 10:09:49.30ID:Smcv6rvz
組み込みでの話らしいよここw
2020/07/06(月) 10:19:56.49ID:cDMo9043
>>57
ユニットテスト以外でメソッドのテストなんかするのか？
2020/07/06(月) 12:11:06.79ID:O9yimTr2
>>3
このコード出すやつが混じってるんだろ？
議論無駄やん
2020/07/06(月) 12:27:52.10ID:aZiq8mUC
>>59
自分であたかも他のテストがあるように言っておいて変な話だがだが、しないね。

単体テストで不具合を見つけた後、デバッガを使って更にどこにバグの原因が潜んでいるのか分析することはあるけど、それはもはやテストではなくデバッグだしね。
2020/07/06(月) 12:34:18.82ID:aZiq8mUC
てかさ、タイトルがカプセル化なのに、なんで>>3も>>5もテストコード実行場所とテストコード実行場所が同じ階層に記述されているのさ。
2020/07/06(月) 12:35:06.14ID:aZiq8mUC
ミス。テストコード実行場所とテスト対象が同じ階層に書かれているのはなんで？だ。
64デフォルトの名無しさん
垢版 |
2020/07/06(月) 14:43:00.46ID:NTI0gQuc
和田理論は穴が多くてお話にならないな。
2020/07/06(月) 14:44:50.89ID:40IRbGG5
では穴のない理論をおなしゃす
66デフォルトの名無しさん
垢版 |
2020/07/06(月) 14:51:45.46ID:NTI0gQuc
和田理論に騙された人は抗議したほうが良い。
2020/07/06(月) 14:54:43.27ID:EsDpeV3E
wadaは忘れろ

privateだからテストしなくていいなどという都合のいい法則は存在しない だがprivateだからとテストをしないキチガイは確実に存在するのだ
68デフォルトの名無しさん
垢版 |
2020/07/06(月) 15:05:56.82ID:NTI0gQuc
和田理論被害者の会ひつよう。
とてもひつよう。
2020/07/06(月) 15:07:12.95ID:bo8iccWD
一意見を急に”理論”とか”法則”とか言う方が頭どうかしてるよ

t-wadaはTDDをわかりやすく解説することに定評があるだけ
ただいろんな会社からテストのコンサルティングで雇われる程度には有能だから
君たちの意見よりは一般には受け入れられやすい
70デフォルトの名無しさん
垢版 |
2020/07/06(月) 15:10:07.70ID:NTI0gQuc
和田理論被害者の会ニューヨーク支部もひつよう。
とてもひつよう。
2020/07/06(月) 15:12:35.36ID:Smcv6rvz
このスレ、脱線が好きだなw
72デフォルトの名無しさん
垢版 |
2020/07/06(月) 15:14:01.90ID:NTI0gQuc
前スレで王家秘伝のレシピ教えたのに。
誰も活用しないんだな。
73デフォルトの名無しさん
垢版 |
2020/07/06(月) 15:17:52.38ID:NTI0gQuc
俺も活用していないわそういえば。
2020/07/06(月) 15:18:30.87ID:9+hHOd2F
>>61
> 単体テストで不具合を見つけた後、デバッガを使って更にどこにバグの原因が潜んでいるのか分析することはあるけど、それはもはやテストではなくデバッグだしね。

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

だろ？

テストの後にするデバッグは、テストではなくデバッグだしねって
あんた何言ってるの？
2020/07/06(月) 16:05:37.75ID:n2uckING
オブジェクト指向の話をしよう

彡ミ
↓↓↓
彡 ⌒ ミ
(´・ω・`) 頭皮、毛髪に触れるものは全て検査する
2020/07/06(月) 17:22:23.35ID:aZiq8mUC
>>74
そんな細かいこと気にすんなよ。揚げ足取りめ。
よく単体テストは不具合を見つけるためにやると言われるが、別視点の考え方もあるって話だ。

ここでは些細な話だったな。
2020/07/06(月) 17:32:36.88ID:zkbL7M3i
>>74
> テストの後にするデバッグは、テストではなくデバッグだしねって
> あんた何言ってるの？

テストの後にするデバッグはデバッグでしかないと思いますけど、どこに突っ込みを入れたいのですか？
2020/07/06(月) 18:58:02.99ID:9+hHOd2F
>>77
↓ここにツッコミを入れたい

> それはもはやテストではなくデバッグだしね。
2020/07/06(月) 20:17:44.91ID:pO1U5uVJ
>>60
そうそれ
あれ書く奴とは1byteたりとも意見交換したくないわ
80デフォルトの名無しさん
垢版 |
2020/07/06(月) 22:24:42.23ID:djm9krVS
まだプログラミング教室やってんのかw
2020/07/07(火) 11:04:13.93ID:hnvy2tUT
>>78
(テストとデバッグは違う作業だと思うのだが)
2020/07/07(火) 11:33:22.19ID:hnvy2tUT
もしかすると、バグの原因を調査する作業はテストに含まれるかどうかって話かな？
83デフォルトの名無しさん
垢版 |
2020/07/07(火) 11:34:20.85ID:zTLocdwC
和田メソッド被害者の会に入れてほしいです。
2020/07/07(火) 11:40:38.80ID:YBf2Aagc
>>81
当たり前だろ？
テストではなくデバッグだしねっていうのが意味わからんって言ってる
テストはテストだろ
2020/07/07(火) 12:01:27.16ID:hnvy2tUT
突っ込みどころねーじゃんw
それなのに喧嘩腰で突っ込んで周囲に突っ込まれただけか。
2020/07/07(火) 12:04:20.99ID:bdixmHft
privateメソッドのチェックもできない奴は出荷すんぞ
2020/07/07(火) 12:25:55.22ID:NlNPNtxk
privateのチェックって具体的にどうやるの？
Queueというクラスをテストするケースを例に教えて。
2020/07/07(火) 12:31:21.40ID:jam8vymb
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.92ID:NlNPNtxk
>>88
おお、俺もそんなのイメージしてた。
そんなコードでpublic経由でprivateを呼び、ついでにカバレッジテストとか済ませる感じかな。

...だったらいいのだが、なーんか、このスレの人達の言動を見ていると怪しいんだよな。
88は別に問題ないけど。
2020/07/07(火) 12:56:17.65ID:NlNPNtxk
カバレッジテストをするかどうかは、ケースバイケース。どちらでもいいとして、一番気にしているのは、そもそも>>1の記事主に批判的な人はオブジェクト指向を理解しているのか？という点。
怪しいというのは、そこね。
2020/07/07(火) 13:10:52.54ID:YBf2Aagc
>>85
デバッグはテストじゃなくてデバッグだしねと言われても当たり前としか言えないし、
デバッグの前にやるテストはテストだしねで終わりだろ？
92デフォルトの名無しさん
垢版 |
2020/07/07(火) 13:14:04.43ID:zTLocdwC
>>91
言いたいことは何となく理解できる。
「私以外はみんな馬鹿」
ってことだろ？

つまりキミの主な言語はJava。
93デフォルトの名無しさん
垢版 |
2020/07/07(火) 13:19:42.71ID:zTLocdwC
どうやら図星だったようですね。

Javaではありがちなんですよ。
2020/07/07(火) 13:28:08.57ID:RJk/Rw+W
>>91
とまぁ、>>92みたいな低能が蔓延るスレで世間の常識は通用しないってことが証明されちゃったわけだ。
立ち去るがいい。
2020/07/07(火) 13:34:52.03ID:YBf2Aagc
>>92
何がいいたいの（笑）
俺を馬鹿にしたいのかな？でも文章がそうなってないな。
96デフォルトの名無しさん
垢版 |
2020/07/07(火) 14:34:40.27ID:zTLocdwC
>>95
Java！Java！
97デフォルトの名無しさん
垢版 |
2020/07/07(火) 14:48:31.09ID:zTLocdwC
>>95
馬鹿にしてない。
馬鹿に付ける薬はない。
98デフォルトの名無しさん
垢版 |
2020/07/07(火) 14:56:56.51ID:zTLocdwC
オブジェクト指向はダメなんじゃないか？というのが前スレの趣旨で、非公開のメンバは何のためにあるの？テストどうするの？という話になった。

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

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

それに対して「オブジェクト指向を知らない」などと抜けたことを言うので、「Java！」という結論が出された。
2020/07/07(火) 15:04:16.80ID:YBf2Aagc
> それに対する回答が「非公開のメンバはブラックボックスとして扱いテストしてはならない」という和田メソッドが示された。

正確には

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

まとめ
繰り返すと、プライベートなメソッドや関数をテストする必要は無いと考えています。
プライベートなメソッドは、実装の詳細であるからです。
100デフォルトの名無しさん
垢版 |
2020/07/07(火) 15:05:58.19ID:O+C/kGAG
実装の詳細ならむしろテストが必要だと思うのだけど
なんで詳細だからテストしなくて良いのだろう
t_wada被害者の会に僕も入会させてください
101デフォルトの名無しさん
垢版 |
2020/07/07(火) 15:07:05.23ID:zTLocdwC
>>99
公開メンバは文字を入力するpush()のみと示されてるだろ。
複数の文字を入力して構文木を得る、ごくありふれた状態機械の話だよ。

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

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

それはパブリックメソッドでも同じこと
内部を観測して、すべての入力の組み合わせをテストしないといけないから
2020/07/07(火) 15:08:26.27ID:YBf2Aagc
>>101
> ほらテストしてみろ。

テストしてもいいが、先にパブリックメソッドの場合の例を書いてくれ
それと同じことをやればいいだけなんだが
104デフォルトの名無しさん
垢版 |
2020/07/07(火) 15:10:39.95ID:O+C/kGAG
単体テストっていう言い方がまずいのかも
インターフェーステストとインプリメンテーションテストに分けるのが良い気がします
105デフォルトの名無しさん
垢版 |
2020/07/07(火) 15:11:55.03ID:zTLocdwC
言い張るんだったら、ほんとに和田メソッド被害者の会作っちゃうよ？
会員200万人目指しちゃうよ？
いいの？
2020/07/07(火) 15:12:43.32ID:YBf2Aagc
>>105
先にパブリックメソッドの場合に
どうするかを答えてくれないか？
107デフォルトの名無しさん
垢版 |
2020/07/07(火) 15:12:43.34ID:zTLocdwC
>>105
言いたいことわかってくれるあなたは、PythonとかC#とかC++ですね？
2020/07/07(火) 15:13:07.34ID:YBf2Aagc
自作自演ｗ

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++ですね？
109デフォルトの名無しさん
垢版 |
2020/07/07(火) 15:13:54.75ID:O+C/kGAG
王家の秘伝メソッド普及委員会を立ち上げよう
110デフォルトの名無しさん
垢版 |
2020/07/07(火) 15:14:08.44ID:zTLocdwC
>>108
>>104 に言いたかっただけだろ。
そのくらいわかれ。
2020/07/07(火) 15:15:25.23ID:YBf2Aagc
>>110
そんなどうでもいいものばっかりにレスしてないで、
先にパブリックメソッドの場合に
どうするかを答えてくれないか？
112デフォルトの名無しさん
垢版 |
2020/07/07(火) 15:15:52.28ID:zTLocdwC
Javaは色々研究して提唱してえらいなあと思う部分もあるんだけど、すぐに宗教化してしまうからな。
そこがJava！なんだよな。
2020/07/07(火) 15:17:07.58ID:YBf2Aagc
どうも都合が悪かったようだな
俺のレスに答えられずひたすら関係ない話を始めた
2020/07/07(火) 15:18:08.09ID:pnFsfYkB
>>108
この自作自演は恥ずかしいなww
115デフォルトの名無しさん
垢版 |
2020/07/07(火) 15:20:13.83ID:zTLocdwC
でも俺が設立するからには、普通じゃダメなんだよ。
俺がやるからには、和田メソッド被害者の会長が和田さん。
ここまでやってこそ本物だと思うんだよね。
まず、和田さんを説き伏せなきゃ。
2020/07/07(火) 15:21:24.14ID:YBf2Aagc
いいだしっぺが説き伏せるそうです。
実際に行動を起こせるか見てみましょう
117デフォルトの名無しさん
垢版 |
2020/07/07(火) 15:22:05.58ID:zTLocdwC
和田さん頼むわ。
見てるんやろ？
2020/07/07(火) 15:22:43.01ID:YBf2Aagc
いきなり他人だよりです（笑）
自分から知らせに行く勇気もないようですね
119デフォルトの名無しさん
垢版 |
2020/07/07(火) 15:24:58.29ID:zTLocdwC
>>104 をベースに考察していけば良いのでは。
2020/07/07(火) 15:26:46.74ID:YBf2Aagc
ではまずいいだしっぺから
なぜパブリックメソッドにすると
すべての入力の組み合わせをテストしなくてよくなるのか？
C1カバレッジって言葉知ってますかね？
2020/07/07(火) 15:27:35.56ID:YBf2Aagc
訂正、すべての入力の組み合わせは2カバレッジだった
2020/07/07(火) 15:27:50.82ID:YBf2Aagc
C2カバレッジだった
2020/07/07(火) 15:39:26.39ID:K/dtFkkl
>>87
一般的にはQueueはprivateのメソッドも単純なロジックしか持たないから
public経由のテストで十分な場合が多いかもしれないけど
それがあらゆる状況に当てはまるわけではない

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

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

もしpublic経由のテストで不十分な場合、それはメソッドの責務が大きいと考えられる
リファクタリングして複数のクラスに分離するのが正しい対応
そうすれば自然publicメソッドとなりテストが可能になる
125デフォルトの名無しさん
垢版 |
2020/07/07(火) 15:53:43.14ID:zTLocdwC
キミも和田メソッドの被害者なんだよ。
可哀そうに。
和田さん本人だとしても、被害者であることに変わりはないんだよ。
2020/07/07(火) 15:59:04.01ID:pnFsfYkB
あれ、説き伏せるんじゃないの？
127デフォルトの名無しさん
垢版 |
2020/07/07(火) 16:00:23.93ID:zTLocdwC
もう和田さんは改心してこっちの味方だから。
2020/07/07(火) 16:09:18.16ID:bdixmHft
ごちゃごちゃうるさい
privateメソッドのテストやれよ
キチガイ
2020/07/07(火) 16:11:54.35ID:cp13iBTj
虚言癖まであるのかな
130デフォルトの名無しさん
垢版 |
2020/07/07(火) 16:15:15.28ID:zTLocdwC
和田メソッド被害者の会、会員募集中。
和田さんも入会して良いんだヨ。
131デフォルトの名無しさん
垢版 |
2020/07/07(火) 17:39:48.25ID:O+C/kGAG
>>123
君は信頼できる
実装をよくわかってる人だ
2020/07/07(火) 17:43:14.92ID:jam8vymb
可視性狂信者ってのはどうしようもないね。テストさえ犠牲にし始める。
13394
垢版 |
2020/07/07(火) 17:55:40.01ID:k9dX6gDH
>>95
今さらだけど、前スレのタイトル(今より酷い)から察せると思うが、あなたの想像を越えるお馬鹿さんがいるから、常識を持つあなたはここから逃げた方がいいよと、お前どっちの味方だよ風に言いたかった。
2020/07/07(火) 18:13:36.60ID:k9dX6gDH
って、何でもないです。
(アンカが自分に向いていたと勘違いしてたとか言えねぇ)
2020/07/07(火) 18:55:11.35ID:B0OaeN7n
>>130
むしろ、こんな気違いに絡まれる和田さんカワイソ
2020/07/07(火) 19:10:49.50ID:BPFgnJg7
結局実際に絡む度胸もないから無害やけどなw
2020/07/07(火) 19:21:49.86ID:zm6NAQZG
>>128
君の言うテストって何？
しかも、君は>>87の質問に答えてないよね？
具体例示してよ。
2020/07/07(火) 19:26:55.85ID:bdixmHft
>>137
は？いいからprivateメソッドのテストやれよ
2020/07/07(火) 19:27:35.46ID:zm6NAQZG
オブジェクト指向を理解している人間の言うprivateをテストせよ/テストするべきではない と、 オブジェクト指向を理解していない人間のprivateをテストせよ/テストするべきでない は論争のレベルが全然違う。
大学生同士の論争と小学生同士の喧嘩くらい違う。

お前はどっちだ？>>138
2020/07/07(火) 19:28:44.77ID:bdixmHft
>>139
ごちゃごちゃうるさい
privateメソッドのテストをやれ
2020/07/07(火) 19:32:38.84ID:bdixmHft
ここで話を整理するけど

privateメソッドだからテストしないとか言ってるやつはキチガイ
早く死んでね
2020/07/07(火) 19:33:11.12ID:zm6NAQZG
具体例もなく、privateのテストやれと言われてもできねーよ、バーカ。
2020/07/07(火) 19:35:50.42ID:bdixmHft
うんうん

わかってない人もいるみたいだからもう一度ここで仕切り直しさせてほしい



privateメソッドだからテストしないとか言ってるやつはキチガイ
早く死んでね



わかったかな？
2020/07/07(火) 19:36:30.14ID:OvditBd9
ほら、>>143は巣に帰りなさい。

【隔離】オブジェクト指向アンチスレ
https://mevius.5ch.net/test/read.cgi/tech/1593926155/
2020/07/07(火) 19:37:40.45ID:rZMsiuSb
頭皮に触れるものは全て検査する

彡ミ
↓↓↓
彡 ⌒ ミ
(´・ω・`) 使ってるシャンプー、肌に合ってるかい？
2020/07/07(火) 19:44:10.79ID:rZMsiuSb
［ケハエール］


彡 ⌒ ミ
(´・ω・`) 確認もせず使うのは馬鹿だ
2020/07/07(火) 22:01:24.32ID:YBf2Aagc
>>143
> privateメソッドだからテストしないとか言ってるやつはキチガイ
だれそれ？

privateメソッドはpublicメソッド経由でテストすると
言ってる人しかいないけど？
2020/07/07(火) 22:08:08.37ID:O+C/kGAG
>>3の下痢便コード書いた人ですよね？
2020/07/07(火) 22:11:43.28ID:bdixmHft
>>147
それテストできてねーからｗ
publicから t=0のときだけprivateの処理が欲しかったとするじゃん？
そのままじゃpublicは仕様でt=0のときしかそのprivateメソッドを呼ばないんだから
そのpublicからしかテストしないんじゃt≠0のときのテストできねーじゃん
やんねーのかよ？
ある時改修で別のpublicメソッドからt=3のときにprivateメソッドを呼ぶことになったらt=0しかテストやってなかったら
普通は怒り狂うもんだよ
っていうか絶対ブッ殺す
2020/07/07(火) 22:27:58.06ID:YBf2Aagc
>>149

> そのままじゃpublicは仕様でt=0のときしかそのprivateメソッドを呼ばないんだから

つまり、publicがt=0のときprivateメソッドを呼ぶんですよね？
publicがt=0のテストをすれば、privateのテストしてるじゃないですかｗ
■ このスレッドは過去ログ倉庫に格納されています
5ちゃんねるの広告が気に入らない場合は、こちらをクリックしてください。