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

[Daniel]

אני מציע שתסתכל במחלקות 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


בהצלחה!