For Reviewer

共通指摘事項

・変数の型を意識させる。処理の途中で変数の型が変わっているものは必ず指摘する。早い段階から @var を書くようにするとよいかも。関数の引数、戻り値の方は必ず指定。
・冗長な条件分岐や条件式は指摘して、極力シンプルな構造となるようにする。
・null チェックなどの例外処理漏れは必ず指摘する。
・Lesson 7以降は、XSSやCSRF、SQLインジェクションは必ず指摘する。

Lesson 1

・MVCの概念について説明し、最低でもビューとロジックの2ファイルに分離
・型の重要性を説明

Lesson 2〜3

・Form タグの役割について説明
・クラスとオブジェクトについて基本的な説明をし、エンティティクラスを作成する

Lesson 4〜5

・GET と POST について説明
・処理の分割(関数レベルでよい)と早期リターン

Lesson 6

・フロントバリデーションとバックエンドバリデーションの両方が必要である理由を説明
・リポジトリについて(集合を表すクラスとして)説明、作成する。

Lesson 7

・Webアプリにおけるセキュリティの重要性を説明。

Lesson 8

・共通化は次項の課題であるため、指摘は不要。

Lesson 9

・DRY原則について説明し、まずは関数レベルの共通化を進めてもらう。 但し、関数の引数、戻り値には必ず型をつける。

Lesson 9′

・モノではなく処理をクラス化することについて説明。
・コントローラをクラス化し、共通処理を基底クラスにする。継承、アクセス修飾子について説明。
・関数化している共通処理について、別途クラス化する(その場合、極力単一責務に)。静的メソッドについて説明。
・オートローダーを作成する。