Modifiers on solidity

Question

So, as part of the bitdegree course on solidity I'm looking to create a modifier named onlyOwner and assign it to changePrice function. I must make sure the modifier allows function to be executed only if the sender's address matches the address of the owner. The sender's address can be obtained using msg.sender.

I have tried entering this to create the modifier but its not working for me and I'm not sure why. Any help/recommended code would be greatly appreciated!

pragma solidity ^0.4.17;

contract ModifiersTutorial {

address public owner;
uint256 public price = 0;
address public sender=msg.sender;

//
modifier onlyOwner(sender){
if (owner==sender);
}
//

// Use your modifier on the function below
function changePrice(uint256 _price) public onlyOwner {
    price = _price;
}

function ModifiersTutorial () {
    owner = msg.sender; // msg.sender in constructor equals to the address that created the contract
}

}

score 1 · Accepted Answer · answered Jan 23 '18 at 19:32

1

Your modifier code is incorrect. You need an underscore to proceed.

modifier onlyOwner(sender){
  if (owner==sender) _; // Note the underscore
}

Also, for security reasons, you really should just use msg.sender instead of passing it in.

modifier onlyOwner() {
  if (owner == msg.sender) _;
}

answered Jan 23 '18 at 19:32

Adam Kipnis

10,175
10
35
48

Thankyou for that, I have a bad habit of making syntax mistakes. I've spent hours 'fixing' small syntax errors. I appreciate the useful response brother! – GG123GG Jan 23 '18 at 19:35

score 1 · Answer 2 · answered Nov 14 '19 at 08:02

Not sure if it conflicts with the spec you've been given, but an alternative practice would be to require(owner == msg.sender) rather than use an if statement -- the former tells the user what happened, while the latter simply fails silently. This is what it could look like:

modifier onlyOwner(){
  require(owner == msg.sender, "error-only-owner");
  _;
}

score -1 · Answer 3 · edited Jul 16 '18 at 17:10

-1

pragma solidity ^0.4.17;

contract ModifiersTutorial {

    address public owner;
    uint256 public price = 0;

    modifier onlyOwner(){
    if( owner == msg.sender ) _;
    }

    function changePrice(uint256 _price) public onlyOwner{
        price = _price;
    }

    function ModifiersTutorial () {
        owner = msg.sender; 
    }
}

edited Jul 16 '18 at 17:10

pushkin

9,575
15
51
95

answered Jul 16 '18 at 16:40

le van hai

1

A blob of code without any introduction or explanation of what it answers the question at hand is not great. Can you provide some context? – Jul 16 '18 at 17:13

Modifiers on solidity

3 Answers3