Isn't section 5.1 "Use object destructuring" conflicting with 3.3 "Use object method shorthand" ?
まだ誰も着手していません。
評価
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 初心者へのやさしさ
- 35/100
- issue の種類
- ドキュメント
- 明瞭さ
- おおむね明確
- 活発さ
- 停滞
- 技術スタック
- javascript
調査の方向性
スタイルガイドのセクション3.3と5.1を読み、その後、issueにあるobject-methodとdestructuringの例を再現してください。コメントの議論を確認して合意された解釈を把握し、ガイダンスが適用される場合を示す例を添えて、明確で矛盾のない推奨事項を文書化してください。2つのセクションが安全でない組み合わせた使用を示唆しなくなれば完了です。
索引モデルが issue の本文から書いたものです。
説明
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?
- 主要言語
- JavaScript
- スター
- 148k
- フォーク
- 26.6k
- PR マージ指標
- 30日以内にマージされた PR はありません
環境構築
このプロジェクトには開発コンテナ、Dockerfile、コントリビューションガイドがありません。まず README を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
airbnb/javascript のほかの issue
-
Severity: Unhandled promise rejection in `whitespace-async.js` when ESLint async path is used対応中かも @bodapatisaikrishna が 24 日前に担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
airbnb/javascript#3237 · コメント 6 件 ·
-
Inconsistent semicolon usage in examples (Arrays vs Functions)対応中かも @Developer-shivamMishra が 15 日前に担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
airbnb/javascript#3152 · コメント 4 件 ·
-
No error handling around execSync + JSON.parse in whitespace.js (ESLint 9 path)対応中かも @dataCenter430 が 219 日前に担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 48/100
airbnb/javascript#3238 · コメント 8 件 ·
-
Upgrading eslint-plugin-react-hooks対応中かも @weihongyu12 が 365 日前に担当しました。 オープン
難易度 3/5 1〜2日 初心者へのやさしさ 38/100
airbnb/javascript#3186 · コメント 3 件 · リアクション 2 件 ·
-
難易度 3/5 1〜2日 初心者へのやさしさ 45/100
airbnb/javascript#3173 · コメント 9 件 · リアクション 1 件 ·
airbnb/javascript の issue をすべて見る
似ている issue
-
type: bug
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
blamechris/chroxy#8579 ·
メンテナーはふだん 1 日以内に返信
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 66/100
modusensus/dsh-mneme#411 ·
-
Flaky (suspected): VideoTimelineEditor bed audio test sees 600px instead of 120px under CI loadオープン
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
メンテナーはふだん 1 日以内に返信
-
Actions on Toastsオープン
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
nextcloud-libraries/nextcloud-dialogs#2669 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
XRPLF/xrpl-dev-portal#4000 ·
メンテナーはふだん 1 日以内に返信