カプセル化の有害性、オブジェクト指向は愚かな考え

**デフォルトの名無しさん** · 2020/06/18(木) 23:47:36.69

カプセル化（英語：encapsulation）とは、オブジェクト指向を構成する概念の一つで、オブジェクト内部のデータを隠蔽したり(データ隠蔽)、オブジェクトの振る舞いを隠蔽したり、オブジェクトの実際の型を隠蔽したりすることをいう。

かつて偏差値の低い学校向けの情報処理系教科書において「カプセル化は大変すばらしいものであり絶対に使うように」と大体的に宣伝された。

一方、カリフォルニア大学バークレー校の有識者を中心とした「インターネットを作った人たち」は「階層化の有害性」として「カプセル化は絶対にやめろ」としている。

https://monobook.org/wiki/%E3%82%AB%E3%83%97%E3%82%BB%E3%83%AB%E5%8C%96(%E3%83%97%E3%83%AD%E3%82%B0%E3%83%A9%E3%83%9F%E3%83%B3%E3%82%B0)

**デフォルトの名無しさん** · 2020/07/05(日) 09:30:39.94

上のチンコの例で言えば、funcSとfuncBのテストをテストケース書いて
コードを書き換えた上でやってドキュメントなりログなりで残すの？

だったらpublicでやれば良いし、そうやってpublicにするなら関数型でも
おんなじ事でしょ。

チンコがnullなのか16以上そこそこでかいのかまでしか関係ないわけで
privateがnullをハンドリングしてないとか無駄なんだよ。もちろん
命に関わるようなところではそれくらいの厳格さが求められる場合も
あるだろうが、普通のビジネスソフトではそこまでやらない。

他のチームが関わる場所を少なくするためにカプセル化するわけで、
そんな全部publicにしたら意味ないんだよ。

**デフォルトの名無しさん** · 2020/07/05(日) 09:41:54.39

弊社はC2カバレッジ100%未満は出荷できませんけどね。

**デフォルトの名無しさん** · 2020/07/05(日) 09:42:35.87

>>854
線引じゃねーよクソ野郎
publicでたまたま呼ばれた1パターンと
privateの網羅テストが同じになってたまるかアホかよ
クラスのなかにあるのでpublicから呼ばれたときだけ動けば
ルーチンとして不出来でもOKなんてあるわけないだろ
お前はクソだからもう死ねよ

**デフォルトの名無しさん** · 2020/07/05(日) 09:45:43.25

>>857
まあじゃあ君はprivateまでテストケース書いてやっとけば良いんじゃない？
誰も止めてないし。大規模ビジネスソフトでは世界的に言って普通ではないというだけで。

僕はあんまり関わることのないレベルの世界だけど、まあ多分一生関わらないので
君が良いならそれで良いと思うよ。

**デフォルトの名無しさん** · 2020/07/05(日) 09:51:39.34

>>857
君は大変なんだねwテストがんばれーwww

**デフォルトの名無しさん** · 2020/07/05(日) 09:54:43.54

>>858
落としどころとしてはそんなところで良いんじゃね？
問題なのは「privateはテストするべきではない」なんて変な教義を押しつける人の方なんで。

**デフォルトの名無しさん** · 2020/07/05(日) 10:02:55.49

>>856
正直QAやってたのは随分前の話だから最近のQAツールは知らんけど、
privateが全部通ってれば100％なわけで、そもそもprivateがある理由は
使われるためなんだから普通は誰も一回も使ってない場合なんかはないわな。
レベルが高くなると通らないケースも出るだろうが。だから線引きってこと。

チンコテストならnullから16までやれば100％通ってる。privateかpublicかって
のは関係ない。ただしprivateはnullをハンドリングしてない。それをどうするかって話。

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

>>854
お前こそ大規模プロジェクトやったことないだろw
それだけ大規模だと逆にテストコード書かんわ(nttデータとかアクセンチュアとかな)
そんなクソプロジェクトを引き合いに出されても知らんわw

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

>>860
「べきでは無い」とは思わないけど、privateをデベロッパーの個人的な
チェックを超えてテストすることを求められるとしたら、何かプロジェクト的に
おかしいとは思う。

**デフォルトの名無しさん** · 2020/07/05(日) 10:08:05.85

>>862
だから僕もNTTデータとかは知らんと上に書いてるけれども。

アメリカの大規模ソフトは中の人だし、他の会社のデベロッパも何人も知ってるので、
実際にアメリカの業界は知ってる。

君の「クソブロジェクト」ではないプロジェクトは会社名は出す必要はないが
例えばアメリカのERPとかそのレベルでなんなの？

**デフォルトの名無しさん** · 2020/07/05(日) 10:09:56.64

だいたい質問を質問で返す人は答えるのに不都合がある人。

数百人のデベロッパーのプロジェクトをやったことなくても全く問題は
ないんだが（ロッキードのミサイルの制御の人とか数百人もいないし）、
「あるよ」とは帰ってこない。

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

>>863
だからそれは別の話なんだからさ。
「privateなら必ずしも単体テストする必要はないけどやりたいならやれば？」
これでいいじゃん。

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

>>866
ちょっと弱いかな。「普通はやらんけどやりたかったりやる必要があるならやれば？」

くらい？privateを全部テストケース書いてエッジやネガティブやるとなると多くの場合
サグラダファミリアになるし、普通はやらない。

でもやりたいならやれば良いとは思うのでだいたいそういう感じか。

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

組み込みおじさんはそこに義務感を感じ、他者に強要するのだろう
難儀なこった

**デフォルトの名無しさん** · 2020/07/05(日) 10:37:41.26

組み込みとかだと多くの場合そもそもOOあんまりいらんと思うのよね。

人工呼吸器の制御とかだときびしくやらないといけないのはわかるし、
やってない人工呼吸器とか俺も付けられたくないけどさw

OOのメソッドとかはちょっと違う話だわな。カプセル化とかは数百人になると
生きてくる。個人的には自分一人の個人プロジェクトでも使うけど。
数年後にまたやる羽目になったりした時に楽なので。

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

>>847
それは僕も思ってた
机上の空論っぽいんだよね

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

>>867
どこに差があるのかよくわからんが、どちらにしても一行目は問題ないにしても
二行目はここに存在しない敵を攻撃している印象。
いちいち関係ない話を絡めなくてもいいじゃんと思うが、二度も書くということは
そっちが主張したい本音ということかな？

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

OOPは理解している人とOOPを理解していない人が単体テストについて不毛な争いをするスレかな？

**デフォルトの名無しさん** · 2020/07/05(日) 10:58:45.46

>>871
いや、元々がpublicじゃなくてprivate単位でやれという話からなんだからその話だよ。

publicでprivateの分岐通ってるなら良いじゃんといなら誰も否定していないのでは？

逆にprivateで一度も通ってない部分があっても良いじゃんとも言っていないような。
if文書くのに絶対通らない物を書く人いるわけないでしょw

ただ、公式なテストとして、privateのレベルでテストケース書いてログ残してコード
書き換えてコンパイルし直してやるというのは少なくとも普通のビジネスのソフトとしては
一般的なベストプラクティスではないし普通はアメリカの大手一流ではやらないということ。
例外はあるとしても。

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

土台が腐ってたら。
全部検査するほうが楽。

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

幸いそういうのには当たったことはないw
上に書いたように例外は認める。

**デフォルトの名無しさん** · 2020/07/05(日) 11:09:23.64

アメリカの大手一流ってどこだろ？

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

アメリカや大手や一流が正しいとは限らないけどね
ボーイングの飛行機があんなことになってるように

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

まあ会社的にはプログラマなら全員知ってるとこ。それ以上は言わん。

**デフォルトの名無しさん** · 2020/07/05(日) 11:13:19.35

>>877
だからボーイングの制御系とかはそもそも全然別の話。

もちろん正しいとは限らないけど標準なのは確かだわな。
日本初のOOより優れたこういうメソッドがある！というなら
もちろん喜んで聞くけどね。

**デフォルトの名無しさん** · 2020/07/05(日) 11:14:37.74

>いや、元々がpublicじゃなくてprivate単位でやれという話からなんだからその話だよ。
>ublicでprivateの分岐通ってるなら良いじゃんといなら誰も否定していないのでは？

そう、それが一致しているならいちいち関係ない話を絡めなくてもいいんでは？
public経由のテストで十分網羅できるのにそれでもprivateメソッド単位のテストが
必要なんて主張している人いたかな？そこに誤解があるんじゃないの？

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

>>880
いるから揉めてるんすよ。

**デフォルトの名無しさん** · 2020/07/05(日) 11:16:31.89

>>881
それはどれ？レスを教えて

**デフォルトの名無しさん** · 2020/07/05(日) 11:17:47.72

アメリカの大手の一流がそうしてるから正しいのだという権威にすがった論理を
展開されておられたのでその会社ってどちらの会社なのって聞いてみたんだけど
そんな会社本当にあるの？

**デフォルトの名無しさん** · 2020/07/05(日) 11:18:34.49

t_wadaの次はアメリカの大手一流か、議論の根拠が権威ばっかりやな

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

例えばこの方。

848デフォルトの名無しさん2020/07/05(日) 08:16:36.05ID:IUsMolpf
publicを通した範囲でしかできないprivateのテストは網羅されてなく
メソッドの単体テストとして十分なテストは行われていません
あなたの意見は採用できません
できれば実際のプロジェクトを5つぐらい経験されてはいかがでしょうか？

メソッド全部ちゃんとエッジケースやネガティブまでやれって業界があるのは
わかるし、必要だとも思うけど、それはOOのビジネス系だでは普通やらないということを
言ってただけなんだけど、バカだアホだお前は素人だといろんな方面から変に噛みつく
やつばかりでもう僕はいやになっちゃった。この人が噛み付いてきたということではないよ。

もちろんまともな人のが多いんだろうけどおかしな人のが目立つからね。

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

いちいち論点ずらすな。
お前ら、何で争ってるんだよ。

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

OOのビジネス系ってどんな仕事？
そんないい加減なことやっていい仕事ある？
トラブってもうちうちでなーなーで済むようなBtoBの仕事？
自社内でしか使わないシステムとか？

**デフォルトの名無しさん** · 2020/07/05(日) 11:24:18.36

アメリカの大手一流が不満なら自分の業界の実例をあげたらいかがだろうか？

privateをテストケース書いて全件やってるとこなんておそらくないぞ。あるというならあげれば良いだけの話。

テストツールどうやって動いてるねん。コード自動で書き換えてコンパイルし直しか。話がおかしいのすぐわかるだろ。

**デフォルトの名無しさん** · 2020/07/05(日) 11:24:52.69

まず、このスレの連中は議論する前に論理学を学ぶことをお勧めするよ。
あと、詭弁について調べろ。

**デフォルトの名無しさん** · 2020/07/05(日) 11:25:24.17

>>888
アメリカの大手一流ってどこの会社なん？

**デフォルトの名無しさん** · 2020/07/05(日) 11:25:52.62

>>887
だからアメリカ製のパッケージとかコンサルとの間でよく揉めてるだろ。これは「バグ」じゃないとか。DBだってよくある話だわな。

君はどんな仕事なの？

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

>>890
君の会社はどこなん？

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

>>891
アメリカ製のパッケージとかコンサルとの間でよく揉めてる仕事なん？
なにそれ？何言うてんの？何の仕事してるんよ

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

>>885
それは網羅してないからダメという意見では？
そこに反論するならば「privateのテストを強制すべきではない」ではなく
「public経由でも網羅することは可能」ではないかね？(できるなら)

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

>>882
>>848
デフォルトの名無しさん2020/07/05(日) 08:16:36.05ID:IUsMolpf
publicを通した範囲でしかできないprivateのテストは網羅されてなく
メソッドの単体テストとして十分なテストは行われていません
あなたの意見は採用できません
できれば実際のプロジェクトを5つぐらい経験されてはいかがでしょうか？

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

>>892
アメリカの大手一流でこうしてるから、こうするのが正解なんやー
言うてたからアメリカの超一流ってどこなんよ？と聞いたんやけど
なんで言えへんの

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

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

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

>>897
なるほどね、わかる気がする

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

大手一流の名前を言うだけで逃げそう（笑）

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

足ガクガクになってまうよー

**デフォルトの名無しさん** · 2020/07/05(日) 11:36:47.20

まぁ、単体テストでprivateを直接呼び出すのは変な話だけどな。
デバッグならともかく、単体テストでリフレクションによるprivate強制実行とか反則でしょ。
デバッグで許される理由は、バグを見つける可能性があるから。
単体テストで駄目な理由は、テストの合否判定が不当に変わる恐れがあるから。

というのが自分の見解だが、皆さんはどう？
スレが加速してからの途中参加だから、自分の発言に自信ないけど。

**デフォルトの名無しさん** · 2020/07/05(日) 11:37:26.26

>>894
そもそもpublicを経由するのが網羅してないから、経由せずにprivate単位で
コード書き換えてテストやるべきという主張自体がOO的にはおかしいわけです。

見せたくないからprivateなわけで。

もちろんデベロッパー個人でコード入れて確認することはあるだろうけれども、
開発手順としてテストケース書いてログ残してというのは違う。

public経由で内部を網羅してるかしてないかは知ったことではない、というのが正解。
上の例で言えばfuncBやfuncSがnullをチェックしてようがしていまいが他のチームには
関係ないわけです。入り口でnullチェックしてれば。

個人的にはサイズがでかければprivateでも入れるけども、それをテストケース書いて
ログ残してと要求されたことはないし、普通はされんはず。少なくともアメリカでは。

**デフォルトの名無しさん** · 2020/07/05(日) 11:38:33.09

GoogleであってもMicrosoftやOracleもバグのないソフトは作れないし
銀の弾丸的な手法があるわけもなく、アメリカ、大手、一流という箔付けに頼ってるだけでしかない

**デフォルトの名無しさん** · 2020/07/05(日) 11:41:45.16

せっかく匿名で議論できるのに
「アメリカの大手一流」とか言い出しちゃうところが哀れやわ
>>805の下痢便コード出た時点でスレ終了やろ普通

**デフォルトの名無しさん** · 2020/07/05(日) 11:43:37.87

>>896
君の会社はどこなの？

アメリカ大手１流でブライベート単位でのテストケース書いたりした網羅的なテストは「していない」という証明は悪魔の証明だから無理だけど。全社で働いたわけじゃないし。

逆にアメリカ大手１流でやっているという証明は一件出すだけなのでGAFAのどこでも、マイクロソフトでもアドビでも簡単にできるのでぜひどうぞ。

ただ、なんかコピペで爆撃されたオートメーションツールでは、特にコンパイル必要な言語は、直接privateテスト出来ないよ。当たり前の話。

**デフォルトの名無しさん** · 2020/07/05(日) 11:45:23.06

>>905
アメリカの大手一流ってどこなんよ？

**デフォルトの名無しさん** · 2020/07/05(日) 11:45:48.03

その会社がやってるからすごいんでしょ、はよ教えてやー

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

もう何でもええから言うて、そしたら僕がすごーいって言うから、もうそれでええやんか

**デフォルトの名無しさん** · 2020/07/05(日) 11:49:23.59

また別IDが下痢便とか発狂し始めた。

お前はほんとにウンコとかそのレベルのことが好きだな。

**デフォルトの名無しさん** · 2020/07/05(日) 11:51:02.27

>>907
やって「ない」という話だけど。
まず日本語学んでやー　よろしくなー

**デフォルトの名無しさん** · 2020/07/05(日) 11:51:58.59

>>910
アメリカの大手一流ってどこなんよ？

**デフォルトの名無しさん** · 2020/07/05(日) 11:52:54.88

たぶんアップルは単体テストやってない、SSLでアホなバグだしてたから、あの会社適当だわ、しらんけど

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

>>911
君が自分の名刺でもあげたら考えとくわー

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

>>913
アメリカの大手一流ではそれが普通なんやーおりゃーってイキっておられましたよね
だから聞いたのに、教えてくれないなんて酷いです、あんなにイキってたのに酷いです

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

>>902
> そもそもpublicを経由するのが網羅してないから、経由せずにprivate単位で
> コード書き換えてテストやるべきという主張自体がOO的にはおかしいわけです。

誰がそんなこと言ってるの？

少なく遠も俺は
1. privateはpublic経由でテストすれば十分
2. もしpublic経由でテストできないほど複雑なら、そもそも設計がおかしい
3. 設計の問題を修正して、責務を分割すれば、自然にpublicになるはず
と言ってる。

テストするためにpublicにするのではなく
問題がある設計を直せばpublicになる。と言っている。

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

>>914
劣等感がひどいな君は。まずはカウンセラーに
かかりなさい。OOの話はそれからしようか。

**デフォルトの名無しさん** · 2020/07/05(日) 11:58:34.47

>>915
いや　だ　か　ら　この人。

848デフォルトの名無しさん2020/07/05(日) 08:16:36.05ID:IUsMolpf
publicを通した範囲でしかできないprivateのテストは網羅されてなく
メソッドの単体テストとして十分なテストは行われていません
あなたの意見は採用できません
できれば実際のプロジェクトを5つぐらい経験されてはいかがでしょうか？

**デフォルトの名無しさん** · 2020/07/05(日) 11:59:21.06

アップルのSSLのバグはこれね

Apple史上最悪のセキュリティバグか、iOSとOS XのSSL接続に危険すぎる脆弱性が発覚──原因はタイプミス？
https://appllio.com/20140223-4899-apple-ios-bug-ssl-goto-fail

**デフォルトの名無しさん** · 2020/07/05(日) 12:06:18.88

IUsMolpfに文句言えばいいのに。

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

>>902
ああつまり、重要度は関数仕様＞ブラックボックステスト＞ホワイトボックステストだから
テストのために仕様変更はすべきでないという主張か。
それはわからんでもない気はするけど、そういうウォーターフォールが絶対というわけでもないしな。

その是非はさておき、そこが論点なら例えばリフレクションのような形でテストできたり、あるいは
テスト仕様を書く本人が関数仕様も決定することができる条件なら全然問題ないわけだ。

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

>>918
正直iOSは全然知らんけどこれは酷いなw
Public Private以前にStaticだからなんにしても直接テストする部分だな。

**デフォルトの名無しさん** · 2020/07/05(日) 12:08:47.51

>>921
せやろｗ

**デフォルトの名無しさん** · 2020/07/05(日) 12:13:26.38

>>919
横から狂犬が噛み付いてくるんだよ。

>>920
ウォーターフォールは関係ないし、重要度をつけてるわけでもないけれども、
普通はprivateはソースコード変えてまでテストするものではないというだけの単純な話。

そもそも「テスト」と言ってもいろんな段階があるわけだし、「ユニットテスト」にしても
プロジェクトによって意味が違うわけで。

なんにしてもprivateを（例外はあるにせよ）全部公式な形でテストしろというのは
OOの手法のプロジェクトでは「普通は」やらんというだけなのになんでこんな揉めるのか。

**デフォルトの名無しさん** · 2020/07/05(日) 12:16:58.82

>なんにしてもprivateを（例外はあるにせよ）全部公式な形でテストしろというのは
>OOの手法のプロジェクトでは「普通は」やらんというだけなのになんでこんな揉めるのか。

全部やれなんて主張している人がいないから。

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

>>924
だからいるだろうが。本人なの？

848デフォルトの名無しさん2020/07/05(日) 08:16:36.05ID:IUsMolpf
publicを通した範囲でしかできないprivateのテストは網羅されてなく
メソッドの単体テストとして十分なテストは行われていません
あなたの意見は採用できません
できれば実際のプロジェクトを5つぐらい経験されてはいかがでしょうか？

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

>>805
こう書いた方が良いと思うの
https://paiza.io/projects/mPhqBnYZnQukkW6HY9LmOQ

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

privateメソッドが正しいことは確認する、その上でpublicのメソッドが正しいかテストするって感じ

**デフォルトの名無しさん** · 2020/07/05(日) 12:30:30.98

>>925
これが「privateのテストを全部やれ」に読めてしまうのか。そこが原因だな。

**デフォルトの名無しさん** · 2020/07/05(日) 12:32:35.17

>>923
> ウォーターフォールは関係ないし、重要度をつけてるわけでもないけれども、
> 普通はprivateはソースコード変えてまでテストするものではないというだけの単純な話。

テスト中に設計の問題が発覚しても
ソースコード書き換えないの？

ソースコード変えてまでテストするものではないというけど
なんでソースコードを作成＆自分が書いたそのコードをテストしてるときに
自分が作成してるソースコードを変えたらだめなの？

**デフォルトの名無しさん** · 2020/07/05(日) 12:36:24.73

普通の開発だったら

0. なにかの修正を行う必要が発生する
1. その修正のためにソースコードを修正する
2. 修正したコードが正しいかテストする
3. 1～2を繰り返し修正したコードに問題がなければ開発完了
4. 統合テストとかより大きなテストを行う

でしょ？

public・privateメソッドをテストしたいっていうのは
この1～2のフェーズで発生することなのに、
なんでテストがやりにくいなと思ったときに
ソースコードを修正したらだめなの？

**デフォルトの名無しさん** · 2020/07/05(日) 12:38:58.81

>>926
スマホでも見やすくなったな。
あー、なるほど。

でも、サンプルみたら少し混乱してきた。
自分はクラスの単体テストをイメージしてたけど、まさか、Mainクラスの中に定義されたprivateメソッドの単体テスト(？)の話をしていたとか？
もう単体テストの意味が少し崩壊してきたが...

**デフォルトの名無しさん** · 2020/07/05(日) 12:46:16.73

どういう流れなのか追うのもめんどいなw

**デフォルトの名無しさん** · 2020/07/05(日) 12:52:51.86

>>928
そもそも300くらいからそういう話だから。

privateはpublicからアクセスして全部網羅できるから問題ないなんて
誰も否定してないしそういうことならもともと荒れてない。

>>929
あのさ、例えばバグが発生したらコード書き換えるのは当たり前だよね？
そんな話では当然ないよね？

privateを直接テストするためには外部から呼べないのだから、呼ぶためには
クラス内に書くしかないわけだよ。で、ソースに書き込んでクラス内に書いて
テストしたあとまたそのコードを取り除いてとかするの？それをドキュメント化
する？

普通はコード書いてる最中にテストコードを入れて確認するのは公式なテストの
メソッドでは無いし、それをドキュメント化したりするのは求められないと
思うが。

>>926
ワロタw

けどそうじゃ無い。privateで直接アクセスした時のみnullがでるということ。
intじゃ無いのは例のためにわざとそうなっている。あとchinTestが呼ばれてないし、
OptionalにisEmpty無いのでは。

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

nullがでる->ヌルポがでる

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

>>933
> privateを直接テストするためには外部から呼べないのだから、呼ぶためには
> クラス内に書くしかないわけだよ。で、ソースに書き込んでクラス内に書いて
> テストしたあとまたそのコードを取り除いてとかするの？それをドキュメント化
> する？

誰がそんな事するって言ってんの？
お前が想像が間違ってる

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

>>933
えっとごめんそのnullの問題は設計が下手なだけだと思ってて、設計を見直すべきかと
privateのテストをどうしますかの例として出したので問題ないっしょ
君はこのプログラムをどうやってテストするん？

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

よくもまあ粉薬をオブラートで包むような話でこれだけのレスで盛り上がれるなー

**デフォルトの名無しさん** · 2020/07/05(日) 12:57:28.17

いやーびっくりだｗ

設計がおかしいから設計を修正するって話をしてるのに
設計を修正した後に修正前に戻すなんでやらないだろ！って言われちゃった（笑）

だーれも修正前に戻すなんて言ってないの
おかしい設計を修正して「開発完了」なの

**デフォルトの名無しさん** · 2020/07/05(日) 12:57:46.31

たぶん、スレタイがスレタイだから、手続き型でプログラムを書く人のイメージする単体テストとオブジェクト指向プログラマーのイメージする単体テストで齟齬が生じているのでは？

そりゃ、サンプルみたいに関数が定義されていたらprivateだろうと関数はテストするべきって発想になるのは不思議じゃないし、オブジェクト指向に基づき設計されたクラスを単体テストする話であれば、privateをテストって、どういう意味なんだろって不思議がるだろうよ。

そして、たぶん、サンプルコードに対して真っ先に突っ込むべき点はOOPでも何でもないコードという点だと思うのだが、そこはいいのかな？

**デフォルトの名無しさん** · 2020/07/05(日) 12:57:47.49

OptionalにisEmptyが無いというのは何言ってるのかわからない
本当になかったらコンパイルに失敗するよ

**デフォルトの名無しさん** · 2020/07/05(日) 13:00:08.78

>>939
一人privateがテストしづらいからpublicに変更してテストして
"元に戻す" 話をしてるやつがいるみたいだが（笑）

**デフォルトの名無しさん** · 2020/07/05(日) 13:00:23.50

>>939
僕はOOPのコードだと思ってるよクラスが定義されてるからね
君が思うOOPのコードはこれだっていうのを君に出していただきたいかな

**デフォルトの名無しさん** · 2020/07/05(日) 13:00:34.24

>>935
もうお前は良いからスレを読め。

>>936
いやそうじゃなくて、publicで呼ばれるprivateを直接テストするのかという
話なので、そのためにわざわざヌルポになるように書いてるわけで、staticなら
そのままメソッド呼べば良い話。

もうこの板嫌。。。

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

>>943
ヌルポになるような設計はおかしいのがそれはコーディング力の稚拙さゆえ。
だから、いまはそのヌルポの問題は置いておこう。
さて、君はこのプログラムをどうテストするのか教えて欲しいっていうのが僕の要望だよ。

**デフォルトの名無しさん** · 2020/07/05(日) 13:06:51.97

>>944
いやだから　そ　う　じ　ゃ　な　く　て

privateでだけヌルポが出るように書いてるの。privateとpublicのテストの話だから。
出さないだけならnullチェック入れれば良いだけの話だが、そういうことじゃ無くて
わざと出るようになってるの。

そのプログラムならstaticなのだからそのまま呼べば良いし、
chinTestも呼ばれてないけどpublicなのだからテストしたければそのまま
呼べば良い話。

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

>>942
スマホだからインデントとか崩れは許してほしいが

class USBMemory ： Storage{
private：
色々定義
public：
USBMemory(ポート設定)
void open(string file)
string read()
void write(テキスト)
void close()
}

文法適当だが勘弁して

組み込みおじさんにも分かりやすいように、
組み込みを例にするが、
OOPってこんな感じのコードでしょ。

で、ユーザーは

USBMemory usb；
usb.open(ファイル指定)
text = usb.read()
usb.close()

こんな風に内部実装を知らなくても呼べる。

**デフォルトの名無しさん** · 2020/07/05(日) 13:11:36.99

>>945
わざと出るように入れたのが現実の実装と乖離してるので、わざと入れる意味が無いので
気にする必要ないですよってこと

君はpublic経由でprivateをテストするんだって立場ですよね
君はどうテストを書きますかってことを教えて欲しいです
君の元のコードでもいいですよ、僕が変更したコードでも良いですよ
君はどうprivateをテストしますか？

**デフォルトの名無しさん** · 2020/07/05(日) 13:12:18.35

残念ながら、クラスを使えばOOPという認識は間違い。

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

>>948
僕は君の認識が間違いだと思ってる、君の方が残念

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

クラスの存在こそがOOPの本質

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

え、そこに反論されるの？
何このスレ。

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

JavaScriptとかpythonとか、元々、クラスはサポートしていないのに？
プロトタイプベースはオブジェクト指向じゃないとでも？

**デフォルトの名無しさん** · 2020/07/05(日) 13:17:49.53

>>951
えじゃねえよ、なめんなよ

**デフォルトの名無しさん** · 2020/07/05(日) 13:18:05.20

>>947
意味がないどころか、そこにしか意味がないんだよ。
それ「だけ」を見せるためのコードなんだから。staticとか
なら話の流れとなんにも関係がないのでそれこそ意味がない。

でもあえていうなら、そもそもまず連打が好きじゃ無い。自分ならループにするけど、
それだけでもない。

まず壊すようにテストすんの。null入れたり、変なオブジェクト入れたり。

で、privateをテストするんじゃなくて、要件通り動くかどうか。nullからint maxまで
回して、要件通りの結果が全てでくるか。変なオブジェクト入れたらどうか。
そういう感じ。

でも最初に書いたように、publicとprivateの話なんだから、そこで
エラーが出るか出ないかがないとこのスレ的にはなんの意味もない。