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

חוקר

@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 יש בזה סיכון לטעויות של הוספת שורה מתחת במחשבה שזה חלק מהבלוק).

חוקר

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

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

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

nigun

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

dovid

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

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

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

nigun

@dovid אני הייתי ממשיל את זה לאופניים חשמליות ורכב בסביבה עירונית

magicode

@nigun
כמובן שהרכב זה PHP בציור הזה.

חוקר

אז לפי הנראה הכי גמיש מבחינתי לכתוב בצורה הזאת:

var user_id;
if (typeof (user_id = read('id')) === 'undefined') return false ;
var name;
if (typeof (name = read('name')) === 'undefined') return false ;
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');
    }
}

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

אהרן

בשורה 4 חסר await

if (typeof (user_id = read('id')) === 'undefined') return false ;

==

if ( !read('id') )  return;

חוקר

@אהרן
user_id אמור לקבל ערך חוזר מ read, לכן לא עשיתי אותו כמו מה שכתבת.
וכן זה לא פעולה של חיבור לבסיס נתונים וכדומה שיש לי כאן מה לחכות, אלא זה פונקצית בדיקה, באם הערך קיים בURL זה מחזיר את ערכו, אחרת זה מדפיס למסך הוראה למערכת של ימות שיחזיר את הערך המבוקש ומפסיק את המשך הריצה (זה היה כל המטרה שלי בדיון כאן, לדעת איך אני עוצר את המשך הריצה במקרה שאני דורש מהמערכת שתחזיר לי ערך שעדיין לא קיים ב URL), ולכן אני לא רואה שום סיבה להכניס כאן await

avr416

@חוקר
לגבי הawait אתה צודק, אך מלבד זה, אהרן הציע לך תחביר מקוצר ופשוט יותר.
אם הערך שחוזר מהפונקציה הtype שלו undefined, זה אומר שגם הערך עצמו הוא undefined.
ממילא אין כל מניעה מלכתוב כמו שאהרון הציע.
כנ"ל return false שקול לreturn, כי בjs אם הערך null , undefined או false זה אותו דבר (כל זמן שאתה לא משתמש ב===)

clickone

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

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

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

זה הפתיע גם אותי, כי גם אני הייתי כותב בשיטה הזו ומאד אהבתי את זה

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

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