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

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

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

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

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

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

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

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

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

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

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

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

**デフォルトの名無しさん** · 2020/07/12(日) 18:53:56.08

>>366
なるほど、つまりTDDはテストが開発を駆動するという考え方で
テストによって実装や設計が導かれるということなのだけれども
設計過剰だったり設計不足だったりする
設計は本来ドメインに関する知識から導かれるものなので
入力、出力を決めたら自動的に最適な設計が見つかりますなんて
そんな都合の良いことが起こるわけもなくTDDは現代では否定されていて
単体テストよりもシステムテストを重視したほうが製品の品質はあがるわけですな

**デフォルトの名無しさん** · 2020/07/12(日) 18:54:49.46

説明に対して全く聞く耳を持たず、和田経典に書いてあると繰り返すだけだから、宗教的なナニカ扱いされてる。

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

t-wadaがお勧めする和田メソッド。

騙されたやつ多いのでは。

**デフォルトの名無しさん** · 2020/07/12(日) 19:10:42.57

>>367
> TDDは現代では否定されていて

どこで否定されてるの？さらっとウソを付かないように。
どうせすぐに俺がバラすんだから意味ないよ（笑）

**デフォルトの名無しさん** · 2020/07/12(日) 19:20:59.87

どこで否定されてるのとか、これまでのみんなの説明が全くの無駄。

和田さんには和田さんの経典があり、みんなの意見には耳を貸さない。

だから宗教と言われてる。

**デフォルトの名無しさん** · 2020/07/12(日) 19:22:03.23

前スレですばらしい分析を見たなあ

897+1 ：デフォルトの名無しさん [↓] ：2020/07/05(日) 11:28:59.86 ID:YdQ981ul
>>870
新しい理論や手法を提唱する人には良くあることだと思うけど、その理論はある前提、ある側面では正しいけど、常に無条件に適用することは正しくないと本人は分かっていて、敢えてそういうことはわざわざ詳しくは説明しない。自分の論が有用な物だと主張したいがため、嘘はつかない範囲で相手が勝手に誤解してすごいと思わせるような物言いをする。
で、その理論に感銘を受けた人の一部が、理論の表面的な効能だけをありがたく受け取って問題点や前提条件などは正しく理解しないまま、受け売りの知識を他所で披露する。
そこで議論になると、本質をちゃんと理解してないまま自説を擁護しようとするから、無理が生じたり話が噛み合わなかったりする。
という流れでイマココなのかなと思う。

◆QZaw55cn4c · 2020/07/12(日) 19:58:09.07

>>372
昔の言い方でいうところの「東大話法」というやつですね…

**デフォルトの名無しさん** · 2020/07/13(月) 00:50:54.57

>>372
素晴らしい分析？それはTDDに関しての分析じゃないよ
その文章はTDDに関して何一つ言及していない。

**デフォルトの名無しさん** · 2020/07/13(月) 00:54:17.24

>>371
ここに和田さんはいないんだから、意見を聞かないはずだという逃げの言葉で
お前の意見を言わないのはおかしい。

他人が意見を聞くかは関係なく、
お前の意見を言えばいい。

言えないなら、反論するすべがないというのがお前に対する結論だ。

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

>>1 がアスペのスレ

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

和田メソッドの弊害

**デフォルトの名無しさん** · 2020/07/13(月) 01:36:28.61

と繰り返えすだけ

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

>>375
イイエ居ます。
みんな説明してます。
見えないことにするなら、みんなの書き込みが無駄です。

和田メソッドはカルト宗教の様相を呈してきた。

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

和田メソッドのせいで日本のITが10年遅れた。

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

和田メソッドって単なるTDDだぞ？

**デフォルトの名無しさん** · 2020/07/13(月) 08:39:56.06

イイエ違います。
和田メソッドは宗教です。

**デフォルトの名無しさん** · 2020/07/13(月) 08:41:17.44

和田さんはこのスレに居ないと書き込んでるのが御本人かもしれない。

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

ステマスレ。

**デフォルトの名無しさん** · 2020/07/16(木) 12:32:21.61

失速したな
TDDはプロトタイピングと組み合わせると良いかもね
テスト書かずに概ね正確に動作するコードを書いた後に
ブラッシュアップする目的でTDDを使って書き直す感じ

**デフォルトの名無しさん** · 2020/07/16(木) 12:34:19.44

TDDを布教する人が何回も同じコード書いてうまくなっていくように
書き捨てたコードの数だけTDDは洗練されるんじゃなかろうかと

**デフォルトの名無しさん** · 2020/07/16(木) 13:59:03.56

和田メソッドのせいで失われた日本ITの20年。

**デフォルトの名無しさん** · 2020/07/16(木) 15:09:03.31

和田さんじゃなくてTDDの話をしよう
詳しい人いないの？

**デフォルトの名無しさん** · 2020/07/16(木) 15:51:48.48

テスト書くタイミングについてなら少しだけ。
わりとだら～と書いて関数なりクラスなり切り出したタイミングで書く感じ。
テストから書くってのは普通の人には難易度高い気がするわ。

**デフォルトの名無しさん** · 2020/07/17(金) 02:41:00.28

最終的なプログラムの詳細設計がパッと頭に浮かぶ人ならテストから書けるんだろうけど
プロトタイプぐちゃぐちゃ作って、やっと構造が決まって、とりあえず動くもの作った後でリファクタリング、な俺には無理だなぁ

**デフォルトの名無しさん** · 2020/07/17(金) 11:27:59.95

>>390
そういうやり方にこそTDD的なアプローチが有効だと思うが

**デフォルトの名無しさん** · 2020/07/17(金) 20:03:07.08

>>390
構造が決まった時に設計が始まるんじゃないのか?

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

和田メソッドの弊害

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

https://i.imgur.com/azm6Xga.jpg

**デフォルトの名無しさん** · 2020/07/20(月) 18:19:36.42

リフレクション使え
終わり

**デフォルトの名無しさん** · 2020/07/24(金) 00:52:33.29

https://cdnegc.trafficfactory.biz/banners/41/ac/bc/67d6672dc5a49cb31207f1e7c17556e2.gif

**デフォルトの名無しさん** · 2020/07/26(日) 01:00:24.09

相変わらずオブジェクト指向信じて
頑張ってるJavaプログラマさん達に
お伺いしたいんだけどさぁ

Javaでオブジェクトをsystem.out.printしても
変な文字列出てくるだけでオブジェクトの中身見れないよね？
これってあんたらの好きな「カプセル化」のせいでしょ？
これ新人にはめっちゃ不人気でみんなJavaディスってんだよね。

一方JavaScriptやPythonは
オブジェクトをconsole.logするとオブジェクトの中身
全部一目瞭然に見れるし▶ボタンで折りたたみとかも出来
ちゃうわけよ。
Pythonは折り畳めないけどそれでも見やすいよ。

こんな簡単な揺るがぬ真実を説明するのに
「コンストラクタ」とか「ポリモーフィズム」とか
「メソッド」「インタフェース」「継承」
「オーバーライド」こんなクソみたいな単語を一切使わなくても
簡単に説明できるわけ。

これでJavaScriptやPythonはイケてる言語で
Javaやオブジェクト思考は糞だって事が分かったね。
誰でも簡単に理解できるよね。

**デフォルトの名無しさん** · 2020/07/26(日) 01:16:15.62

おっ、そうだな

**デフォルトの名無しさん** · 2020/07/26(日) 01:16:16.56

わかりやすいREPLが使える言語のほうが学習効率が高いのは確かだが
標準でオブジェクトの中身がprintされるかどうかはカプセル化とは関係ない

**デフォルトの名無しさん** · 2020/07/26(日) 02:14:47.15

p obj

Ruby なら、これでインスタンスの内容が分かる

この時に、Object ID みたいな謎の文字が表示されて困る場合は、
Object#inspect をオーバーライドして、好きな内容に変える

これは、たいていの言語の「Effective 何々」という本に書いてある。
まず最初に、表示メッセージを分かりやすいように、オーバーライドしろって

**デフォルトの名無しさん** · 2020/07/26(日) 10:27:42.72

>>397
> Javaでオブジェクトをsystem.out.printしても
> 変な文字列出てくるだけでオブジェクトの中身見れないよね？

JavaScriptのオブジェクトは連想配列

というわけで、Javaで連想配列を作ってprintしてみた
https://paiza.io/projects/OnW9KR89v5m-CY5JXIV2tw

> これってあんたらの好きな「カプセル化」のせいでしょ？

オブジェクトをprintしたときはそのオブジェクトのtoStringメソッドが呼ばれる
これは「ポリモーフィズム」のおかげ

toStringメソッドはObjectクラスで宣言されている

すべてのオブジェクトはObjectクラスを「継承」しているため
toStringメソッドを「オーバーライド」して任意の値を返却できる

「カプセル化」は関係ない

> 一方JavaScriptやPythonは
> オブジェクトをconsole.logするとオブジェクトの中身
> 全部一目瞭然に見れるし?ボタンで折りたたみとかも出来
> ちゃうわけよ。

すてきなデバッガです

**デフォルトの名無しさん** · 2020/07/26(日) 10:36:11.38

PythonやJavaScriptがイケてる言語であることは間違いない
Javaよりも開発者多いのじゃないかな

PythonはAIで使われているし大学の授業でも使われていて
それまでC言語で教えてた内容をPythonで教えるようにしたら
学生の成績が飛躍的に上がったという話がある
人間の認知能力に最適な言語

JavaScriptはWebのクライアントでもサーバでも使われていて
日進月歩で進化している最も熱い言語

**デフォルトの名無しさん** · 2020/07/26(日) 10:45:55.24

Javaは昔はオブジェクト指向一筋の堅物みたいな感じだったけど
最近はラムダ式やStream、CompletableFutureの導入など
どちらかというと関数型の方に傾倒していて、オブジェクト指向と関数型の
良いとこ取りをしようとしてる

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

javaに限った話ではない

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

そもそも単体で平置き出来る関数が出来るまでバカみたいに長い年月が掛かったからその間に他の言語に抜かされていった

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

オブジェクト指向もテストの文脈で語られるようになって、
だいぶ胡散臭い馬鹿が減ったと思う。

**デフォルトの名無しさん** · 2020/07/27(月) 13:33:10.26

胡散臭くない小奇麗なバカは減って無い、ってことだな

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

論理的には間違ってる

**デフォルトの名無しさん** · 2020/07/27(月) 21:43:04.96

>>405
「体で平置き出来る関数」ってどんなの？

**デフォルトの名無しさん** · 2020/07/27(月) 21:54:44.65

TDDに限定するならともかく、一般論として「テストの文脈で」語ろうとするのは
逆にそっちの方が胡散臭いなぁ。

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

「体で平置き」とか体育会系っぽいな

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

平置き出来る関数って何？

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

第一級関数の事じゃね

**デフォルトの名無しさん** · 2020/07/29(水) 00:11:33.88

class/object instance scopeにあるmethodではない関数または関数objectのことではないかいな

**デフォルトの名無しさん** · 2020/07/29(水) 02:31:20.73

無能は言葉を知らない

**デフォルトの名無しさん** · 2020/07/29(水) 07:34:13.23

俺様定義でしゃべるヤツが多いけどな

**デフォルトの名無しさん** · 2020/07/29(水) 07:49:48.28

「平置き出来る関数」なんてどこの村で通じる方言なん？

**デフォルトの名無しさん** · 2020/07/29(水) 08:44:34.08

文脈でわかると思うけどね
自分の辞書にないからわからないというやつはただのアホ

**デフォルトの名無しさん** · 2020/07/29(水) 08:52:18.07

慎吾ママが両手を広げて「おっはー」と言ったらおはようの意味だとわかるし
貴乃花親方がドアを開けて「あーす」と言ったらおはようございますの意味だとわかるだろ
「平置きできる関数」がわからない人は自分が八角理事長だと自覚したが良い

**デフォルトの名無しさん** · 2020/07/29(水) 09:00:12.23

その流れなら「ひっらー」だろ
平沖出来る関数は

**デフォルトの名無しさん** · 2020/07/29(水) 10:11:11.33

他人と議論をしようと言う時に文脈でしかわからない言葉を使うな無能が

**デフォルトの名無しさん** · 2020/07/29(水) 11:23:16.72

>>421
文脈がわからない人間が議論に参加するべきではないと思うの

他人の言葉には寛容に、自分の言葉は厳密にが会話の基本
議論は会話の積み重ねなんだから会話の基本ができない人間は議論ができない人
他人の言葉に厳しい態度とっても良いことなんて無いよ

**デフォルトの名無しさん** · 2020/07/29(水) 11:26:28.92

プログラムでもそうだよね
メソッドを宣言するときは引数はできるだけ広い型
戻り値はできるだけ狭い型
にすることで他のオブジェクトと協調しやすいプログラムができる
なんでもかんでもインターフェイス使おうとするJavaの習慣は誠に遺憾

**デフォルトの名無しさん** · 2020/07/29(水) 11:27:35.62

他人には厳しいみなさん

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

自分の説明能力の無さを相手が低脳だからと責任転嫁するヤシ

**デフォルトの名無しさん** · 2020/07/29(水) 11:36:20.63

我々人間もメソッドなのかも知れませんね

**デフォルトの名無しさん** · 2020/07/29(水) 11:38:34.39

>>422
> 自分の言葉は厳密にが会話の基本
いやだよばーかｗ
俺の言葉は寛容にだ。平置きでわからんやつが悪い
お前は俺の言葉に寛容になれな。

**デフォルトの名無しさん** · 2020/07/29(水) 11:38:43.76

自分の理解能力の無さを相手が無能だからと責任転嫁するココナツ

**デフォルトの名無しさん** · 2020/07/29(水) 11:39:49.67

>>427
僕は平置きでわかる派ですよ
完全に理解した

**デフォルトの名無しさん** · 2020/07/29(水) 11:41:33.11

で、結局のところ
Javaで「単体で平置きできる関数」って何なの？

コードで示してくれるといいんだか

**デフォルトの名無しさん** · 2020/07/29(水) 11:42:06.25

>>430
() -> "coconut"

**デフォルトの名無しさん** · 2020/07/29(水) 11:43:59.64

>>431
んん？それどこに置いてるの？

**デフォルトの名無しさん** · 2020/07/29(水) 11:54:07.92

>>430
単体でおける関数ってわかりますよね？
それを平置きするんですよ

**デフォルトの名無しさん** · 2020/07/29(水) 12:47:42.81

>>432
どこだと思う？

**デフォルトの名無しさん** · 2020/07/29(水) 13:01:46.48

無限ココナツ
Stream.generate(() -> "coconut").forEach(System.out::println);

**デフォルトの名無しさん** · 2020/07/29(水) 13:03:28.35

変数にセット可能、引数として渡すことができる、戻り値にセットできる
これが平置き三原則

**デフォルトの名無しさん** · 2020/07/29(水) 13:06:58.34

Javaはクラスに包んで間接的に置くしかない
これが最大の欠陥

**デフォルトの名無しさん** · 2020/07/29(水) 13:29:06.56

単体でもなければ平置きでもなくて草生える

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

用語はともかくラムダ式のことだと分かってスッキリした

**デフォルトの名無しさん** · 2020/07/29(水) 16:49:02.49

二つ折りじゃないですか!

**デフォルトの名無しさん** · 2020/07/29(水) 18:21:34.93

第一級函数のことかと思ったよ俺は
馬鹿は1つしかモノを知らないから適当に言っても1つの意味にしかならないが他人はお前よりモノを知ってるから第一級函数なのかラムダ式なのかそれとも自分が知らない別の概念なのか判断できないんだよ

**デフォルトの名無しさん** · 2020/07/29(水) 18:32:59.18

結局単にラムダ式のことを言おうとしてたん？
それだけのことをなぜ平置きとか言い出しちゃったのかなｗ

見苦しいからもうこれ以上レスいらんぞ

**デフォルトの名無しさん** · 2020/07/29(水) 18:34:25.78

平置き式のこと？

**デフォルトの名無しさん** · 2020/07/29(水) 18:35:43.73

平置きとかメルカリ用語じゃねーのか

**デフォルトの名無しさん** · 2020/07/29(水) 18:40:24.30

単体で置ける関数のことを単体関数と言い、それを平置きして式化したものが平置き式
ラムダ式なんて西洋かぶれのバズワードを使わなくても説明できるのができるプログラマ

**デフォルトの名無しさん** · 2020/07/29(水) 18:43:48.50

必死かコイツ

**デフォルトの名無しさん** · 2020/07/29(水) 18:59:31.61

みーっけた！
こうゆう関数併記するスタイルの事じゃないの？
プログラミングのお題スレ Part18
http://itest.5ch.net/mevius/test/read.cgi/tech/1594702426/124
https://github.com/katahiromz/3D

function add_line(x0, y0, z0, x1, y1, z1) {
}
function set_pos(x, y, z) {
}
function set_dir(dx, dy, dz) {
}
function walk(delta) {
}
function turn_dir(dx, dy, dz) {
}
function turn_angle(angle) {
}
function init() {
}
function koch(degree, step) {
}
function draw() {
}
function animate() {
}

**デフォルトの名無しさん** · 2020/07/29(水) 19:19:06.70

それは関数を単体でおいただけ
平置きとは違う

**デフォルトの名無しさん** · 2020/07/29(水) 20:44:56.64

フォルダに平積みする事か？
A.class
B.class
C.class
D.class
E.class

**デフォルトの名無しさん** · 2020/07/29(水) 20:48:45.31

本の平積みと同じ意味だよ
平にするって所を強調したいんだ

**デフォルトの名無しさん** · 2020/07/29(水) 23:30:32.60

ポリモーフィズムって押し付けなんだよね
汎用的な機能を提供する
与える引数だけが戻り値を決めるような
純粋な関数を1つくればDRYになっていいものを、

何らかの意図を持った特化した関数を
幾つも定義しなければならない
意味がわからない。

インスタンスには極力状態を持たせない方が
圧倒的にバグが少ない

UI制御の状態ならGUI上だけで
本当に重要な状態ならDB上やセッションとかで
持つくらいで大体は事足りる
クラスのフィールドなんてほとんど使わんよ

引数と戻り値
配列連想配列をとことん突き詰めれば
大抵の事はなんでも出来る。
これが構造化プログラミング

**デフォルトの名無しさん** · 2020/07/29(水) 23:41:24.99

ttps://i.imgur.com/qRx3DMV.gif

**デフォルトの名無しさん** · 2020/07/30(木) 00:11:51.67

>>451
それが設計意図がキチンとドキュメントにまとまっていれば
こうなったときに引数増やすしか無いかとかわかるんだけど
だいたいドキュメント書かないバカがノリで作ったようなポリモだから
頭にくるんだよね
作ったやつのおバカなドヤ顔に拳をブチ込んでその場で正座させながらドキュメント書かせたい

**デフォルトの名無しさん** · 2020/07/30(木) 08:29:08.61

>>451
単純にモジュール切り分けテクニックの一つなだけだよ。
変な切り分けすればそりゃ可読性悪くなるわ。
オブジェクト指向をいつでもすべしみたいな馬鹿が誤解をひろめたのがいかんだけで有用なテクニックの一つではある。

**デフォルトの名無しさん** · 2020/07/30(木) 10:11:39.63

ドキュメントとセットでの運用なら、
それ自体では完結してない、つまりは欠陥品ってことでしょ

ドキュメントの有無や書いた奴の頭の程度で決定したりはしない

**デフォルトの名無しさん** · 2020/07/30(木) 10:58:41.66

>>455
visualstudioがWordやExcelと連携してくれれば楽なのにね

**デフォルトの名無しさん** · 2020/07/30(木) 12:33:29.09

コードそのものが「文芸的」になる必要がある

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

ソリューションエクスプローラーなんて芋っポイのやめてWordの項目一覧でいいよ
そこの項目に対応したソースを記述すればいい

**デフォルトの名無しさん** · 2020/07/30(木) 13:21:46.37

ソリューションエクスプローラのCMakeターゲットビュー見てみろよ。
そんなこと言えなくなるだろ。

**デフォルトの名無しさん** · 2020/07/30(木) 13:24:30.25

ドキュメント主導型にして欲しい

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

それはちょっと違うかもしれないな。
プログラミング言語は自然言語と比較して非常に制約がきつく、それは機械が読めるほどだ。
自然言語で書くよりもプログラミング言語で書く方が誤解の余地が少ない優れた文書になるのではないか。

**デフォルトの名無しさん** · 2020/07/30(木) 14:36:27.17

data Status = Dead | Alive

you == not Alive

お前はもう死んでいる

**デフォルトの名無しさん** · 2020/07/31(金) 00:38:15.14

https://img.gifmagazine.net/gifmagazine/images/710205/original.gif

**デフォルトの名無しさん** · 2020/07/31(金) 00:57:13.70

http://imgcc.naver.jp/kaze/mission_anm/USER/20140905/93/994463/52/250x186x42178393ac27032c611b2149.gif

**デフォルトの名無しさん** · 2020/07/31(金) 01:01:24.69

http://imgcc.naver.jp/kaze/mission_anm/USER/20140905/93/994463/79/326x205x7ffb3f5b8763642d29df7d79.gif

**デフォルトの名無しさん** · 2020/07/31(金) 07:44:49.12

>>455
プログラムで表現しずらいものをドキュメントで補うんだよ。
SIerに提出するような糞アリバイドキュメントばっかりつくってると理解できないだろうけれど。