bug in addMonths method about shamsi_date HOT 5 CLOSED

من یه نگاه به کد پکیجتون انداختم ...
مثلا وقتی شما روز رو لازم ندارین حتی دستی دارین ۱ مینویسین توی استرینگ
توی ریدمی هم توضیح دادم فک کنم

j.withDay(1).addMonths(1)

می تونین بزنین
این کاملا سیف هست

یه پیشنهاد دیگه هم این که
اگه فقط شمسی و میلادی رو ساپورت می کردین
می تونستین مستقیما از خود Jalai و Gregorian استفاده کنین، که ساب گلس Date هستن هر دو
نیازی به استرینگ کردن و ‌... نبود
این طوری سیف تر هم بود
این طوری کدتون هم خیلی کوتاه تر میشد
پرفورمنس هم بالا می‌رفت احتمالا

اون لایبرری php رو هم یه نگاه انداختم
همون حالت دوم هست که گفتم تصحیح می‌کنه خودش، میشه یه سری متد با اجازه دادن اورفلو اضافه کرد تو شمسی دیت هم.
مثلا addMonthsSafe
مشابه مال اون
باید یه بررسی کنم سر فرصت ...

from shamsi_date.

ممنون کلوز کردین
شرمنده من دوباره کدتون رو یه نگاه انداختم
به جای
int.parse(date.formatter.y)
می تونین خیلی راحت
date.year
بزنین
تو ریدمی هم گفتم
خیلیا این اشتباه رو می کنن !
سلامت باشید انشاءالله

from shamsi_date.

سلام
دقیقا داکیمنتیشنش اشاره کردم همین مطلب رو. و هم توی ریدمی.
کار دیگه ای به ذهنم نمیرسه ... شما اگه پیشنهاد بهتری دارین بگین.
چون مثلا اگه منظورتون این بود که به جای یک ماه ۳۰ روز اضافه کنم که خوب ۳۰ روز خودتون با + یا متد اضافه کردن روز اضافه می کردین.
اگر هم که نه من از کجا بدونم که مثلا یک ماه بعد از ۳۱ شهریور منظورتون کی هست ؟ باید برم ۳۰ مهر یا مثلا ۱ آبان ؟ مشخص نیست دیگه.
توی ریدمی کامل بخونین یه سری توضیح دادم
باید بگین کاربردتون دقیق چیه تا راهنمایی کنم.
این جور موارد رو دو سری اپروچ من دیدم براش٫ یه سری مثل این لایبرری خطا میدن٫ یه سری سعی می کنن تصحیحش کنن٫ من خطا دادن رو ترجیح میدم. چون اولا جواب غیر قابل پیش بینی نامیده لایبرری ثانیا باعث میشه یوزر اون چیزی که می خواهد رو دقیق مشخص کنه و جلوی باج های لاجیکال رو می‌گیرد.

from shamsi_date.

@FatulM

ممنون از پاسختون. حرف شما درسته و ما میتونیم توی پکیجمون راه حلی برای این قضیه پیدا کنیم
ولی برای این چالش فکر میکنم باید یه ذره بیشتر توسعه اش بدیم و حالت های مختلف رو در نظر بگیریم
مثلا فایل زیر رو ببینید
https://github.com/briannesbitt/Carbon/blob/master/src/Carbon/Carbon.php
و داخل این فایل addMonth رو جستجو کنید

from shamsi_date.

@FatulM
خیلی ممنون از وقتی که گذاشتید . حتما از پیشنهادی که دادید استفاده میکنیم

@HoseinHaqiqian
پیشنهاد امیررضا برای روز اول ماه بهتر از کاریه که الان میکنیم

from shamsi_date.