タグ

リファクタリングに関するjuve534のブックマーク (2)

  • privateメソッドをテストしたい - 日々常々

    と思うのは、とてもいいこと。 前置き もし行いたいテストが外的振る舞いを示すものであれば(少なくともテストにより観測できる見通しがなければ「テストしたい」とは思わないだろうから、何かしら外から観測可能なものではある可能性は高い)、それがprivateに閉じていていいものではないと言う気づきのきっかけになる。 と言うのは教科書的回答だけど、外には見せたくないけれど複雑なロジックを包含していて、入念かつ局所的にテストしたいと思うこともある。 この動機はすごく自然。きっとそこはテストしなかったらバグってるし。テストしてもバグが見つからないと言うのもよくあるんだけど。 この手のがどうあるのがいいのかはチーム体制も含めたプロダクトによると思っている。 綺麗な考え方は、独立したコンポーネントとして関心ごとや複雑性を閉じ込め、テストしたいと思った内容にもっと高い格を与える。「格」なんて表現は他で使ったこ

    privateメソッドをテストしたい - 日々常々
    juve534
    juve534 2022/03/14
    "表出した事象を叩き潰してはいけない。" 引用元のt-wadaさんの記事にあるように、publicメソッド経由でテストをすれば良くて、それでは事足りない場合は、そもそもコードを別クラス等に切り出したりすれば良いんだよね。
  • PR TIMESにおけるリファクタリングデー | PR TIMES 開発者ブログ

    こんにちは、業務委託でPR TIMESにJOINしているuzulla (”うずら”, twitter, GitHub)です。エントリではPR TIMESで行っているリファクタリングデーについてお話したいと思います。 リファクタリングデーとは?なぜ必要か? PR TIMESは歴史あるサービスです。サービス・機能は追加され、削除され、結果古いコードが大量に存在します。今後もスピード感を維持しつつ成長を続けるためにはそれらの整理・改善・削除、つまりリファクタリングが重要なことはいうまでもありません。 日々の業務においてリファクタリングが行われていれば問題ありませんが、日々サービスの成長にフォーカスしていくとやはり後回しになりがちで、タイミングを失います。 その為に「リファクタリングデー」という「リファクタリング作業を定期的に実施する日」をとりきめて実施しています。現在は月に一回程度開催され、3

    PR TIMESにおけるリファクタリングデー | PR TIMES 開発者ブログ
    juve534
    juve534 2021/12/24
    お祭り感があって楽しそう。一連の流れが記載されているので、「自分たちの場合はどう組み込むか?」も考えやすい。
  • 1