为何这段Solidity代码中Ether无法在账户间转账？

你的Solidity合约存在几个核心问题，导致ETH无法正常转账，以下是具体分析和修复方案：

1. transferEther 函数的 payable 修饰符逻辑错误 #

该函数的设计目的是从合约向目标地址转ETH，但你给它添加了payable修饰符——这意味着调用该函数时，调用者需要主动向合约转入ETH，而非使用合约已有的余额完成转账，完全搞反了逻辑，必须去掉这个payable。

2. 未检查合约余额，转账易触发异常 #

调用transferEther时，如果合约当前ETH余额小于指定的amt，transfer方法会直接抛出异常并回滚交易（transfer内置了余额检查逻辑）。需要在函数内显式添加余额校验，避免无意义的失败。

3. 缺失权限控制（安全隐患） #

当前任何地址都能调用transferEther转走合约内的ETH，属于高危漏洞，建议添加权限控制，限制仅合约部署者可调用转账功能。

修复后的完整合约代码 #

// SPDX-License-Identifier: MIT
pragma solidity >0.5.0 < 0.9.0;

contract TestPay {
    address public owner;

    // 构造函数，初始化合约部署者为权限拥有者
    constructor() {
        owner = msg.sender;
    }

    // 用户向合约存入ETH，payable修饰符正确
    function payEther() payable public {}

    // 修正转账逻辑：移除payable、添加权限与余额校验
    function transferEther(address payable _to, uint amt) public {
        require(msg.sender == owner, "仅合约部署者可调用");
        require(address(this).balance >= amt, "合约余额不足");
        _to.transfer(amt);
    }

    // 辅助函数：查看合约当前ETH余额
    function getContractBalance() public view returns(uint) {
        return address(this).balance;
    }
}

额外注意事项 #

• 调用payEther时，必须在调用时附带ETH（比如在Remix编辑器的Value字段填写金额），否则合约余额不会增加。
• 如果目标地址是合约且包含复杂的fallback/receive函数，transfer的2300 gas限制可能导致转账失败，此时可改用call方法并手动校验结果：

  (bool success, ) = _to.call{value: amt}("");
  require(success, "转账失败");
  

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