أفضل الممارسات (Best Practices)

-- جيد: أسماء واضحة ووصفية
CREATE TABLE employee (
    employee_id INT PRIMARY KEY,
    first_name VARCHAR(50),
    last_name VARCHAR(50)
);

CREATE TABLE customer_order (
    order_id INT PRIMARY KEY,
    customer_id INT,
    order_date DATE
);

-- سيء: أسماء غامضة أو مختصرة جداً
CREATE TABLE emp (  -- غير واضح
    id INT,
    fn VARCHAR(50),  -- ما معنى fn؟
    ln VARCHAR(50)
);

CREATE TABLE tbl1 (  -- اسم عديم المعنى
    data VARCHAR(100)
);

-- جيد: أسماء واضحة مع بادئة للمفاتيح الأساسية
CREATE TABLE product (
    product_id INT PRIMARY KEY AUTO_INCREMENT,
    product_name VARCHAR(200) NOT NULL,
    product_code VARCHAR(50) UNIQUE,
    unit_price DECIMAL(10, 2),
    stock_quantity INT DEFAULT 0,
    is_active BOOLEAN DEFAULT TRUE,
    created_at DATETIME DEFAULT CURRENT_TIMESTAMP,
    updated_at DATETIME ON UPDATE CURRENT_TIMESTAMP
);

-- سيء: أسماء غير متسقة
CREATE TABLE product (
    ID INT,              -- غير متسق مع باقي الأعمدة
    Name VARCHAR(200),   -- أحرف كبيرة
    pCode VARCHAR(50),   -- camelCase غير متسق
    Price DECIMAL(10,2), -- مختلط
    qty INT              -- مختصر جداً
);

-- جيد: أسماء وصفية مع بادئات
CREATE TABLE employee (
    employee_id INT PRIMARY KEY,
    email VARCHAR(100),
    department_id INT,
    
    -- فهارس مع بادئة idx_
    CONSTRAINT idx_employee_email UNIQUE (email),
    
    -- مفاتيح أجنبية مع بادئة fk_
    CONSTRAINT fk_employee_department 
        FOREIGN KEY (department_id) 
        REFERENCES department(department_id),
    
    -- فهارس عادية
    INDEX idx_employee_department (department_id)
);

-- إنشاء فهارس منفصلة
CREATE INDEX idx_employee_name ON employee(first_name, last_name);
CREATE INDEX idx_employee_hire_date ON employee(hire_date);

-- جيد: منسق ومقروء
SELECT 
    e.employee_id,
    e.first_name,
    e.last_name,
    d.department_name,
    e.salary
FROM employee e
INNER JOIN department d 
    ON e.department_id = d.department_id
WHERE 
    e.is_active = TRUE
    AND e.salary > 5000
    AND d.location = 'الرياض'
ORDER BY 
    e.last_name ASC,
    e.first_name ASC
LIMIT 100;

-- سيء: غير منسق وصعب القراءة
SELECT e.employee_id,e.first_name,e.last_name,d.department_name,e.salary FROM employee e INNER JOIN department d ON e.department_id=d.department_id WHERE e.is_active=TRUE AND e.salary>5000 AND d.location='الرياض' ORDER BY e.last_name ASC,e.first_name ASC LIMIT 100;

-- جيد: أسماء مستعارة واضحة
SELECT 
    e.employee_id,
    e.first_name,
    e.last_name,
    d.department_name,
    COUNT(p.project_id) AS total_projects,
    AVG(p.budget) AS average_project_budget
FROM employee e
LEFT JOIN department d ON e.department_id = d.department_id
LEFT JOIN project p ON e.employee_id = p.manager_id
GROUP BY e.employee_id;

-- سيء: أسماء مستعارة غامضة
SELECT 
    t1.id,
    t1.n,
    t2.dn,
    COUNT(t3.id) AS c
FROM employee t1
LEFT JOIN department t2 ON t1.did = t2.id
LEFT JOIN project t3 ON t1.id = t3.mid
GROUP BY t1.id;

-- جيد: استخدام CTE لتقسيم المنطق
WITH active_employees AS (
    SELECT 
        employee_id,
        first_name,
        last_name,
        department_id,
        salary
    FROM employee
    WHERE is_active = TRUE
),
department_stats AS (
    SELECT 
        department_id,
        COUNT(*) AS employee_count,
        AVG(salary) AS avg_salary
    FROM active_employees
    GROUP BY department_id
)
SELECT 
    d.department_name,
    ds.employee_count,
    ds.avg_salary
FROM department_stats ds
JOIN department d ON ds.department_id = d.department_id
WHERE ds.employee_count > 5
ORDER BY ds.avg_salary DESC;

-- جيد: تعليقات توضح السبب والسياق
-- تحديث رواتب الموظفين بناءً على تقييم الأداء السنوي
-- ملاحظة: يتم تطبيق هذا فقط على الموظفين الذين أكملوا سنة كاملة
UPDATE employee
SET salary = salary * 1.10
WHERE 
    hire_date <= DATE_SUB(CURRENT_DATE, INTERVAL 1 YEAR)
    AND performance_rating >= 4
    AND is_active = TRUE;

-- سيء: تعليقات تكرر ما يفعله الكود
-- تحديث جدول الموظفين
UPDATE employee
-- ضرب الراتب في 1.10
SET salary = salary * 1.10
-- حيث تاريخ التوظيف أقل من أو يساوي التاريخ الحالي ناقص سنة
WHERE hire_date <= DATE_SUB(CURRENT_DATE, INTERVAL 1 YEAR);

DELIMITER //

/*
 * اسم الإجراء: calculate_employee_bonus
 * الوصف: حساب مكافأة نهاية العام للموظف بناءً على الأداء والأقدمية
 * 
 * المعاملات:
 *   - p_employee_id (IN): معرف الموظف
 *   - p_bonus_amount (OUT): المكافأة المحسوبة
 *   - p_success (OUT): نجاح أو فشل العملية
 * 
 * المنطق:
 *   - الموظفون الجدد (أقل من سنة): 5% من الراتب
 *   - الموظفون (1-5 سنوات): 10% من الراتب
 *   - الموظفون الأقدم (أكثر من 5 سنوات): 15% من الراتب
 *   - مضاعفة المكافأة للموظفين ذوي الأداء الممتاز (rating >= 4.5)
 * 
 * المؤلف: أحمد محمد
 * التاريخ: 2024-01-15
 * آخر تعديل: 2024-02-01
 */
CREATE PROCEDURE calculate_employee_bonus(
    IN p_employee_id INT,
    OUT p_bonus_amount DECIMAL(10, 2),
    OUT p_success BOOLEAN
)
BEGIN
    DECLARE v_salary DECIMAL(10, 2);
    DECLARE v_years_of_service INT;
    DECLARE v_performance_rating DECIMAL(3, 2);
    DECLARE v_bonus_percentage DECIMAL(5, 2);
    
    -- معالج للأخطاء
    DECLARE EXIT HANDLER FOR SQLEXCEPTION
    BEGIN
        SET p_success = FALSE;
        SET p_bonus_amount = 0;
    END;
    
    -- جلب بيانات الموظف
    SELECT 
        salary,
        YEAR(CURRENT_DATE) - YEAR(hire_date),
        performance_rating
    INTO 
        v_salary,
        v_years_of_service,
        v_performance_rating
    FROM employee
    WHERE employee_id = p_employee_id;
    
    -- حساب نسبة المكافأة بناءً على الأقدمية
    IF v_years_of_service < 1 THEN
        SET v_bonus_percentage = 5.00;
    ELSEIF v_years_of_service <= 5 THEN
        SET v_bonus_percentage = 10.00;
    ELSE
        SET v_bonus_percentage = 15.00;
    END IF;
    
    -- حساب المكافأة
    SET p_bonus_amount = v_salary * v_bonus_percentage / 100;
    
    -- مضاعفة للأداء الممتاز
    IF v_performance_rating >= 4.5 THEN
        SET p_bonus_amount = p_bonus_amount * 2;
    END IF;
    
    SET p_success = TRUE;
END //

DELIMITER ;

-- جيد: أنواع بيانات مناسبة
CREATE TABLE product (
    product_id INT UNSIGNED AUTO_INCREMENT PRIMARY KEY,
    product_name VARCHAR(200) NOT NULL,
    product_code CHAR(10) UNIQUE,  -- طول ثابت
    unit_price DECIMAL(10, 2) NOT NULL,  -- للمبالغ المالية
    stock_quantity SMALLINT UNSIGNED DEFAULT 0,
    is_active BOOLEAN DEFAULT TRUE,
    category_id TINYINT UNSIGNED,  -- للقيم الصغيرة
    created_at TIMESTAMP DEFAULT CURRENT_TIMESTAMP
);

-- سيء: أنواع بيانات غير مناسبة
CREATE TABLE product (
    product_id VARCHAR(50),  -- مبالغة، INT كافٍ
    product_name TEXT,  -- مبالغة، VARCHAR أفضل
    unit_price FLOAT,  -- غير دقيق للمبالغ المالية
    stock_quantity INT,  -- مبالغة، SMALLINT كافٍ
    is_active VARCHAR(10),  -- يجب أن يكون BOOLEAN
    created_at VARCHAR(50)  -- يجب أن يكون TIMESTAMP
);

-- جيد: قيود شاملة لضمان سلامة البيانات
CREATE TABLE employee (
    employee_id INT UNSIGNED AUTO_INCREMENT PRIMARY KEY,
    email VARCHAR(100) NOT NULL UNIQUE,
    first_name VARCHAR(50) NOT NULL,
    last_name VARCHAR(50) NOT NULL,
    salary DECIMAL(10, 2) NOT NULL CHECK (salary >= 3000),
    hire_date DATE NOT NULL,
    department_id INT UNSIGNED NOT NULL,
    manager_id INT UNSIGNED,
    
    CONSTRAINT fk_employee_department
        FOREIGN KEY (department_id)
        REFERENCES department(department_id)
        ON DELETE RESTRICT
        ON UPDATE CASCADE,
        
    CONSTRAINT fk_employee_manager
        FOREIGN KEY (manager_id)
        REFERENCES employee(employee_id)
        ON DELETE SET NULL,
        
    CONSTRAINT chk_hire_date
        CHECK (hire_date <= CURRENT_DATE)
);

// جيد: Prepared Statement آمن
$stmt = $pdo->prepare("
    SELECT * FROM employee 
    WHERE department_id = :dept_id 
    AND salary > :min_salary
");
$stmt->execute([
    'dept_id' => $department_id,
    'min_salary' => $min_salary
]);

// سيء: عرضة لـ SQL Injection
$query = "SELECT * FROM employee 
          WHERE department_id = $department_id 
          AND salary > $min_salary";
$result = $pdo->query($query);

-- أضف فهارس على:
-- 1. المفاتيح الأجنبية
CREATE INDEX idx_employee_department ON employee(department_id);

-- 2. الأعمدة المستخدمة في WHERE كثيراً
CREATE INDEX idx_employee_email ON employee(email);
CREATE INDEX idx_employee_active ON employee(is_active);

-- 3. الأعمدة المستخدمة في ORDER BY
CREATE INDEX idx_employee_hire_date ON employee(hire_date);

-- 4. فهارس مركبة للاستعلامات الشائعة
CREATE INDEX idx_employee_dept_salary 
    ON employee(department_id, salary);

-- migrations/2024_02_01_create_employee_table.sql

-- UP Migration: إنشاء الجدول
CREATE TABLE employee (
    employee_id INT UNSIGNED AUTO_INCREMENT PRIMARY KEY,
    first_name VARCHAR(50) NOT NULL,
    last_name VARCHAR(50) NOT NULL,
    email VARCHAR(100) NOT NULL UNIQUE,
    hire_date DATE NOT NULL,
    created_at TIMESTAMP DEFAULT CURRENT_TIMESTAMP
);

-- DOWN Migration: التراجع عن التغيير
-- DROP TABLE IF EXISTS employee;

# Database Changelog

## [1.2.0] - 2024-02-01
### Added
- إضافة جدول employee_performance لتتبع الأداء
- إضافة فهرس على employee.department_id

### Changed
- تعديل نوع بيانات employee.salary من INT إلى DECIMAL(10,2)
- تحديث إجراء calculate_bonus لدعم المكافآت الربع سنوية

### Fixed
- إصلاح قيد FOREIGN KEY في جدول project

## [1.1.0] - 2024-01-15
### Added
- إضافة جدول department
- إضافة علاقة employee-department

-- 1. اختبر الاستعلام مع SELECT قبل UPDATE/DELETE
-- بدلاً من:
DELETE FROM employee WHERE department_id = 5;

-- افعل:
SELECT * FROM employee WHERE department_id = 5;  -- تحقق أولاً
-- إذا كانت النتائج صحيحة:
DELETE FROM employee WHERE department_id = 5;

-- 2. استخدم EXPLAIN لتحليل الأداء
EXPLAIN SELECT * FROM employee WHERE department_id = 5;

-- 3. اختبر في بيئة التطوير أولاً
-- لا تختبر مباشرة في الإنتاج!

-- جيد: استخدام معاملة للعمليات المترابطة
START TRANSACTION;

UPDATE account SET balance = balance - 1000 WHERE account_id = 1;
UPDATE account SET balance = balance + 1000 WHERE account_id = 2;
INSERT INTO transaction_log (from_account, to_account, amount) 
VALUES (1, 2, 1000);

-- تحقق من النتائج قبل COMMIT
SELECT * FROM account WHERE account_id IN (1, 2);

-- إذا كانت صحيحة:
COMMIT;
-- وإلا:
-- ROLLBACK;

-- تفعيل سجل الاستعلامات البطيئة
SET GLOBAL slow_query_log = 'ON';
SET GLOBAL long_query_time = 2;

-- مراجعة الاستعلامات البطيئة بانتظام
-- وتحسينها بإضافة فهارس أو إعادة كتابتها

-- تحسين الجداول (شهرياً)
OPTIMIZE TABLE employee;
OPTIMIZE TABLE department;

-- تحليل الجداول (أسبوعياً)
ANALYZE TABLE employee;

-- فحص الجداول (شهرياً)
CHECK TABLE employee;

-- إصلاح الجداول (عند الحاجة)
REPAIR TABLE employee;