Using Solidity ^0.5.0, I'm actually trying to implement an old code working in ^0.4.17, but it throws an error on Remix. I checked syntax and everything but just can't catch what's wrong. Any help highly appreciated.
pragma solidity ^0.5.0;
contract Lottery{
address public manager;
address[] public players;
constructor() public {
manager = msg.sender;
}
function participate() public payable {
require(msg.value > .01 ether);
players.push(msg.sender);
}
function pseudoRandom() private view returns(uint){
return uint(keccak256(abi.encodePacked(block.difficulty, now, players)));
}
function pickWinner() public {
uint index = pseudoRandom() % players.length;
players[index].transfer(address(this).balance);
}
}
Here is the error message:
browser/Lottery.sol:22:8: TypeError: Member "transfer" not found or not visible after argument-dependent lookup in address.
players[index].transfer(address(msg.sender).balance);
^---------------------^
Please see Solidity v0.5.0 Breaking Changes.
As pointed out here:
The address type was split into address and address payable, where only address payable provides the transfer function.
See my answer here for more details.
Change this line:
address[] public players;
To this:
address payable[] public players;
EDIT
Thanks smarx! 😄
You are missing the payable modifier on the initial declaration. Change
address[] public players;
to
address payable[] public players;
If you love us? You can donate to us via Paypal or buy me a coffee so we can maintain and grow! Thank you!
Donate Us With