Merge pull request #31 from Snowstorm05/matching

Refactor OnlyAuthorized contract for better error handling

Author: Snowstorm08

src/OnlyAuthorized.sol

@@ -1,22 +1,28 @@
 // SPDX-License-Identifier: UNLICENSED
-
 pragma solidity ^0.8.0;
 
 contract OnlyAuthorized {
-    address public owner = msg.sender;
+    address public owner;
+    uint256 public n;
+
+    error OnlyOwner();
+    error ZeroAddress();
+
+    constructor() {
+        owner = msg.sender;
+    }
 
     modifier onlyOwner() {
-        require(owner == msg.sender, "Only owner");
+        if (msg.sender != owner) revert OnlyOwner();
         _;
     }
 
-    function changeOwner(address _newOwner) public onlyOwner {
-        owner = _newOwner;
+    function changeOwner(address newOwner) external onlyOwner {
+        if (newOwner == address(0)) revert ZeroAddress();
+        owner = newOwner;
     }
 
-    uint256 public n;
-
-    function setN(uint256 n_) public {
-        n = n_;
+    function setN(uint256 newN) external {
+        n = newN;
     }
 }