search

haushinka2dx / atmosphere-web

1 stars 2 forks source link

Refactoring201310 #15

Closed haushinka2dx closed 11 years ago

haushinka2dx commented 11 years ago

主に js/atmos.js をリファクタしました。 やろうと思えば他の部分も出来ますがとりあえずここまでで。

ちなみにテストケース（BDD的に何て言う？）も追加しました。

kannokanno commented 11 years ago

いいですね！

テストケース（BDD的に何て言う？）

「テストを書いた」というニュアンスで「スペック(Spec)を書いた」というのを見たことがありますが、 私は特に意識せず普通に「テスト(テストケース)」という単語のまま使っている気がします。 (ので、あんま分かっていない)

haushinka2dx commented 11 years ago

確かにspec（仕様）ですね。 余談ですが初めてのspecで（慣れないなw）バグ見つけていきなり恩恵にあずかりました。 BDD良いね。

kannokanno commented 11 years ago

テストを書くと自分自身が安心できるので好きです。

あとでレビューしまーす

haushinka2dx commented 11 years ago

よろしくです！

kannokanno commented 11 years ago

以上でーす

haushinka2dx commented 11 years ago

レビューありがとうございました。 質問で返しまくってしまっていてすみませんが意見いただけると嬉しいです。

kannokanno commented 11 years ago

:+1:

haushinka2dx commented 11 years ago

レビュー結果を反映しました。問題なければマージしまする。

haushinka2dx commented 11 years ago

そこまで理解して使ってないです。 と言うかそこまでいくと再帰的にディープコピーするようにしなきゃなくなりませんか？ 不理解すんません。ホントは調べてから書きたいんですが。

kannokanno commented 11 years ago

おっしゃるとおりです。 前のコメントで書いたjQueryのextendは再帰処理にてディープコピーするようになっています。

認識合わせ程度のものですので、このままでもいいと思います。

haushinka2dx commented 11 years ago

なるほど！ 再起の方に書き換えて後でマージします！