שלום לכולם, כתבתי פונקציה פשוטה לבדיקה האם שבת

הפונקציה לוקחת את המיקום המשוער של המשתמש ובודקת אם יום שישי אן שבת אצלו
ואם כן בודקת שקיעה או חושך
מה דעתכם?

https://github.com/djboy/MSisS

בעייתי אם לאתר יש יותר 4 היטים לשנייה בממוצע במשך יותר מדקה (המשתמש בסקריפט שלך שישתמש ב API שאתה משתמש בשירות שלו בקוד - יחסם)... ולא תומך בשום מנגנון של caching - ואני בכלל לא מדבר על זה שעבור כל גולש שנכנס, אתה מתחבר לאתר אחר, שזה דבר נורא מבחינת ביצועים...

יש סיבה מיוחדת שאתה ממציא את הגלגל מחדש ולא משתמש ב GeoIP החינמי? (ואולי על הדרך תומך בקאשינג בזיכרון על כל אחת מהשכבות - מיפוי IP ל TZ והאם ב TZ כעת שבת או לא, כדי לחסוך בחישוב?)

כמו כן, אמנם הגדרת את זה מראש כ"האם שבת", אבל בד"כ למי שאיכפת לדעת אם עכשיו שבת או לא, איכפת יהיה גם לדעת האם שאר מועדי ישראל: ראש השנה (יומיים), יום כיפור, סוכות, שמיני עצרת / שמחת תורה (בחו"ל), חג ראשון של פסח, חג שני של פסח, שבועות... אז אולי כדאי שתחשוב על זה גם.

my 2c

מעבר לשיקולים התכנותיים ששימי ציין, החישוב שלך לקביעת השבת אינו נכון. תוספת השבת לפני כניסתה ולאחר יציאתה אינה שווה סתם ל־"10 דקות".

תוספת השבת בכניסה נעה בין 20 ל־40 דקות לפני השקיעה. תלוייה במנהג/בעיר.
ברוב הערים היא 20 דקות, בחלקן (טבריה, חיפה, אשקלון, בית־שמש ועוד..) נוהגים חצי־שעה, ובבודדות, כמו ירושלים ופ״ת מוסיפים 40 דקות.
התוספת בצאת השבת, לאחר צאת הכוכבים, היא כמעט באופן גורף 18 דקות.

ואם כבר.. סוף דמדומים אזרחיים הם לא בדיוק "צאת הכוכבים" ההלכתי.
צאת הכוכבים היא בדר״כ 18 דקות מהשקיעה (להוציא שיטת ר״ת).

תוספים לפורום
קורס לטינית בלעדי לפרש, בפורום שפות ובלשנות.