Buenas señores de nuevo por aca tengo una duda !!!!

yo tengo esta condicion de este pedazo de codigo :

 
if(Delimitador){
                            if(Buscar_IpsBaneadas(ip)){
                    //String Mensaje ="Su ip a sido Bloqueada";
    %>
 
    <script language="javascript" type="text/javascript">
        alert("<%="Su ip a sido Bloqueada"%>");
        setTimeout("location.href='192.168.1.37'", 5000);
    </script>
 
    <%!
            return "";
            }else{
            return ip;
            } 
 
}
           else {
 
                    return ip;
           }
 

La duda es porque siempre me lee el trozo de javascript aunque no entre en la condicion !!!!
el lencuaje es jsp y empotre ese pedazo de codigo de javascript al

Esa porción de código parece un método, estas declarando un método dentro de un archivo JSP??? Eso es una MUY MALA PRACTICA.

Para lo demás, basta con utilizar el debuger para verificar cuales y porque razón se están ejecutando algunas sentencias. O bueno hacer un println a Buscar_IpsBaneadas(ip) para verificar que retorna, pero te recomiendo que utilizes el debuger que es lo ideal.

Otro dato muy importante, si en un método que retorna un String, NUNCA debes devolver un string vació, como en la linea 13 del código que haz posteado. En ese tipo de caso, debes devolver un null, la razón es porque si retornas un String vació, de igualmanera se estaría creando un objeto de Tipo string en la zona de memoria Heap, en cambio si retornas un null no pasara esto.

Saludos.