Open knakazawa99 opened 3 years ago
コメントありがとうございます。
env.phpはDBの設定値なのでアップしませんでした。
送っていただいた記事を参考にもう少し良くできるよう直してみたいと思います。 laravelの方も学習しておきます。
2020年10月22日(木) 14:31 KensukeNakazawa notifications@github.com:
CRMセンターの中澤です。
ソースコードを見たので、何点かコメントをさせてもらいます。
InquiryLogicクラスは非常に良く出来ていると思います。 クラス内部のメソッドもstaticが使われていたり、PHPDocがきちんと書かれていてよかったです。 もう少し良くするのであれば、メソッドの引数に型を宣言してあげると良いかもしれません。 その際は下記の記事なんかが参考になると思います。 https://qiita.com/hikaru_/items/2c12068eafea75e51cce
env.phpを使っていたのですが、それがリポジトリ内に無いのが少し気になりました。 何か事情があってアップ出来ないとかであれば、書いていただけると良いかと。
今後、実際に開発する上では、PHPを直接書く事はあまり無いと思われるので、フレームワークの学習もしてみると良いかもしれません。 PHPであればlaravelが情報も多くやりやすいかもしれません。 laravelのドキュメントを載せておきます。 https://readouble.com/laravel/6.x/ja/lifecycle.html
全体的に最初に自力で書いた物としてはかなりよく出来ていると思います! 頑張ってください!
— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/yuu830/inquiry_form/issues/1, or unsubscribe https://github.com/notifications/unsubscribe-auth/AQ2MAOSAFP776EUI6BUDP2TSL67URANCNFSM4S2WDM6Q .
フレームワークの勉強の方は追々で大丈夫ですよ!
月曜日楽しみにしています。
CRMセンターの中澤です。
ソースコードを見たので、何点かコメントをさせてもらいます。
InquiryLogicクラスは非常に良く出来ていると思います。 クラス内部のメソッドもstaticが使われていたり、PHPDocがきちんと書かれていてよかったです。 もう少し良くするのであれば、メソッドの引数に型を宣言してあげると良いかもしれません。 その際は下記の記事なんかが参考になると思います。 https://qiita.com/hikaru_/items/2c12068eafea75e51cce
env.phpを使っていたのですが、それがリポジトリ内に無いのが少し気になりました。 何か事情があってアップ出来ないとかであれば、書いていただけると良いかと。
今後、実際に開発する上では、PHPを直接書く事はあまり無いと思われるので、フレームワークの学習もしてみると良いかもしれません。 PHPであればlaravelが情報も多くやりやすいかもしれません。 laravelのドキュメントを載せておきます。 https://readouble.com/laravel/6.x/ja/lifecycle.html
全体的に最初に自力で書いた物としてはかなりよく出来ていると思います! 頑張ってください!