🎄teratailクリスマスプレゼントキャンペーン2024🎄』開催中!

\teratail特別グッズやAmazonギフトカード最大2,000円分が当たる!/

詳細はこちら
Vue.js

Vue.jsは、Webアプリケーションのインターフェースを構築するためのオープンソースJavaScriptフレームワークです。

ESLint

ESLintは、JavaScriptのための構文チェックツール。全検証ルールを自由に on/offでき、独自のプロジェクトに合わせたカスタムルールを容易に設定することが可能。公開されている様々なプラグインを組み込んで使用することもできます。

JavaScript

JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

Q&A

解決済

1回答

8917閲覧

【JavaScript/ESLint】Use array destructuringエラーが発生している(処理的には問題ない)

frtne

総合スコア29

Vue.js

Vue.jsは、Webアプリケーションのインターフェースを構築するためのオープンソースJavaScriptフレームワークです。

ESLint

ESLintは、JavaScriptのための構文チェックツール。全検証ルールを自由に on/offでき、独自のプロジェクトに合わせたカスタムルールを容易に設定することが可能。公開されている様々なプラグインを組み込んで使用することもできます。

JavaScript

JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

0グッド

1クリップ

投稿2019/11/28 13:39

標題の件、Vue.jsで以下のコードを記述時に
Use array destructuringのエラーが発生しています。

動作的には、問題なく動いているのですが
なぜこの記述がESlintエラーとされているのか、腑に落ちない点があったため質問させていただきました。

// 対象のコード hoge(e) { this.hogeState = e.target.files[0]; },

【質問】
①:なぜ破壊的な処理だとされているのか?
②:このルールの正しいコードの例(ESLint公式参照)
:// With array enabled
var [ foo ] = array;
var foo = array[someIndex];
上記のように書いたら、1行増えるし、冗長かと感じています
どうでしょうか?
③:②はvue.js(this.hogeState)に置き換えると
どう記述するのが良いでしょうか?

(参考)ESLint公式(Use array destructuringについて)
https://eslint.org/docs/rules/prefer-destructuring

よろしくお願いいたします。

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

think49

2019/11/29 03:35 編集

試行錯誤する過程で概ね理解しましたが、質問が正しく伝わる為にも、「再現可能なコード」を開示することをお勧めします。 コードの一部を切り出したものでは、再現できません。
guest

回答1

0

ベストアンサー

ESLint

[prefer-destructuring] を有効にする(初期設定はオフ)

Use array destructuring

①:なぜ破壊的な処理だとされているのか?

"Use array destructuring" は

「配列の**分割代入(Destructuring assignment)**を使用せよ」

の意です(ESLintのドキュメントからそのように読み取れます))。
「破壊」ではありません。
ESLintがエラーを返すのは、分割代入を強制するオプションを有効に設定しているからです。

再現コード

プロパティアクセサ + 代入式。

JavaScript

1'use strict'; 2var files = ['ready','execute','complete']; 3var result = {state: ''}; 4 5result.state = files[0]; // Use array destructuring 6console.log(result); // {state: "ready"}

分割代入

分割代入の使用例1 (Google Chromeで正常動作するも、ESLintでParsing error)

JavaScript

1'use strict'; 2var files = ['ready','execute','complete']; 3var result = {state: ''}; 4 5[result.state] = files; // 5:1 - Parsing error: Assigning to rvalue 6console.log(result); // {state: "ready"}

分割代入の使用例2 (Google Chromeで正常動作するも、ESLintでParsing error)

JavaScript

1'use strict'; 2var files = ['ready','execute','complete']; 3var result = {state: ''}; 4 5{"0":result} = files; // 5:1 - Parsing error: Assigning to rvalue 6console.log(result); // {state: "ready"}

まとめ

下記ポイントを踏まえて、コードを保守している仲間内で最適回を相談して決めてください。

  • 「分割代入を強制すること」の何が利点なのか(「分割代入 > プロパティアクセサ」の優位性はどこにあるのか)
  • ESLintのパーサが解釈できる構文を探すこと

少なくとも、私はそこに意義を見出せません。

Re: frtne さん

投稿2019/11/28 16:04

編集2019/11/28 16:07
think49

総合スコア18189

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

frtne

2019/11/29 13:18

ありがとうございます! ケースバイケースではありますが、現時点で一般的には、 [prefer-destructuring] を有効にすることが最適解になり得るということですかね 大変参考になりました!ベストアンサーとさせていただきます。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.36%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問