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

[Ben21]

PHP קוד:

<?php
class mysql
{
    var $connect_id,$db;
    var $host,$user,$pass,$dbname;
    function db_login($host, $user, $pass, $dbname)
    {
        $this->connect_id = mysql_connect($host, $user, $pass)or die(mysql_error());
        $this->db = mysql_select_db($dbname, $this->connect_id)or die(mysql_error());
    }
}

class galery
{
    //Function For Add New Album
    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"];
        if(empty($user) || empty($name) || empty($description) || empty($main_img))
        {
            echo "Error";
        }
        else
        {
            echo "Good";
            mysql_query("INSERT INTO `albums` (`user`, `name`, `description`, `main_img`) VALUES
            ('".$user."', '".$user."', '".$description."', '".$main_img."')") or die(mysql_error());
        }
    }
    
    //Function For Delete Images In Albums
    var $from, $delete, $delete_id;
    function delete_from($from, $delete, $delete_id)
    {
        global $db;
        $delete = $_POST["del"];
        $delete_id = implode(",",$delete);
        if(empty($delete_id) || empty($from))
        {
            echo "Error";
        }
        else
        {
            echo "Good";
            mysql_query("DELETE FROM `".$from."` WHERE `id` IN (".$delete_id.")") or die(mysql_error());
        }
    }
}
$db = new mysql();
$db->db_login("localhost", "root", "", "cms");
$g = new galery();
?>

מספר שאלות
שאלה ראשונה:
האם את משתני הPOST ותנאיי הIF לעשות לאחר הטופס עצמו בHTML של המערכת או בבדיקה בפונקצייה בCLASS ואז בHTML אתה פשוט קורא לפונקצייה הרצוייה מהמערך וזה מבצע את הבדיקות לבד..

שאלה שנייה:
האם המחלקה יעילה?
אני מתכנן לפתח CMS ענקית ע"י שימוש בOOP

תודה רבה על שיתוף הפעולה, ושבת שלום!

[AlmogBaku]

אני מציע מראש לשקול שימוש ב ActiveRecord..

[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


בהצלחה!

[astricks]

אתה צריך ללמוד OOP לפני שאתה מפתח CMS כזאת.. זה סתם פונקציות בתוך מחלקה.
מציע לך לקרוא קצת חומר, לא חסר

[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


בהצלחה!

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