ERC-721 智能合约一次铸造 2 个 NFT

Posted 2023-02-16

【中文标题】ERC-721 智能合约一次铸造 2 个 NFT

【英文标题】：ERC-721 Smart contract is minting 2 NFTs at a time

【发布时间】：2021-11-23 05:43:02

【问题描述】：

我从 Hash Lip 的 github 获得了这个智能合约，据我所知，我应该一次铸造 1 个，但每次都铸造 2 个。代码如下：

设置代码：

// SPDX-License-Identifier: GPL-3.0

// Created by HashLips
// The Nerdy Coder Clones

pragma solidity ^0.8.0;

import "@openzeppelin/contracts/token/ERC721/extensions/ERC721Enumerable.sol";
import "@openzeppelin/contracts/access/Ownable.sol";

contract TestBoxes is ERC721Enumerable, Ownable 
  using Strings for uint256;

  string public baseURI;
  string public baseExtension = ".json";
  uint256 public cost = 0.01 ether;
  uint256 public maxSupply = 3;  //there should only be 3 minted (I have 3 image files to test)
  uint256 public maxMintAmount = 3;
  bool public paused = false;
  mapping(address => bool) public whitelisted;

然后合约铸币的部分如下。如你所见，我将最大值设置为 3，在接下来的部分中，构造函数执行后，它为所有者铸造了 1 个 NFT。

  constructor(
    string memory _name,
    string memory _symbol,
    string memory _initBaseURI
  ) ERC721(_name, _symbol) 
    setBaseURI(_initBaseURI);
    mint(msg.sender, 1); //should mint 1 at deployment but mints 2...
  

  // internal
  function _baseURI() internal view virtual override returns (string memory) 
    return baseURI;
  

  // public
  function mint(address _to, uint256 _mintAmount) public payable 
    uint256 supply = totalSupply();
    require(!paused);
    require(_mintAmount > 0);
    require(_mintAmount <= maxMintAmount);
    require(supply + _mintAmount <= maxSupply);

    if (msg.sender != owner()) 
        if(whitelisted[msg.sender] != true) 
          require(msg.value >= cost * _mintAmount);
        
    

    for (uint256 i = 0; i <= _mintAmount; i++)  //start the index at 0
      _safeMint(_to, supply + i);
    
  

【问题讨论】：

我不想对 hashlips 说任何坏话，但您可能应该为您的智能合约代码找到不同的来源。

【参考方案1】：

mint() 函数内的 for 循环存在逻辑错误。

for (uint256 i = 0; i <= _mintAmount; i++)

示例：_mintAmount 是 1（与从构造函数中传递的相同）。

第一次迭代：

i 是 0，小于或等于 1

=> 执行迭代并执行_safeMint()

第二次迭代：

i 是1，小于或等于 1

=> 它仍然执行迭代并执行_safeMint()

---

通过将条件更改为i < _mintAmoun（i 小于）来修复它。然后它只会执行一次（对于mintAmount 值1）。