Poor Style: Confusing Naming

コントラクトが、曖昧で誤用されやすい、シャドウされた変数を使用しています。

Solidity では、開発者は状態変数を曖昧に宣言できます。つまり、2 つの異なるコンテキストで 2 つの異なる変数を同じ名前で宣言することは可能ですが、それらを使用すると混乱や誤用につながる可能性があることを意味します。

これは、関数レベルと継承レベルの両方で発生する可能性があります。たとえば、Contract1 が var1 を宣言し、同じく var1 という名前の変数を宣言する Contract2 を継承すると、変数は曖昧となり、後でスマート コントラクトを実行する際に混同しやすくなります。

例 1: 次のコードは、継承を使用して、両方のスマート コントラクトで同じ名前の状態変数を宣言します。どれが実際のトークン セールの hardcap かを見極めるのは困難になる可能性があります。

contract Tokensale {
    uint hardcap = 10000 ether;

    function Tokensale() { }

    function fetchCap() public constant returns(uint) {
        return hardcap;
    }
}

contract Presale is Tokensale {
    uint hardcap = 1000 ether;

    function Presale() Tokensale() { }
}