冗长的代码行与可读性

Question

这是非常好的C#代码,并提供正确的URL提供良好.但是,通过降低代码的可读性,一切都只在一行完成.

这是代码:

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

我只是想知道开发人员对这种编写代码的简短方式的看法

Answer 1

将它推入一个名字很好的方法,然后将其分解,以便单个标准延伸几行.我也可能使用WebClient:

return new WebClient().DownloadString(urlName);

Answer 2

不,它不是真正完美的C#代码.你应该处理StreamReader,所以至少要有一个using声明:

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

该代码可以通过将其划分为更多行来获得一点可读性,但不是很多.

通常我更喜欢紧凑代码之前的可读代码.在每一行上都有一个语句,使代码更易于阅读和理解.例如:

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

这在一行中的if语句和单独的行内的代码变得更易读,if语句总是有括号:

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

即使在紧凑的形式中,此代码当然也是可读的,但代码越复杂,将每个语句放在单独的行上就越多.