Anti‑patterns typiques en C#

1. Architecture & Conception

🔴 God Class / God Service

• Classe énorme, fait tout, dépend de tout.
• Tests difficiles, couplage fort.
• Correction : extraire plusieurs services, appliquer SRP, utiliser Strategy/Command.

🔴 Service Locator

var service = ServiceLocator.Resolve<ICache>();

• Masque les dépendances réelles.
• Correction : injection par constructeur avec DI.

🔴 Primitive Obsession

public void CreateUser(string name, string email) { }

• Trop de primitives (string, int).
• Correction : utiliser des Value Objects (records C#).

🔴 Feature Envy / Data Clumps

Groupe récurrent de paramètres → dépasser en record ou classe.

2. Asynchronisme & Threading

🔴 async void (hors événements UI)

async void DoWork() { await ... }

• Exceptions perdues, pas d'attente possible.
• Correction : utiliser async Task.

🔴 Bloquer async (Result, Wait)

var data = GetAsync().Result;

• Risque de deadlock.
• Correction : toujours utiliser await.

🔴 Task.Run dans ASP.NET

Pollution du thread pool → utiliser I/O async natif.

🔴 Pas d'utilisation d'IProgress<T>

Utiliser Invoke partout n’est plus nécessaire, Progress<T> publie automatiquement sur le contexte UI.

🔴 Mauvais locks

lock(this) { }
lock("string") { }

Utiliser un verrou privé :

private readonly object _gate = new();
lock(_gate) { }

3. Exceptions & Gestion d’erreurs

🔴 Exceptions avalées

try { } catch { }

🔴 Catch trop large

Préférer les exceptions spécifiques, utiliser when filters.

🔴 Exceptions utilisées comme flux normal

Préférer TryParse et équivalents.

4. Ressources & IDisposable

🔴 Oublier Dispose

var conn = new SqlConnection(cs);

Correction : using

using var conn = new SqlConnection(cs);

🔴 Mauvais pattern IDisposable

Utiliser le pattern standard + GC.SuppressFinalize(this).

🔴 Fuites d’événements

publisher.Changed += OnChanged;

5. API & Méthodes

🔴 Retourner null pour une collection

Correction : Array.Empty<T>() ou new List<T>().

🔴 Paramètres publics non validés

Correction : ArgumentNullException.ThrowIfNull(arg).

🔴 Equals / GetHashCode incohérents

Solution : utiliser des record pour gérer automatiquement.

6. LINQ, Performance & Style

🔴 Concaténation de chaînes dans une boucle

var s = "";
for(...) s += part;

Correction : StringBuilder.

🔴 LINQ abusif dans les hot paths

🔴 Magic strings / numbers

Correction : const, enum, nameof.

7. Internationalisation & Dates

🔴 DateTime.Now pour des données persistées

Correction : DateTimeOffset.UtcNow.

🔴 Formatage sans culture explicite

Utiliser CultureInfo.InvariantCulture ou culture choisie.

8. Tests & Maintenabilité

🔴 Code non testable (new partout)

🔴 Méthodes trop longues / multifonctions

Appliquer SRP, découper.

9. Détection automatique

• Analyzers .NET (Roslyn)
• StyleCop Analyzers
• SonarQube / SonarCloud
• .editorconfig

10. Exemples avant / après

a) async void → async Task

// ❌
async void Process() { await DoAsync(); }

// ✅
async Task ProcessAsync() { await DoAsync(); }

b) .Result / .Wait → await

// ❌
var data = GetAsync().Result;

// ✅
var data = await GetAsync();

c) Magic strings → nameof

// ❌
throw new ArgumentNullException("user");

// ✅
throw new ArgumentNullException(nameof(user));