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

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

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

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

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

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

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

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

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

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

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

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

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

**デフォルトの名無しさん** · 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
要するに君のスペックや君が手掛けてきたシステムは
オブジェクト指向を検討する程のものじゃなかったし
今後もオブジェクト指向を検討するようなシステムに
君が係ることもなんだろうということは理解できた