Dies ist ein Ajax-Handler, der prüft, ob alle übergebenen Parameter in Ordnung sind. Wird dies als schlecht angesehen, und gibt es einen besseren Weg dies zu schreiben, oder ist es nicht so wichtig?
Angenommen, Sie verwenden die einzelnen bool -Variablen nicht anderswo, könnten Sie das als:
Vielleicht möchten Sie int.TryParse(context.Request.QueryString[name], out variable in eine separate Methode extrahieren, sodass Sie etwas wie folgt erhalten:
Alternativ könnten Sie alle diese Kontextdaten mit einer TryParse-Methode in einen neuen Typ kapseln, sodass Sie etwa Folgendes haben:
%Vor%Das ist natürlich mehr Arbeit, aber ich denke, das würde den Code sauberer machen.
Eine Sache, die Sie tun können, ist dies zu ersetzen:
%Vor%Mit diesem
%Vor% Sie könnten einen Helfer schreiben, der den hässlichen out -Übergabe-Stil von TryParse beseitigt, wie zum Beispiel:
Und dann (vorausgesetzt, Sie sind nur an der Gültigkeit interessiert):
%Vor%Wenn Sie die einzelnen Werte benötigen:
%Vor%Dies behält fast die gleichen Informationen wie zuvor bei, außer dass die feinkörnigen Informationen eine Suche und nicht einen Zugriff auf lokale Variablen benötigen.
Tags und Links c# asp.net readability