שאלות של מתחיל - מקצועי ב nodejs

[יוסף בן שמעון]

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

[חוקר]

@יוסף-בן-שמעון אמר בשאלות של מתחיל - מקצועי ב nodejs:

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

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

[יוסף בן שמעון]

@חוקר כלומר אתה מבצע שתי התקשרויות לצ'רדי עבור כל בקשה, אחת כדי לקבל מידע אישי ואחת בשביל מידע כללי נכון? אם כן, את המידע האישי לא תוכל לחסוך ותצטרך להמשיך לעשות אותו עבור כל בקשה, וכשהוא יבקש מידע כללי תוכל לתת לו את הנתון ששמרת בזיכרון.
השאלה היא האם אתה צריך לשלוח 15 בקשות כל שתי שניות ולשמור 15 משתנים, זה נראה לי קצת מוגזם, יבואו המומחים ויגידו אם ה נכון לעבוד כך או עדיף לעשות את זה רק כשהקליינט מבקש

[clickone]

@יוסף-בן-שמעון
@חוקר
זה תלוי בגול הקמפיין.
עד 100 מתרימים הAPI שלהם מאפשר לקבל את כל הנתונים בבקשה אחת.
מעל זה זה כבר להוסיף בקשות.

לכאורה ברגע שיש בקשה למספר מסויים שיעדכן רק אותו בליסט שמור

[חוקר]

@יוסף-בן-שמעון אמר בשאלות של מתחיל - מקצועי ב nodejs:

@חוקר כלומר אתה מבצע שתי התקשרויות לצ'רדי עבור כל בקשה, אחת כדי לקבל מידע אישי ואחת בשביל מידע כללי נכון? אם כן, את המידע האישי לא תוכל לחסוך ותצטרך להמשיך לעשות אותו עבור כל בקשה, וכשהוא יבקש מידע כללי תוכל לתת לו את הנתון ששמרת בזיכרון.
השאלה היא האם אתה צריך לשלוח 15 בקשות כל שתי שניות ולשמור 15 משתנים, זה נראה לי קצת מוגזם, יבואו המומחים ויגידו אם ה נכון לעבוד כך או עדיף לעשות את זה רק כשהקליינט מבקש

אכן.
זה מה שעשיתי בינתיים.
עשיתי מערך של קמפיינים שרלוונטים עבורי, והם מתעדכנים כל 2 שניות, והמידע הנשלח למאזין הינו לפי הקמפיין המופיע בGET.
מה אפשר לעשות יותר טוב?

[חוקר]

@clickone אמר בשאלות של מתחיל - מקצועי ב nodejs:

לכאורה ברגע שיש בקשה למספר מסויים שיעדכן רק אותו בליסט שמור

שוב שאלת תם
מה הכוונה בליסט.

תכניסו לראש שאני באמצע לצאת מהריבוע של ב PHP בו הייתי, @dovid (אולי בדרך לאסטריסק דרך נוד)

[clickone]

@חוקר ליסט זה במקום משתנה רגיל ש @magicode הציע לשמירה על הסכום.
במידה ויש לך כמה קמפיינים
אז אתה צריך לעשות רשימה
עם כל הקמפיינים הפעילים שלך
ולשמור לדוגמא:
מספר מערכת (שווה ערך למזהה קמפיין אצלך) => סכום שנאסף
ואת זה לעדכן כל 2 שניות

[חוקר]

@clickone אמר בשאלות של מתחיל - מקצועי ב nodejs:

@חוקר ליסט זה במקום משתנה רגיל ש @magicode הציע לשמירה על הסכום.
במידה ויש לך כמה קמפיינים
אז אתה צריך לעשות רשימה
עם כל הקמפיינים הפעילים שלך
ולשמור לדוגמא:
מספר מערכת (שווה ערך למזהה קמפיין אצלך) => סכום שנאסף
ואת זה לעדכן כל 2 שניות

זה מה שעשיתי במה שכתבתי קודם, אבל השתמשתי במערך, שמקבל ID של הקמפיין => הJSON שלו, מה העניין בליסט?

[clickone]

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

[חוקר]

שאלה חדשה, בנוגע לכתיבת הקוד.
אם יש לי כזה קוד

    if (!campaign_id || campaign_id === ''){
        res.end("id_list_message=" + 't-לא הוגדר מספר קמפיין, לא ניתן לבצע פעולה זו.' + '&' + "go_to_folder=/" + 'hangup');
    }else{
        if (!campaigns_data[campaign_id]){
            res.end("id_list_message=" + 't-קמפיין לא מורשה במערכת, לא ניתן לבצע פעולה זו.' + '&' + "go_to_folder=/" + 'hangup');
        } else{
            campaign_stats = campaigns_data[campaign_id];
        }
    }
    var total = campaign_stats.data.total;

כאשר אין מספר קמפיין, או מספר הקמפיין אינו מופיע במערך של הקמפיינים הפעילים, אני מקבל בלוג

ReferenceError: campaign_stats is not defined

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

[חוקר]

@חוקר אמר בשאלות של מתחיל - מקצועי ב nodejs:

מה היא הדרך הנוחה ביותר לסדר קוד כעין זה שיהיה תקין?

אולי הכי קל לעבוד עם IF ו ELSE?
או כן להתרגל לעבוד עם משהו שהוזכר לעיל בנוגע לעבודה אסינכרונית?

[יוסף בן שמעון]

@חוקר זה לא קשור לאיך נוד קורא את הדף, זה פשוט משתנה שמעולם לא אותחל אז הוא undefind, אתה יכול לכתוב כך:

    var total = (campaign_stats && campaign_stats.data.total);

זה יבדוק אם המשתנה קיים ואם לא הוא יחזיר מיד את הundifind ולא יכשיל את הפונקציה

[חוקר]

@יוסף-בן-שמעון אמר בשאלות של מתחיל - מקצועי ב nodejs:

@חוקר זה לא קשור לאיך נוד קורא את הדף, זה פשוט משתנה שמעולם לא אותחל אז הוא undefind, אתה יכול לכתוב כך:

    var total = (campaign_stats && campaign_stats.data.total);

זה יבדוק אם המשתנה קיים ואם לא הוא יחזיר מיד את הundifind ולא יכשיל את הפונקציה

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

[יוסף בן שמעון]

אולי תפתח אשכול חדש לכל נושא, זה מכביד 3 עמודים לנושא אחד...

[חוקר]

@יוסף-בן-שמעון אמר בשאלות של מתחיל - מקצועי ב nodejs:

אולי תפתח אשכול חדש לכל נושא, זה מכביד 3 עמודים לנושא אחד...

כעת יש לי עוד 17 הודעות עד לדף הבא, ואח"כ נעבור נושא.
או שעד אז הכל כבר יהיה מסודר בעזה"י..

[אהרן]

@יוסף-בן-שמעון אמר בשאלות של מתחיל - מקצועי ב nodejs:

@חוקר זה לא קשור לאיך נוד קורא את הדף, זה פשוט משתנה שמעולם לא אותחל אז הוא undefind, אתה יכול לכתוב כך:

    var total = (campaign_stats && campaign_stats.data.total);

זה יבדוק אם המשתנה קיים ואם לא הוא יחזיר מיד את הundifind ולא יכשיל את הפונקציה

@יוסף-בן-שמעון
לדעתי בשלב זה עדיף לכתוב את הדוגמאות עם תחביר הכי פשוט
בדוגמא זו

if(campaign_stats) {
    var total = campaign_stats.data.total
}

[חוקר]

@חוקר אמר בשאלות של מתחיל - מקצועי ב nodejs:

כאשר אין מספר קמפיין, או מספר הקמפיין אינו מופיע במערך של הקמפיינים הפעילים, אני מקבל בלוג
ReferenceError: campaign_stats is not defined

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

[חוקר]

@אהרן @יוסף-בן-שמעון
בעצם הכי פשוט להכניס הכל ב IF הקודם.
כך:

if (!campaign_id || campaign_id === ''){
    res.end("id_list_message=" + 't-לא הוגדר מספר קמפיין, לא ניתן לבצע פעולה זו.' + '&' + "go_to_folder=/" + 'hangup');
}else if (!campaigns_data[campaign_id]){
    res.end("id_list_message=" + 't-קמפיין לא מורשה במערכת, לא ניתן לבצע פעולה זו.' + '&' + "go_to_folder=/" + 'hangup');
} else{
    campaign_stats = campaigns_data[campaign_id];
    var total = campaign_stats.data.total;
    ...
}

[חוקר]

תודה רבה לכל אלו שעזרו וסייעו
@david @יוסף-בן-שמעון @אהרן @magicode @clickone
מקוה שלא פיספסתי מישהו.
ב"ה הייתה התקדמות משמעותית היום, השלמתי בניית מערכת שעובדת בצורה חלקה, כמובן בעזרתכם.
עדיין לא גמרתי, אני לא למדתי עדיין מספיק את השפה, אבל כבר היה מספיק בסיסי כדי לעשות משהו שזז באמת..
במיוחד תודה ל @magicode על הדוגמה הראשונה שנתן, זה נתן לי פריצת דרך רצינית, וכן לימדה אותי פרק נוסף בנושא.
וכן ל @david שדחף אותי להיכנס לזה..
תודה

נ.ב. אני מקוה שסיימתי לאשכול זה, הנושא הבא כבר יהיה באשכול חדש..

[יוסף בן שמעון]

@חוקר עכשיו נפל לי האסימון (במסגרת 17 ההודעות שנותרו לאשכול...) כשאתה קורא ל res.end אתה אמור לעצור את הריצה של הפונקציה, היא לא אמורה להמשיך לרוץ, אתה צריך לכתוב כך

    if (!campaign_id || campaign_id === ''){
        return res.end("id_list_message=" + 't-לא הוגדר מספר קמפיין, לא ניתן לבצע פעולה זו.' + '&' + "go_to_folder=/" + 'hangup');
    }else{
        if (!campaigns_data[campaign_id]){
            return res.end("id_list_message=" + 't-קמפיין לא מורשה במערכת, לא ניתן לבצע פעולה זו.' + '&' + "go_to_folder=/" + 'hangup');
        } else{
            campaign_stats = campaigns_data[campaign_id];
        }
    }
    var total = campaign_stats.data.total;