לוגי שגיאה בnodejs וyemotrouter מה הגורם לקריסה?
-
אהה כנראה שהוא ניתק והשרת של ימות המשיח פשוט שלח &ApiHangupExtension=1/1/1/1/1&hangup=yes& ואז הcatch תפס את השגיאה והודיע על שגיאה למרות שהוא כבר ניתק וזה מה שגרם לאפליקציה לקרוס.
לקחתי את הurl של השגיאה, בלי הסוף (החלק של הניתוק) והרצתי בpostman ולא קרה כלום, רק אחרי שהוספתי את החלק של הניתוק פתאום הוא קרס פעם אחר פעם.
מה אומר @צדיק-תמים יכול להיות?
ומה הפתרון?
@צדיק-תמים @dovid אתם בעצם מציעים להוריד לגמרי את הtry/catch? -
@eido כתב בלוגי שגיאה בnodejs וyemotrouter מה הגורם לקריסה?:
הtry/catch הארוך הוא כי עשיתי שינוי על שינוי לטובת הלקוח. אז נשאר כזה ארוך, סתם להבין מה רע בזה?
1 - בגדול זה שזה ארוך זה לא ממקד אותך בבעיה
2 - הוא כתב שעדיף בלי, כי אם אתה לא מטפל בשגיאה אז זה נזרק הלאה ויש לו בספריה טיפול בשגיאות.
אז אתה יכול לטפל שם בשגיאות..@יהודי-טוב כתב בלוגי שגיאה בnodejs וyemotrouter מה הגורם לקריסה?:
2 - הוא כתב שעדיף בלי, כי אם אתה לא מטפל בשגיאה אז זה נזרק הלאה ויש לו בספריה טיפול בשגיאות.
יותר מזה, ברגע שהוא עושה try catch על הכל הוא "בולע" את השגיאה שנועדה להשמיד את הסקופ של הפונקציה, וממילא הקוד שלו חשוף לבאג כזה שהוא מנסה להחזיר תשובה פעמיים
ואם תאמר - מה אם תהיה שגיאה לא צפויה? אז בשביל זה יש את הקאלבק הגלובלי כדי שהאפליקציה לא תקרוס, המשתמש יקבל תשובה, ואם תרצה תוכל גם לשלוח לעצמך מייל על השגיאה וכדומה... -
@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 כתב בלוגי שגיאה בnodejs וyemotrouter מה הגורם לקריסה?:
אבל במקום זה חוזר בחזרה לchooseStreet שקראה לו, בלי להחזיר ערך לstreet, ככה שהוא null.
הוא מגיע לשורה הזו:ומתחיל את הלולאה מחדש.
ואז מגיע שוב לשורה:ופה זורק שגיאה שא"א לשלוח עוד פעם הודעה למאזין.
ככה אני מניח.
במקום לנחש תלמד להשתמש בדיבאגר של VSC ולראות את הזרימה של הקוד "בעיניים"
תחסוך לעצמך המון כאב ראש עתידי -
@יהודי-טוב כתב בלוגי שגיאה בnodejs וyemotrouter מה הגורם לקריסה?:
2 - הוא כתב שעדיף בלי, כי אם אתה לא מטפל בשגיאה אז זה נזרק הלאה ויש לו בספריה טיפול בשגיאות.
יותר מזה, ברגע שהוא עושה try catch על הכל הוא "בולע" את השגיאה שנועדה להשמיד את הסקופ של הפונקציה, וממילא הקוד שלו חשוף לבאג כזה שהוא מנסה להחזיר תשובה פעמיים
ואם תאמר - מה אם תהיה שגיאה לא צפויה? אז בשביל זה יש את הקאלבק הגלובלי כדי שהאפליקציה לא תקרוס, המשתמש יקבל תשובה, ואם תרצה תוכל גם לשלוח לעצמך מייל על השגיאה וכדומה...@צדיק-תמים אז אני יכול למחוק את כל הtry/catch מהפרוייקט?
-
@צדיק-תמים כתב בלוגי שגיאה בnodejs וyemotrouter מה הגורם לקריסה?:
@dovid הכי טוב היה בלי try בלבד חוץ מנקודתי בשורה שרוצים לטפל בשגיאות שלה בצורה שונה
יש לספריה קאלבק מיוחד כדי לטפל בשגיאות לא צפויות בצורה מרוכזת במקום לפזר בקוד try catchאהה חזק, עכשיו הבנתי מה התכוונת.
אבל זה תופס את כל החריגות?
איך זה עובד?@eido כתב בלוגי שגיאה בnodejs וyemotrouter מה הגורם לקריסה?:
אבל זה תופס את כל החריגות?
כל החריגות שלא נתפסו קודם - uncaughtErrorHandler
איך זה עובד?
כיוון שכל הפונקציה של השיחה זה בעצם פונקציה אסינכרונית שהראוטר (הספריה) מריץ, אז הראוטר מריץ אותה בתוך try catch כדי שיהיה אפשר לטפל בצורה נאותה בשגיאות לא צפויות, נניח אם יש לך בקוד נסיון קריאה ל API חיצוני שלא זמין
-
@eido כתב בלוגי שגיאה בnodejs וyemotrouter מה הגורם לקריסה?:
אבל במקום זה חוזר בחזרה לchooseStreet שקראה לו, בלי להחזיר ערך לstreet, ככה שהוא null.
הוא מגיע לשורה הזו:ומתחיל את הלולאה מחדש.
ואז מגיע שוב לשורה:ופה זורק שגיאה שא"א לשלוח עוד פעם הודעה למאזין.
ככה אני מניח.
במקום לנחש תלמד להשתמש בדיבאגר של VSC ולראות את הזרימה של הקוד "בעיניים"
תחסוך לעצמך המון כאב ראש עתידי -
@צדיק-תמים איך אפשר לדבאג קוד שאמור לרוץ על שרת?
הוא הרי אמור לקבל בקשות, לא?@eido אתה יכול לדבג ישירות על השרת עם Remote SSH (תוסף של מייקרוסופט ל VSC)
אבל אתה אמור להיות מסוגל להגיש את הפיתוח המקומי שלך החוצה ולדבג מקומית מול בקשות שמגיעות מימות, יש כל מיני כלים שמאפשרים להנגיש שירות לוקאלי החוצה - יש אפילו אחד מובנה ב VSC, זה דרך המלך -
@eido אתה יכול לדבג ישירות על השרת עם Remote SSH (תוסף של מייקרוסופט ל VSC)
אבל אתה אמור להיות מסוגל להגיש את הפיתוח המקומי שלך החוצה ולדבג מקומית מול בקשות שמגיעות מימות, יש כל מיני כלים שמאפשרים להנגיש שירות לוקאלי החוצה - יש אפילו אחד מובנה ב VSC, זה דרך המלך -
@צדיק-תמים נו, תשתף אותי מה שמו.
-
@eido אתה יכול לדבג ישירות על השרת עם Remote SSH (תוסף של מייקרוסופט ל VSC)
אבל אתה אמור להיות מסוגל להגיש את הפיתוח המקומי שלך החוצה ולדבג מקומית מול בקשות שמגיעות מימות, יש כל מיני כלים שמאפשרים להנגיש שירות לוקאלי החוצה - יש אפילו אחד מובנה ב VSC, זה דרך המלך@צדיק-תמים כתב בלוגי שגיאה בnodejs וyemotrouter מה הגורם לקריסה?:
יש אפילו אחד מובנה ב VSC, זה דרך המלך
שלו
ניסיתי עכשיו לדבאג דרך הvsc ולא הוא לא נותן שום נתון -
@צדיק-תמים כתב בלוגי שגיאה בnodejs וyemotrouter מה הגורם לקריסה?:
יש אפילו אחד מובנה ב VSC, זה דרך המלך
שלו
ניסיתי עכשיו לדבאג דרך הvsc ולא הוא לא נותן שום נתון -
לא יודע לא מצליח לי
https://drive.google.com/file/d/1TEiEr9******0dx2YkX5-hNkouUyecCmU9Voaz/view -
@צדיק-תמים לא הבנתי איך אני אמור לטפל בשגיאה? נניח ואני רוצה לתפוס שגיאה ולהחזיר בשבילה למאזין פלט ספציפי, איך עושים את זה? הרי אח"כ הקוד ימשיך לרוץ כי הid_list_message לא עוצר את הריצה.
-
@צדיק-תמים אבל יש שגיאה, ואם אחצא כרגיל זה עלול להסתבך כמו שקרה בקוד שלי.
-
@צדיק-תמים תוכל רגע לענות לי, אני רוצה להבין דבר יסודי בספריה, האם הid_list_message עוצר את הריצה או לא? ואם כן, למה זה ממשיך לרוץ?
@eido הid_list_message בלי פרמטר prepend to next זורק שגיאה
אם אתה לא עושה עליה try catch אז הפונקציה נעצרת והשגיאה מטופלת על ידי הראוטר
אם אתה עושה try catch ולא בודק את סוג השגיאה, זה בעיהבכללי אין סיבה בדוגמאות שהבאת לעשות try catch כי יש הנדלר גלובלי לשגיאות לא צפויות
אם היה לך נניח קוד שמנסה API אחד ואם הוא נכשל מנסה אחר, זה היה הגיוני, אבל רק מסביב הקוד הספציפי הזה ולא לעטוף את כל הקוד -
@eido הid_list_message בלי פרמטר prepend to next זורק שגיאה
אם אתה לא עושה עליה try catch אז הפונקציה נעצרת והשגיאה מטופלת על ידי הראוטר
אם אתה עושה try catch ולא בודק את סוג השגיאה, זה בעיהבכללי אין סיבה בדוגמאות שהבאת לעשות try catch כי יש הנדלר גלובלי לשגיאות לא צפויות
אם היה לך נניח קוד שמנסה API אחד ואם הוא נכשל מנסה אחר, זה היה הגיוני, אבל רק מסביב הקוד הספציפי הזה ולא לעטוף את כל הקוד -
@צדיק-תמים גם אם אני עושה רק try/finally זה יתנגש עם ההנדלר הזה?
