あまりにも多くのガス消費

Question

私は最初のEth契約をプログラミングしていますが、私は問題に直面しています。buyメソッドのガス消費量は本当に高くなります（「最大ガス - あなたが所有できる複数のゾーンを持つ地図（2Dマップ）があります（ここではユニットと呼ばれています。これが私が "unitsToState"を維持する理由です） "明らかなマッピング）。 - 隣接する複数のゾーンを一度に購入することができるため、「ブロック」が作成されます。 - 新しいブロックを購入するとき、契約は内部のすべてのユニットが空であるかどうかを確認する必要があります（unitsToState [x] == 0）。ブロックを購入すると、これらの状態は1に設定されます。

私はこの問題が主に「Solidity」の悪いアルゴリズムプログラミングであると推測していますので、あまり詳しく説明しません。

この方法は、小ゾーンを表すX、Y、X、YからYまでのarround 500kガスで実行することができますが、遠く離れているとガス見積もり中に「Maximum gas allowance exceeded」エラー...だから、本当に私は問題があると思います。..

`` ` ...

struct Block { 
    address owner; 
    uint fromX; 
    uint fromY; 
    uint toX; 
    uint toY; 
    string imageUrl; 
    string redirectUrl; 
    string text; 
    bool removed; 
} 

uint size = 100; 
mapping (uint => uint) unitsToState; 
Block[] public blocks; 
uint public areaPrice; 
uint public areaPerUnit; 

... 


function buy(uint fromX, uint fromY, uint toX, uint toY, string imageUrl, string redirectUrl, string text) payable public { 
    require(fromX >= 0); 
    require(fromY >= 0); 
    require(fromX <= toX); 
    require(fromY <= toY); 
    require(toX < size); 
    require(toY < size); 

    // Here do check of collisions. 
    for (uint i = fromX; i <= toX; i++) { 
     for (uint j = fromY; j <= toY; j++) { 
      require(getUnitsToState(i*size*size + j) == 0); 
     }  
    } 

    uint width = toX - fromX + 1; 
    uint height = toY - fromY + 1; 
    uint areaCount = width * height * areaPerUnit; 
    uint price = areaCount * areaPrice; 
    require(msg.value >= price); 

    Block memory b = Block(
     msg.sender, 
     fromX, 
     fromY, 
     toX, 
     toY, 
     imageUrl, 
     redirectUrl, 
     text, 
     false 
    ); 
    blocks.push(b); 

    // Registrer units states. 
    for (i = fromX; i <= toX; i++) { 
     for (j = fromY; j <= toY; j++) { 
      unitsToState[i*size*size + j] = 1; 
     }  
    } 
} 

... 

` ``

Answer 1

特にネストされたループと

大きなループ、ソリディティに危険です。各反復でどの論理を実行しているのか注意する必要があります。

あなたのケースでは、それはガス使用量の巨大なスパイクを引き起こしているunitsToStateフラグを設定している2番目のループです。ゼロ以外のデータを格納する呼び出しには、2万のガス（ゼロの場合は5,000）がかかります。ループが繰り返されるたびにそのコストが発生します。

Block構造体の内部で衝突チェックを行う必要があるようです。あなたはそれを使うことができますか？

for (uint8 i = 0; i < blocks.length; i++) { 
    Block storage bTest = blocks[i]; 

    bool xIntersect = int(fromX) - int(bTest.toX) <= 0 && int(toX) - int(bTest.fromX) >= 0; 
    bool yIntersect = int(fromY) - int(bTest.toY) <= 0 && int(toY) - int(bTest.fromY) >= 0; 

    require (!(xIntersect && yIntersect)); 
} 

ガス代はAppendix G of the yellow paperを参照してください。