Có phải nó không đúng cách / Thiết kế xấu để đặt một chủ đề / công nhân nền trong một lớp?

Tôi có một lớp sẽ đọc từ Excel (C # và .Net 4) và trong lớp đó tôi có một nhân viên nền sẽ tải dữ liệu từ Excel trong khi UI vẫn có thể phản hồi. Câu hỏi của tôi là như sau: Có phải là một thiết kế xấu khi có một nhân viên nền trong một lớp học? Tôi có nên tạo lớp của mình mà không có nó và sử dụng một nhân viên nền để vận hành trên lớp đó không? Tôi không thể thấy bất kỳ vấn đề nào thực sự tạo ra lớp học của mình theo cách này nhưng sau đó tôi lại là một người mới nên tôi nghĩ rằng tôi sẽ đảm bảo trước khi tiếp tục.

Tôi hy vọng rằng câu hỏi này có liên quan ở đây vì tôi không nghĩ nó nên có trên stackoverflow khi mã của tôi hoạt động, đây chỉ là một vấn đề thiết kế.

Tôi có nên tạo lớp của mình mà không có nó và sử dụng một nhân viên nền để vận hành trên lớp đó không?

Vâng, bạn nên. Và tôi sẽ cho bạn biết lý do - bạn đang vi phạm Nguyên tắc Trách nhiệm duy nhất . Bằng cách kết hợp chặt chẽ lớp truy cập tài liệu excel với cách nó truy cập tài liệu excel, bạn loại bỏ khả năng mã "trình điều khiển" (bất kỳ mã nào sử dụng mã này) để thực hiện theo cách khác. Làm thế nào khác nhau, bạn có thể yêu cầu? Điều gì xảy ra nếu mã điều khiển có hai thao tác mất nhiều thời gian nhưng muốn chúng là tuần tự? Nếu bạn cho phép bộ điều khiển khả năng xử lý luồng, nó có thể thực hiện cả hai tác vụ chạy dài cùng nhau trong một luồng. Điều gì xảy ra nếu bạn muốn truy cập tài liệu excel từ ngữ cảnh không phải UI và không cần phải xử lý luồng?

Bằng cách chuyển trách nhiệm phân luồng ra cho người gọi, bạn cho phép mã linh hoạt hơn, làm cho nó có thể tái sử dụng nhiều hơn.

Đó là thiết kế tốt để có các hoạt động UI hoạt động trong một luồng riêng biệt từ các tác vụ nền. Nếu không, UI trở nên không phản hồi khi ứng dụng bận.

Nếu bạn có thể tách phần hoạt động trong luồng nền sang lớp riêng của nó, mã sẽ sạch hơn.

Tôi sẽ tách UI của bạn khỏi tác vụ nền của bạn bằng các lớp riêng biệt. Làm như vậy khuyến khích sự tách biệt các mối quan tâm. Mã UI và logic nghiệp vụ không nên trộn lẫn.

Từ những gì tôi nhớ về BackgroundWorkers là họ cung cấp một số phương thức liên kết như khả năng gửi các bản cập nhật tiến độ tới UI. Không có quy tắc nào nói rằng bạn không thể sử dụng nó từ một lớp khác.

Ngoài ra, nếu bạn đang thực hiện việc lặp mà không yêu cầu các mục được xử lý theo một thứ tự cụ thể, hãy xem xét sử dụng ThreadPool thay thế (hoặc nếu bạn đang trên .NET 4 sử dụng Thư viện song song tác vụ ).