View difference between Paste ID: KWbDpeJX and yTDuRjCH

 <?php
 class Firewall{
 	public function __construct() {
-	public function __construct(){
+		// O isset elimina a necessidade da supressão de erros(@)
-		if(@$_POST['firewall'] == 'cancelar'){
+		// Invertendo a condição elimina a necessidade de um if vazio
-			//NÃO FAZ NADA
+		if(isset($_POST['firewall']) and $_POST['firewall'] != 'cancelar') {
-		}else{
+
-
+
 			} else if ($_GET) {
 				$this->get();
-			} else if ($_GET){
+
 		}
 	}
 
     // Definindo visibilidade para todos os métodos (boa prática)
     public function post() {
-	function post(){
+        // aqui provavelmente poderíamos trocar esses loops aninhados por algo como um array_map, eliminando o else
 		foreach ($_POST as $chave => $valor){
 			if(is_array($valor)) {
 				foreach ($valor as $chave2 => $valor2) {
 					$_POST[$chave2] = $this->antiInjection($valor2);
 				}
-			}else{
+			} else {
 				$_POST[$chave] = $this->antiInjection($valor);
 			}
 
-		}return;
+        }
 
         return;
-	function get(){
+
 
 	public function get() {
 		foreach ($_GET as $chave => $valor){
 
 			if(is_array($valor)){
 				foreach ($valor as $chave2 => $valor2){
 					$_POST[$chave2] = $this->antiInjection($valor2);
 				}
 			}
 			else { $_GET[$chave] = $this->antiInjection($valor);
-		} return;
+
 
         }
-	function antiInjection($valor) {
+
         return;
 	}
 
     public function antiInjection($valor) {
         /* Algumas colocações:
          * 1 - esqueça a linha do preg_replace. Ela se baseia em blacklist e não é uma boa idéia.
          * 2 - Para prevenir SQL Injection a melhor saída é usar um prepared statemant (via PDO, por exemplo)
          * 3 - Seria nesse método que usaríamos a filter
          */
 		$valor = preg_replace("/(from|select|insert|delete|where|drop table|show tables|show databases|describe|left join|right join|inner join|#|\*|--|\\\\)/i", "", $valor);
 		$valor = trim($valor);
 		$valor = strip_tags($valor);
 		$valor = addslashes($valor);
 
 		return $valor;
 	}
 }
 
 $fw = new Firewall();