שימוש ב-MongoDB בנוד

[dovid]

מה שהייתי עושה זה פוקנציה, getDb().
נראה לי שכל דבר אחר יהיה בו חוסר אלגנטיות ותחזוקה מעצבנים יותר.

[yossiz]

@dovid זה מה שרציתי לחסוך
ה-getDb הרי יחזיר פרומיס ולא DB ואז צריך לשרשר כל דבר עם then-ים...

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

toplevel await היה עוזר הרבה כאן.

[dovid]

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

אכן נשגב מבינתי למה נוד לא עושים toplevel await (וגם למה הם לא הופכים הכל הכל במודולים שלהם מקאלבק לפרומייז).

[dovid]

כעת אני מחפש ורואה פתרונות טובים יותר (כלומר בטוחים לגמרי)
https://stackoverflow.com/a/49400334/1271037

[dovid]

כעת אני חושב שלמה לא להכניס את כל טעינת האפליקציה לקאלבק?
זה לגיטמי, אני מאשר את זה

[yossiz]

@dovid אמר בשימוש ב-MongoDB בנוד:

כעת אני חושב שלמה לא להכניס את כל טעינת האפליקציה לקאלבק?

בדיוק באתי לכתוב את זה... תודה רבה!

[אהרן]

@yossiz אמר בשימוש ב-MongoDB בנוד:

שלא קיים toplevel await

בדרך
אבל זה לא דבש מוחלט.

[אהרן]

@yossiz אמר בשימוש ב-MongoDB בנוד:

הפיתרון הוא לכאורה להחזיר אובייקט שיעטוף את ה-db. כל פונקציה שנקרא על העטיפה יעשה קודם await לאובייקט ה-db האמיתי ואז יעביר את הפונקציה הלאה לאובייקט האמיתי.

האם יש דרך לעשות עטיפה כזאת בלי יותר מדי boilerplate? פשוט להגיד ל-JS שכל פונקציה שאני יקרא על האובייקט לא משנה מה השם שלו יעבור דרך קוד העטיפה?

לא מבין את הבעיה
הקלאס מיד חוזר מכיל בתוכו את ה-DB
ופונקצית insert({1:2}) יכיל בתוכו בדיקה אם ה-DB מחובר, ומחזיר תמיד פרומיז.

כך תמיד עושים?!

[אהרן]

@dovid אמר בשימוש ב-MongoDB בנוד:

למה לא להכניס את כל טעינת האפליקציה לקאלבק?

לחזור לטרום ימי הפרומיז?

[dovid]

@אהרן לא התכוונתי דוקא קלאבק, התכוונתי שהאפליקציה כולה תיטען מתוך פונקציה.
ולגבי הלא דבש שבדרך, הרי נוד מאמצת ממש לאט את ההתקדמויות האלה, לא כן? נראה לי הדפדפנים יעקפו אותה עוד מעט.

[אהרן]

@dovid אמר בשימוש ב-MongoDB בנוד:

נוד מאמצת ממש לאט את ההתקדמויות האלה

שמח שהם מאפשרים לדנו לעקוף אותם בקלות

[אהרן]

@dovid אמר בשימוש ב-MongoDB בנוד:

לא התכוונתי דוקא קלאבק, התכוונתי שהאפליקציה כולה תיטען מתוך פונקציה.

לא הבנתי מדוע לא להשתמש ב-await
(בפונקציית insert שאמור להיקרא מתוך פונקציה, לא ב-require)?

[yossiz]

@אהרן אמר בשימוש ב-MongoDB בנוד:

ופונקצית insert({1:2}) יכיל בתוכו בדיקה אם ה-DB מחובר, ומחזיר תמיד פרומיז.

הפונקציה insert מוגדר כבר על ה-db. לא רציתי לעבור על כל הפונקציות של ה-db ולהעתיק אותם לקלאס. בשביל זה ביקשתי משהו שחוסך boilerplate.
בשפות דינאמיות אחרות יש אפשרות להגדיר פונקציה שירוץ אם קוראים לפונקציה לא קיימת על אובייקט. אם זה היה קיים ב-JS הייתי יכול לעשות פרוקסי ל-DB בקלות עם מעט מאוד קוד.

[yossiz]

@dovid הבעיה היחידה שיש לי עכשיו היא ש-vscode לא מזהה את הטייפ של require('./db').db כנראה בגלל שנותנים לו ערך רק בתוך callback, ולכן אני לא מקבל השלמה אוטומטית.

האמת היא שהגיע הזמן שאלמד typescript...

[yossiz]

@yossiz אמר בשימוש ב-MongoDB בנוד:

בשפות דינאמיות אחרות יש אפשרות להגדיר פונקציה שירוץ אם קוראים לפונקציה לא קיימת על אובייקט. אם זה היה קיים ב-JS הייתי יכול לעשות פרוקסי ל-DB בקלות עם מעט מאוד קוד.

זה קיים!
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Proxy

[dovid]

@yossiz יפה מאוד! זה ממש יכול להיות שימושי להמון דברים.
אני כעת חושב שזה יכול לפתור את בעיית הundefined ועוד כמה רעיונות.

[dovid]

@dovid אמר בשימוש ב-MongoDB בנוד:

אני כעת חושב שזה יכול לפתור את בעיית הundefined ועוד כמה רעיונות.

אני לא הייתי ברור, אני מתכוון שהרבה פעמים אני עושה שימוש באובייקט בjs כדי לעשות קיבוץ או פעולות אחרות של set (של key value). אם יש כזה דבר אפשר לעשות פוש לאיבר בלי לדעת אם הוא קיים הוא לא כאילו יש שמה כבר מערך (הפרוקסי ידאג ליצור אותו) וגם לבדוק count וכאלה דברים.
תוך כדי כתיבה אני חושב שזה לא כזה מציאה כי אחרי הכל זה קוד, ועם פונקציות יכולתי לעשות הכל, אני אחשוב בהמשך למה נראה לי שזה יעזור לי .