規則優先==超過=

Question

我只是想知道這將是更好地做到這一點：

if((fd = open(filename, O_RDWR)) == -1) 
{ 
fprintf(stderr, "open [ %s ]\n", strerror(errno)); 

return 1; 
} 

或本

fd = open(filename, O_RDWR); 
if(fd == -1) 
{ 
    fprintf(stderr, "open [ %s ]\n", strerror(errno)); 

    return 1; 
} 

非常感謝您的任何建議，

Answer 1

Yuck，把它分開。通過將所有內容混合在一行中獲得什麼？讓我們來比較和對比：

單行：

• 優點：
• 缺點：難以閱讀，容易出錯。 （考慮你的第一次修訂。）

多行：

• 優點：易於閱讀，容易出錯少。
• 缺點：

我認爲這是明顯的。 :)

---

「有時候把它放在一個行更有意義，例如：while ((c=getchar())!=EOF)」

這很好，但這裏是不是這樣的。有分別是時不分解更合理，但一般，不要。

---

，當一行被殺死你看到的功能的能力「這樣可以節省更多的垂直空間」

，你需要1）買與分辨率的顯示器比640×480高，2）寫小功能。

真的，我已經從未理解爲任何參數，函數應該容易適合在任何屏幕上，無論一行差異。

---

「多行使它看起來複雜」

不是真的，順手就在同一行無疑是難以閱讀和更復雜的期待。將事物拆分可以簡化一次處理一個數據的過程，但不應該假設兩行使其複雜一倍。

Answer 2

它是一種風格的東西 - 你不是問優先級（不是主持人）。

很多人會認爲後者的例子更清晰。

Answer 3

這是一個風格問題，是主觀的。他們做同樣的事情。我傾向於更喜歡後者，因爲我發現它更易於閱讀，並且更容易在調試器中設置斷點/檢查變量。

Answer 4

也許在括號使排序明顯嗎？

if((fd = open(filename, O_RDWR)) == -1) 

Answer 5

除了標準的成語 - 那些是很常見的，每個人都立即得到你正在嘗試做的事情 - 我會避免做在有條件的分配。首先，閱讀起來比較困難。其次，在條件檢查中，通過使用錯誤的賦值運算符來創建錯誤（至少在弱類型語言中將零解釋爲false和非零）。

Answer 6

第二個爲了可讀性更好，但我知道我經常做第一個。優先考慮=運營商，尤其是因爲您將其用引號括起來，從而允許指定的值返回&，與==運營商進行比較。

Answer 7

(-1 == __whatever__) 

減少錯字

Answer 8

有幾個人贊成第二種的主張。我不同意他們。雖然（顯然）最初有一個小問題=與==在第一個，我認爲這是一個小問題。

一個更大的問題是，人們（尤其是如果他們急於）忽略錯誤檢查 - 完全忽略if (whatever == -1)，通常根據他們正在工作的理論上快速，一次性代碼和檢查錯誤是不是真的需要。這真是壞習慣;我幾乎可以保證每個讀過這個的人都看到真的這樣的代碼跳過錯誤檢查，即使它確實應該有它。

在這樣的代碼中，試圖打開文件並檢查是否有錯誤，因此應該與不可約束地綁定在一起。將兩者置於同一聲明中反映了正確的意圖。將兩者分開顯然是錯誤的 - 他們應該以任何方式在任何時間出於任何原因以任何方式分離而不是。這應該被編碼爲單個操作，因爲它應該是是的單個操作。它應該始終被認爲和編碼爲一個單一的操作。

在我看來，這樣做的藉口是相當薄弱的。實際情況是，任何使用C 的人都需要才能閱讀將條件測試與條件測試相結合的代碼。只是一個明顯的例子，像while ((ch=getchar()) != EOF)幾乎需求循環寫入作爲組合的分配和測試 - 在嘗試測試EOF分別通常會導致代碼，只是不能正常工作，如果你這樣做使它工作正常，代碼要複雜得多。

同樣，-與==的問題。由於我沒有看到缺陷開始，所以我不確定這兩個人爲了避免問題而做了多少分離，但我的直接猜測是，它可能幾乎沒有任何區別。編譯器會提醒你什麼時候應該是一個條件只包含一個任務已經存在多年（如gcc）。在大多數情況下，無論如何，症狀幾乎立即顯而易見 - 簡而言之，事實上，您在本帖的某一部分中犯了特定錯別字，但沒有證明其中的任何部分（甚至無法表明）兩者的困難。基於這種證據，我顯然認爲「不」比「立即」更難打字，因爲我只是在沒有問題的情況下鍵入「立即」，但必須糾正「不」（兩次以上），沒有更少），然後纔出現在前面的句子中。我很確定，如果我們錯誤地多次輸入它，「這個」是英語中最難的單詞。

Answer 9

在這個例子中，我會加入合唱團，說第二種方法更好。

更艱鉅的情況是，當它在一個循環中，如：

while ((c=getchar())!=-1) 
{ 
    ... do something ... 
} 

與

while (true) 
{ 
    c=getchar(); 
    if (c==-1) 
    break; 
    ... do something ... 
} 

在這樣的情況下，我寧願做在同一行，因爲那時它明確是什麼控制循環，我認爲這會覆蓋賦值和測試複雜組合的缺點。

Answer 10

第一種情況是非常正常在編寫輸入循環時是正常的，因爲替代方法是必須將輸入命令寫兩次 - 一次在循環之前，一次在循環結尾。

while ((ch=getchar()) != -1){ 
    //do something with it 
} 

我認爲第二種方法對於if語句更爲正常，在這種情況下，您沒有同樣的擔憂。