C # Antimuster

To cut a long story short: ich finde die Java-antipatterns eine unverzichtbare Ressource. Für Anfänger genauso wie für Profis. Ich habe noch etwas finden, wie diese für C#. Also ich werde eröffnen diese Frage als community-wiki und laden alle ein, Ihr wissen zu teilen. Als ich bin neu in C# bin, bin ich stark interessiert, aber nicht starten mit einige antipatterns :/

Hier sind die Antworten die ich finden insbesondere gilt für C# und nicht in anderen Sprachen.

Ich nur copy/Paste! Betrachten Sie werfen einen Blick auf die Kommentare auf diese als gut.

Werfen Sie die `NullReferenceException`

Werfen der falsche Ausnahme:

if (FooLicenceKeyHolder == null)
    throw new NullReferenceException();

Eigenschaften und public-Variablen

Public-Variablen in Klassen (die Nutzung einer Immobilie statt).

Es sei denn, die Klasse ist eine einfache Daten-Transfer-Objekts.

Nicht zu verstehen, dass bool ein echter Typ, nicht nur eine Konvention

if (myBooleanVariable == true)
{
    ...
}

oder, noch besser

if (myBooleanVariable != false)
{
    ...
}

Konstrukte wie diese sind oft von C und C++ Entwicklern, wo die Idee, ein boolean-Wert war nur eine Konvention (0 == false, alles andere true); ist dies nicht notwendig (oder wünschenswert), die in C# oder anderen Sprachen, die echte booleans.

Mit `()`

Nicht verwenden using gegebenenfalls:

object variable;
variable.close(); //Old code, use IDisposable if available.
variable.Dispose(); //Same as close.  Avoid if possible use the using() { } pattern.
variable = null; //1. in release optimised away.  2. C# is GC so this doesn't do what was intended anyway.

InformationsquelleAutor der Frage |

anti-patterns c#

Erneute auslösen der exception falsch. Zu rethrow eine Ausnahme :

try
{
    //do some stuff here
}
catch (Exception ex)
{
    throw ex;  //INCORRECT
    throw;     //CORRECT
    throw new Exception("There was an error"); //INCORRECT
    throw new Exception("There was an error", ex); //CORRECT
}

InformationsquelleAutor der Antwort

40

GC.Collect() zu sammeln, statt Vertrauen in den garbage collector.

InformationsquelleAutor der Antwort
30

Ich sehe das nicht ein viel zu viel, sowohl in Java und C#...
```
if(something == true){
  somethingelse = true;
}
```
bonus-Punkte, wenn Sie auch
```
else{
  somethingelse = false;
}
```
InformationsquelleAutor der Antwort
25
```
using Microsoft.SharePoint;
```
'nuff said

InformationsquelleAutor der Antwort

Sehe ich folgenden code viel:

if (i==3)
       return true;
else
       return false;

werden sollte:

       return (i==3);

InformationsquelleAutor der Antwort

Beleidigung des law of Demeter:

a.PropertyA.PropertyC.PropertyB.PropertyE.PropertyA = 
     b.PropertyC.PropertyE.PropertyA;

InformationsquelleAutor der Antwort

Werfen NullReferenceException:

if (FooLicenceKeyHolder == null)
    throw new NullReferenceException();

InformationsquelleAutor der Antwort

16

Dies ist wahr, ich habe es gesehen mit meinen eigenen Augen.
```
public object GetNull()
{
     return null;
}
```
Es tatsächlich verwendet wurde, in der app, und hatte sogar eine gespeicherte Prozedur, die mit ihm zu gehen, auch eine sp_GetNull das würde null zurückgeben....

machte meinen Tag.

Ich denke, die sp war für eine classic asp Website .. etwas zu tun mit einem ResultSet. die .net war jemand Ahnung von "konvertieren" Sie den code in .net...

InformationsquelleAutor der Antwort

int foo = 100;
int bar = int.Parse(foo.ToString());

Oder allgemeineren Fall:

object foo = 100;
int bar = int.Parse(foo.ToString());

InformationsquelleAutor der Antwort

Ich habe festgestellt, dass dies in unserem Projekt, und fast brach der Stuhl...

DateTime date = new DateTime(DateTime.Today.Year, 
                             DateTime.Today.Month, 
                             DateTime.Today.Day);

InformationsquelleAutor der Antwort

11

Ganz oft stolpere ich über diese Art von var-Missbrauch:
```
var ok = Bar();
```
oder noch besser:
```
var i = AnyThing();
```
Mit var so keinen Sinn macht und gewinnt nichts. Es macht den code schwerer zu Folgen.

InformationsquelleAutor der Antwort
10
- Fehlende null-test vor der Delegat-Aufruf.
- Nicht wissen, Wann und wie die Verwendung von 'as' mit einem null-check gegen eine Besetzung mit einer try/catch.
- "throw exception' vs. 'werfen', in einem catch-block.
- Instanziierung einer großen Anzahl von Zeichenfolgen anstelle der Verwendung von StringBuilder.
- Tiefe Verschachtelung der Blöcke.
InformationsquelleAutor der Antwort
10

Nicht zu verstehen, dass bool ein echter Typ, nicht nur eine Konvention
```
if (myBooleanVariable == true)
{
    ...
}
```
oder, noch besser
```
if (myBooleanVariable != false)
{
    ...
}
```
Konstrukte wie diese sind oft von C und C++ Entwicklern, wo die Idee, ein boolean-Wert war nur eine Konvention (0 == false, alles andere true); ist dies nicht notwendig (oder wünschenswert), die in C# oder anderen Sprachen, die echte booleans.

Aktualisiert: Also der Letzte Absatz zu verbessern die Klarheit.

InformationsquelleAutor der Antwort
9

Public-Variablen in Klassen (die Nutzung einer Immobilie statt).

Es sei denn, die Klasse ist eine einfache Daten-Transfer-Objekts.

Siehe Kommentare unten für die Diskussion und Klärung.

InformationsquelleAutor der Antwort
8

Habe ich tatsächlich gesehen.
```
bool isAvailable = CheckIfAvailable();
if (isAvailable.Equals(true))
{ 
   //Do Something
}
```
beats die isAvailable == true anti-Muster, die Hände nach unten!

So dass dies ein super-anti-pattern!

InformationsquelleAutor der Antwort

object variable;
variable.close(); //Old code, use IDisposable if available.
variable.Dispose(); //Same as close.  Avoid if possible use the using() { } pattern.
variable = null; //1. in release optimised away.  2. C# is GC so this doesn't do what was intended anyway.

InformationsquelleAutor der Antwort

6

Private automatisch implementierte Eigenschaften:
```
private Boolean MenuExtended { get; set; }
```
InformationsquelleAutor der Antwort
6

Deklarieren und initialisieren allen lokalen Variablen am Anfang jeder Methode ist so hässlich!
```
void Foo()
{
    string message;
    int i, j, x, y;
    DateTime date;

    //Code
}
```
InformationsquelleAutor der Antwort

Zwei string-anti-patterns

Anti-Pattern # 1

Prüfen von Zeichenfolgen für null oder leer

//Bad
if( myString == null || myString == "" )
OR
if( myString == null || myString.Length == 0 )

//Good
string.IsNullOrEmpty(myString)

Anti-Pattern # 2 (nur für .NET 4.0)

Prüfen von Zeichenfolgen für null oder leer ist oder Leerzeichen

//Bad
if( myString == null || myString == "" || myString.Trim() == "")

//Good
string.IsNullOrWhiteSpace(myString)

InformationsquelleAutor der Antwort

Unnötig casting (bitte Vertrauen Sie den compiler):

foreach (UserControl view in workspace.SmartParts)
{
  UserControl userControl = (UserControl)view;
  views.Add(userControl);
}

InformationsquelleAutor der Antwort

if(data != null)
{
  variable = data;
}
else
{
  variable = new Data();
}

besser geschrieben als

variable = (data != null) ? data : new Data();

und noch besser geschrieben als

variable = data ?? new Data();

Letzten code-listing arbeiten .NET 2.0 und höher

InformationsquelleAutor der Antwort

4

Sprechen mit einem Akzent immer holte mich ein.

C++ - Programmierer:
```
if (1 == variable) { }
```
In C# dadurch erhalten Sie einen compiler-Fehler, wenn Sie die Art if (1 = variable)so dass Sie den code schreiben, der die Art und Weise Sie meinen, es anstatt sich Gedanken über die Dreharbeiten selbst in den Fuß.

InformationsquelleAutor der Antwort

Nicht mit ternären ist etwas, was ich sehen, konvertiert c# gelegentlich

sehen Sie:

private string foo = string.Empty;
if(someCondition)
  foo = "fapfapfap";
else
  foo = "squishsquishsquish";

statt:

private string foo  = someCondition ? "fapfapfap" : "squishsquishsquish";

InformationsquelleAutor der Antwort

Zugriff auf geänderte Verschlüsse

foreach (string list in lists)
{
        Button btn = new Button();
        btn.Click += new EventHandler(delegate { MessageBox.Show(list); });
}

(siehe link für Erklärung und fix)

InformationsquelleAutor der Antwort

3

Für concating beliebige Anzahl von strings die string-Verkettung anstelle von string builder

Exampls
```
foreach (string anItem in list)
    message = message + anItem;
```
InformationsquelleAutor der Antwort
2

ist dies als Allgemeinen ?
```
public static main(string [] args)
{
  quit = false;
  do
  {
  try
  {
      //application runs here .. 
      quit = true;
  }catch { }
  }while(quit == false);
}
```
Ich weiß nicht, wie zu erklären, aber es ist wie jemand, der das abfangen einer Ausnahme und wiederholt den code immer und immer in der Hoffnung es funktioniert später. Wie wenn eine IOException Auftritt, Sie versuchen einfach, über und über, bis es funktioniert..

InformationsquelleAutor der Antwort
2

Dem Projekt bin ich auf hatte fünfzig Klassen Erben von der gleichen Klasse, die alle definiert:
```
public void FormatZipCode(String zipCode) { ... }
```
Entweder legen Sie es in der übergeordneten Klasse, oder eine utility-Klasse aus, die Seite. Argh.

Haben Sie als das surfen durch The Daily WTF?

InformationsquelleAutor der Antwort
1

Massiv über-komplizierte "Page_Load" Methoden, die alles tun wollen.

InformationsquelleAutor der Antwort
1

Mithilfe von Eigenschaften für etwas anderes, als einfach einen Wert Abfragen oder möglicherweise eine kostengünstige Kalkulation. Wenn Sie den Zugriff auf eine Datenbank aus Ihrer Immobilie, sollten Sie es zu einem Methodenaufruf. Entwickler erwarten, dass Methodenaufrufe können teuer werden, Sie erwarten nicht, dass dies von Eigenschaften.

InformationsquelleAutor der Antwort

Fand diese ein paar mal in einem system, das ich geerbt...

if(condition){
  some=code;
}
else
{
  //do nothing
}

und Umgekehrt

if(condition){
  //do nothing
}
else
{
  some=code;
}

InformationsquelleAutor der Antwort

Schreibe einen Kommentar

Du musst angemeldet sein, um einen Kommentar abzugeben.

Werfen Sie die NullReferenceException

Eigenschaften und public-Variablen

Nicht zu verstehen, dass bool ein echter Typ, nicht nur eine Konvention

Mit ()

Werfen Sie die `NullReferenceException`

Mit `()`