problema con funciones

[[seoane]]

Bueno roman, yo empiezo poniendo esa linea, luego si la cosa se tuerce se quita y en paz

[dec]

Hola,

Yo suelo hacer lo que hace Seoane, ciertamente. La primera línea es una instrucción: "Result := false" o "Result := true", según si se supone que la función ha de retornar "false" o "true" por defecto. Sin embargo, últimamente, programando en PHP, las cosas cambian amigos.

¿Cómo? Pues porque en PHP no vale algo como "return false" al principio de la línea, puesto que el "return" impedirá que se ejecute lo que venga después, es decir, que no es lo mismo que el "result" de Delphi. Lo siento.

Ahora, aún así sigo utilizando a veces algo como:

Código PHP:

function LoqueSea(){
  $resultado = false;
  /* */
  return $resultado;
} 


Que es como copiar lo que hago en Delphi: asegurarme de que la función tiene un resultado por defecto. Sin embargo, es verdad que a veces ocurre lo que menciona Román, y bueno, que, últimamente también estoy tirando por el lado de no tener un resultado por defecto, aunque, eso sí, asegurándome de que lo que la función X retorna tiene sentido en cualquier caso. ¡O eso intentamos!

[roman]

Hola,

A decir verdad me extraña un poco este tipo de técnicas, o la necesidad de ellas. Parece que damos un valor por defecto para "cubrirnos las espaldas" ante algún posible caso no considerado. Pero el valor por defecto no arreglará tal omisión- de haberla -, de tal suerte que el problema seguirá latente. En el caso de Delphi, al menos, prefiero que el compilador me notifique que hay un problema, a ocultar el mensaje con un valor por defecto.

// Saludos

[dec]

Hola,

Bueno. Tú eres mucho más experto que yo, Román, empero, yo no trato de ocultar ningún problema o mensaje de error... sino que, simplemente, asigno un valor de retorno por defecto para una función: nada más y nada menos.

Cuando escribo una función, digamos, "EsUsuarioAdministrador()", a veces (así lo he hecho más a menudo en Delphi que en PHP, eso sí), nada más comenzar la función, escribo un "Result := False", por ejemplo. Es una forma de decir "ningún usuario será considerado administrador a no ser que se cumplan X condiciones... en un primer momento ningún usuario es considerado administrador".

No sé. Tal vez incluso sea una forma de hacer más legible la función, no sé si me explico. Pero, ya digo, la intención no es esconder errores, ni ocultarlos, ni nada de eso.

[roman]

Empezado por dec

Bueno. Tú eres mucho más experto que yo, Román, empero, yo no trato de ocultar ningún problema o mensaje de error... sino que, simplemente, asigno un valor de retorno por defecto para una función: nada más y nada menos.

No me di a entender. No digo que hagan eso con la intención de ocultar el error. Digo que al dar un valor por defecto, corro el riesgo de no darme cuenta de que hay un error, porque el compilador no me lo marca.

// Saludos

[[seoane]]

Empezado por roman

No me dí a entender. No digo que hagan eso con la intención de ocultar el error. Digo que al dar un valor por defecto, corro el riesgo de no darme cuenta de que hay un error, porque el compilador no me lo marca.

Exacto, cuando yo doy un valor por defecto, normalmente es un valor que a mi me indica un error, es decir, si la función me devuelve ese valor se que ha ocurrido algo. Por ejemplo, si la función me tiene que devolver el tamaño de un archivo y le pongo de valor por defecto -1, se que si devuelve ese valor a ocurrido un fallo.

Lo dicho, son manías que uno tiene, pero que no tengo problemas en cambiar si la función lo requiere

[dec]

Hola,

Creo que todos estamos más o menos de acuerdo. Es cierto que no en todos los casos un valor de retorno por defecto puede no ser del todo "intuitivo"... yo creo que me limito a preparar un valor de retorno por defecto cuando por el contrario es intuitivo... ya digo, creo que me sirve para "leer" mejor la función... pero, ya digo que esto es algo que surje de un modo más o menos natural... no es forzado.

De hecho alguna vez se me ocurrió que todas las funciones tuvieran un valor de retorno por defecto: todas, todas las funciones. Y esto no funcionó. Es decir, que no programo de esa manera, sino que a veces lo hago así, y otras veces no. Tengo cierta confianza en que sé elegir qué función es menester de un valor de retorno por defecto y cuál no.

PD. Y no tengo abuela.

[roman]

Empezado por dec

Cuando escribo una función, digamos, "EsUsuarioAdministrador()", a veces (así lo he hecho más a menudo en Delphi que en PHP, eso sí), nada más comenzar la función, escribo un "Result := False", por ejemplo. Es una forma de decir "ningún usuario será considerado administrador a no ser que se cumplan X condiciones... en un primer momento ningún usuario es considerado administrador".

Lo que describes es un caso muy particular, donde la propia semántica de la función requiere un valor por defecto, pero que yo ponga:

Código Delphi [-]
function esNumeroPrimo(n: Integer): Boolean;
begin
  Result := false;
  
  {
    algoritmo para determinar si n es primo
  }
end;

no me sirve, porque si mi algoritmo no cubre todos los casos, el valor por defecto tendrá igual de posibilidades de ser adecuado que inadecuado.

// Saludos

[[egostar]]

Yo creo que no es necesario inicializar el valor de retorno, la regla es que no haya ningún "vacio" en el retorno de la función.

Código Delphi [-]
function TForm1.algo(dato:String):Bool;
begin
  If dato = 'e32er433432e' then
     Result := True
  else Result := False; // Else
end;

Código Delphi [-]
function TForm1.algo(dato:word):integer;
begin
  case dato of
     0 : Result := 1;
     1 : Result := 0;
     else Result := -1; // Else
  end;
end;

De esta manera no hay ningún "vacio" para el retorno de la función de cualquier tipo.

Salud OS.

[dec]

Hola,

Estamos de acuerdo egostar. Pero también puede verse como distintas formas de codificación... porque una cosa es lo mismo que la otra al fin y al cabo:

Código Delphi [-]
function TForm1.algo(dato:word):integer;
begin
  case dato of
     0 : Result := 1;
     1 : Result := 0;
     else Result := -1; // Else
  end;
end;

Código Delphi [-]
function TForm1.algo(dato:word):integer;
begin
  Result := -1;
  case dato of
     0 : Result := 1;
     1 : Result := 0;
  end;
end;

Ignoro si una forma es más correcta, elegante o aconsejable que la otra, pero, las dos formas son posibles y al cabo se consigue lo mismo... Así que es cuestión de la costumbre de uno, o de si se han establecido ciertas normas al respecto,... qué sé yo.

Yo no abogo ni por una forma ni por otra. Creo que esto significa que es cuestión de gustos... en este caso creo que no me equivoco si digo esto, vamos.