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

مشکل تابع initJalaliDate #68

Closed
mahramane opened this issue Jun 26, 2022 · 5 comments
Closed

مشکل تابع initJalaliDate #68

mahramane opened this issue Jun 26, 2022 · 5 comments

Comments

@mahramane
Copy link

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

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

@samanzamani
Copy link
Owner

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

@mahramane
Copy link
Author

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

@mahramane
Copy link
Author

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

samanzamani added a commit that referenced this issue Jul 1, 2022
@samanzamani
Copy link
Owner

samanzamani commented Jul 1, 2022

سلام @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
Copy link
Author

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

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

No branches or pull requests

2 participants