Tối ưu laravel

Muốn load tất cả các cấp dùng relationship dạng parent_id sẽ bị như này vì nó đệ quy. Chuyển qua nested set hoặc bạn đang dùng mysql 8 thì có hỗ trợ query đệ quy đó.

public function childrenCategories()
		{
        return $this->hasMany('App\DaiLy', 'nguoigt', 'affID')->with('childrenCategories')->select('id','affID','nguoigt');
		}

Có thể đoạn đệ quy này gây ra việc tốn nhiều query đấy

Bạn có thể thử truy vấn với WITH RECURSIVE (sử dụng CTE) để truy vấn mọi đại lý cấp dưới, và cấp dưới của cấp dưới,... của 1 user.

WITH RECURSIVE a AS (
  SELECT *, 0 AS level, CAST(affID AS VARCHAR) AS path
  FROM daily 
  WHERE nguoigt = 0
  
  UNION ALL
  
  SELECT b.*, a.level + 1 AS level, CAST(CONCAT(a.path, '/', b.affID) AS VARCHAR) AS path
  FROM a 
  INNER JOIN daily b
  ON a.affID = b.nguoigt
)
SELECT * FROM a ORDER BY path;

Đây là dữ liệu giả + ví dụ: http://sqlfiddle.com/#!17/e8210f/4

CTE hỗ trợ cho mariaDB từ bản 10.2.2 trở lên nên bạn cần phải update phiên bản mariaDB mới có thể gọi câu query trên.

Mình chưa nhìn được toàn bộ code nên cũng chưa biết có chỗ nào trong code gây n+1 query nữa không nhưng hiện tại theo code trên thì có đoạn khai báo relations:

public function childrenCategories()
{
    return $this->hasMany('App\DaiLy', 'nguoigt', 'affID')
        ->with('childrenCategories')
        ->select('id','affID','nguoigt');
}

Khi code Laravel bạn nên lưu ý không dùng query sql hoặc eager loading trong getter / accessor, hoặc trong các class có nhiệm vụ transform dữ liệu trước khi response... nó sẽ là nguyên nhân gây ra vấn đề n+1 query. Bạn nên bỏ cái with('childrenCategories') này đi.

Mình đọc comment phía dưới, có vẻ bạn muốn truy vấn danh sách danh mục đa cấp. Bạn thử chuyển đổi code qua các cách dưới đây xem sao nhé:

• Cách 1: Trả về danh sách danh mục gốc, Web UI sẽ có nút chức năng tải chi tiết các danh mục con của danh mục tương ứng. Khi click vào thì bạn sẽ gọi api để lấy ra danh các danh mục con.
• Cách 2: Thêm một trường danh string để lưu đường dẫn đa cấp của một danh mục nested_path - là chuỗi các ID của danh mục mà nó kế thừa (id của danh mục cha). VD:

__ ID 1 - Category 1=> nested_path = null
    |__ ID 3 - Category 1.1 => nested_path = "1"
    |     |__ ID 5 - Category 1.1.1 => nested_path = "1_3"
    |     |__ ID 6 - Category 1.1.2 => nested_path = "1_3"
    |
    |__ ID 4 - Category 1.2 => nested_path = "1"

Khi muốn lấy danh mục con của Category 1, bạn sử dụng regex trong SQL để query nested_path ^= "1_", khi đó chỉ mất 1 truy vấn có thể lấy được toàn bộ children, bây giờ dùng code để sắp xếp nó thành dạng đệ quy nếu bạn cần. Do ít query nên nó cũng sẽ nhanh hơn đó.

Bạn tham khảo nhé.

433 queries thì rõ ràng là không được r. Ở bên phía view bạn có gọi ra quan hệ nào của nó mà chưa load ra k?

cách này chắc là tối ưu nhất về mặt hiệu năng:

1. Select all từ bảng daily
2. Ở php build tree bằng recursive Nếu bảng daily to quá thì cân nhắc dùng chuck để buffer ko bị tràn Cách này bị dư thừa ở chỗ lấy all bảng daily nhưng sẽ tránh đc việc phải query nhiều