Más de un return's?

[diego_g_09]

Buenas me he encontrado con la disyuntiva de si es correcto tener en un método varios return's o no..

La duda concretamente es si esta bien que un método posea más de un return?, yo he trabajado siempre devolviendo una variable que es seteada dependiendo la lógica del procedimiento y es esta la que devuelvo en un único return, pero he visto varios métodos con varios return y no se si es valido hacerlo, que se puede se puede, pero se debe?

Saludos!!

[CRauda]

Esos metodos con varios return es porque poseen condicionales (normalmente if-else) que determinan cual return se realizara, como sabras en el momento en que se hace un return desde un metodo este hace un salto a la ultima linea y sale del mismo, es decir, nunca hara 2 return al mismo tiempo, el return que hagas primero determinara donde termina la ejecucion del metodo.

[seroons]

Lo que hay que tener claro que return puede haber muchos, pero siempre "uno" por condicional, es decir por cada if, o else.

[diego_g_09]

Si, tengo claro como funciona el return, pero mi pregunta pasa no por un tema de funcionamiento del método sino si es correcto o no hacerlo, un ejemplo de mi duda...

Ejemplo 1:

Código Java:

private boolean sonIguales(String a String b) {
 
        boolean iguales = false;
        
        if (a.equals(b)) {
            iguales = true;
        }
        return iguales;
    }
 

Ejemplo 2:

Código Java:

private boolean sonIguales(String a String b) {
        
        if (a.equals(b)) {
            return true;
        } else {
            return false;
        }
    }
 

Que beneficios tiene un método sobre otro?, el resultado es el mismo, pero la forma de programar es muy distinta, cual es correcta si es que una lo es?

Saludos

[CRauda]

cuando usas varios returns el beneficio viene si usas diferentes variables, por ejemplo

Código Java:

if(texto.equals("N")){
   return var1;
else if(texto.equals("Y")){
   return var2;
}else{
   return var3;
}
 

[diego_g_09]

Iniciado por CRauda

cuando usas varios returns el beneficio viene si usas diferentes variables, por ejemplo

Código Java:

if(texto.equals("N")){
   return var1;
else if(texto.equals("Y")){
   return var2;
}else{
   return var3;
}
 

CRauda, la variable puede ser seteada (una variable general al método) ya que el tipo de dato a devolver debe ser el mismo en los tres casos que estas planteando, por lo tanto para el caso que estas planteando usar diferentes variables no determina el uso de más de un return's o no.. . es más al tener más variables tienes por lo tanto mas memoria utilizada..., la pregunta planteada es muy fina, y es más que nada para saber si es correcto una o otra forma de programar.

[cazador1992]

No, se recomienda tener un solo camino de salida en el método. Si queres le podes poner 25 returns o returns en algun bulce, pero se recomienda que solo tengas un único return.

[diego_g_09]

Iniciado por cazador1992

No, se recomienda tener un solo camino de salida en el método. Si queres le podes poner 25 returns o returns en algun bulce, pero se recomienda que solo tengas un único return.

Ok, es lo que yo tenía entendido, pero cual es la argumentación?, si no se recomienda tener más de uno por que es que no se recomienda?, performance, prolijidad, etc..???

[CRauda]

diego_g_09 Lo que te puse era solo con el fin de ejemplificar, ambas formas son correctas para realizar el return, en el caso de los booleanos es mejor la forma de los 2 return ya que evitas la declaracion de una variable, pero en casos diferentes donde requieres el retorno de variables que no son booleanas te dan el beneficio de poder hacer returns dependiendo de lo que requieras que devuelva, por ejemplo en mi caso tengo estas funciones:

Código Java:

private String ejecutaAccion(int accion, String valor,String valor2,String valor3,String Tabla,String Tabla2) throws SQLException{
            String sql = "";
            String valorReturn = null;
            if (accion == 1){
               sql = "SELECT CTIME campo FROM " + Esquema + "." + Tabla + " WHERE CLABRQ IN (select CLABRQ from " + Esquema + "." + Tabla2 + " where cjbcrd='" + valor + "')";
            }
            if(accion == 2){
                sql = "select ATMPOR campo from " + Esquema2 + "." + Tabla + "  where asplit='" + valor + "'";
            }
            if(accion == 3){
                sql = "select JESTTM campo from " + Esquema + "." + Tabla + " where JWRKO2= " + valor2 + " and JWRKO1= " + valor;
            }
            if(accion == 4){
                sql = "select YTMVAL campo from " + Esquema2 + "." + Tabla + " where ysplit ='" + valor + "'";
            }
            try {
                stmt = as.connection.createStatement();
                rs = stmt.executeQuery(sql);  
                while(rs.next()){
                    if (rs.getString("campo") == null || rs.getString("campo").equals("") || rs.getString("campo").equals("null")){
                        return "0";
                    }else{
                        valorReturn = (rs.getString("campo")).toString();
                        if(isNumeric(valorReturn)){
                            if(Integer.parseInt(valorReturn) < 10){
                                return "0" + valorReturn;
                            }else{
                                return valorReturn;
                            }
                        }
                    }
                }
            } catch (SQLException e) {
                e.printStackTrace();
                return "ND";
            }   
        }
 
    public boolean isNumeric(String input) {
      try {
        Integer.parseInt(input);
        return true;
      }
      catch (NumberFormatException e) {
        //e.printStackTrace();
        return false;
      }
    }
 

De no ser por los multiples return tendria que hacer muchas mas cosas para poder obtener lo que necesito segun sea el caso. Espero que con esto se te aclare mejor tu duda. Saludos.

[diego_g_09]

Ok, entiendo tu punto, pero no veo porque al caso que expusiste no se pueda sustituir los return por un seteo de una variable String que luego es la que devolverás...
Yo tengo el concepto de como dice cazador1992 " se recomienda tener un solo camino de salida en el método. ", pero porqué?

[CRauda]

Como te dije, ambas maneras son correctas, pero entre menos variables crees y menos valores setees menor uso de memoria tendras, es por eso que si te fijas hago return con valores directos, a menos que sea necesario, como en el caso de verificar si es un valor numerico y si es menor de 10 ponerle un cero al inicio, en ese caso si utilice una variable, pero en las demas hago un return directamente con el valor que se refiere a la respuesta obtenida, sino me quedaria de esta forma:

Código Java:

private String ejecutaAccion(int accion, String valor,String valor2,String valor3,String Tabla,String Tabla2) throws SQLException{
            String sql = "";
            String valorReturn = null;
            if (accion == 1){
               sql = "SELECT CTIME campo FROM " + Esquema + "." + Tabla + " WHERE CLABRQ IN (select CLABRQ from " + Esquema + "." + Tabla2 + " where cjbcrd='" + valor + "')";
            }
            if(accion == 2){
                sql = "select ATMPOR campo from " + Esquema2 + "." + Tabla + "  where asplit='" + valor + "'";
            }
            if(accion == 3){
                sql = "select JESTTM campo from " + Esquema + "." + Tabla + " where JWRKO2= " + valor2 + " and JWRKO1= " + valor;
            }
            if(accion == 4){
                sql = "select YTMVAL campo from " + Esquema2 + "." + Tabla + " where ysplit ='" + valor + "'";
            }
            try {
                stmt = as.connection.createStatement();
                rs = stmt.executeQuery(sql);  
                while(rs.next()){
                    if (rs.getString("campo") == null || rs.getString("campo").equals("") || rs.getString("campo").equals("null")){
                        valorReturn = "0";
                    }else{
                        valorReturn = (rs.getString("campo")).toString();
                        if(isNumeric(valorReturn)){
                            if(Integer.parseInt(valorReturn) < 10){
                                valorReturn = "0" + valorReturn;
                            }
                        }
                    }
                }
            } catch (SQLException e) {
                e.printStackTrace();
                valorReturn = "ND";
            }   finally{
                return valorReturn;
            }
        }
 
    public boolean isNumeric(String input) {
      boolean Numerico = false;
      try {
        Integer.parseInt(input);
        Numerico = true;
      }
      catch (NumberFormatException e) {
        //e.printStackTrace();
       
      }
      return Numerico;
    }
 

Que al final es lo mismo, pero con mas uso de memoria, imagina un caso real, 600 usuarios conectados al mismo tiempo en el sistema usando los recursos del servidor... entre mas recursos consumas, mas lento se pondra el sistema y mas probabilidades de saturar el servidor tendras. Saludos.

[diego_g_09]

Ok, bien esa argumentación me gusta más, y entiendo, para estos casos entonces es mejor el uso de muchos return (poniendonos finos en el uso de la memoria)...