質問: PHPの学習としまして、簡単な会員制の交流サイトを作成致しました。 作成しましたページは、TOPページである、「index.php」会員が自由に書き込みができる掲示板のページである、「bbs.php」会員がログインする為の画面である、「login.php」、ログアウトさせる「logout.php」新規ユーザーの登録をする為の、「create_user.php」、掲示板に投稿した、コメントを削除パスワードを入力することによって削除させる、「delete.php」、掲示板に書き込まれた情報を、データベースに保存する為の、「write.php」です。 一応なのですが、調べながら、メジャーなセキュリティー対策は施してみました。 もし、セキュリティー対策だったり、コードで冗長な部分がありましたら、ご指摘頂きたいのですが、お願いできますでしょうか...? コードなのですが、長くなってしまうため、Githubにてアップロードさせていただきました。 他の機能などもつけてみた為、上記のファイルも以外のものもアップロードされているのですが、コード量や見て頂く点も増えてしまいますし、今回は上記のファイルに関しまして、ご指摘頂けましたら幸いです...
回答: 思いつくまま・・・ * classを作りましょう * $_POSTをfilter_inputで変数に受けましょう(直接流し込みはNG) * SELECT *→ 面倒でもカラムは羅列しましょう * logoutするときはsession_destroyしてもよいかも * なんでもかんでもloginをincludeするのは微妙 * ファイルのアップロード時はファイル名を調整しないと参照できなくなったり競合したりします