Llamada de miembro virtual en el constructor

Question

En mi aplicación, estoy ejecutando el mismo winform en diferentes contextos para controlar la visibilidad de botones, enabeling de campos de texto y el texto del encabezado de winform. La forma en que decidí hacer esto es simplemente pasando una cadena al constructor de formulario y verificarlo con un par de instrucciones if que a su vez contienen los ajustes de winform deseados.

if (formContext == "add") 
{ 
    Text = "Add member"; 
} 
if (formContext == "edit") 
{ 
    Text = "Change role"; 
    userTextBox.Enabled = false; 
    searchButton.Visible = false; 
} 

Esto funciona bien, sin embargo las palabras clave "texto" Obtener una blue Squigly línea agregada por ReSharper con el siguiente mensaje: Call Viritual en el constructor. ¿Es esto un problema potencial o simplemente algún tipo de mensaje ReSharper demasiado entusiasta?

Cualquier aclaración o sugerencia para mejorar mi implementación sería muy apreciada.

Answer 1

Call virtual en la clase base ctor podría causar algo de lógica para ejecutar en la subclase antes de llamar al subclase ctor (y por lo tanto antes de que el objeto tenga la oportunidad de inicializarse a un estado coherente).

Es solo un recordatorio para que sepa que está haciendo algo que podría causar un comportamiento desagradable inesperado.

Answer 2

Además de las respuestas existentes, para las formas que podría añadir un controlador de eventos de carga:

Load += delegate 
{ 
    if (formContext == "add") 
    { 
     Text = "Add member"; 
    } 
    if (formContext == "edit") 
    { 
     Text = "Change role"; 
     userTextBox.Enabled = false; 
     searchkButton.Visible = false; 
    } 
}; 

Answer 3

yo sugeriría que volver a escribir la clase de la siguiente manera:

public partial class Form1 : Form 
{ 
    public enum FormContextMode 
    { 
     Add, 
     Edit 
    } 

    private FormContextMode m_mode = FormContextMode.Add; 

    public Form1(FormContextMode mode) 
    { 
     InitializeComponent(); 
     m_mode = mode; 
     Load += delegate { UpdateForm(); }; 
    } 

    private void UpdateForm() 
    { 
     if(m_mode == FormContextMode.Add) 
     { 
      Text = "Add member";  
     } 
     else if(m_mode == FormContextMode.Edit) 
     { 
      Text = "Change role"; 
      userTextBox.Enabled = false; 
      searchkButton.Visible = false; 
     } 
    } 
} 

Answer 4

Sólo sellar su clase.