カプセル化■プライベートメソッドをテストする方法
■ このスレッドは過去ログ倉庫に格納されています
■短い回答 https://t-wada.hatenablog.jp/entry/should-we-test-private-methods クソコード例。こんなコード書いてるやつが、privateのテストで https://mevius.5ch.net/test/read.cgi/tech/1592491656/805 983 名前：デフォルトの名無しさん[] 投稿日：2020/07/05(日) 14:18:27.81 ID:9F15TCk0 [68/74] >>805 のチンコテストのfuncSとfuncBはどうやってテストするの？ >>4 への回答 >>805 https://paiza.io/projects/mPhqBnYZnQukkW6HY9LmOQ >>5 の補足、これはまだプライベートですが >>6 僕が書いたように同じオブジェクトにテストコードも書いちゃえば良いと思います >>7 >>9 >>10 >>11 >>12 プライベートをテストしたいっていうのは >>16 僕は下痢便コードをこうするべきって修正したよ まだやってたのかｗ >>5 みたいな下痢便コードが出てきた時点で こうするべきだと思うならやれば良いがな Mainに他のクラスの処理を全て同居させる ×>>5 みたいな下痢便コードが出てきた時点で >>3 みたいな下痢便コードが出てきた時点で >>24 おそらく設計とは何かを知らずに、 >>27 >>26 >>28 謙虚になれや >>27 このスレの主旨的にはprivateメソッドのテストコードを書きたいんだよね？ 結局のところ何でもあり スレタイと書いてあることがチグハグで趣旨が解りづらいな。 >>1 の発言を見るとprivateの呼び出しは推奨しないように見える。 t-wadaさんがどういう人か調べてみたところ、日本が誇るJavascript使いなんだな。 例えば多くのパーサーは公開メンバとしてpush()を持つ。 パーサーの話ついでに、もう一つアイデアを提供しよう。 詳細を知ってはならないということは、どの状態を経てその記号が生み出されたのか知ってはならないということであり、これは非常につらい。 システム利用者がシステム作者に「詳細を検査しちゃだめじゃないか！」と怒っているように見える。 クリスマスプレゼントが何かを知るために箱を振って音を聞くよりも、箱を開けて直接見るほうが早い。 箱を振って「プレゼントはプリキュア」と観測できたとする。 privateなメソッドの動きが仕様書で定義されてたらテストしなくちゃいけないし書いてなかったら書く必要無い >>47 >>50 書いてない関数を勝手に作るなんて禁止に決まってるだろ 関数を作るのに許可がいるなんてすごいことだからとてもすごいと思いました >>1 にはテストとしか書かれていないけど、記事元はユニットテスト(単体テスト)の話だよね？ >>57 >>3 >>59 てかさ、タイトルがカプセル化なのに、なんで>>3 も>>5 もテストコード実行場所とテストコード実行場所が同じ階層に記述されているのさ。 ミス。テストコード実行場所とテスト対象が同じ階層に書かれているのはなんで？だ。 wadaは忘れろ 一意見を急に”理論”とか”法則”とか言う方が頭どうかしてるよ 和田理論被害者の会ニューヨーク支部もひつよう。 前スレで王家秘伝のレシピ教えたのに。 >>61 オブジェクト指向の話をしよう >>74 >>74 >>77 >>60 >>78 もしかすると、バグの原因を調査する作業はテストに含まれるかどうかって話かな？ >>81 突っ込みどころねーじゃんw privateメソッドのチェックもできない奴は出荷すんぞ privateのチェックって具体的にどうやるの？ queue = Queue() >>88 カバレッジテストをするかどうかは、ケースバイケース。どちらでもいいとして、一番気にしているのは、そもそも>>1 の記事主に批判的な人はオブジェクト指向を理解しているのか？という点。 >>85 >>91 どうやら図星だったようですね。 >>91 >>92 みたいな低能が蔓延るスレで世間の常識は通用しないってことが証明されちゃったわけだ。 >>92 >>95 オブジェクト指向はダメなんじゃないか？というのが前スレの趣旨で、非公開のメンバは何のためにあるの？テストどうするの？という話になった。 > それに対する回答が「非公開のメンバはブラックボックスとして扱いテストしてはならない」という和田メソッドが示された。 実装の詳細ならむしろテストが必要だと思うのだけど >>99 >>98 >>101 単体テストっていう言い方がまずいのかも 言い張るんだったら、ほんとに和田メソッド被害者の会作っちゃうよ？ >>105 >>105 自作自演ｗ >>105 >>108 >>104 に言いたかっただけだろ。 >>110 Javaは色々研究して提唱してえらいなあと思う部分もあるんだけど、すぐに宗教化してしまうからな。 どうも都合が悪かったようだな でも俺が設立するからには、普通じゃダメなんだよ。 いいだしっぺが説き伏せるそうです。 いきなり他人だよりです（笑） ではまずいいだしっぺから 訂正、すべての入力の組み合わせは2カバレッジだった >>87 >>123 キミも和田メソッドの被害者なんだよ。 ごちゃごちゃうるさい 和田メソッド被害者の会、会員募集中。 >>123 可視性狂信者ってのはどうしようもないね。テストさえ犠牲にし始める。 >>95 って、何でもないです。 >>130 >>128 >>87 の質問に答えてないよね？ >>137 オブジェクト指向を理解している人間の言うprivateをテストせよ/テストするべきではない と、 オブジェクト指向を理解していない人間のprivateをテストせよ/テストするべきでない は論争のレベルが全然違う。 >>138 >>139 ここで話を整理するけど 具体例もなく、privateのテストやれと言われてもできねーよ、バーカ。 うんうん 頭皮に触れるものは全て検査する ［ケハエール］ >>143 >>147 >>149 > ある時改修で別のpublicメソッドからt=3のときにprivateメソッドを呼ぶことになったら >>149 はホント意味不明だな >>151 設計書かコメントに >>153 >>154 は？最小-1、最小、中間、最大、最大+1通すだろフツー だが、お前のやり方ではどうやっても通すことはできないんだよ っていうかここまで自分の間違いを認めない意味はなんかあるの？ >>157 >>159 bdixmHftはQueueという言葉とオブジェクト指向という言葉が理解できずに発狂した説 詳細に関する知識が無ければ検査できないなら、公開メンバを介して検査するのと、非公開メンバを直接検査するのは同じことですよ。 アンクルボブの理論に対する賛否両論から議論を始めるべきなのかもしれない。 >>149 和田さんはアンクルボブを理論のベースにしています。 詳細に関する知識を持ってはならないというのが和田理論の根幹です。 和田理論を素直に受け入れられるのは、プログラミングとはAPIを呼び出すことであり、決してアルゴリズムを実装することではないからですよ。 >>167 「和田理論」も同じな。「アンチ和田」はなにか理由があって >>172 >>173 アンクルボブをロバートと読み替えても通じますし >>159 >>3 の下痢便コード書いた人だよ 関数が取りうる値で全てでテストしろと言ってるアホがいるスレはここですか？ privateメソッドのテストを具体的にどうやるのか未だに分からん。 巨大なメソッド自体が悪じゃね ステートメントの入力により無限通りの構文木が出力されるパーサ？ >>179 そもそも、privateメソッドを定義すること事態、滅多にないよ。 このスレの過去の発言を見ると誤解を招きそうだから補足するけど、別に内部処理をpublicにしたからprivateは無いという意味じゃないからね？ >>181 >>183 >>186 >>187 >>189 最小、最大、中間値って聞いたことがないけど 俺もそう思ってたけど、そもそもスレタイともずれてきてるよねっていう 詳細を知ってはならないということは、詳細の知識、つまり境界を知ってはならないということ。 >>195 zTLocdwC以外、Javaなんて誰も言ってないけど？何が言いたいんだ？ オブジェクト指向のすばらしさを語っている人が、一番オブジェクト指向を知らない。 >>196 >>200 >>1 に書いてあるのはただの常識で 常識じゃん。むしろ、常識を疑うその心を疑え。それって経験不足では？ ただ、記事を書いた和田さん？も、記事を読むとその危険性を理解しているみたいだから対立する気はないよ。 そりゃC++への嫉妬だろ 黒魔術って言われ方を誉め言葉だと思ってる馬鹿ってほんとにいるんだな。。 オブジェクトが隠蔽・カプセル化するものだからといってホワイトボックステストまで否定しちゃうのは変な話。 ホワイトボックステストを否定する気はないけど...例えば >>88 のコードを借りるけど あー...うん、俺の言い方が悪かったかも。 ユーザーが直接public methodを使うわけじゃなければ まあね、処理の複雑さによるのだろうね >>213 そこで、テスト駆動開発ですよ。 >>220 >>219 >>220 >>213 >>221 >>224 privateではなくてパッケージプライベートにした方がええのかもわからんね それで破綻するならパッケージの設計がよろしくないということで >>227 なんでテストケースが作れるのに、publicにしないのかわからん privateでかっこいいメソッドなんだろうがよ! >>222 和田メソッドによると「非公開メンバをテストすると品質が下がるので絶対するな！」 意識が高くなりすぎて幽体離脱した感のある和田メソッドをよろしく！ >>22 >>241 >>242 >>242 >>244 >>243 >>242 結局クラス分けするなりしてpublicにする部分を多くする、 「privateメソッドがテストできないんですがどうしたらいいですか？」という疑問に対して、 >>248 だって可視性とテストするしないの関係は自明じゃないじゃん。 >>231 publicにしたら全然関係ないパッケージからもアクセスされてしまうからね >>3 のような下痢便コードを書いてしまうんだね >>252 >>253 >>254 >>255 テストから呼び出せるのが問題なんじゃなくて だからパッケージプライベートなら良いだろうと言ってるわけ >>256 パッケージプライベートだと同じパッケージからならアクセスして良いというメッセージを >>257 >>260 >>261 >>264 >>263 RustやGoはprivateもテストできるよって言ってる人いたから >>265 >>266 >>267 privateにあくせすできる言語でも さすがです >>3 の下痢便コード書くだけありますね >>3 のコードを書いたのは別の人。 >>3 のコードをクソコードと言って >>275 同じ人が同じこと言ってるだけのスレ たとえばこの先Javaが進化したとしてテストオブジェクトのみを特別視して いまの言語仕様での最適解は >>5 ですね >>248 テストコードは設計の外にあるものだからクラスの可視性とか直接関係ないだろ。 >>281 クラス内を調査してprivateメソッドを列挙して >>5 のように書けますから >>285 世の中にある全てのカバレッジ計測ツールは >>285 を見る限り何もわかってない気がしますねｗ >>286 >>288 >>285 みたいな的はずれなことを言ってるんだねｗ >>289 >>290 >>284 >>291 僕はいままで手作業による結合テストだけで生き抜いてきた生粋のブラックボックスマンです 僕は単体テスト書いたことありますよ >>5 で書きました >>292 > 最終的な設計ドキュメントはコードから生成するものだからね 正直、詳細なクラス図は、設計図の中で一番不要だと思ってる。 >>297 単体テストという言葉の曖昧さがpublic派か、private派かの分裂の原因です >>301 >>302 頑張って書いた、これはもう論文と言って良いのでは >>304 >>5 で単体テスト書きましたよ 288 返信：デフォルトの名無しさん[] 投稿日：2020/07/11(土) 12:15:29.92 ID:JFnadz6+ [26/35] >>286 君はレトリックな部分に引きづられる傾向があるから 僕はJavaしか知らないと言ったけどC#に関するあっと言わせるような知識と考察を垣間見せたでしょ TDD信奉者がprivateメソッドのテストは必要ないと主張するのは当たり前 和田さんの話にも飽きたので、レガシーコード改善ガイドという本の話でもしようか？ https://bmf-tech.com/posts/%E3%83%AC%E3%82%AC%E3%82%B7%E3%83%BC%E3%82%B3%E3%83%BC%E3%83%89%E6%94%B9%E5%96%84%E3%82%AC%E3%82%A4%E3%83%89 https://maku.blog/p/p6awy3z/#%E7%AC%AC10%E7%AB%A0-%E3%81%93%E3%81%AE%E3%83%A1%E3%82%BD%E3%83%83%E3%83%89%E3%82%92%E3%83%86%E3%82%B9%E3%83%88%E3%83%8F%E3%83%BC%E3%83%8D%E3%82%B9%E3%81%A7%E5%8B%95%E3%81%8B%E3%81%99%E3%81%93%E3%81%A8%E3%81%8C%E3%81%A7%E3%81%8D%E3%81%BE%E3%81%9B%E3%82%93 北朝鮮行ったことあるか？ >>317 話をしようかと言い、本の内容をコピペしただけとかゲンナリするよね 本に書いてあることを自分なりに実践してみた結果大失敗して 実践した結果コードがきれいになり 車の運転がうまい人は車の危険性を知ってる人 ググってみたけどTDDに否定的な考え方は世界的にあるみたいね まるで○○は危険でしょうと言うだけで >>325 >>286 >>327 https://www.infoq.com/news/2008/01/private-methods-tdd-design/ >>329 > 実践した結果コードがきれいになり >>334 >>336 >>337 世の中に銀の弾丸は無いからね >>340 つまり ちなみに僕は常識に価値があるとも思ってない >>342 >>344 >>345 >>347 >>348 > >>337 君は疑いをかけられてると思っていた もうお互い大変でしょう、僕と仲直りしましょうか？ 和田メソッドの真骨頂は、バグを作り込まなければテストする必要が無いことにある。 クソみたいな議論ばっかに見えるがプログラム技術板の中ではわりかしまともな方になってるぞ。 >>355 それは恐ろしい話だけど、このスレにもちらほらいるね。 そういう人に、どういう被害を受けたかを説明できればね（笑） オウム真理教の信者は被害者でもあり加害者でもある。 上の方でも書いたけど「例え」は何かを批判するときに使ってはだめ 和田メソッドの信者は被害者でもあり加害者でもある。 しかしその理由は何一つ言えない。中身が無いからだ。 >>366 説明に対して全く聞く耳を持たず、和田経典に書いてあると繰り返すだけだから、宗教的なナニカ扱いされてる。 t-wadaがお勧めする和田メソッド。 >>367 どこで否定されてるのとか、これまでのみんなの説明が全くの無駄。 前スレですばらしい分析を見たなあ >>870 >>372 >>372 >>371 >>375 和田さんはこのスレに居ないと書き込んでるのが御本人かもしれない。 失速したな TDDを布教する人が何回も同じコード書いてうまくなっていくように 和田さんじゃなくてTDDの話をしよう テスト書くタイミングについてなら少しだけ。 最終的なプログラムの詳細設計がパッと頭に浮かぶ人ならテストから書けるんだろうけど >>390 >>390 相変わらずオブジェクト指向信じて わかりやすいREPLが使える言語のほうが学習効率が高いのは確かだが p obj >>397 https://paiza.io/projects/OnW9KR89v5m-CY5JXIV2tw PythonやJavaScriptがイケてる言語であることは間違いない Javaは昔はオブジェクト指向一筋の堅物みたいな感じだったけど そもそも単体で平置き出来る関数が出来るまでバカみたいに長い年月が掛かったからその間に他の言語に抜かされていった オブジェクト指向もテストの文脈で語られるようになって、 胡散臭くない小奇麗なバカは減って無い、ってことだな >>405 TDDに限定するならともかく、一般論として「テストの文脈で」語ろうとするのは class/object instance scopeにあるmethodではない関数または関数objectのことではないかいな 「平置き出来る関数」なんてどこの村で通じる方言なん？ 文脈でわかると思うけどね 慎吾ママが両手を広げて「おっはー」と言ったらおはようの意味だとわかるし 他人と議論をしようと言う時に文脈でしかわからない言葉を使うな無能が >>421 プログラムでもそうだよね 自分の説明能力の無さを相手が低脳だからと責任転嫁するヤシ >>422 自分の理解能力の無さを相手が無能だからと責任転嫁するココナツ >>427 で、結局のところ >>430 無限ココナツ 変数にセット可能、引数として渡すことができる、戻り値にセットできる Javaはクラスに包んで間接的に置くしかない 用語はともかくラムダ式のことだと分かってスッキリした 第一級函数のことかと思ったよ俺は 結局単にラムダ式のことを言おうとしてたん？ 単体で置ける関数のことを単体関数と言い、それを平置きして式化したものが平置き式 みーっけた！ http://itest.5ch.net/mevius/test/read.cgi/tech/1594702426/124 https://github.com/katahiromz/3D フォルダに平積みする事か？ 本の平積みと同じ意味だよ ポリモーフィズムって押し付けなんだよね ttps://i.imgur.com/qRx3DMV.gif >>451 >>451 ドキュメントとセットでの運用なら、 >>455 ソリューションエクスプローラーなんて芋っポイのやめてWordの項目一覧でいいよ ソリューションエクスプローラのCMakeターゲットビュー見てみろよ。 それはちょっと違うかもしれないな。 data Status = Dead | Alive >>455 プライベートメソッドのドキュメントを作るんだろ？ プライベートだろうとパブリックだろうと、構造がコードでわかりにくけりゃ作るよ。 要するに馬鹿にはプログラミングさせるなということだな 普通は表面的＝形式的なモンで判断するんでしょこの業界 構造図みたいなのはプログラム内には書けないから >>397 >>397 ■ このスレッドは過去ログ倉庫に格納されています
