Langwierige Codezeilen vs Lesbarkeit

Question

Das ist völlig in Ordnung, C # -Code und funktioniert gut korrekte URL zur Verfügung gestellt. Aber das alles ist nur in einer Zeile erfolgt durch die Lesbarkeit des Codes zu reduzieren.

Hier ist der Code:

         return new StreamReader(WebRequest.Create(urlName).GetResponse().GetResponseStream()).ReadToEnd();

Ich frage mich, was sind die Meinungen von anderen Entwicklern auf diese Art von Abkürzung Art des Schreibens Code

Answer 1

Schieben Sie es in einem gut benannte Methode, und es vielleicht brechen, so dass einzelne statment erstreckt sich über ein paar Zeilen. Ich würde auch wahrscheinlich verwenden WebClient:

return new WebClient().DownloadString(urlName);

Answer 2

Nein, es ist nicht wirklich völlig in Ordnung, C # -Code. Sie sollten die Stream verfügen, so dass zumindest eine using Aussage haben:

using (StreamReader reader = new StreamReader(WebRequest.Create(urlName).GetResponse().GetResponseStream()) {
   return reader.ReadToEnd();
}

Dieser Code kann ein wenig Lesbarkeit gewinnen, indem sie in mehr Linien, aber nicht sehr viel geteilt wird.

Im Allgemeinen ziehe ich lesbaren Code vor kompaktem Code. eine Anweisung in jeder Zeile zu haben, macht den Code leichter zu lesen und zu verstehen. Zum Beispiel:

if (i <= 4) i = 4 - i;

Dies wird besser lesbar mit der if-Anweisung in einer Zeile und der Code innerhalb sie in einer separaten Zeile, mit der if-Anweisung immer Klammern mit:

if (i <= 4) {
   i = 4 - i;
}

Dieser Code ist natürlich ziemlich lesbar, auch in der kompakten Form, aber je komplexer der Code, in einer separaten Zeile desto mehr gewinnt aus jeder Aussage setzen.

Answer 3

... Igitt.

Ich werde manchmal ein paar Dinge in eine Zeile kombinieren, in der Regel, wenn ich Sachen in einen Stream bin Dumping, aber noch nie so viel.

Die meisten Compiler (c ++ Compiler mindestens) wird oft Inline Variablendefinitionen, wenn die Definition nur einmal verwendet wird, wenn Sie also einen einmaligen Gebrauch machen, werfen Variable entfernt. Ihr C # Compiler wird wahrscheinlich nur ersetzen seinen Namen mit seiner Definition.

Answer 4

Neben der Lesbarkeit Problem, sollten Sie jede IDisposble Objekt entsorgen Sie verwenden.

Answer 5

Eine Aussage! = Eine Zeile können Sie die Lesbarkeit verbessern, indem die Formatierung des Codes zu verbessern. Natürlich sollten Sie nicht andere Menschen Monitore verwenden hohe Auflösung übernehmen.