General Thread for practicing question formatting

[angelataylorllc]

Hi, here's some code that's wrong:

function withdraw() onlyOwner {
        /* inside parentheses = starting index, ending index, step amount */
       for(uint256 funderIndex = 0; funderIndex < funders.length; funderIndex++){
           address funder = funders[funderIndex];
           addressToAmountFunded[funder] = 0;
       } 

And here is the error:

SyntaxError: No visibility specified. Did you intend to add "public"?
  --> contracts/FundMe.sol:30:5:
   |
30 |     function withdraw() onlyOwner {
   |     ^ (Relevant source part starts here and spans across multiple lines).

Just practicing :)

[notmhyr]

Hi,

Functions and state variables have to declare whether they are accessible by other contracts.

Functions can be declared as

• public - any contract and account can call
• private - only inside the contract that defines the function
• internal- only inside contract that inherits an internal function
• external - only other contracts and accounts can call

in your case you should declare your function as public so anyone can interact with it so it should look like:

function withdraw() public onlyOwner {
        /* inside parentheses = starting index, ending index, step amount */
       for(uint256 funderIndex = 0; funderIndex < funders.length; funderIndex++){
           address funder = funders[funderIndex];
           addressToAmountFunded[funder] = 0;
       } 

check here to learn more https://docs.soliditylang.org/en/v0.8.17/contracts.html#visibility-and-getters