Let`s move - sui move开发实战-dao(6)反馈

本文主要是介绍Let`s move - sui move开发实战-dao(6)反馈，希望对大家解决编程问题提供一定的参考价值，需要的开发者们随着小编来一起学习吧！

引言

我将前文实现的简单的dao参与了https://dacade.org/communities/sui的挑战，得到了反馈。这是我得到的第二个反馈，我认为它为解决我的dao经济模型中一个很重要的问题提供了一个解决思路

反馈详情

这个反馈主要分为两个方面

增加提案持续时间的字段

在我原有的设计之中，dao的提案只能由核心成员手动关闭，然后根据赞成和反对票判断提案是否被接受。

反馈中做出的更改

增加一个提案持续时间的字段

    const PROPOSAL_DURATION: u64 = 7 * 24 * 60 * 60;

设置了一个提案持续时间的结构体

struct ProposalDuration has key, store {id: UID,start_time: u64,end_time: u64,}

核心成员可以设置提案持续PROPOSAL_DURATION后自动关闭

     // Function to set the proposal durationpublic fun set_proposal_duration(core_cap: &CoreCap, proposal: &mut Proposal, ctx: &mut TxContext) {check_corecap_role(core_cap, ctx);assert!(!proposal.is_closed, EProposalClosed);let start_time = tx_context::epoch(ctx);let end_time = start_time + PROPOSAL_DURATION;let proposal_duration = ProposalDuration {id: object::new(ctx),start_time,end_time,};transfer::share_object(proposal_duration);}// Function to check if the proposal is still open for votingpublic fun is_proposal_open(proposal: &Proposal, proposal_duration: &ProposalDuration): bool {let current_time = tx_context::epoch(ctx);current_time >= proposal_duration.start_time && current_time < proposal_duration.end_time}

但是我认为这个修改有一点问题：

ProposalDuration与proposal并没有被很好的关联起来，要关联它们，我们需要对ProposalDuration增加一个字段，也就是id指针，储存对于的提案id
时间的设置不够灵活，在set_proposal_duration函数中可以提供一个参数，而不是使用固定的const，这样可以增加系统的灵活性

经济模型

这是我感觉一个比较重要的反馈点

在我原有的经济模型中，所有dao代币在合约初始化的过程中会被mint，储存在金库中，之后这个代币供应量将不会再更改。

有以下几个方法可以从金库获得dao代币：

社区任务奖励
提案奖励
取回投票凭证

有以下几个方法需要向金库支付dao代币：

提案费用
投票

由于在提案结束后，投票者都可以取回自己的dao代币，因此，投票者不会支付dao代币成本
那么金库的dao代币收入就仅仅只有提案费用

而且提案被接受的话金库将会支出更多的dao代币
金库的收支不平衡，一旦某一天金库的dao代币耗尽，就可能产生坏账，系统可能无法正常运行

该反馈提出了一个方法来缓解问题：
根据成员数量增发dao代币

    public fun update_total_supply(dao: &mut Dao<DAO>, new_members: u64, treasury: &mut Treasury<DAO>, ctx: &mut TxContext) {let new_supply = coin::mint_balance<DAO>(&mut treasury.supply, new_members * TOTAL_SUPPLY / dao.total_members);let new_total_supply = coin::supply_value(&new_supply) + coin::supply_value(&dao.total_supply);dao.total_supply = coin::create_supply(new_total_supply);coin::destroy_supply(dao.total_supply);dao.total_supply = new_total_supply;}