Thiết kế lớp học Python - Chia lớp lớn thành nhiều lớp thành các chức năng nhóm

Question

OK Tôi có 2 lớp học thực sự lớn> 1k dòng mỗi lớp mà tôi hiện đã chia thành nhiều lớp. Sau đó, họ được tái kết hợp bằng cách sử dụng nhiều thừa kế. Bây giờ tôi tự hỏi, nếu có bất kỳ cách làm sạch hơn/tốt hơn để làm điều này. Hoàn toàn bao thanh toán chúng ra sẽ dẫn đến số lượng vô tận của các cuộc gọi self.otherself.do_something, mà tôi không nghĩ là cách nó nên được thực hiện.

Để làm cho mọi việc rõ ràng đây là những gì nó hiện trông giống như:

from gui_events import GUIEvents # event handlers 
from gui_helpers import GUIHelpers # helper methods that don't directly modify the GUI 

# GUI.py 
class GUI(gtk.Window, GUIEvents, GUIHelpers): 
    # general stuff here stuff here 

Một vấn đề đó là kết quả của việc này là pylint phàn nàn cho tôi hàng nghìn tỷ "init không được gọi là"/"thuộc tính không xác định"/"thuộc tính truy cập trước khi định nghĩa "cảnh báo.

CHỈNH SỬA:
Bạn có thể muốn xem mã, để tạo cho mình một bức tranh về toàn bộ thực tế.
http://github.com/BonsaiDen/Atarashii/tree/next/atarashii/usr/share/pyshared/atarashii/

Xin lưu ý, tôi thực sự cố gắng để giữ điều này càng tốt, tôi đang sử dụng thông tin để phát hiện trùng lặp mã, điều duy nhất nó phàn nàn là nhập.

Answer 1

Nếu bạn muốn sử dụng nhiều thừa kế để kết hợp mọi thứ vào một lớp lớn (có thể làm điều này), thì bạn có thể cấu trúc lại từng lớp cha để mọi phương thức và thuộc tính riêng tư (bắt đầu bằng ' __ ') hoặc có tiền tố 2-3 ký tự ngắn duy nhất cho lớp đó. Ví dụ: tất cả các phương thức và thuộc tính trong lớp GUIEvents của bạn có thể bắt đầu bằng ge_, mọi thứ trong GUIHelpers có thể bắt đầu bằng gh_. Bằng cách này, bạn sẽ đạt được một số sự rõ ràng của việc sử dụng các cá thể lớp phụ riêng lẻ (self.ge.doSomething() vs self.ge_doSomething()) và bạn sẽ tránh xung đột tên thành viên, đây là rủi ro chính khi kết hợp các lớp lớn thành một.

Answer 2

Tôi nghĩ đây là vấn đề về thiết kế OO chung hơn là vấn đề về Python. Python khá nhiều cung cấp cho bạn tất cả các công cụ OOP cổ điển, được đóng gói thuận tiện. Bạn sẽ phải mô tả vấn đề chi tiết hơn (ví dụ: các lớp học GUIEvents và GUIHelpers có chứa gì?)

Một khía cạnh cụ thể của Python cần xem xét như sau: Python hỗ trợ nhiều mô hình lập trình và thường là giải pháp tốt nhất không OOP. Đây là có thể là trường hợp ở đây. Nhưng một lần nữa, bạn sẽ phải ném thêm chi tiết để có được một câu trả lời có ý nghĩa.

Answer 3

Mã của bạn có thể được cải thiện đáng kể bằng cách triển khai thiết kế Model-View-Controller. Tùy thuộc vào cách GUI và công cụ của bạn được thiết lập, bạn cũng có thể hưởng lợi từ việc "widget hóa" các phần của GUI, để thay vì có một Model-View-Controller khổng lồ, bạn có một Model-View-Controller chính quản lý một loạt các Các Model-View-Controllers nhỏ hơn, mỗi phần cho các phần riêng biệt của GUI của bạn. Điều này sẽ cho phép bạn chia nhỏ công cụ và GUI thành nhiều lớp và bạn có thể sử dụng lại các phần của nó, giảm tổng số mã bạn cần duy trì.

Trong khi python hỗ trợ nhiều mô hình lập trình, cho các công cụ GUI, giải pháp tốt nhất sẽ gần như luôn luôn là một thiết kế hướng đối tượng.

Answer 4

Bắt đầu bằng cách tìm các lớp học mô hình khái niệm thế giới thực mà ứng dụng của bạn cần làm việc. Đó là những ứng cử viên tự nhiên cho các lớp học.

Cố gắng tránh nhiều thừa kế càng nhiều càng tốt; nó hiếm khi hữu ích và luôn luôn hơi khó hiểu. Thay vào đó, hãy xem sử dụng thành phần chức năng (mối quan hệ "HAS-A") để cung cấp các thuộc tính phong phú cho các đối tượng của bạn được tạo từ các đối tượng khác.

Hãy nhớ đặt từng phương thức làm một điều nhỏ, cụ thể; điều này nhất thiết đòi hỏi phải phá vỡ các phương pháp làm quá nhiều thứ thành các phần nhỏ hơn.

Refactor trường hợp bạn tìm thấy nhiều phương pháp như vậy đang nhân bản chức năng của nhau; đây là một cách khác để tìm các bộ sưu tập tự nhiên về chức năng xứng đáng nằm trong một lớp riêng biệt.

Answer 5

Một khả năng là để gán chức năng nhập khẩu vào lớp thuộc tính:

Trong file a_part_1.py:

def add(self, n): 
    self.n += n 
def __init__(self, n): 
    self.n = n 

Và trong tập tin lớp học chính:

import a_part_1 

class A: 
    __init__ = a_part_1.__init__ 
    add = a_part_1.add 

Hoặc nếu bạn không muốn để cập nhật tệp chính khi các phương thức mới được thêm vào:

class A: pass 

import a_part_1 
for k, v in a_part_1.__dict__.items(): 
    if callable(v): 
     setattr(A,k,v)