בניתי מחלקה, הצעות לשיפור

[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


בהצלחה!

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