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

Discussion: persistent StateHolder #3

Open
Tomucha opened this issue Jan 1, 2021 · 12 comments
Open

Discussion: persistent StateHolder #3

Tomucha opened this issue Jan 1, 2021 · 12 comments

Comments

@Tomucha
Copy link
Contributor

Tomucha commented Jan 1, 2021

Co kdyby mel kazdy StateHolder prideleny svuj hive box:

https://github.com/hivedb/

A prubezne by se pri zmene ukladal? Volitelne by to slo vypnout ... Prijde mi to jako celkem mocny a zajimavy.

Co myslis, @petrblahos ?

@petrblahos
Copy link

Já jsem uvažoval i o tom, že při každé změně by se automaticky uložilo i to json.
Ten hivedb vypadá dobře, ukládat menší kousky - ne všechna data naráz - by taky bylo lepší.

Jak by se to dalo udělat? Přes ChangeNotifier.addListener(saveMe) ?

@Tomucha
Copy link
Contributor Author

Tomucha commented Jan 4, 2021

Spíš jsem uvažoval o tom, že by to byla primo vlastnost StateHolderu. Kazdej state lze serializovat do JSONu a StateHolder vi presne kdy se meni, tak by se ulozil do hive box-u, kterej by dostal v konstruktoru. A taky by to v konstruktoru slo volitelne vypnout / neposkytnout tomu uloziste.

(Cca, mozna to nebude primo box, ale jen nejakej save callback ...)

@petrblahos
Copy link

Vypadá to, že nebude jednoduché zkombinovat built_value a Hive. Zkusil jsem to a neuspěl, Každopádně, zkusil jsem upravit kostlivce tak, aby serializoval to hive. Můžeš mrknout sem:

Mě je ukládání do hive rozhodně sympatičtější než do json, zároveň se mi líbí ten immutable state co měl kostlivec.
Takže zatím nevím, co je lepší. Přemýšlím, jestli mi ten built_value přináší dost výhod na to, abych ho preferoval
před tím hivem. Ještě si s tím asi trochu pohraju...

@Tomucha
Copy link
Contributor Author

Tomucha commented Feb 14, 2021 via email

@petrblahos
Copy link

Asi se nechápeme. Teď tam plácám celej objekt naráz.

@Tomucha
Copy link
Contributor Author

Tomucha commented Feb 15, 2021 via email

@petrblahos
Copy link

Aha, tak takhle. Využít built_value serializer a nedělat adapter pro ten ukládanej objekt,
ale místo toho uložit tu serializovanou hodnotu. Jo?

@Tomucha
Copy link
Contributor Author

Tomucha commented Feb 15, 2021 via email

@petrblahos
Copy link

Tak tohle je opravdu mnohem jednodušší:

https://github.com/petrblahos/flutter_kostlivec/tree/hive_db_serialize_built

@Tomucha
Copy link
Contributor Author

Tomucha commented Feb 16, 2021 via email

@petrblahos
Copy link

petrblahos commented Feb 16, 2021 via email

@Tomucha
Copy link
Contributor Author

Tomucha commented Feb 16, 2021 via email

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

No branches or pull requests

2 participants