A뭐
Общее для каждого пользователя
Size: a a a
2021 August 08
MD
это геморно
A뭐
Победил Вася - добавь в mapping Васе циферку
A뭐
Почему?
SM
я понял
маппинг address -> lotteryId -> amount
И суммировать по всем лотереям
маппинг address -> lotteryId -> amount
И суммировать по всем лотереям
MD
потому что можно ошибиться если кешировать общую сумму выигрыша вместо того чтобы просто по раундам результаты фиксировать
A뭐
Ну можно же тесты написать
SM
только клеймить тогда он должен ВСЕ, что имеется на текущий момент
A뭐
MD
тесты то можно написать, но чем проще контракт тем лучше)
SM
чтобы не вычитать кусками из разных раундов
MD
а чем меньше кеширование важных данных тем контракт проще)
MD
это дублирование данных, которых лучше избегать
SM
вот потому я и думал - закрылась лотерея. ушла рассылка, обнулился контракт
A뭐
Пользователю придётся тогда идти по раундам старым, псли он проступил и искать. Либо делать индексатор какой то с API
MD
если бы тесты показывали бы все проблемы контрактов то у нас уязвимости бы не находили)
SM
никакой привязки к действиям юзеров
A뭐
Ну такую простую штуку тестами покрыть можешь без проблем же
MD
лучше индексатор чем усложнять контракт)