2017-07-14 90 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而不是编程。 –