如何避免浪費屏幕空間編寫稀疏的C＃代碼？

Question

普遍接受的方法來格式化C＃代碼似乎如下：

namespace SomeNamespace 
{ 
    namespace SomeSubNamespace 
    { 
     class SomeClass 
     { 
      void SomeFunction() 
      { 
       using (var someFile = new StreamWriter(somePath)) 
       { 
        try 
        { 
         lock(someCriticalSection) 
         { 
          using (var someDisposableThing1 = new DisposableThing()) 
          { 
           DoSomething();        
           using (var someDisposableThing2 = new DisposableThing()) 
           { 
            lock(someOtherCriticalSection) 
            { 
             DoSomethingMore(); 
            } 
           } 
          } 
         } 
        } 
        catch(Exception e) 
        { 
         Log(e); 
        } 
      } 
     } 
    } 
} 

這浪費了大量的屏幕空間，水平和垂直方向。我當然不是第一個注意到的人。我的問題是：您是否和它一起生活，或者您是否開發了不同的格式樣式以避免空白空間過剩？

PS：請注意，我甚至沒有使用單個if語句！

Answer 1

我傾向於使用較小的嵌套層次編寫小類，簡短的方法。您不僅可以看到屏幕上的所有內容，而且代碼也更好！

Answer 2

我有一個24" 寬屏顯示器。我非常喜歡‘漂亮打印’在回收空間。簡明，擠在一起的代碼難以閱讀和傷害了我的大腦。

Answer 3

1. 我不經常看到命名空間以這種方式嵌套 - 也許它發生，而不是我工作的地方
2. 我生活在一起，大多不寫這種深層次的代碼。具有較短的方法或嵌套較少的方法肯定會減少水平空白問題，並且將方法分解成更小的部分意味着您可以在給定的垂直空間中獲得更重要的信息。

Answer 4

嗯，我買了一臺高清顯示器;）

不過從使用較少的縮進是將一些代碼放到自己的方法除了對付它的唯一途徑。

Answer 5

如果我要嘗試/抓住一些東西，我會結合任何相鄰的使用塊。同樣，我會集結命名空間，所以更接近於：

namespace SomeNamespace.SomeSubNamespace 
{ 
    class SomeClass 
    { 
     void SomeFunction() 
     { 
      StreamWriter someFile; 
      try 
      { 
       someFile = new StreamWriter(somePath); 

       lock(someCriticalSection) 
       { 
        using (var someDisposableThing1 = new DisposableThing()) 
        { 
         DoSomething();        
         using (var someDisposableThing2 = new DisposableThing()) 
         { 
          lock(someOtherCriticalSection) 
          { 
           DoSomethingMore(); 
          } 
         } 
        } 
       } 
      } 
      catch(Exception e) 
      { 
       Log(e); 
      } 
      finally 
      { 
       if(someFile != null) 
       { 
        someFile.Dispose(); 
       } 
      } 
     } 
    } 
} 

Answer 6

不要認爲它看起來那麼糟糕;） 使短但是功能確實是一個點。特別是，它是很好，如果你可以做的東西一樣更換

if (condition) 
{ 
    // bla 
} 
else 
{ 
    // blub 
} 

通過

void someFunction() 
{ 
    if (condition) 
    { 
    // bla 
    return 
    } 
    // blub 
} 

Answer 7

讓我來解釋一下這個樣子。如果你碰巧碰到一個嵌套的代碼結構，就像你上面展示的一樣深，那麼問題可能不是格式，而是代碼結構。

你應該考慮一下這個：Refactoring by Martin Fowler

通過extracting methods你不僅是「浪費」了屏幕空間改善，但顯着提高可讀性太;）

另一種解決方案是總是：Alt + Shift + 在Visual Studio中輸入即可進入全屏模式

Answer 8

我同意我基本生活的其他答案 - 對於我的團隊，我們要麼有高分辨率顯示器，要麼有雙顯示器（或兩者），所以它不像我們在800x600上開發的那樣。

但有一點要記住的是，如果您使用實際間距，因爲某些開發人員使用2個空格作爲製表符，大約4個，大約6個，因此您的製表符會更好。因此，格式化的代碼在一個開發人員的IDE，它可能不太適合其他人。 （我已經看到了一些可怕的結果。）

VS2010有一個很好的插件來自擴展管理器，叫做生產力工具，它可以檢測到你的文件中有多個製表符和空格混合，並允許你轉換一切到標籤或空格（「修復混合標籤」）。清理文件真的很有幫助（即使它看起來沒有任何作用！）。如果你在團隊之間共享代碼，這很好。

Answer 9

工具>選項>文本編輯器>所有語言>標籤>縮進大小> 1
不是一個很好的解決方案...：P

Answer 10

使用CodeRush， 它會通過你的代碼更好的使導航

你會得到這樣的事情

Answer 11

我已經看到了一個代碼塊的開始行開括號，如：

namespace SomeNamespace { 
    class SomeClass { 
     void SomeFunction() { 
      using (var someFile = new StreamWriter(somePath)) { 
       try { 
        lock(someCriticalSection) { 
         using (var someDisposableThing1 = new DisposableThing()) { 
          DoSomething();        
         } 
        } 
       } catch(Exception e) { 
        Log(e); 
       } 
      } 
     } 
    } 
} 

房地產不多，但我覺得它很醜。

Answer 12

您應該閱讀Bob Martin的Clean Code。它有助於解決這個問題。在這種情況下，SomeFunction()做了不止一件事。把它在自己的方法中做的每件事分開。

​​

另外，不要抓住System.Exception;你永遠不知道什麼會失敗，你不應該試圖捕捉你無法處理的異常。讓他們起死回生並死得可怕。

Answer 13

除了別人在這裏所說的，我做的一件事是減少效果，就是使用比例間隔字體。文字更具可讀性，但差別不大。它也迫使你使用標籤在空間對齊（這當然是爲什麼好主給我們標籤首先）

我個人使用漫畫無傷害MS，但這真的驅動同事瘋狂....

Answer 14

using語句是非常方便的，但是如果代碼變得難於閱讀，你也離不開它：

namespace SomeNamespace.SomeSubNamespace 
{ 
    class SomeClass 
    { 
     void SomeFunction() 
     { 
      StreamWriter someFile = new StreamWriter(somePath); 
      DisposableThing someDisposableThing1 = null; 
      DisposableThing someDisposableThing2 = null; 

      try 
      { 
       lock (someCriticalSection) 
       { 
        someDisposableThing1 = new DisposableThing(); 
        DoSomething(); 
        someDisposableThing2 = new DisposableThing(); 
        lock (someOtherCriticalSection) 
        { 
         DoSomethingMore(); 
        } 
       } 
      } 
      catch (Exception e) 
      { 
       Log(e); 
      } 
      finally 
      { 
       // requires SafeDispose extension method 
       someFile.SafeDispose(); 
       someDisposableThing1.SafeDispose(); 
       someDisposableThing2.SafeDispose(); 
      } 
     } 
    } 
} 

Answer 15

一些總體思路：

• 結合的命名空間塊爲一體。如果可以的話，
• 收集using聲明到一個地方。
• 如果你不是一個狂熱者「即使單個語句需要附上{} s！」，您可以擺脫using和try..catch塊之間的括號。
• 想想是否真的需要嵌套lock，或者最外層的lock就足夠了。

這些可以結果在下面的代碼（留意，你假設的例子代碼的含義可能不完全保留）：

namespace SomeNamespace.SomeSubNamespace 
{ 
    class SomeClass 
    { 
     void SomeFunction() 
     { 
      using (var someFile = new StreamWriter(somePath)) 
      try 
      { 
       lock(someCriticalSection) 
       { 
        using (var someDisposableThing1 = new DisposableThing()) 
        using (var someDisposableThing2 = new DisposableThing()) 
        { 
         DoSomething();        
         DoSomethingMore(); 
        } 
       } 
      } 
      catch(Exception e) 
      { 
       Log(e); 
      } 
     } 
    } 
} 

（你甚至可以移動兩個using s到如果您的代碼允許，您可以進一步替換using語句，並在finally塊中顯式調用Dispose，如其他答案中的建議。）