Isn't section 5.1 "Use object destructuring" conflicting with 3.3 "Use object method shorthand" ?
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức phù hợp với người mới
- 35/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ệ
- javascript
- Lĩnh vực
- documentation
Hướng nghiên cứu
Đọc các phần 3.3 và 5.1 của hướng dẫn về style, sau đó tái hiện ví dụ về object-method và destructuring từ issue. Kiểm tra phần thảo luận trong các bình luận để tìm cách diễn giải đã được thống nhất và ghi lại một khuyến nghị rõ ràng, không mâu thuẫn, kèm một ví dụ cho thấy khi nào hướng dẫn được áp dụng. Công việc được xem là hoàn tất khi hai phần này không còn gợi ý việc sử dụng kết hợp không an toàn.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
In section 5.1 it is stated the following:
Use object destructuring when accessing and using multiple properties of an object.
In section 3.3, it is stated the following:
Use object method shorthand.
Let's look at the following example:
const foo = {
bar: 5,
// applying 3.3
computeSomething() {
return this.bar + 5;
}
};
// applying 5.1
const {bar, computeSomething} = foo;
console.log(bar);
computeSomething();
The last instruction throws the following error:
TypeError: Cannot read properties of undefined (reading 'bar')
So, applying both 3.3 and 5.1 can lead to an incorrect code. Either we have to use arrow functions in objects definition; or we have to not use destructuring; but both 3.3 and 5.1 can't be recommended together without creating potential issues. At least, that's what I'm feeling.
What do you think? Did I miss the point or a clarification in the specs about that?
- Ngôn ngữ chính
- JavaScript
- Star
- 148k
- Fork
- 26.5k
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Chuẩn bị môi trường
Dự án này không cung cấp dev container, Dockerfile hay hướng dẫn đóng góp, nên bạn cần tự thiết lập môi trường: hãy bắt đầu từ README và xem hướng dẫn đóng góp lần đầu của chúng tôi để biết các bước chung.
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 airbnb/javascript
-
Severity: Unhandled promise rejection in `whitespace-async.js` when ESLint async path is usedCó thể đã có người làm @bodapatisaikrishna đã nhận 27 ngày trước. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
airbnb/javascript#3237 · 6 bình luận ·
-
Inconsistent semicolon usage in examples (Arrays vs Functions)Có thể đã có người làm @Developer-shivamMishra đã nhận 18 ngày trước. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
airbnb/javascript#3152 · 4 bình luận ·
-
No error handling around execSync + JSON.parse in whitespace.js (ESLint 9 path)Có thể đã có người làm @dataCenter430 đã nhận 222 ngày trước. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 48/100
airbnb/javascript#3238 · 8 bình luận ·
-
Upgrading eslint-plugin-react-hooksCó thể đã có người làm @weihongyu12 đã nhận 368 ngày trước. Đang mở
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 38/100
airbnb/javascript#3186 · 3 bình luận · 2 reaction ·
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 45/100
airbnb/javascript#3173 · 9 bình luận · 1 reaction ·
Tất cả issue của airbnb/javascript
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/100
woocommerce/woocommerce-google-analytics-integration#699 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/100
Maintainer thường phản hồi trong vòng 1 ngày
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
Maintainer thường phản hồi trong vòng 1 ngày
-
t0ggles
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
nolimits4web/swiper#8240 · 1 bình luận ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/100
Inter-Actief/amelink#131 ·