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

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

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

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

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

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

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

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

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

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

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

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

**デフォルトの名無しさん** · 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に提出するような糞アリバイドキュメントばっかりつくってると理解できないだろうけれど。

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

プライベートメソッドのドキュメントを作るんだろ？
欠陥品じゃん

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

プライベートだろうとパブリックだろうと、構造がコードでわかりにくけりゃ作るよ。
そういう表面的なことでしか判断できないのは頭が悪い証拠だと思うよ。

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

和田信者は頭が欠陥品

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

要するに馬鹿にはプログラミングさせるなということだな
馬鹿が書くと変なものをprivateにしてしまって大混乱
プライベートメソッドをテストせざるを得なくなる

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

普通は表面的＝形式的なモンで判断するんでしょこの業界
だって形式しかねーもん

**デフォルトの名無しさん** · 2020/08/01(土) 01:42:55.39

構造図みたいなのはプログラム内には書けないから
それは文章化しておくしかないから仕方がない

**デフォルトの名無しさん** · 2020/08/01(土) 13:40:53.00

https://i.imgur.com/HrbzgM9.gif

**デフォルトの名無しさん** · 2020/08/03(月) 01:10:10.92

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

**デフォルトの名無しさん** · 2020/08/30(日) 15:04:18.01

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

ならば「チンポがシコシコする」という日本語表現は、文法的に正しいのか？

チンポ「を」シコシコするのではなくて、チンポ「が」シコシコする。この場合、「チンポ」は主語となる。

オブジェクト指向で言う「集約」は２種類あって、全体（俺）と部分（チンポ）が繋がっている場合と、
全体（俺）と部分（チンポ）が別々になっている場合とが考えられる。けれども「チンポ」はそれ自体
が独立した生き物であり、所有者の意思とは無関係に、勃起して「シコシコする」。
例えば寝てる時にエロい夢みて朝起きてみたらチンコが勃起して射精してたとか。

違うか？

「胸がドキドキする」は良いが、「チンポがシコシコする」はダメな理由を、５０字以内で述べろ！

**デフォルトの名無しさん** · 2020/11/16(月) 10:13:57.47

>>397
要するに君のスペックや君が手掛けてきたシステムは
オブジェクト指向を検討する程のものじゃなかったし
今後もオブジェクト指向を検討するようなシステムに
君が係ることもなんだろうということは理解できた