I tipi di nidificazione sono considerati pratiche scorrette?

I tipi annidati non sono male. L'avviso che stai ricevendo non sta suggerendo di non avere mai un tipo annidato. Sta semplicemente indicando che il tuo tipo annidato dovrebbe usare un modificatore di accesso e un percorso di codice appropriati.

Se il tipo annidato è realmente usato solo all'interno della classe contenente (cioè è un contenitore di dati interno o indicatore di stato), quindi imposta il suo modificatore di accesso su privato.

Se il tipo annidato fa parte di una o più firme del metodo, allora non è in effetti locale alla classe contenitore. Rappresenta un messaggio passato o proveniente da istanze della classe contenente. In questo caso, è decisamente meglio spostare il tipo nidificato fuori dalla classe contenente e dargli un modificatore di accesso più aperto come interno o pubblico.

In breve, sembra che l'avviso consigli di rendere privati i tipi locali e i tipi condivisi dovrebbero essere indipendenti.

Se è usato esclusivamente da quella classe, dovrebbe essere reso privato e questo è esattamente ciò che il messaggio suggerisce.

No

Un buon esempio in C # è IEnumerable in cui altre classi non hanno bisogno di conoscere la classe esatta restituita, solo che è un IEnumerator. Quindi ha senso renderlo una classe nidificata altrimenti potresti avere un sacco di piccole classi che fluttuano in intellisense che implementa IEnumerator

L'avvertimento sui tipi di nidificazione era uno dei primi "suggerimenti" che ho ricevuto qualche tempo fa dopo aver abilitato l'analisi del codice. È stato anche il motivo per cui l'ho spento.

Alcuni suggerimenti sono davvero strani come se provenissero da un altro pianeta.

Metto l'enumerazione all'interno delle classi per mantenere logicamente le cose insieme.

Pensa in questo modo: se i tipi di nidificazione fossero stati il male assoluto in tutti i casi, perché i progettisti linguistici l'avrebbero implementato in primo luogo? Perché è utile in molti casi e, a tale proposito, il suddetto avvertimento è semplicemente una precisazione che qualcosa nel tuo codice potrebbe non essere ottimale. Se si applica o meno dipende da te decidere.