
23-06-2006, 15:17
|
|
|
|
חבר מתאריך: 25.08.03
הודעות: 9,114
|
|
בהחלט אך לא הסתכלכלל לא בדקתי את הקוד מכיון שראיתי שהוא ארוך, אם היית מציין בהודעה שהבעיה מצויינת בתוך הקוד כנראה הייתי רואה זאצ כבר בזמן ההודעה הראשונה 
אצלי זה עובד ורואים את מה שכתבת "כל מה שאני כותב פה לא מופיע ..", תנסה שוב ותסתכל גם בקוד מקור מה כן מופיע, יתכן שבממהלך ריצת קוד ה-PHP הקוד "נחתך" ולכן אתה לא מקבל את התוכן שכתוב לאחריו.
לא קראתי את הקוד אבל עיניי צדו מספר שורות הקרובות להערה שכתבת ושאותה קראתי, לכן אני מעיר רק לגבי מספר שורות בסוף.
קוד PHP:
// instead $query3 = "TRUNCATE TABLE `feedback` "; mysql_query($query3); // use mysql_query('TRUNCATE TABLE `feedback`');
1. היות ואתה לא עושה שימוש חוזר במשתנה אתה סתם מבזבז זכרון מיותר (אם כי כמו שציינתי מספר פעמים בעבר, מדובר בבזבוז שנדיר שיורגש).
2. היות ואתה לא מנצל את יתרונותיה של תיחום מחרוזת במרכאות כפולות תשתמש במרכאות בודדות המבזבזות פחות משאבים.
3. die זה פאסה !
תתאר בעצמך את התהליך כשהפונקציה תרוץ, ניקח לדוגמה את הדף שלך, את החלק של תג ה-html, head, body תגי מטא שונים, אולי גם תסריטים ועיצוב CSS, ומי יודע אם בדף אחר שאתה גם משתמש בו עם פונקציה דומה (exit עושה אותה פעולה בדיוק) לא הספקת כבר להדפיס תוכן . ופתאום... die !
כאילו מה ? למה שלא תבצע בדיקה מסודרת אם הפעולה הצליחה ואם כן או לא תדפיס ללקוח אותה בהתאם, אבל למה לחתוך את התסריט במהלך הריצה כדי שהמידע שהתקבל בדפדפן יהיה חצי דף HTML ?!
אפילו בדף שגיאה 404 משקיעים בעיצוב מותאם אישית, אז למה כאן במקום לתת ללקוח הודעה מסודרת מציגים מלל לקוני שלא מכיל אף תפריט או כפתור לחזור לדף הקודם (הלחצן "הקודם" בדפדפן יהווה בעיה מסויימת כשהמידע בדף הקודם התקבל בעקבות בקשת POST).
4. אתה משלב הדפסה של איברים במערך במחרוזת התחומה במרכאות כפולות.
אמנם ניתן להכליל משתנים בין מרכאות כפולות, אך אם מדובר במערך צריך להוסיף מסביב למערך סוגריים מסולסלים כמו בדוגמה הבאה:
5. תכתוב קוד עם הזחות.
את ההזחות תעשה באמצעות טאבים בלבד ! ולא עם רווחים אחרים.
נערך לאחרונה ע"י ישראל K בתאריך 23-06-2006 בשעה 15:20.
|