C#: Finden Sie alle leeren Catch-Blöcke

Ich überprüfe einige Code.C#: Finden Sie alle leeren Catch-Blöcke

Ich habe einige leere Catch-Blöcke bemerkt. Keine gute Idee, da etwas nicht funktioniert und man nicht sehen kann warum.

Gibt es eine einfache Möglichkeit, alle leeren Try-Catch-Blöcke in einer Lösung zu finden?

EDIT

Danke für alle Antworten.

Verwendet das Beispiel von Stefan gegeben. Musste ein paar Variationen machen, gibt es viele Möglichkeiten, leere Catch-Blöcke zu schreiben. Auch eine einfache Suche nach catch (Exception) hat mehrere Fehler gefunden.

Quelle

2009-09-28 Shiraz Bhaiji

Nur ein FYI - nicht alle leeren Catch-Blöcke sind schlecht. Manchmal fängt der Entwickler genau nach einer bestimmten Ausnahme, so dass die Ausnahme ignoriert werden kann. –

@Matt, Danke, stimme zu, aber dann haben wir eine Regel, dass der Entwickler einen Kommentar abgeben muss, um zu bestätigen, dass es mit Absicht gemacht wurde. –

@Matt - aber fangen (Ausnahme) ist fast immer schlecht. – TrueWill

Verwenden globalen Suchen-Dialog verwenden, aktivieren Sie reguläre Ausdrücke und suchen Sie nach:

catch:b*\([^)]*\):b*\{:b*\}

Quelle

2009-09-28 11:01:48 Stefan

Danke, das ist ein echter Zeitsparer –

FxCop oder ReSharper wäre für nur leerer Versuch übertrieben/fangen, gutes Denken! – jrummell

Die obige Antwort funktionierte nicht für mich, aber das "catch (\ (. * Exception. * \)) *: Wh * \ {: Wh * \}" ohne die Anführungszeichen. – UnhandledExcepSean

Haben Sie ReSharper? Dies sollte die im Code gefundenen Probleme verbessern.

Quelle

2009-09-28 10:50:26

Ich habe nicht gefunden, wo oder wie das in ReSharper ... wie würdest du es tun? – yeyeyerman

Es erscheint als eine Warnung mit ReSharper, aber ich denke, Sie müssen die Code-Datei geöffnet haben, um es zu sehen. – jrummell

Seltsamerweise habe ich neulich eines davon in einem Code gefunden.Wir benutzen sowohl FxCop als auch ReSharper; jemand hatte die FxCop-Warnung explizit unterdrückt (so dass es nicht auf dem Continuous Integration-Build angezeigt wurde) und den ReSharper ignoriert. Nicht viel kannst du dagegen tun. : P – TrueWill

FxCop wird sie zusammen mit vielen anderen möglichen Problemen finden.

Quelle

2009-09-28 10:51:23 jrummell

Können Sie die Regel benennen, die leere Catch-Blöcke abfängt? – slolife

Hier ist ein regulärer Ausdruck, die auch innerhalb von Blöcken mit nur Kommentare fangen findet:

catch:b*\([^)]*\)[:b\n]*\{([:b\n]|(\/\*[^*]*\*\/)|(//[^\n]*))*\}

Quelle

2010-10-19 14:27:56 Toto

Dank an Stefan für den Regex-Vorschlag. Ich fand, dass die vorgeschlagene regex nicht catch-Blöcke nicht findet, wo die Ausnahme nicht angegeben ist, das heißt:

catch { }

ich gezwickt Stefans leicht optional die Ausnahme Klammer zu machen:

catch:b*(\([^)]*\))*:b*\{:b*\}

Quelle

2011-04-21 15:14:32 Matt

weiter ausgebaut die drei Lösungen vorstehenden Klauseln enthalten, wo die geschweiften Klammern nicht auf der gleichen Linie wie der Fang sind und wo die Klausel Fang nur einzelne Zeile enthält zitiert:

catch:b*(\([^)]*\))*:b*[ \n\r\t]*\{:b*([ \n\r\t.]|(\/*[^]*\/)|(//.*$))*\}

Quelle

2011-06-01 13:54:46

+1 Danke und willkommen zu stackoverflow –

Drücken Sie Strg + Umschalt + F. Erweitern Suchoptionen. Aktivieren Sie Reguläre Ausdrücke Fügen Sie diesen Regex ein.

catch\s*(\(\s*Exception(\s*\w+)?\))?\s*\{\s*\}

Quelle

2015-09-01 13:13:02 bobah75

Ich fügte die Details hinzu, um zu antworten. – bobah75

Antwort

Verwandte Themen