Lange Codezeilen vs. Lesbarkeit

Question

Dies ist völlig in Ordnung C#-Code und funktioniert gut, sofern richtige URL. Aber die alles ist nur in einer Zeile durch die Verringerung der Lesbarkeit des Codes getan.

Hier ist der Code:

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

Ich frage mich nur, welche Meinung andere Entwickler zu dieser Art von Abkürzung beim Schreiben von Code haben

Answer 1

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

return new WebClient().DownloadString(urlName);

Answer 2

Nein, es handelt sich nicht wirklich um einwandfreien C#-Code. Sie sollten den StreamReader entsorgen, also zumindest eine using Erklärung:

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

Der Code wird vielleicht etwas lesbarer, wenn er in mehr Zeilen aufgeteilt wird, aber nicht sehr viel.

Im Allgemeinen ziehe ich lesbaren Code einem kompakten Code vor. Wenn in jeder Zeile eine Anweisung steht, ist der Code leichter zu lesen und zu verstehen. Zum Beispiel:

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

Dies wird lesbarer, wenn die if-Anweisung in einer Zeile steht und der Code darin in einer separaten Zeile, wobei die if-Anweisung immer in Klammern steht:

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

Dieser Code ist natürlich auch in der kompakten Form recht gut lesbar, aber je komplexer der Code ist, desto mehr profitiert er davon, jede Anweisung in eine eigene Zeile zu setzen.

Answer 3

...IGITT.

Manchmal fasse ich ein paar Dinge in einer Zeile zusammen, normalerweise, wenn ich etwas in einen Stream leite, aber nie so viel.

Die meisten Compiler (zumindest C++ Compiler) werden oft Variablendefinitionen einbinden, wenn die Definition nur einmal verwendet wird, wenn Sie also eine einmalig verwendete Variable wegwerfen. Ihr C#-Compiler wird wahrscheinlich nur ihren Namen durch ihre Definition ersetzen.

Answer 4

Abgesehen von dem Problem der Lesbarkeit sollten Sie jedes IDisposble-Objekt, das Sie verwenden, entsorgen.

Answer 5

Eine Anweisung != eine Zeile , können Sie die Lesbarkeit verbessern, indem Sie die Formatierung Ihres Codes verbessern. Natürlich sollten Sie nicht davon ausgehen, dass andere Leute hochauflösende Monitore verwenden.