2017-07-14 101 views
-1

得了-1格里特因爲我用下面的命令包括:​​什麼是正確的包括訂單

#include <aaa/b.h> 
#include <aaa/d.h> 
#include <aaa/f.h> 
#include <aaa/h.h> 
#include <aaa/bbb/c.h> 
#include <aaa/bbb/ccc/ddd/e.h> 
#include <aaa/bbb/ccc/ddd/eee/a.h> 
#include <aaa/bbb/ccc/ddd/eee/g.h> 

我審稿指望我用下面的字母(.h文件)命令:

#include <aaa/bbb/ccc/ddd/eee/a.h> 
#include <aaa/b.h> 
#include <aaa/bbb/c.h> 
#include <aaa/d.h> 
#include <aaa/bbb/ccc/ddd/e.h> 
#include <aaa/f.h> 
#include <aaa/bbb/ccc/ddd/eee/g.h> 
#include <aaa/h.h> 

哪訂單是否正確?我相信我的方式更好,但我得到了-1。

+1

在寫得很好的標題中沒有「正確」的順序。而且我們不能分辨這些是否寫得很好。 –

+1

沒有「正確」的順序,你應該遵循你的小組的風格。因此,這個問題是無關緊要的,因爲任何答案都是基於意見。 –

+0

我不確定這是「主要」基於意見的。任何答案的主要部分都應該集中在這樣一個事實,即可以按照任何順序包含寫得很好的標題。這是事實。 – Bathsheba

回答

2

標頭文件通常會寫入,因此您可以在使用時按照您喜歡的任何順序包含它們。 C和C++標準庫標頭符合該原則。

如果情況並非如此,那麼證明文件應該這樣說。

Alpabeticisation只是您審稿人工作方式的表達方式,我認爲這不是特別有用。

對於它的價值,我傾向於將更多的「通用」包含物放在列表的上方(例如,頂部的stdlib.hstdio.h)。

+0

我傾向於從早期的特定到通用的依賴問題,但這不是一個規則。 – stefaanv

0

我怕你的審稿人編程知識低。我的建議 - 做他想做的事(通常你的技能水平很低,就像大自我一樣),但忘記了字母順序。訂單必須合乎邏輯而不是按字母順序排列

+1

*我怕你的審稿人編程知識低*不!他正在一個團隊中工作。如果球隊有他需要遵循的風格。這不是審稿人的錯。我相信這裏涉及的各方都知道頭文件包含順序對行爲無關緊要。但請注意,問題是關於gerrit而不是編程。 –