Longues lignes de code vs lisibilité

Question

C’est parfaitement du code C # et fonctionne bien si l’URL correcte est utilisée. Mais tout se fait en une seule ligne en réduisant la lisibilité du code.

Voici le code:

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

Je me demande simplement quels sont les avis des autres développeurs sur ce type de raccourci pour écrire du code

Answer 1

Poussez-le dans une méthode bien nommée et mettez-le en place de manière à ce qu'un seul énoncé s'étende sur quelques lignes. Je voudrais aussi probablement utiliser WebClient:

return new WebClient().DownloadString(urlName);

Answer 2

Non, le code C # n’est pas parfaitement parfait. Vous devez disposer de StreamReader afin d’avoir au moins une instruction utilisant :

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

Ce code peut gagner un peu en lisibilité en le divisant en plusieurs lignes, mais pas beaucoup.

Généralement, je préfère un code lisible avant un code compact. Le fait d'avoir une déclaration sur chaque ligne facilite la lecture et la compréhension du code. Par exemple:

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

Cela devient plus lisible avec l’instruction if sur une ligne et le code qu’elle contient sur une ligne séparée, l’instruction if ayant toujours des crochets:

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

Ce code est bien sûr assez lisible, même sous une forme compacte, mais plus le code est complexe, plus il gagne à placer chaque instruction sur une ligne séparée.

Answer 3

... beurk.

Je vais parfois combiner plusieurs éléments dans une même ligne, généralement lorsque je décharge des éléments dans un flux, mais jamais autant.

La plupart des compilateurs (au moins les compilateurs c ++) utilisent souvent des définitions de variable en ligne si la définition est utilisée une seule fois. Par conséquent, si vous utilisez une seule fois, jetez la variable. Votre compilateur C # remplacera probablement son nom par sa définition.

Answer 4

Outre le problème de lisibilité, vous devez disposer de tout objet IDisposble que vous utilisez.

Answer 5

Une déclaration! = une ligne, vous pouvez améliorer la lisibilité en améliorant le formatage de votre code. Bien entendu, vous ne devez pas supposer que d’autres personnes utilisent des moniteurs haute résolution.