עזרה עם async await


  • תכנות

    אולי הכותרת צריכה להיות: הבנת השגיאה Promise resolver undefined is not a function

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

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

    async function getValue() {
    	let value = 0;
    	let keyes = ['cow', 'deer'];
    	
    	if(any_condition) {
    		await Promise.all( keyes.map( row => {
    			return new Promise((reslove) => {
    				db.get('value', (err, newValue) => {
    					if(err)	console.log(err);
    					
    					value += newValue;
    				});
    				reslove();
    			});
    		}));
    	}
    	return new Promise().reslove(value)
    }
    
    

  • תכנות

    function getValue () {
      let value = 0;
      let keyes = ['cow', 'deer'];
    
      return new Promise((resolve, reject) => {
        if (any_condition) {
          return resolve(Promise.all(keyes.map((row) => {
            return new Promise((resolve, reject) => {
              db.get('value', (err, newValue) => {
                if (err) console.log(err); // return reject?
                return resolve(value += newValue);
              });
            });
          })));
        }
        return resolve(value);
      })
        .then((result) => {
          return result;
        });
    }
    
    


  • יש שתי טעויות, הבנה והקלדה.
    טעות בהבנה, פונקציית await לא אמורה להחזיר פרומייז אלא ערך ישיר.
    כלומר במקום return new Promise תחזיר סתם return value;

    טעות הקלדה, הreslove ללא פרמטר צריך להיות בתוך הבלוק של הdb.get שורה מתחת לvalue += newValue;

    אגב, במידה וכן היית צריך פרומייז אפשר לכתוב Promise.reslove(value) ללא new.

    הנה קוד עובד:
    https://stackblitz.com/edit/js-d1yhhp

    אני מניח שאתה יודע שאתה יכול לעשות באותה מידה את אותו קוד ללא async וawait אלא שאתה מעדיף ללמוד על הדרך העדכנית יותר.



  • @zvizvi לthen האחרון אין משמעות, בכל מקרה אתה מחזיר פרומיז.
    כמו"כ הdb.get מחזיר ערך ישירות (undefined, אלא"כ הנחת שזה פרומייז, שזה מידע שאין לי),
    אז אם אני צודק אתה בעצם לא מחכה לתשובה שלו.


  • תכנות

    @dovid אמר בעזרה עם async await:

    @zvizvi לthen האחרון אין משמעות, בכל מקרה אתה מחזיר פרומיז.

    נכון, רק בשביל ההבנה שהכל מגיע לפה.

    @dovid אמר בעזרה עם async await:

    כמו"כ הdb.get מחזיר ערך ישירות (undefined, אלא"כ הנחת שזה פרומייז, שזה מידע שאין לי),

    צודק, תיקנתי.


  • תכנות

    @dovid
    ראשית, תודה גדולה על ההשקעה!!

    טעות בהבנה, פונקציית await לא אמורה להחזיר פרומייז אלא ערך ישיר.

    ידוע, אני לא צריך כאן ערך חוזר, אלא לעדכן לעדכן את הערך, השימוש בפרומיס (נכון יותר ב-await הוא בשביל שהשלב הבא ימתין לסיום כל עדכוני הערך.

    כלומר במקום return new Promise תחזיר סתם return value;

    1 לא ברור לי ההקשר ואיך זה מסביר את השורה הקודמת
    2 הסיבה שמספיק להחזיר ערך הינו בגלל שכל מה שאחרי await עטוף בפרומיס?

    אגב, במידה וכן היית צריך פרומייז אפשר לכתוב Promise.reslove(value) ללא new.

    טוב לדעת, תודה!!

    הנה קוד עובד:
    https://stackblitz.com/edit/js-d1yhhp

    תודה!!

    אני מניח שאתה יודע שאתה יכול לעשות באותה מידה את אותו קוד ללא async וawait אלא שאתה מעדיף ללמוד על הדרך העדכנית יותר.

    לא. למעשה זו אותה דילמה כמו ב-await.



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



  • @dovid לקחתי את הקוד שלך, וערכתי אותו בצורה שהוא יותר אלגנטי וקריא לדעתי.
    https://stackblitz.com/edit/js-wcyk6m

    מעוניין לשמוע חוו"ד אם יש שם טעויות או מקום לעוד שיפורים


  • תכנות

    @dovid אמר בעזרה עם async await:

    בשביל המתנה משתמשים בפרומייז בדיוק באותה המידה של הawait.

    איך אני כותב א"ז בפרומיס
    אם יש לי התניה

    if(...) {
       update from async function
    }
    return value
    

    כך?

    if(...) {
       update from async function
       reslove(value)
    }
    reslove(value)
    

    א"כ רציתי להימנע מלהפעיל את reslove בנפרד ל-2 האפשרויות (משפט מאוד).


  • תכנות

    @dovid אמר בעזרה עם async await:

    @אהרן לא הבנתי אותך אבל אתה בודאי לא הבנת אותי.

    נירגע שניה מחלוקת הציונים ונדון ענינית.

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

    ההגיון ימצא במידה ורוצים לעשות לתשובה .then()



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



  • @אהרן אמר בעזרה עם async await:

    נירגע שניה מחלוקת הציונים ונדון ענינית.

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



  • @dovid אמר בעזרה עם async await:

    אתה דבר ראשון "ידעת" שהdb.get זה פרומייז

    לא, כתבתי פונקציה promisifiedDbGet שעוטף את ה-db.get המקורי בפרומייז.



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


  • תכנות

    @dovid אמר בעזרה עם async await:

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

    db.get לא מחזיר פרומיס
    בשביל שPromise.all ימתין שכולם יסתיימו, עטפתי את הקירות לdb בפרומיס.



  • @אהרן אמר בעזרה עם async await:

    db.get לא מחזיר פרומיס

    באשכול השני כתבת שבלי callback זה מחזיר promise



  • @yossiz קראתי, נהדר.
    את הreduce יש להחליף ב.join(''), במקרה הזה מדובר בשרשור פשוט וזה בדיוק התפקיד של Array.join.


  • תכנות

    @yossiz אמר בעזרה עם async await:

    @אהרן אמר בעזרה עם async await:

    db.get לא מחזיר פרומיס

    באשכול השני כתבת שבלי callback זה מחזיר promise

    אבל בדוגמא כאן כתבתי עם

    db.get('value', (err, newValue) => {
    

  • תכנות

    @dovid אמר בעזרה עם async await:

    @yossiz קראתי, נהדר.
    את הreduce יש להחליף ב.join(''), במקרה הזה מדובר בשרשור פשוט וזה בדיוק התפקיד של Array.join.

    הוא צודק במידה ורוצים ערך מספרי.


  • תכנות

    @dovid
    אפי' שקראתי שוב ושוב [ושוב], עדיין לא מסכים עם הביקורת שלא קראתי בעיון. לא חשוב.

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

    השאיפה שלי היתה לא להחזיר את הערך בכל א' משני האפשרויות אלא אחרי כל ההתניה, ועם זאת שימתין לעדכון הא-סינכרוני (רק בתנאי שמתבצע עדכון א-סינכרוני)
    לכן עטפתי את הפונקציה העדכון הא-סינכרוני עם פרומיס לתוך Promise.all עם הקידומת await ע"מ שכל מה שאחרי העדכון (החזרת הערך) ימתין לסיום כל העדכונים. את הערכים שחזרו מהפרומיס'ס אני לא צריך.

    async function getValue() {
    	let value = 0;
    	let keyes = ['cow', 'deer'];
    	
    	if(any_condition) {
    		await Promise.all( keyes.map( row => {
    			return new Promise((reslove) => {
    				db.get('value', (err, newValue) => {
    					if(err)	console.log(err);
    					
    					value += newValue;
    					reslove();
    				});
    			});
    		}));
    	}
    	return new Promise().reslove(value);
    }
    

    הטעות שמעלה את השגיאה היא

    return new Promise().reslove(value);
    

    והיא שגויה גם אם נכתוב אותה כפי שמחזירים reslove באופן תקין, כך

    return Promise().reslove(value);
    

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

    reslove(Promise().reslove(value))
    

    או משהו דומה, שזה שגוי.

    כפי ש @dovid כתב:

    טעות בהבנה, פונקציית await לא אמורה להחזיר פרומייז אלא ערך ישיר.

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

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

    שוב תודה ל @dovid @zvizvi @yossiz על הדוגמאות וההסברים המושקעים ומפורטים!



  • @אהרן לא הבנתי כל מה שאתה כותב, אבל השגיאה שאתה מקבל הוא פשוט שה-constructor של Promise אמור לקבל פונקציה שעושה את ה-resolve או reject,

    הודעת השגיאה מאוד מובנת, Promise resolver undefined is not a function
    פי', promise resolver זה הפונקציה שהיית אמור לספק ל-constructor
    undefined, - מכיון שלא סיפקת פונקציה, אז הפרמטר קיבלה את ה(אי-)ערך undefined
    is not a function - הוא לא פונקציה

    מה שהתכוונת לכתוב זה זה:

    return Promise.resolve(value)
    

    בלי הקריאה ל-constructor
    זו פונקציה שמחזירה promise שהוא כבר במצב resolved עם הערך - value

    זה שווה לזה:

    return new Promise(resolve=>resolve(value));
    

  • תכנות

    @yossiz אמר בעזרה עם async await:

    return Promise.resolve(value)

    להחזיר

    return Promise.resolve(value)
    

    במקרים אחרים, זה לא שגיאה



  • @אהרן אמר בעזרה עם async await:

    להחזיר

    return Promise.resolve(value)
    במקרים אחרים, זה לא שגיאה

    אתה קובע או שואל?


  • תכנות

    @yossiz אמר בעזרה עם async await:

    @אהרן אמר בעזרה עם async await:

    להחזיר

    return Promise.resolve(value)
    במקרים אחרים, זה לא שגיאה

    אתה קובע או שואל?

    מספר עובדה.



  • כשמגיבים בפורום אמור להיות לתגובה ניגון כל שהוא: שאלה/קריאה/ויכוח/הסתייגות.
    אחרת זה משרה תחושה של בלבול.


  • תכנות

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


  • תכנות

    @חוקר אמר בעזרה עם async await:

    לעבוד עם פרומיס או לעבוד עם await ולכתוב את הקוד לאחריו.

    await זה פרומיס מאחורי הקלעים, רק עם סינטקס נוח יותר.



  • אני ממליץ לעבוד עם async\await, אם בכלל, רק אחרי שליטה והבנה של פרומייז, כי אחרת נהיה בראש חוסר בהירות.


התחבר כדי לפרסם תגובה
 

45
מחובר

1.8k
משתמשים

6.0k
נושאים

69.4k
פוסטים

סלולר כשר במחירים הנמוכים ביותר עם ובלי ניוד.

כנסו לאתר לקניה מקוונת, או בדקו סניף קרוב לביתכם.

מוצרים עם הנחה

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

בהגעה דרך הלינק, ישנה הנחה בשלב התשלום בקופה!

בא תתחבר לדף היומי!