¿Hay alguna solución para FxCop warning CA1006?

Question

Tengo problemas con FxCop warning CA1006, Microsoft.Design "DoNotNestGenericTypesInMemberSignatures". Específicamente, estoy diseñando una clase ReportCollection<T> que hereda de ReadOnlyCollection<Report<T>>, y su constructor public toma un IList<Report<T>> como parámetro.

La sugerencia para la fijación de esta advertencia no es muy útil:

"Para el montaje de una violación de esta regla, cambiar el diseño para eliminar el argumento de tipo anidado". Hay dos maneras en que puedo ver lo que va a cambiar el diseño como se sugiere:

1. hacer que el constructor internal. Esto no funciona en mi caso. El constructor debe ser public porque esta clase de colección necesita ser instanciable por código fuera del ensamblado.
2. Haga que el constructor tome Report<T>[] en lugar de IList<Report<T>>. Esto no es óptimo porque el código externo debe tener la flexibilidad de usar estructuras de datos de tamaño dinámico como List<T> en lugar de matrices de tamaño fijo.

En este punto, he renunciado y suprimido esta advertencia. ¿Hay una mejor solución?

Answer 1

Estoy de acuerdo, otro buen momento para hacer caso omiso de esta regla es cuando se necesita decir:

Func<IEnumerable<T>> 

, por supuesto, se puede utilizar el IEnumerable no genérico, pero entonces cualquier tipo se puede utilizar siempre y cuando implementa IEnumerable (no genérico). El objetivo de los genéricos (en parte) es restringir los tipos que son permisibles para un conjunto determinado de tipos.

Creo que esta regla es muy estúpida. solo lo necesita si tiene múltiples tipos genéricos anidados. una capa de anidación es más que segura.

Por cierto, creo que muchos de las funciones nido LINQ tipos genéricos, así, así que si lo hace MS, podemos también :)

Answer 2

Tomaría las advertencias de FxCop como si fueran sugerencias de un compañero de trabajo extremadamente atento al anal. Está perfectamente bien ignorar (suprimir) algunas de las cosas que sugiere.

Answer 3

Estoy de acuerdo que se puede ignorar la advertencia CA1006 en el caso de

Func<IEnumerable<T>> 

también se puede simplificar el código mediante el uso de delegados y evitar la CA1006:

public delegate IEnumerable<T> ChildrenDel<T>(T parent); 

// was: GetDescendants<T>(this T item, Func< T, IEnumerable<T> > children) 

public static IEnumerable<T> GetDescendants<T>(this T item, ChildrenDel<T> children) 
{ 
    var stack = new Stack<T>(); 
    do { 
     children(item).ForEach(stack.Push); 

     if(stack.Count == 0) 
      break; 

     item = stack.Pop(); 

     yield return item; 
    } while(true); 
}