ERC721 NFT ETH交易函数正确性验证及市场展示逻辑问询

你的实现逻辑完全错误，核心问题出在两个认知偏差上：

• address(0)不是ETH的代币地址，ETH本身不属于ERC20代币，你强行将零地址转为IERC20接口调用transferFrom，本质是调用零地址的合约方法，而零地址不存在部署的合约，这个调用必然失败，会直接导致整个buy函数永久回滚，无法完成任何交易。
• 你参考的原回答是针对ERC20支付场景的gas异常问题，ETH支付场景完全不需要额外做这类校验。ETH是以太坊原生代币，用户调用payable函数时，require(msg.value == price, 'Incorrect value')已经完成了支付确认，用户发交易时对应ETH已经转入合约；如果交易出现gas耗尽的情况，整个交易的所有操作都会原子性回滚，不会出现ETH没转成功、NFT却被转走的边缘问题。

你可以直接删除错误的ERC20调用逻辑，修正后的代码如下：

function buy(uint256 _tokenId) external payable {
    uint256 price = tokenIdToPrice[_tokenId];
    require(price > 0, 'This token is not for sale');
    require(msg.value == price, 'Incorrect value');
    address seller = ownerOf(_tokenId);
    // 若要兼容卖家为合约地址的场景，可把transfer换成call方法避免gas不足
    (bool success, ) = payable(seller).call{value: msg.value}("");
    require(success, "ETH transfer failed");
    _transfer(seller, msg.sender, _tokenId);
    tokenIdToPrice[_tokenId] = 0;
    emit NftBought(seller, msg.sender, msg.value);
}

• NFT交易市场展示某系列全量在售NFT，不强制依赖合约提供的全量查询接口。主流NFT市场通用方案是通过链下索引的方式，抓取链上的上架事件、售卖事件、Transfer事件等数据，自主整理出全量在售NFT的ID、URI、价格等信息，这种方案性能更高，也不需要占用昂贵的链上存储资源。如果不想做链下索引，才需要合约额外提供对应的查询接口。
• OpenZeppelin的ERC721标准实现仅自带单个NFT的tokenURI(uint256 tokenId)查询接口、以及查询用户持有的NFT列表的相关接口，没有自带全量NFT/全量在售NFT的返回接口，如果你需要链上直接查询全量在售列表，需要自行开发：比如维护一个在售NFT的动态数组，上架时写入、售出/下架时移除，再对外提供查询这个数组的接口即可。

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