פרוייקט אבן הטוען

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

@chv אמר בפרוייקט אבן הטוען:

טבלאת אבידות - שם החפץ, תיאור, מיקום האבידה - עיר, טלפון\מייל\חשבון (ID של חשבון באתר), timestamp של יצירת האייטם בDB
טבלאת מציאות - כנ"ל בערך. + סטטוס של האייטם - אם 'נמסר' כבר או לא

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

---

https://github.com/chaim-chv/even-hatoan/blob/master/app.js#L7
עשית חיבור אחד למסד, זה לא טוב, צריך ליצור מאגר של חיבורים, במקום createConnection צריך createPool
https://stackoverflow.com/a/37102397

[chv]

@יוסף-בן-שמעון אמר בפרוייקט אבן הטוען:

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

צודק בהחלט

@יוסף-בן-שמעון אמר בפרוייקט אבן הטוען:

עשית חיבור אחד למסד, זה לא טוב, צריך ליצור מאגר של חיבורים, במקום createConnection צריך createPool

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

[chv]

אגב שכחתי להסביר קצת על המבנה (לא שבניתי משהו סבוך) הנה זה בערך:
האפליקציה בנויה מnodeJS (נטו נוד + אקספרס) וגם חבילה של mysql לnodeJS.
אפליקציית הנוד מורצת על הפלטפורמה הרוקו heroku בחינם.
ה-DB הוא jawsDB MySQL כתוסף לאפליקציית הרוקו. תוכנית חינמית - גודל הDB מוגבל ל-5 מגהבייט בלבד נראה מתיי זה יגיע לגבול..

האפליקציה שרצה בפועל - שואבת את המקור שלה (הכל עובד על GIT) ישירות מהריפו בגיטהב
בכל פוש לריפו (כל pull request שמאושר - הוא גם push) קורה תהליך אוטומטי של בנייה מחדש של האפליקציה על פי הריפו העדכני.

כך שכל שינוי שם משתקף מיידית וניתן לצפייה.

collabration:

מי שמעוניין להיכנס לעניינים בזה ורוצה לערוך וכו' בריפו הנ"ל בלי שיגעון של fork וזה - תשלחו לי במייל ואצרף אתכם כ-collabrators של הריפו.

[yossiz]

@chv שאפו על העבודה!

כמה הערות:

• אם הבקשה ל-‎/data/lost לא עושה שינויים בשרת, מקובל להשתמש ב-GET.
• עכשיו זה זמן מצויין ללמוד על SQL Injection
  מה יקרה אם במקרה נאבד למישהו '; drop from found;
  ניסיתי לבדוק ו... הממ... משהו נשבר באתר שלך

[yossiz]

@yossiz אמר בפרוייקט אבן הטוען:

מה יקרה אם במקרה נאבד למישהו '; drop from found;

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

[yossiz]

מן הענין להביא כאן את הקומיק המפורסם:

[clickone]

@yossiz אמר בפרוייקט אבן הטוען:

האמת שזה לא SQL תקין, לא ברור לי מה מה בדיוק קרה שם ולמה האתר נשבר...

הגיוני שאם הSQL לא תקין, תיווצר שגיאה, ומן הזמן היא לא מטופלת....

(לא עברתי על הקוד, רק ממה שנראה מלמעלה.)

[yossiz]

@clickone אמר בפרוייקט אבן הטוען:

הגיוני שאם הSQL לא תקין, תיווצר שגיאה, ומן הזמן היא לא מטופלת....

אתה צודק, הורדתי את הקוד ובדקתי. כאשר יש שגיאה כל האפליקציה נופלת.
@chv, נעשה pull request

[חראשו]

פוסט זה נמחק!

[Y.Excel.Access]

פוסט זה נמחק!

[חראשו]

פוסט זה נמחק!