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

[yossiz]

אני רוצה לעשות מודול שיחזיר לי מופע של MongoClient.db שאוכל להשתמש בו מיד.
אופן השימוש יהיה כך:

db = require('./db')
db.collection('xyz').insert({1:2});

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

נסיון נאיבי שלא עובד:

const MongoClient = require('mongodb').MongoClient;

const url = 'mongodb://localhost:27017';
const dbName = 'dbname';

const client = new MongoClient(url);
let db;

client.connect()
  .then(client => {
    console.log('Connected!');
    db = client.db(dbName);
  })
  .catch(err => {
    console.log('Failed to connect:', err);
  });

module.exports = db;

[yossiz]

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

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

[dovid]

ראשית אני לא כ"כ משתמש בנוד, אז אני לא יודע דברים בסיסיים.
אני מבין ראשית כל שכל השאלה נשענת על כך שא"א לעשות await require.

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

const MongoClient = require('mongodb').MongoClient;
 
const url = 'mongodb://localhost:27017';
const dbName = 'dbname';
 
const client = new MongoClient(url);
let container = {};
 
client.connect()
  .then(client => {
    console.log('Connected!');
    container.db = client.db(dbName);
  })
  .catch(err => {
    console.log('Failed to connect:', err);
  });
 
module.exports = container;

db = require('./db').db;

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

[yossiz]

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

אני מבין ראשית כל שכל השאלה נשענת על כך שא"א לעשות await require.

אפשר, אבל צריך לעטוף אותו בפונקציה בגלל שלא קיים toplevel await (אולי קיים בצורה נסיונית, לא בדקתי).

בפתרון הנאיבי שלך הexport קורה לפני החיבור, ממילא הexport מוציא null.

בדיוק

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

כן, אבל לא אוכל להשתמש בו בצורה עיוורת כי לא אוכל להיות בטוח שזה מאותחל. אני רוצה לקבל מהמודול אובייקט שאוכל להשתמש בו בלי לדאוג שהוא לא מאותחל.

[dovid]

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

כעת אני קולט שגם הקוד שלי לא טוב,
כי הrequire קורה מייד.
אז בכל מקרה צריך פונקציה עם await בכל דף שתשתמש (לא בכל שימוש רק אחת בתחילת הדף להשמת האובייקט). אכן מעצבן ומזכיר שJS היא עדיין JS...

[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