samanzamani / PersianDate

Persian date for android
BSD 3-Clause "New" or "Revised" License
326 stars 40 forks source link

مشکل تابع initJalaliDate #68

Closed mahramane closed 2 years ago

mahramane commented 2 years ago

سلام تابع initJalaliDate مشکل داره مثلا تاریخ initJalaliDate(1401, 4, 1, 0, 0, 0)

رو تنظیم میکنی توش بعد همونو میگیری 3 تیر رو بر میگردونه در حالی که من 1 تیر زدم. لطفا بررسی کنید ممنونم.

samanzamani commented 2 years ago

سلام ممنون از گزارش تون بررسی می کنم

mahramane commented 2 years ago

یه مشکل دیگه dayOfWeek() اشتباه میگه مثلا امروز که پنجشنبه هست باید 6 برگردونه ولی مینویسه 5 خواهشا اینارو درست کنید کارم گیره ممنونم

mahramane commented 2 years ago

تابع subDays و بقیه تابع های sub به صورت int ورودی میگیرن اما addDay و بقیه تابع های add به صورت long

samanzamani commented 2 years ago

سلام @mahramane در مورد تابع ‍‍initJalaliDate من تابع رو تست کردم و نتیجه درست رو داد لطفا اول کتابخانه رو به آخرین نسخه بروز کن و بعد کد زیر اجرا کن ببین چه نتیجه‌ای می گیری.

    PersianDate pDate = new PersianDate();
    pDate.initJalaliDate(1401, 4, 1, 0, 0, 0);
    Log.i("PersianDate", pDate.toString());//چهارشنبه 1 تیر 1401 

نتیجه ای که به من داد ‍‍چهارشنبه 1 تیر 1401 بود.

تابع dayofWeek همون طور که در مستندات ذکر شده از صفر شروع میشه. یعنی شنبه صفر میشه و جمعه ۶. بازه تغییراتش بین ۰ تا ۶ هست و تابع به درستی کار می کنه.

در مورد int بودن پارامترهای ورودی همون طور که به درستی اشاره کردی ورودی ها long تنظیم شده بود که در کامیت آخر اصلاح شد.

لطفا موارد بررسی کن اگر ابهامی نبود بگو تا نسخه جدیدی منتشر کنم.

باتشکر از وقتی گذاشتی و برای بهبود این کتابخانه مشکلات گزارش دادی.

mahramane commented 2 years ago

بله درسته ممنون بابت پیگیریتون زنده باشید