2017-07-14 17 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++標準ライブラリヘッダーは、その原則に準拠しています。

これが当てはまらない場合は、その裏付け文書にそのように記載する必要があります。

アルパベティシゼーションは、あなたの批評家の心が働いている方法の表現に過ぎず、私はそれが特に役立つとは思わない。

価値のあることについては、リストの上位に「generic」のインクルーシブを追加する傾向があります(たとえば、stdlib.hstdio.h)。

+0

私は以前から特定の依存関係の問題を発見する傾向がありますが、 。 – stefaanv

0

あなたの批評家のプログラミングに関する知識は低いです。私のアドバイス - 彼が何を望むか(通常はあなたのスキルは自我と同じくらい)ですが、アルファベット順を忘れてしまいます。順序は論理的でアルファベット順でなければなりません

+1

*あなたの批評家がプログラミングの知識が低いことを恐れています* NO!彼はチームで働いています。チームにスタイルがある場合は、それに従う必要があります。査読者の間違いではありません。私はここに関与している両当事者が、ヘッダーのインクルード順序がその動作に関係してはならないことを知っていると確信しています。しかし、問題はプログラミングではなく、gerritに関することです。 –

関連する問題