Fujita Shu comments

Results 33 comments of


                                            Fujita Shu

非機能要件（可読性・拡張性・設計）問題点・改善点の整理

## component の中身が全体的に荒れている感ありその他，`page/index.vue` 上の component, chart が全体的に荒れている印象があるので，dry で統一感のある構成にしたい．

非機能要件（可読性・拡張性・設計）問題点・改善点の整理

## JSONファイルとVueの間に中間層を設ける `data/data.json` などからデータを取得する際に，中間層を設けることで型推論を可能とし、コードの保守性を上げる． cf. JSONファイルとVueの間に中間層を設ける (#1100) 実装すると宣言した方がいらっしゃるが，音沙汰なしの状況．

非機能要件（可読性・拡張性・設計）問題点・改善点の整理

## `tool/convert.php` について Nuxt.js のプロジェクトなので，JavaScript（または TypeScript）に書き換えたい．型をしっかり付けるなら「JSONファイルとVueの間に中間層を設ける」とも関連してくるかも？ **追記 (2020-08-15)**: `tool/convert.php` は削除されたため対応不要

## `data.json` について ### 分割すべきでは？ * 現在 7727 行と肥大化している．将来的にデータが溜まっていくとさらに肥大化が進み，管理しにくくなると予想される． * 他県版に東京版のコードを merge しようとすると conflict が多発する．`data.json` を分割しなくても conflict することには変わりないが，conflict 解消時のミスの危険性が増すと思われる． * 各グラフのデータの更新タイミングが異なるので，`data.json` を分割した方がファイルごとの責務が明確になり，Git で管理するときに差分が確認しやすい． * ただ，`tool/convert.php` の書き換えも含め，工数が大きく，データの信頼性を担保したまま分割するのは大変． ### property の日本語を英語にすべきでは？ *...

非機能要件（可読性・拡張性・設計）問題点・改善点の整理

## template に data.json をそのまま放り込んでいる箇所がある template に渡す値は `` 内で絞り込みたい． **例**: `components/cards/ConfirmedCasesAttributesCard.vue` ```vue import Data from '@/data/data.json' // （略） export default { // （略） data() { // （略） const data =...

非機能要件（可読性・拡張性・設計）問題点・改善点の整理

@khsacc > 非機能要件ということで、各vueファイルのの言語がjs, tsで統一されていないのが少し気になっているのですが、どうお考えでしょうか……？ > 個人的にはすべてtsに書き換えたい気がします。 `` を機械的に `` に置き換え，型情報が足りず警告が出るような箇所には型を追加する，というのであれば手早く対応できると思います．

非機能要件（可読性・拡張性・設計）問題点・改善点の整理

@mikkame > 他のプロジェクト（本プロジェクトは全く関係ない）がdata.jsonを引っ張って使っている可能性それについては Slack でアナウンスして，数日後にサポートを切るということにすれば十分だと思います．それ以上は面倒見切れないですよね… `tool/convert.php` を見たところ，Excel ファイルをパースして JSON を作っていますが，それは別途管理している現在も変わらないでしょうか？ `data.json` の中にはどこからも使われていない無駄なデータもある気がしますので，そういうものを消していく意味でも，あるいは他のデータを追加するときに誰でも手早く変換スクリプトが書けるようにするためにも，セキュリティが絡むところと絡まないところは適切に分離して，安全な部分については GitHub で管理して頂けるとよろしいかと思います．

非機能要件（可読性・拡張性・設計）問題点・改善点の整理

@mikkame > 所定の場所からエクセルをダウンロードする機能が追加されております。（アクセス権部分） > その元のエクセルを公開することが難しい一般的な設計論の話になりますが，ファイルを * (1) 任意の URL にあるファイルをダウンロードする機能 * (2) (1) を利用して（公開できない Excel ファイルの URL を指定して）Excel ファイルをダウンロードする機能 * (3) (2) でダウンロードした Excel ファイルをパースして JSON に展開する機能に分離して，(1) だけ（Excel...

非機能要件（可読性・拡張性・設計）問題点・改善点の整理

@halsk こちらの issue，discussion タグを付けてください．よろしくお願いします．

非機能要件（可読性・拡張性・設計）問題点・改善点の整理

@mikkame > (3) (2) でダウンロードした Excel ファイルをパースして JSON に展開する機能の件，Excel ファイルの表の構造は現状 `tool/convert.php` から推測できるので既にバレバレという気もします… ちょっと今から一旦流れを整理しますね．