6 replies to this topic

[poliburro]

A todos alguna vez nos ha aparecido el warning en delphi al compilar:

"Variabe 'X' mightnot have been initialized"

Eso sucede ya que al declarar la clase se procede a su creación sin ningún trámite adicional. El warning desaparece asignandole a la clase 'X' el valor Nil. Estuve meditando sobre ello y quisiera conocer sus puntos de vista al respecto. En el  sentido más purista se debe inicializar con Nil las variables de clase para evitar el warning? O debido a que no provocará problema alguno se puede omitir?


Que piensan al respecto?

[cHackAll]

...Que piensan al respecto?

delphi

procedure TForm1.FormCreate(Sender: TObject);
var Trial: Integer; {$warn use_before_def off}
begin
 if False {or ProductoRegalado} then
  Inc(Trial);
 if Trial >= 10 then
  Halt;
end;

[Al González]

A todos alguna vez nos ha aparecido el warning en delphi al compilar:

"Variabe 'X' mightnot have been initialized"

Eso sucede ya que al declarar la clase se procede a su creación sin ningún trámite adicional. El warning desaparece asignandole a la clase 'X' el valor Nil. Estuve meditando sobre ello y quisiera conocer sus puntos de vista al respecto. En el  sentido más purista se debe inicializar con Nil las variables de clase para evitar el warning? O debido a que no provocará problema alguno se puede omitir?


Que piensan al respecto?

Hola Edgar, un gusto poder darte mi opinión. 

Hace falta matizar algunas cosas.  Al compilar, Delphi muestra esa advertencia (warning) si detecta que el código hace referencia de lectura a una variable local que "aparentemente" (según el propio código fuente) no recibirá valor antes de ejecutarse dicha referencia.

Cuando dices "clase X", creo que te refieres más bien a un "objeto X".  Pero como fuera, la advertencia aplica a muchos tipos de variables, no sólo clases u objetos.  De hecho, puede resultar curioso notar que el compilador lo reportará con una variable Integer, pero no con una variable String, y esto se debe a que las variables de tipos que usan contadores de referencias (como String) sí­ son inicializadas de manera implí­cita (en este mensaje de Club Delphi profundizo sobre la lógica detras de ello: http://www.clubdelph...25&postcount=29).

Resulta muy útil que el compilador lance la advertencia, ya que en muchos casos habrá sido por algún descuido de nuestra parte.  Pero hay casos especiales, donde el compilador no logra determinar si la variable está siendo inicializada o no, pero nosotros, los creadores del código, sabemos que sí­ lo estamos haciendo.  Tal vez no de una manera convencional que el compilador pueda detectar fácilmente, pero sí­ que le estamos dando un valor inicial.

Para estos últimos casos (que en la práctica son poco frecuentes), es de mucha ayuda desactivar la directiva de compilación "Warn Use_Before_Def", que señaló Javier.  Salvo que debes hacerlo sólo para la rutina en cuestión, ya que de otra manera quedarí­a desactivada también para las rutinas siguientes del archivo de código.

delphi

{$Warn Use_Before_Def Off}
Procedure Proc1;
Var
  MasterField :TField;
  SavedModified :Boolean;
  SavedState :TDataSetState;
Begin
  ...
End;
{$Warn Use_Before_Def On}  // volvemos a activar la directiva

Uno no debe inicializar una variable sólo por evitar que aparezca esa advertencia.  Mas bien, en caso de aparecer la advertencia, hay que revisar si estamos haciendo algo mal (como leer una variable local que sólo hemos declarado pero no otorgado ningún valor), o, por el contrario, estamos seguros de que el código le habrá asignado un valor a la variable antes de ser leí­da y por tanto proceder con seguridad a desactivar temporalmente la directiva.

Por último, siempre que aparezca esta advertencia del compilador, debemos revisar nuestro código.  Ignorarla o desactivar la directiva correspondiente sin hacer ningúna revisión es a lo menos comprometedor.  En mi opinión, nunca debemos obviar este tipo de mensajes del compilador.

Mi contribución.

Saludos.

Al González.

[egostar]

Hola, me parece que este hilo bien podria estar en otro foro, por el momento lo paso a General, si alguno de los moderadores considera moverlo hagalo con libertad.

Salud OS

[poliburro]

Coincido con todo lo que mencionas Al, solo que mi post estaba orientado a si se debe o no inicializar las variables de clase con Nil tal y como mencionan la documentación o en su defecto puede ser omitida ya que no genera problema alguno.

Omitir warnings para variables de tipo nativo (notese influecia dot net ) es una grave práctica.

Saludos.

[Al González]

Coincido con todo lo que mencionas Al, solo que mi post estaba orientado a si se debe o no inicializar las variables de clase con Nil tal y como mencionan la documentación o en su defecto puede ser omitida ya que no genera problema alguno...

Hola de nuevo.  ¿Podrí­as mostrarnos esa documentación?  ¿De qué compilador estamos hablando?

Hay una regla muy sencilla: Una variable local debe ser inicializada solamente si puede darse el caso de que lo primero que le ocurra dentro de la rutina sea una lectura y no una escritura.

delphi

Procedure Proc1 (Const Crear :Boolean);
Var
  F :TForm1;
Begin
  If Crear Then
    F := TForm1.Create (Nil);
 
  If F <> Nil Then  // Puede que llegue aquí­ sin haberse asignado nada a F (¡cuidado!)
    F.Show;
End;
 
Procedure Proc2 (Const Crear :Boolean);
Var
  F :TForm1;
Begin
  If Crear Then
    F := TForm1.Create (Nil)
  Else
    F := Nil;  // Así­ está mejor
 
  If F <> Nil Then  // Esto ya es seguro
    F.Show;
End;
 
// Y en este caso no hay problema porque siempre ocurre la asignación primero
Procedure Proc3 (Const Crear :Boolean);
Var
  F :TForm1;
Begin
  F := FuncionForm ();
 
  If F <> Nil Then
    F.Show;
End;

Espero haber despejado alguna duda.

Saludos.

Al.

[cHackAll]

...Para estos últimos casos (que en la práctica son poco frecuentes), es de mucha ayuda desactivar la directiva de compilación "Warn Use_Before_Def", que señaló Javier.  Salvo que debes hacerlo sólo para la rutina en cuestión, ya que de otra manera quedarí­a desactivada también para las rutinas siguientes del archivo de código...

Aunque yo solo mostraba un ejemplo de lo que sucederí­a si ignoraramos por completo dicho mensaje ("Run-Time Check Failure #3"), me parece que lo que comentas es muy importante.

...Una variable local debe ser inicializada solamente si puede darse el caso de que lo primero que le ocurra dentro de la rutina sea una lectura y no una escritura...

El resumen del hilo