Add section on inappropriate intimacy between classes
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 2/5
- Thời gian dự kiến
- 1-3 giờ
- Mức phù hợp với người mới
- 48/100
- Loại issue
- Tài liệu
- Độ rõ ràng
- Khá rõ ràng
- Mức độ hoạt động
- Đình trệ
- Công nghệ
- python
- Lĩnh vực
- documentation
Hướng nghiên cứu
Bắt đầu bằng cách xác định phần trong tài liệu khóa học đề cập đến thiết kế lớp hoặc việc vẽ biểu đồ, vì issue không nêu tên tệp hay điểm bắt đầu. Thêm phần giải thích được cung cấp và các ví dụ về MIKE IO vào đó, đồng thời giữ nguyên hướng dẫn về các API công khai và thao tác dữ liệu; công việc được coi là hoàn tất khi phần mới được xuất bản trong tài liệu khóa học phù hợp.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Classes should interact through public APIs, not by reaching into each other's private attributes.
Example from MIKE IO plotting refactor:
Before (inappropriate intimacy):
# Plotter reaches into geometry internals for subsetting
values = values[self.da.geometry.top_elements]
geometry = self.da.geometry.geometry2d
# Plotter uses private attribute to check for time axis
if self.da._has_time_axis:
return self.da.values[0]
After (using public API):
# Use sel/isel for subsetting
da = da.sel(layers="top")
geometry = da.geometry
# Use public dims property
da = self.da.isel(time=0) if "time" in self.da.dims else self.da
Key principles:
- Never access private attributes (prefixed with
_) of another class - Use public methods like
sel,isel, and public properties likedims - If you need to access private state, that's a signal the class is missing a public API
- Plotters should plot, not subset — keep data manipulation in the data layer
- Ngôn ngữ chính
- Jupyter Notebook
- Star
- 8
- Fork
- 1
- Merge trung bình
- 4 phút
- Pull request đã merge (30 ngày)
- 1
Hướng dẫn đóng góp
Chưa lập chỉ mục được hướng dẫn đóng góp cho kho mã nguồn này
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của DHI/python-package-development
-
Độ khó 2/5 1-2 ngày Mức phù hợp với người mới 88/100
-
Độ khó 2/5 1-2 ngày Mức phù hợp với người mới 72/100
DHI/python-package-development#37 · 1 bình luận ·
-
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 35/100
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 48/100
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 48/100
Tất cả issue của DHI/python-package-development
Issue tương tự
-
Độ khó 1/5 1-3 giờ Mức phù hợp với người mới 78/100
Mintplex-Labs/anything-llm#6490 ·
-
bug needs response
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
Adyen/adyen-dotnet-api-library#1869 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
w3c/webdriver-bidi#1168 ·
-
Templating diagram is misleading Đang mởarea/documentation
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
rancher/fleet-product-docs#393 ·
-
Type/Improvement
Độ khó 1/5 1-3 giờ Mức phù hợp với người mới 90/100
OpenNSW/nsw-srilanka#522 ·