Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

メインプロセスのEngineInfoとVuexのEngineInfoが同期する/1つになる仕組みを作る #1312

Open
Hiroshiba opened this issue May 9, 2023 · 0 comments

Comments

@Hiroshiba
Copy link
Member

内容

エンジンはメインプロセスで動いていますが、その情報をなんとかしてVuex側に伝える必要があります。
現状はVuex側からメインプロセスの情報をGETしてstateに登録しています。

この場合、メインにEngineInfoの更新が入った後に再登録を忘れるとバグに繋がります。
実例↓

2つの値が同期する仕組み、あるいは情報源を1つにする仕組みを作りたいです。

Pros 良くなる点

予期せぬバグを防げる

Cons 悪くなる点

特になし

実現方法

情報源を1つにする場合、Vuex側のstateを信用する形しか取れないと思います。
(Vuex.stateがデータバインディングに用いられるため、メインプロセスから非同期にデータ取得できないから)
となると、メインプロセス側でエンジン情報が必要になったとき、Vuexを一度介してから再度メインプロセスにあるエンジンの情報を得る形になってちょっと不毛感があります。

2つの値を同期する場合、メインプロセス側にEngineInfos用のsetterを用意し、そのsetter内でVuex側に値をセットするようにすれば解決しそうです。

その他

いわゆるリファクタリングですが、結構事情が面白いのでelectronの勉強のためなどで興味がある方いたらぜひ。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant