NODEJS עצירת המשך ביצוע פעולות

dovid

אתה רוצה יציאה מפונקציה, זה עושים עם return.

res.end(data);
return;

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

return res.end(data);

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

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

dovid

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

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

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

$send_to_printer($big_content);
echo "now print";

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

חוקר

@dovid אמר בNODEJS עצירת המשך ביצוע פעולות:

אתה רוצה יציאה מפונקציה, זה עושים עם return.
res.end(data);
return;
זה בעצם מחזיר void (ביטוי שאומר כלום) למי שמצפה לתוצאה מהפונקציה (במקרה שאתה שואל מן הסתם אין מישהו כזה ולכן לא משנה מה מחזירים העיקר שהפונקציה הופסקה).
תוכל גם לקמבן זאת בשורה בודדת:
return res.end(data);
(זה בעצם יחזיר את תוצאת הפוקנציה res.end ויעצור, וזה לא אמור להיות אכפת לך מה באמת חוזר מres.end שהרי אין מי שעושה שימוש בתוצאת הפונקציה שלך).

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

אכן אם כותבים למשל כך:

var get_he_full_date1 = get_he_full_date('2019-05-1');
    res.end(get_he_full_date1);
    return;
    console.log(get_he_full_date1);

ההדפסה לקונסול לא מבוצעת כי ה server/app נעצר.
הבעיה היא כזו:
כפי שזה בנוי בימות המשיח ניתן לשלוח תשובה read שיחזיר לי נתון מסויים, שזה אומר כך, בתחילה השיחה אני מקבל קריאת URL לשרת שלי אם נתונים קבועים כגון מספר הטלפון של המתקשר וכו' בצורה כזו
http://domain.com/aaa?ApiDID=033080222&ApiRealDID=033080222&ApiPhone=0521234567&ApiExtension=/1
עכשיו אני רוצה לקבל את מספר הזהות של המתקשר, אני שולח תשובה read עם ההגדרות שצריך, וכעת אני מקבל קריאה חדשה + id
http://domain.com/aaa?ApiDID=033080222&ApiRealDID=033080222&ApiPhone=0521234567&ApiExtension=/1&id=222333444
מה שאני עושה הוא בעצם פונקציה שבודקת האם הid קיים בURL, אם כן, הוא מכניס אותו למשתנה והשרת ממשיך לנתונים הבאים, אחרת הוא שולח לימות read=הקש זהות=id
משהו בסגנון

var user_id = read('id');
var name = read('name');
fetch(`http://aaa.com?name=${name}&id=${id}`);
res.end('id_list_message=נשלח בהצלחה');
function read(var_name){
    if (query[var_name]){
        return query[var_name];
    }else{
        return res.end('read=הקש זהות=id');
    }
}

בדוגמא הנ"ל הראיתי שלאחר קליטת הנתונים מהמשתמש אני שולח אותם למשל לשרת אחר.
לכן אני חייב שרק לאחר אכלוס שני המשתנים name ו id אני יבצע את מה שבהמשך.
כאן אני בבעיה, כי אפילו אם אני שם return בפונקציה של read זה לא יעבוד לעצור את server/app, כי זה מסיים את הפונקציה עצמה.
זה הבעיה שלי.

yossiz

@חוקר אמר בNODEJS עצירת המשך ביצוע פעולות:

כי ה server/app נעצר

למה לא כתבת פשוט "כי הפונקציה נעצרת"?

כאן אני בבעיה, כי אפילו אם אני שם return בפונקציה של read זה לא יעבוד לעצור את server/app, כי זה מסיים את הפונקציה עצמה.

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

function deal_with_request(query, res) {
   if (var user_id = query['id']) {
       var name = read('name');
       fetch(`http://aaa.com?name=${name}&id=${id}`);
       res.end('id_list_message=נשלח בהצלחה');
    }
    else  res.end('read=הקש זהות=id');
}

חוקר

@yossiz אמר בNODEJS עצירת המשך ביצוע פעולות:

   if (var user_id = read('id')) {
       var name = read('name');
       fetch(`http://aaa.com?name=${name}&id=${id}`);
       res.end('id_list_message=נשלח בהצלחה');
    }
    else  res.end('read=הקש זהות=id');
}

function read(var_name){
    if (query[var_name]){
        return query[var_name];
    }else{
        return null;
    }
}

הבעיה היא כאשר יש לך עשר נתונים לקלוט שחלק מהם תלויים אחד בשני אם הקשת 1 השלב הבא הוא א, ואם הקשת 2 השלב הבא הוא ב וכו'.
לעשות הכל ב if מאוד מסרבל.
לעומת זאת ב PHP אתה שם בפונקציה של read את exit()
לדוגמא: (לא רלוונטית כי זה לא בPHP, אבל הרעיון)

function read(var_name){
    if (query[var_name]){
        return query[var_name];
    }else{
        res.end('read=הקש זהות=id');
        exit();
    }
}

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

yossiz

@חוקר בנוד אין כזה דבר שעוצר את כל התהליך, (חוץ מ-process.exit וזה אתה לא רוצה)

בשפת אחרות היה אפשר להשתמש ב-goto אבל ב-JS גם זה לא קיים

צריך לבנות את זה בצורה אחרת. (משהו כעין state machine?)

dovid

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

חוקר

@dovid אמר בNODEJS עצירת המשך ביצוע פעולות:

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

כל פעם הוא שולח את כל ה URL פלוס הערך האחרון שנוסף, אם אתה שולח לו שאתה צריך שוב ID כגון לתקן וכדומה הוא יישלח לך
http://domain.com/aaa?ApiDID=033080222&ApiRealDID=033080222&ApiPhone=0521234567&ApiExtension=/1&id=222333444&id=111444555
ואח"כ תבקש את השם הוא ייתן לך
http://domain.com/aaa?ApiDID=033080222&ApiRealDID=033080222&ApiPhone=0521234567&ApiExtension=/1&id=222333444&id=111444555&name=name&lest=2
וכן הלאה
(ברעיון זה טירוף כי בעצם כדי לקבל את הערך החמישי שאני צריך בשיחה זו, יש חמש בקשות הלוך ושוב בין השרתים, ובכל שלב הURL מתארך ולפי הנתונים ב URL על השרת שלי לבצע את ההתקדמות לשלב הבא. ואז הURL מתארך בפרמטר נוסף וכו'.)
כעת ברור יותר?

dovid

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

var props = ['id', 'name', 'tel', 'last4cerdit']; //רשימת נתונים נדרשים 

for (var key of props) 
  if(!query[var_name])
    return res.send('read=???=' + var_name);

//בשלב זה אתה יודע שכל המאפיינים קיימים

fetch(`http://aaa.com?name=${query.id}&id=${query.name}`);
res.end('id_list_message=נשלח בהצלחה');

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

dovid

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

//רשימת נתונים נדרשים 
var props = [                          
  {name: 'id', client_msg: 'הקש זהות'},
  {name: 'name', client_msg: 'הקש שם'},
  {name: 'tel', client_msg: 'אכול דגים'},
  {name: 'last4cerdit', client_msg: 'כל טוב'},
]; 

for (var key of props) 
  if(!query[key.name])
    return res.send(`read=${key.client_msg}=${key.name}`);

//בשלב זה אתה יודע שכל המאפיינים קיימים

fetch(`http://aaa.com?name=${query.id}&id=${query.name}`);
res.end('id_list_message=נשלח בהצלחה');

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

חוקר

@dovid אמר בNODEJS עצירת המשך ביצוע פעולות:

בשביל ההודעה ללקוח אתה צריך להתאים למאפיין החסר, אז תוכל לעשות קל יותר לתחזוקה ככה:
//רשימת נתונים נדרשים 
var props = [                          
  {name: 'id', client_msg: 'הקש זהות'},
  {name: 'name', client_msg: 'הקש שם'},
  {name: 'tel', client_msg: 'אכול דגים'},
  {name: 'last4cerdit', client_msg: 'כל טוב'},
]; 

for (var key of props) 
  if(!query[key.name])
    return res.send(`read=${key.client_msg}=${key.name}`);

//בשלב זה אתה יודע שכל המאפיינים קיימים

fetch(`http://aaa.com?name=${query.id}&id=${query.name}`);
res.end('id_list_message=נשלח בהצלחה');
אני מבין שיש הרבה מאפיינים שלא תמיד נדרשים במתודות שונות אז כדאי להעביר את הרשימה מחות לפונקציה לשימושים חוזרים וחלקיים.

ברור.
הפונקציה של read שציטטי כאן הייתה רק לפורום ולא הפונקציה המלאה שיש הרבה אפשרויות, אך לא כל קוד ניתן להעתקה..

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

אך כעת נפל לי האסימון שאם כל הבעיה הייתה ש return res.end לא עבד לי' כי זה היה בתוך הפונקציה של read, א"כ איפה שאני חייב שהוא לא יעבור הלאה לפני נתון מסויים, שם אני יכול להשתמש ב return res.end בקוד עצמו ולא בתוך פונקציה.
אצטרך לבדוק האם אני צודק.
בכל מקרה תודה על כל היעוץ

dovid

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

חוקר

@dovid אמר בNODEJS עצירת המשך ביצוע פעולות:

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

א"כ הייתי שולח לך דוגמה בפרטי.. (בתקוה להבנה)
לשלוח למייל?

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

    let press_date = read('הקשתאריך', 'date');
    now = new Date(press_date.substr(4,4) + '-' + press_date.substr(2,2) + '-' + press_date.substr(0,2));

כאשר עדיין הוא לא הקיש תאריך, אלא כעת נשלח לימות
read=date=הקשתאריך
אני מקבל בינתיים שגיאה בקונסול

TypeError: press_date.substr is not a function

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

dovid

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

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

חוקר

@dovid אמר בNODEJS עצירת המשך ביצוע פעולות:

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

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

nigun

@חוקר
אני עדיין לא הבנתי מה אתה מרוויח מלעבוד עם NODE בשביל ימות המשיח
יכול להיות שאני טועה
אבל לכאורה אפשר לבנות הכל בPHP ואם יש החלקים שאתה צריך בNODE תבנה אותם בנפרד
ותעשה הפניה מהסקריפט של PHP

magicode

מה שדוד כתב פה
https://tchumim.com/post/59303
זה מושלם.
אני גם התעסקתי עם זה ואני ממש חושב שזה מושלם.
אחרי שאתה יודע שכל הנתונים קיימים אז תתחיל לעבד נתונים.
ואם יש לך צורך לאמת נתון.
אז תעשה עוד מאפיין באובייקט של כל נתון
שיכיל פונקציה שבודקת שהנתון נכון. אם היא מחזירה ריק סימן שאין בעיה. אם היא מחזירה טקסט של שגיאה. אתה שולח את זה ללקוח.

dovid

@nigun תהיה בריא. אתה מנסה לחסוך לו חבלי קליטה ואומר לו שלא צריך לעבור למגדלי יוקרה אפשר לחיות בביקתה...

חוקר

@dovid
כרגיל אני אוהב ללמוד בין השורות
אני רואה פה תחביר חדש ולא מוכר.. אני רוצה להבין אותו.
במקום לכתוב הכל עם סוגריים מסולסלות ניתן להשים הכל באותו שורה (הוי אומר לפני ה ; שמפריד לנו שורה בקוד, ואז הכל רץ כאילו בתוך סוגריים, משהו כזה
אני מתכווין לחלק הזה ממה שכתבת

for (var key of props) 
  if(!query[key.name])
    return res.send(`read=${key.client_msg}=${key.name}`);

(ככה זה כשלא לומדים ליבה..)

dovid

גם בPHP זה ככה...
כל המקרים בהם יש {} לבלוק פקודות (כמו if, while וכו'), אפשר לוותר עליו אם זה שורה בודדת (למעט try).
אני מאוד מחבב את זה כשאפשר כי זה מוסיף קריאות לדעתי (אבל כשלא משתמשים בIDE יש בזה סיכון לטעויות של הוספת שורה מתחת במחשבה שזה חלק מהבלוק).

תחומים - פורום חרדי מקצועי

NODEJS עצירת המשך ביצוע פעולות