で…
l何がまずい?
-PHP的には$REQUEST_も少々危険
-SQL InjectionとXSSがナチュラルに存在
l新規なら?
-ちゃんとエスケープ処理用の集約点を作る&する
lリファクタの時は?
-まず「危ない箇所」を洗い出す(grep可能なコメント
-(場当たりだけど)各個撃破。XSSなどは「テンプレートエンジン」使ってないと特に大変だけど、やる!!
-せめて「関数化」で集約点つくるか、本質的には、全体的に設計をやりなおす