みてみた

[Tomosuke0930]

1. 型が違うという箇所 👉 from block.timstamp to 日にちだからここのtypeは　”Date”という認識でよろしいでしょうか？

　2.provider の代わりにここで signer 入れとけんかね？？ 　こちらは下記みたいにしたらという提案でしょうか？

　　const lotContract = new ethers.Contract(contractAddr, contractAbi, provider) 　　const lotContract = new ethers.Contract(contractAddr, contractAbi, signer)

　ちなみにsignerについては以下の認識です！

　signerはethers.jsにおいてtxを送る際に必要

　逆にblockchain上の情報を取得する際はいらないやつ 　参考文献はこちら 　https://docs.ethers.io/v5/getting-started/#getting-started--glossary

　その他のレビューは提案の通りだなと思いました！！

[noy4]

”Date”という認識でよろしいでしょうか？

ホバーしたらわかるけどmomentの独自の型みたいなのを入れようとしてるくね

signer あっても取得できるから初期化の時にsigner入れといて全部それ使ったら書く量減るんじゃねと思った

[Tomosuke0930]

const lotContract = new ethers.Contract(contractAddr, contractAbi, provider)　 こちらに関してはコントラクトのインスタンスを作成しているという認識でした！

そしてsignerは下記のコードで記述されます。お前、providerまだねえじゃねえか！と怒られているのかな？と思いsignerは各関数ごとに宣言しておりました！ const signer = lotContract.connect(provider!.getSigner())

👇 怒られた時のコード

[noy4]

ethersのドキュメント見たらコントラクトインスタンス作る時にprovider渡したらRead-Onlyなってsigner渡したらRead-Writeなるって書いとると思う providerで初期化したインスタンスに後からsignerをコネクトするのと多分一緒 provider?.getSigner()でどうだろう

[noy4]

https://docs.ethers.io/v5/api/contract/contract/#Contract-connect connectの返り値Contractだから signerって変数に格納するの変かも

[Tomosuke0930]

こちら見ておりませんでした！後で確認させていただきます！