Skip to content
This repository has been archived by the owner on Jun 1, 2023. It is now read-only.

組み込みオブジェクトのオーバーライド(DataView) #4176

Open
goki90210 opened this issue May 15, 2020 · 1 comment
Open

組み込みオブジェクトのオーバーライド(DataView) #4176

goki90210 opened this issue May 15, 2020 · 1 comment
Labels
bug バグ。本来あるべき動作をしていないもの

Comments

@goki90210
Copy link
Contributor

起こっている問題 / The Problem

  • components/cards/ConfirmedCasesDetailsCard.vue, components/cards/TestedCasesDetailsCard.vue で
import DataView from '@/components/DataView.vue'

としているが、DataViewはJavaScriptの組み込みオブジェクトでもあるためオーバーライドの状態になっています。

スクリーンショット / Screenshot

期待する見せ方・挙動 / Expected Behavior

  • import DataViewではなく別の名前にすべきです。

起こっている問題の再現手段 / Steps to Reproduce

SonarQubeで静的コード解析を行った結果です。

動作環境・ブラウザ / Environment

動作環境・ブラウザに依存しません。

@goki90210 goki90210 added the bug バグ。本来あるべき動作をしていないもの label May 15, 2020
@goki90210 goki90210 changed the title 組み込みオブジェクトのオーバーライド 組み込みオブジェクトのオーバーライド(DataView) May 15, 2020
@mcdmaster
Copy link
Contributor

たしかに、

mcdmaster@localhost:~/covid19> find . -name "*.js" -print | xargs grep DataView | wc -l -
grep: ./node_modules/@types/chart.js: Is a directory
grep: ./node_modules/asn1.js: Is a directory
grep: ./node_modules/big.js: Is a directory
grep: ./node_modules/bn.js: Is a directory
grep: ./node_modules/browserify-sign/node_modules/bn.js: Is a directory
grep: ./node_modules/chart.js: Is a directory
grep: ./node_modules/des.js: Is a directory
grep: ./node_modules/hash.js: Is a directory
grep: ./node_modules/html-webpack-plugin/node_modules/big.js: Is a directory
grep: ./node_modules/ipaddr.js: Is a directory
grep: ./node_modules/md5.js: Is a directory
grep: ./node_modules/sha.js: Is a directory
351 -

というシェルコマンドの実行結果が得られますね。
hogehoge is a directory のような無効 grep 結果があるとしても、DataView がサードパーティモジュールなどの JS でかなり多く使われていることが想定されます。
だとしたら、ご指摘のように混乱を避ける目的で DataView.vue のファイル名を変更した方が良さそうです。

これによって想定されるリスクは、既存 JS に使われている DataView と混同されるリスクに比べればいささか小さいはずです。
あとは、都民をはじめとするオーディエンスの皆さんに不利益が生じないならという前提で、私も DataView.vue ファイルの名前変更に賛同します。

ちなみに、以下のようなコマンドも打ってみました。

mcdmaster@localhost:~/covid19> find . -name "*.vue" -print | xargs grep DataView | wc -l -
50 -

Vuetify ファイルだけがサーチ対象だと 50 件、まあまあの件数ですね…。
力作業で直すしかないでしょうか?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug バグ。本来あるべき動作をしていないもの
Projects
None yet
Development

No branches or pull requests

2 participants