View Single Post
ישן 05-09-10, 17:52   # 5
Hagaibl
משתמש - היכל התהילה
 
מיני פרופיל
תאריך הצטרפות: Jul 2010
הודעות: 214

Hagaibl לא מחובר  

ציטוט:
נכתב במקור על ידי MasterT צפה בהודעה
אני מציע שתסתכל במחלקות MySQL וללמוד מהם.

אין צורך להגדיר " var $host,$user,$pass,$dbname;" ודומיהם.

ובהמשך להערה האחרונה - תסתכל על הקוד שלך..
PHP קוד:
    var $user$name$description$main_img;
    function 
add_album($user$name$description$main_img)
    {
        global 
$db;
        
$name $_POST["name"];
        
$user $_POST["user"];
        
$description $_POST["description"];
        
$main_img $_POST["main_img"]; 
אתה מצד אחד הגדרתי properties של המחלקה (ואתה לא משתמש בהם - ז"א $this->name),
אחר כך אתה מגדיר שלפונקציה קוראים עם נתונים מסויימים - אבל אתה אחר כך מגדיר אותם מחדש.


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

גלריה כותבים עם 2 L - ז"א gallery.

אם אתה מעוניין להתחיל ליישם את עקרונות התכנות מונחה עצמים, אני שוב פעם - ממליץ להסתכל על מחלקות קיימות.

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


אני חושב שלפני שתתחיל בלתכנת מערכת גדולה כדאי שתקרא מחדש את הדפים ב-PHP.NET שמסבירים על פונקציות ועל OOP. יש דף מעולה שמסביר על פונקציות ב-PHP.NET: http://www.w3schools.com/PHP/php_functions.asp


בהצלחה!
מחזק כל מילה. בנוסף אתה עושה פונקציה של יצירת אלבום אבל פונקציה של מחיקת תמונה? במחלקה של גלריה? כאילו למה לעשות סלט? תעשה קובץ שיכיל את הפונקציות שאחראיות על התמונות, אחת על האלבומים. זה ריקני מידי. לא עושים לכל דבר מחלקה. לפעמים זה לא כדאי.
__________________
בברכה,
חגי בלוך גדות.
  Reply With Quote