ASYNC/AWAIT

[מוטי אורן]

@yossiz נראה לי היא מתתכוונת לקוד שנמצא בהערה (שורה 24). @שרה-רחל הוא לא אמור להיות עם await בכלל לפי מה שאני מבין פה.

עריכה: נראה לי שטעיתי. על פניו כן צריך את ה await

[שרה רחל]

@מוטי-אורן
תקנתי את הקוד
הRESULT מודפס לפני MEM

[שרה רחל]

@שרה-רחל
הMEMBER מובא ע"י REST API

[yossiz]

@שרה-רחל צריך להוסיף await או return לפני promise.all

[שרה רחל]

@yossiz
נכון זה עזר
האמת שהPROMISE.ALL
כבר היה חלק מהפתרון
מה הוא בעצם פותר ?
הוא לא אמור לבוא כשיש כמה קריאות אסיכרוניות?
בפונקציה זו יש קריאה אחת בתוך הסוגריים של PROMISE.ALL
אז איך זה פתר את הבעיה?

[dovid]

@שרה-רחל יש בקוד בעיה נוספת, הפונקציהבPromise.all נקראת עבור כל אחד מהפריטים ברשימה, ומחזירה עבור כל אחד או not-found או אובייקט משותף לכלל האיברים.
הפונקציה הקוראת בודקת אם הערך הוא not-found אבל הוא לעולם לא יהיה not-found כי התוצאה של Promise.all היא תמיד מערך של תוצאות (בכל תוצאה יכול להיות not-found).
בקיצור הפונקציה getSsForMembers צריכה כתיבה מחדש, אשמח לעזור אם אדע בגדול מה היא מנסה לעשות.

[dovid]

@שרה-רחל הפונקציה Promise.all מטפלת במקרה שיש הרבה await לעשות ואת רוצה לחכות שכולם יתבצעו ולקבל את התוצאות כמערך.

[שרה רחל]

@dovid
אז איך במקרה זה מתאימה הפונקציה
יש כאן רק קריאה אחת של AWAIT

[dovid]

@שרה-רחל ss.map הופך את מערך הss למערך של פונקציות await, כל אחת מהסוג שכתובה בסוגריים... כלומר הטיפול של הפונקציה יתבצע מול כל פריט בעוד הוא נכתב כנראה במקור לטיפול בכלל המערך ביחד. אשמח לדעת מה הפונקציה עושה, נראה שהיא מאחדת בין שני רשימות שונות, ובנוסף מבקשת מהשרת השלמת מידע.

[שרה רחל]

@dovid
MEMBER:

Name: String,
Email: String,
City: String

Subscriptions

MemberId: String,
Movies: [String]

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

[dovid]

@שרה-רחל אני לא לגמרי מבין, כתבתי שלד קוד שלכאורה עושה כל מה שקיים כעת, תוכלי לומר מה חסר בו?

const getAllMembers = async function () {
    let members = await membersRest.getAllMembers();

    let membersArr = members.data.map(z => {
        return {
            _id: z._id,
            Name: z.Name,
            Email: z.Email,
            City: z.City,
            Movies: []
        }
    });

    let ss = await subscriptionRest.getSubscriptions();
    let ssData = ss.data;

    for (const sub of ss.data) {
        var memberFind = membersArr.find(x => x._id == sub.MemberID);
        if (memberFind)
            memberFind.Movies = sub.Movies;
        else
            console.log(`the member ${sub.MemberID} not found in the members list, but have subscrition!`)
    }

    return membersArr;
}

[dovid]

@dovid גם הקוד הזה ארוך מידי, כי אני שם לב שהMap הראשון לא עושה כלום, הקוד הזה שקול:

const getAllMembers = async function () {
    let membersArr = await membersRest.getAllMembers();

    let ss = await subscriptionRest.getSubscriptions();
    let ssData = ss.data;

    for (const sub of ss.data) {
        var memberFind = membersArr.find(x => x._id == sub.MemberID);
        if (memberFind)
            memberFind.Movies = sub.Movies;
        else
            console.log(`the member ${sub.MemberID} not found in the members list, but have subscrition!`)
    }

    return membersArr;
}

[שרה רחל]

@dovid
נראה נכון
אני כבר בודקת
רק מענין אותי למה בקוד הראשוני ששמתי (תקנתיקצת)
הfinalMmArr מדפיס תוכן
בזמן שהRESULT מדפיס אחרי כן UNDEFINDED

[dovid]

חיסרתי את העיקר לכאורה שזה הawait membersRest.getMember(memId); אבל עשיתי זאת במודע, כי אני לא מבין מה הוא מביא (מה היה קודם ומה הוא מחליף/מוסיף).

[dovid]

@שרה-רחל אמר בASYNC/AWAIT:

@dovid
נראה נכון
אני כבר בודקת
רק מענין אותי למה בקוד הראשוני ששמתי (תקנתיקצת)
הfinalMmArr מדפיס תוכן
בזמן שהRESULT מדפיס אחרי כן UNDEFINDED

לפני התיקון של @yossiz (לשים return או await) זה לא הגיוני שהוא הדפיס not-found.
אחרי התיקון, הוא היה אמור להדפיס את זה אבל בתוך מערך [.... .... "not-found"]

[שרה רחל]

@dovid
לפני הGETMEMBER
יש רק את הID שלו
ואני רוצה ליצור JSON שמכיל גם את הפרטים שלו

[dovid]

@שרה-רחל לא היה לפני כן City וכל זה? ורק למי שיש הרשמה לMovie צריך את הפרטים או לכולם?
לא עדיף שהשרת בgetAllMembers ייתן פרטים מלאים?

[שרה רחל]

@dovid
זה הדרישה של הפרויקט

[dovid]

@שרה-רחל ענית לי רק על השאלה האחרונה...

[שרה רחל]

@dovid
בSUSCRIPTION
יש את הID של הMEMBER
ומערך של MOVIES שמכיל רק את הID'S
וחוץ מזה יש גם
MOVIES שמכיל ID ועוד כמה פרטים
וMEMBER שמכיל ID
CITY
EMAIL
NAME