לוגי שגיאה בnodejs וyemotrouter מה הגורם לקריסה?

[eido]

@צדיק-תמים אם אני מבין נכון, לא עושים בכאלו מקרים לא await ואין צורך בreturn, או שלא הבנתי נכון.

[יהודי טוב]

@eido כתב בלוגי שגיאה בnodejs וyemotrouter מה הגורם לקריסה?:

הtry/catch הארוך הוא כי עשיתי שינוי על שינוי לטובת הלקוח. אז נשאר כזה ארוך, סתם להבין מה רע בזה?

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

[dovid]

@eido אחרי שההשמצות האלה יגרמו לך להשתפר, תבקש לקבל עוד.
היינו מכוונים אותך יותר ספציפית, אבל זה סטיה מהנושא.
עניינית אתה צריך להוסיף return או לשנות את מבנה הקוד, שהשורה id_list_message לא תהיה בתוך בלוק של try.

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

[eido]

@יהודי-טוב כתב בלוגי שגיאה בnodejs וyemotrouter מה הגורם לקריסה?:

@eido כתב בלוגי שגיאה בnodejs וyemotrouter מה הגורם לקריסה?:

הtry/catch הארוך הוא כי עשיתי שינוי על שינוי לטובת הלקוח. אז נשאר כזה ארוך, סתם להבין מה רע בזה?

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

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

[eido]

@dovid כתב בלוגי שגיאה בnodejs וyemotrouter מה הגורם לקריסה?:

@eido אחרי שההשמצות האלה יגרמו לך להשתפר, תבקש לקבל עוד.

היינו מכוונים אותך יותר ספציפית, אבל זה סטיה מהנושא.
עניינית אתה צריך להוסיף return או לשנות את מבנה הקוד, שהשורה id_list_message לא תהיה בתוך בלוק של try.

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

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

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

[eido]

@צדיק-תמים כתב בלוגי שגיאה בnodejs וyemotrouter מה הגורם לקריסה?:

@dovid הכי טוב היה בלי try בלבד חוץ מנקודתי בשורה שרוצים לטפל בשגיאות שלה בצורה שונה
יש לספריה קאלבק מיוחד כדי לטפל בשגיאות לא צפויות בצורה מרוכזת במקום לפזר בקוד try catch

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

[dovid]

@eido כתב בלוגי שגיאה בnodejs וyemotrouter מה הגורם לקריסה?:

היא להודיע ללקוח ולצאת באלגנטיות.

אז תצא באלגנטיות!! במקטע הcatch... המצב הוא שאתה לא יוצא בcatch, והtry בתוך לולאה.

[eido]

@dovid למה? סיכמנו שהid וכו מודיע הודעה למאיזן ויוצא מהשלוחה, מה יותר אלגנטי מזה?

[eido]

@dovid אני מתחיל לחשוב שהid_list_message לא סוגר באמת את החיבור.
כי כל הלוגים מראים בעיה בשורה 7 בקוד הראשון:

while(okStreet!="1")
    {
->        let ans = await call.read([{ type: 'file', data: "/ivr_records/רחוב1" }], "record");

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

Error: HangupError: the call was hangup by the caller
    at EventEmitter.<anonymous> 

ושמשום מה התווסף לurl

&ApiHangupExtension=1/1/1/1/1&hangup=yes&

)
מפעיל את

call.id_list_message([{ type: 'text', data: "אירעה שגיאה" }]);

שלכאורה אמור לסיים את הקוד (ככה אני מבין מהתיעוד)

this.send(this.#responsesTextQueue.pull() + responseTxt + '&');
            throw new ExitError(this, {
                target: goToFolderMessageIndex !== -1 ? messages[goToFolderMessageIndex].data : `parent of /${this.extension}`,
                caller: goToFolderMessageIndex !== -1 ? 'go_to_folder' : 'id_list_message'
            });

אבל במקום זה חוזר בחזרה לchooseStreet שקראה לו, בלי להחזיר ערך לstreet, ככה שהוא null.
הוא מגיע לשורה הזו:

if (street==null) {continue;}

ומתחיל את הלולאה מחדש.
ואז מגיע שוב לשורה:

while(okStreet!="1")
    {
->        let ans = await call.read([{ type: 'file', data: "/ivr_records/רחוב1" }], "record");

ופה זורק שגיאה שא"א לשלוח עוד פעם הודעה למאזין.

ככה אני מניח.

אבל זה לא מסביר לי למה רק השגיאה השלישית הקריסה את האפליקציה - עובדה שאחריה כתוב node is starting - ואילו השאר לא הקריסו.

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

[eido]

אהה כנראה שהוא ניתק והשרת של ימות המשיח פשוט שלח &ApiHangupExtension=1/1/1/1/1&hangup=yes& ואז הcatch תפס את השגיאה והודיע על שגיאה למרות שהוא כבר ניתק וזה מה שגרם לאפליקציה לקרוס.
לקחתי את הurl של השגיאה, בלי הסוף (החלק של הניתוק) והרצתי בpostman ולא קרה כלום, רק אחרי שהוספתי את החלק של הניתוק פתאום הוא קרס פעם אחר פעם.
מה אומר @צדיק-תמים יכול להיות?
ומה הפתרון?
@צדיק-תמים @dovid אתם בעצם מציעים להוריד לגמרי את הtry/catch?

[eido]

כולם התאדו לי כאן באמצע המתח

מחקתי את הtry/catch ובאמת הוא זורק שגיאה אבל לא קורס!
זה גם לא מצב אופטימלי לדעתי, אבל לפחות לא קורס...

[צדיק תמים]

@יהודי-טוב כתב בלוגי שגיאה בnodejs וyemotrouter מה הגורם לקריסה?:

2 - הוא כתב שעדיף בלי, כי אם אתה לא מטפל בשגיאה אז זה נזרק הלאה ויש לו בספריה טיפול בשגיאות.

יותר מזה, ברגע שהוא עושה try catch על הכל הוא "בולע" את השגיאה שנועדה להשמיד את הסקופ של הפונקציה, וממילא הקוד שלו חשוף לבאג כזה שהוא מנסה להחזיר תשובה פעמיים
ואם תאמר - מה אם תהיה שגיאה לא צפויה? אז בשביל זה יש את הקאלבק הגלובלי כדי שהאפליקציה לא תקרוס, המשתמש יקבל תשובה, ואם תרצה תוכל גם לשלוח לעצמך מייל על השגיאה וכדומה...

[צדיק תמים]

@eido כתב בלוגי שגיאה בnodejs וyemotrouter מה הגורם לקריסה?:

אבל במקום זה חוזר בחזרה לchooseStreet שקראה לו, בלי להחזיר ערך לstreet, ככה שהוא null.
הוא מגיע לשורה הזו:

ומתחיל את הלולאה מחדש.
ואז מגיע שוב לשורה:

ופה זורק שגיאה שא"א לשלוח עוד פעם הודעה למאזין.

ככה אני מניח.

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

[eido]

@צדיק-תמים אז אני יכול למחוק את כל הtry/catch מהפרוייקט?

[צדיק תמים]

@eido כתב בלוגי שגיאה בnodejs וyemotrouter מה הגורם לקריסה?:

אבל זה תופס את כל החריגות?

כל החריגות שלא נתפסו קודם - uncaughtErrorHandler

איך זה עובד?

כיוון שכל הפונקציה של השיחה זה בעצם פונקציה אסינכרונית שהראוטר (הספריה) מריץ, אז הראוטר מריץ אותה בתוך try catch כדי שיהיה אפשר לטפל בצורה נאותה בשגיאות לא צפויות, נניח אם יש לך בקוד נסיון קריאה ל API חיצוני שלא זמין

[eido]

@צדיק-תמים איך אפשר לדבאג קוד שאמור לרוץ על שרת?
הוא הרי אמור לקבל בקשות, לא?

[צדיק תמים]

@eido אתה יכול לדבג ישירות על השרת עם Remote SSH (תוסף של מייקרוסופט ל VSC)
אבל אתה אמור להיות מסוגל להגיש את הפיתוח המקומי שלך החוצה ולדבג מקומית מול בקשות שמגיעות מימות, יש כל מיני כלים שמאפשרים להנגיש שירות לוקאלי החוצה - יש אפילו אחד מובנה ב VSC, זה דרך המלך

[eido]

@צדיק-תמים נו, תשתף אותי מה שמו.

[צדיק תמים]

@eido של מי?

[eido]

@צדיק-תמים כתב בלוגי שגיאה בnodejs וyemotrouter מה הגורם לקריסה?:

יש אפילו אחד מובנה ב VSC, זה דרך המלך

שלו
ניסיתי עכשיו לדבאג דרך הvsc ולא הוא לא נותן שום נתון