ERC-20代币水龙头合约无法提取代币问题咨询

问题排查与修复方案 #

核心问题定位 #

你删除所有require校验后仍然报错，说明问题出在withdraw方法中唯一剩下的_token.transfer调用逻辑上，常见原因如下：

1. 水龙头合约构造函数传入的代币地址错误

部署Faucet合约时需要传入ERC20代币的合约地址，若该地址填错、或者和你后续交互的MyToken地址不匹配，调用transfer时会直接触发revert。

• 验证方案：将合约中IERC20 private _token;改为IERC20 public _token;，重新部署后在truffle控制台执行FaucetDeployed._token()，对比输出地址和token.address是否一致。

2. ERC20代币实现不符合IERC20标准

OpenZeppelin的IERC20接口定义transfer方法需要返回bool类型值，如果你的自定义ERC20代币的transfer函数没有声明returns(bool)、或者没有正确返回布尔值，接口调用时会因为返回值格式不匹配触发revert。

• 修复方案：检查你的ERC20代币实现，确保transfer方法符合标准：

function transfer(address to, uint256 amount) external override returns (bool) {
    // 你的转账逻辑
    return true;
}

3. 未校验transfer的返回结果

你当前代码直接调用_token.transfer没有处理返回值，即使转账失败也不会抛出明确错误，优化代码如下即可拿到具体失败原因：

// 替换原来的_token.transfer行
require(
    _token.transfer(msg.sender, withdrawalAmount),
    "Token transfer failed, please check token contract status"
);

其他隐藏bug修复 #

你当前的setWithdrawalAmount方法存在重复计算小数的问题：

function setWithdrawalAmount(uint256 amount) public onlyOwner {
    require(amount <= 10000 * (10 ** 18));
    // 这里重复乘了10**18，如果你传入的amount已经是带小数位的数值，会导致金额异常
    withdrawalAmount = amount * (10 ** 18); 
}

如果该方法期望传入的是不带小数位的代币数量（比如传50代表50个代币），则代码没问题；如果期望传入的是已经计算过18位小数的数值，需要删掉后面的乘法逻辑。

临时验证方案 #

你可以先在truffle控制台执行以下命令确认调用参数没问题：

// 确认当前默认账户是正常地址
console.log(accounts[0])
// 指定发起方调用withdraw
FaucetDeployed.withdraw({from: accounts[0]})

内容的提问来源于stack exchange，提问作者jspru