pragma solidity ^0.4.23;

contract HodlFraud {
    uint public ownerAmount;  //合約擁有者的金錢
    uint public  numberOfPayouts; //次數
    address public owner;  //合約的擁有者
 


    struct HoldRecord {
        uint amount;  //儲存的金錢
        uint unlockTime;  //解鎖的時間
    }

    mapping (address => HoldRecord) public balance;  //地址  => 儲存的金錢，時間

    //建構函式初始化
    function HodlFraud
 
 () public payable {
        owner = msg.sender;
        ownerAmount = msg.value;
    }

    //某一個使用者儲存金錢，
    //@param  holdTime 代表的是解鎖時間
    function payIn(uint holdTime) public payable {
        require(msg.value > 0);


        HoldRecord  newRecord;


        newRecord.amount += msg.value;
        newRecord.unlockTime = now + holdTime;
        balance[msg.sender] = newRecord;
    }

    //轉賬，使用這筆錢
    function withdraw () public {
        require(balance[msg.sender].unlockTime < now && balance[msg.sender].amount > 0);

        msg.sender.transfer(balance[msg.sender].amount);

        balance[msg.sender].amount = 0;
        numberOfPayouts++;
    }

    //合約的擁有著，轉移自己的錢
    function ownerWithdrawal () public {
        require(msg.sender == owner && ownerAmount > 0);
        msg.sender.transfer(ownerAmount);
        ownerAmount = 0;
    }
}

// 這個例子是要舉出在 smart contract 中 storage 預設行為的危險性。其實只要維持一個原則就可以避免這個問題。
// 養成明確定義使用 storage 還是 memory 的好習慣
// 一般來說，指定 storage 時就直接給初始值；而在 function 裡面需要用到的快取器都用 memory，除非想要直接修改鏈上的值。現在 compiler 都會很聰明的提醒開發者要定義 storage 還是 memory，而當 storage pointer 沒有初始值時也會提醒開發者。