C++で意図しないコンストラクタが呼ばれるのを何とかしたいです

[こんた]

コンテナクラスのlistにループ情報も加えたClistクラスを作り、そのコンストラクタを作っているのですが

コード:

template<class _Ty, class _Alloc = allocator<_Ty> > 
class Clist {
private:
	bool isLoop = false;
	list<_Ty> myList;

public:
	Clist() { myList = list<_Ty>; }
	Clist(bool loop, size_type _Count) { isLoop = loop; myList = list<_Ty>(_Count); }
	Clist(bool loop, size_type _Count, const _Ty& _Val) { isLoop = loop; myList = list<_Ty>(_Count, _Val); }

	template<class _Iter, class = typename enable_if<_Is_iterator<_Iter>::value, void>::type>
	Clist(bool loop, _Iter _First, _Iter _Last) { isLoop = loop; myList = list<_Ty>(_First, _Last); }

	Clist(const _Myt& _Right) { isLoop = _Right.isLoop; myList = list<_Ty>(_Right.myList); }
	Clist(bool loop, _XSTD initializer_list<_Ty> _Ilist, const _Alloc& _Al = allocator_type()) { isLoop = loop; myList = list<_Ty>(_Ilist, _Al); }
}

コードのように6つのコンストラクタを作ったのですが
例えば
Clist<int> first(true, { 1, 2, 3 });
Clist<int> second(first);
Clist<int> third({ 1, 2, 3 });
のようにするとfirstは６つめのコンストラクタが呼ばれ、secondは５つめのコピーコンストラクタが呼ばれ、ここまでは意図した通り動いてくれているのですが
thirdのようにした場合なぜかコンパイルが成功し、５つめのコピーコンストラクタが呼ばれてしまいます。
thirdのようにした場合はエラーがでるようにするにはどうすればいいでしょうか。よろしくお願いします。

[nullptr]

{ 1, 2, 3 }が_Mytに暗黙変換可能なのでしょう。_Mytの型がわからないのでそれしか言えません。

[こんた]

すみません。今回のコードに関係ありそうなtypedef書いておきます。

typedef Clist<_Ty, _Alloc> _Myt;
typedef list<_Ty, _Alloc> _Mybase;
typedef typename _Mybase::size_type size_type;
typedef typename _Mybase::value_type value_type;

よろしくお願いします。

[nullptr]

おっと、返信が遅れてすいません。忘れてました。
third({1,2,3})と書いた場合、コピーコンストラクタが呼ばれます。
コンパイラはおそらくthird(Clist{1,2,3})のように推論しているからです。
なぜ{ 1, 2, 3 }がClistに変換できると判断されるかというと、おそらく3番目のコンストラクタと一致すると判断されているのでしょう。1はboolに、2はsize_typeに、3は_Tyに暗黙変換されています。
これらの回避策は複数考えられますが、根本的にClistの暗黙変換を禁止するのが手っ取り早いと思います。
explicit Clist(bool loop, size_type _Count, const _Ty& _Val)
のように宣言して試してみて下さい。他のコンストラクタも暗黙変換を禁止しても良いと思います。

[こんた]

返信ありがとうございます。
アドバイス通りexplicitをつけるとコンパイルエラーが出るようになりました。
ありがとうございました。