23-06-2006, 08:57
|
|
|
חבר מתאריך: 25.08.03
הודעות: 9,114
|
|
ראשיצ, הבעיה העיקרית שנתקלתי בה:
אתה פונה לאיבר במערך כך:
היות ו-member אינו קבוע אלא מחרוזת אתה צריך לתחום את השם הזה במרכאות ולפנות לאיבר כפי שתראה בדוגמת הקוד שתיקנתי והציג להלן.
מעבר לזה ניתקלתי רק בבעיות של יעילות וסירבול שמשום מה להרבה מפתחים נדמה שזה נוחות, אבל אין צורך ביותר ממאמץ מוחי קטן כדי להשכיל להבין שמדובר בבזבוז מיותר (אם כי בזבוז שהשרת בוודאי לא ירגיש ולא יתרגש ממנו) והנוחות לא בהכרח מפיקה תועלת, תסתכל באופן שרשור המשתנים בדוגמה להלן.
קשה להבין מה אתה רוצה לעשות, מדוע אתה מגיש שאילתה על מידע קיים ומיד מוסיף אותו חזרה למסד נתונים, אתה בעצם מכפיל רשומות ללא טעם - לפחות מהקצת קוד שהצגת בהודעה האחרונה ושבה הסתכלתי.
כשיש הודעות שגיאה - אל תשכח להציג לנו גם את הודעת השגיאה !
כשאתה משרש מחרוזת - אל תשכח להדפיס (echo/print) את המחרוזת הסופית כדי שנדע איך נראית המחרוזת ולעלות על הבעיה (אם תקפיד לעשות זאת אפילו לא תצטרך לחכות עד שיעזרו לך, אתה בעצמך תעלה על הבעיה במהירות).
קוד PHP:
while($row2 = mysql_fetch_array($result2)) { $query = " INSERT INTO downloads ( name, comment, picture, link, feedback, username ) VALUES ( {$row2['name']}, {$row2['comment']}, {$row2['picture']}, {$row2['link']}, {$row2['feedback']}, {$row2['username']})"; }
על מנת לשפר ביצועים צריך לתחום את המחרוזת במרכאות בודדות ולשרשר את המשתנים באמצעות האופרטור נקודה (.) כמובן.
אני מקווה שההזחות כאן לא ישתבשו, אחרת אשתדל לעלות קובץ עם הקוד.
|