Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

between 関数を実行すると内部に保持されている祝日データセットの値が変化する #36

Open
camelmasa opened this issue Jun 16, 2022 · 1 comment

Comments

@camelmasa
Copy link
Member

camelmasa commented Jun 16, 2022

https://github.com/holiday-jp/holiday_jp-js/blob/master/lib/holiday_jp.js#L22

この実行によって、祝日データセットの値が date が文字列型から Date 型の値へ変化する。
between 関数は Date 型に変更してオブジェクトを返す意図があると思われるが、値の変更自体は意図しているでしょうか? 🤔

var holiday_jp = require(".")
holiday_jp.holidays['2010-09-20'].date //  '2010-09-20'
var holidays = holiday_jp.between(new Date('2010-09-14'), new Date('2010-09-21'));
holiday_jp.holidays['2010-09-20'].date // 2010-09-20T00:00:00.000Z

何らかの方法で shallow copy ではなく deep copy の処理をすることで、変数が変更される事は避けられそうです。

@k1LoW
Copy link
Member

k1LoW commented Aug 1, 2022

値の変更自体は意図しているでしょうか?

意図はしていないと思います

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants