おやつ前のTDD

あひさめ @a_hisame

TDDなう。 内部メソッドのテストをしたいんだけれど、そのメソッドのパラメタがコンストラクタの物をそのまま利用するものであった場合、テストコードに迷い中。

あひさめ @a_hisame

Hoge hoge = new Hoge( new String[]{"abc", "bbb"} ); hoge.method( /* コンストラクタの引数*/ ); で methodはHogeのコンストラクタ内で呼ばれる場合、どんなコードを記述すべきだろうか

あひさめ @a_hisame

もちろん、hoge.methodは独立して作業を行えるので 適当な初期化をするか、フィールドに持たせて流用するか……

ぐるぐる系SQL @bleis

@a_hisame そのメソッドをテストしたいのかコンストラクタをテストしたいのか。そのメソッドがprivateなら無理してテストする必要ないんじゃ？ [TDDBC名古屋7/10,11予定]

あひさめ @a_hisame

@bleis 正直privateでいいんだけれどねー。 テストした方が安全な気がして・・・・・・この辺りのテストする/しないの勘所が難しい

あひさめ @a_hisame

@bleis まあ、あとはpackage privateレベルにして、private無しでも良いかなと思いつつ処理してるのもある。 現状privateなし

ぐるぐる系SQL @bleis

@a_hisame なら設計を見直す機会ととらえるか [TDDBC名古屋7/10,11予定]

あひさめ @a_hisame

@bleis どうなのかなー。 設計としてはコンストラクタで投げて各種初期化 => 使う だけなんだけれど

あひさめ @a_hisame

@bleis この初期化に色々と関数を切り出しているから、そこを外から個別テストして正当性を確かめたいってだけなんだけれど。

あひさめ @a_hisame

いっそ、テストを作ってテスト＞消す(private化)でも良いのかな。。。

ぐるぐる系SQL @bleis

@a_hisame 今のクラスの、そのメソッドで初期化するフィールドをまとめてクラスに抽出すればおｋ [TDDBC名古屋7/10,11予定]

ぐるぐる系SQL @bleis

@a_hisame コンストラクタ内からメソッド呼んじゃう時点で、イミュータブル性の保障できないし(finalフィールドに出来ない)、まとめてクラスにしちゃってもいいんじゃないかな。いや、詳細知らんからわからんけど [TDDBC名古屋7/10,11予定]

あひさめ @a_hisame

@bleis いや、フィールドはメソッドの結果を受けて初期化するだけだよ？

あひさめ @a_hisame

@bleis finalフィールドにするために結果をメソッドに丸投げ＝＞コンストラクタでfinalフィールドに設定 をしてる

ぐるぐる系SQL @bleis

@a_hisame ん？各種初期化ってのは？ [TDDBC名古屋7/10,11予定]

あひさめ @a_hisame

@bleis そのクラスの全てのフィールドを(少なくとも見た目上は)イミュータブルに初期化する

あひさめ @a_hisame

見た目上はってのは、ユーザから見てって意味ね。 変更操作は一切ユーザには与えない。

ぐるぐる系SQL @bleis

@a_hisame hogeみたいなメソッドがたくさんあるってこと？ [TDDBC名古屋7/10,11予定]

あひさめ @a_hisame

@bleis 高々２，３個だとは思うけれど。 そんなに数多くはない。

ぐるぐる系SQL @bleis

@a_hisame それでも、コンストラクタ内で仕事しすぎかな、って感じが [TDDBC名古屋7/10,11予定]

あひさめ @a_hisame

@bleis んー。まあ今回は自分の考えで通してみる。 また機会があったら（疑似）コード見て議論しよう。多分そっちの方が良い。

rf @rf0444

@a_hisame なんか複雑になってるなぁ。俺のすげぇ単純になったけど・・・

あひさめ @a_hisame

@rf0444 ああ、テストの書き方について思うところがあっただけ。 書いてる記述自体はかなり単純。

なかわんくま/nakawankuma/ @nakawankuma

. @a_hisame @bleis コンストラクタでは今は処理しないのがいいよ。

rf @rf0444

@a_hisame 1クラスで、公開コンストラクタ1個と公開メソッド1個と公開データ構造1個。packageアクセスのヘルパが3つ。