Usando "como" y esperando un resultado nulo

Question

Por ejemplo. Digamos que tenemos un stackpanel en un formulario. Está lleno de Grids y Labels. Quiero recorrer todas las Grids y hacer alguna operación con ellas, pero dejo intactas las Lables. Por el momento lo estoy haciendo de esta manera.

foreach(UIElement element in m_stacker.Children) 
{ 
    Grid block = element as Grid; 
    if(block != null) 
    { 
     //apply changes here 
    } 
} 

Así que estoy usando el hecho de que "como" devuelve nulo si no puede convertirse en el tipo requerido. ¿Es esto correcto o hay una mejor solución para este problema?

Answer 1

Sí, es la forma correcta de hacerlo (teniendo en cuenta "aplicar cambios" tendrá que utilizar el resultado del molde).

Sin embargo, si no va a utilizar el valor de retorno del molde, simplemente puede usar is.

Usando is y luego casting again with the cast operator (parenthesis) is frowned upon.

Answer 2

Funcionará bien. Sin embargo, lo que puedes hacer es usar "es".

foreach(UIElement element in m_stacker.Children) 
{ 
    if(element is Grid) 
    { 
     //apply changes here 
    } 
} 

Answer 3

¿Qué hay de OfType()?

foreach(var grid in m_stacker.Children.OfType<Grid>()) { ... } 

Este bucle sólo sobre los hijos de tipo rejilla , por lo que no hay necesidad de emitir o comprobar el tipo en absoluto.

Answer 4

Si va a utilizar block como Grid, esta es la forma correcta de codificarlo.

Comprobación utilizando is (en mi opinión) produce código menos legible como se ilustra en los siguientes dos malas ejemplos:

if (element is Grid) 
{ 
    Grid block = element as Grid; 
    // Do stuff 
} 

if (element is Grid) 
{ 
    Grid block = (Grid)element; 
    // Do stuff 
} 

No hacer cualquiera de estas.

Usted está diciendo que si el elemento es una cuadrícula, entonces empújelo.