zAuction - pot. initialization fronrunning and unnecessary init function
The `zAuction` initialization method is unprotected and while only being executable once, can be called by anyone. This might allow someone to monitor the mempool for new deployments of this contract and fron-run the initialization to initialize it with different parameters.\\nA mitigating factor is that this condition can be detected by the deployer as subsequent calls to `init()` will fail.\\nNote: this doesn't adhere to common interface naming convention/oz naming convention where this method would be called `initialize`.\\nNote: that zNS in contrast relies on ou/initializable pattern with proper naming.\\nNote: that this function might not be necessary at all and should be replaced by a constructor instead, as the contract is not used with a proxy pattern.\\n```\\nfunction init(address accountantaddress) external {\\n    require(!initialized);\\n    initialized = true;\\n    accountant = zAuctionAccountant(accountantaddress);\\n}\\n```\\n
The contract is not used in a proxy pattern, hence, the initialization should be performed in the `constructor` instead.
```\\nfunction init(address accountantaddress) external {\\n    require(!initialized);\\n    initialized = true;\\n    accountant = zAuctionAccountant(accountantaddress);\\n}\\n```\\n