Powtórzę oczywiste rozwiązanie polegające na „konieczności zrobienia tego samemu”. To jest zwięzła wersja kodu w C ++ 11, która działa zarówno z prostymi klasami, jak i szablonami klas:
#define DECLARE_SELF(Type) \
typedef Type TySelf; /**< @brief type of this class */ \
/** checks the consistency of TySelf type (calling it has no effect) */ \
void self_check() \
{ \
static_assert(std::is_same<decltype(*((TySelf*)(0))), \
decltype(*this)>::value, "TySelf is not what it should be"); \
} \
enum { static_self_check_token = __LINE__ }; \
static_assert(int(static_self_check_token) == \
int(TySelf::static_self_check_token), \
"TySelf is not what it should be")
Możesz to zobaczyć w akcji w ideone . Geneza prowadząca do tego wyniku jest poniżej:
#define DECLARE_SELF(Type) typedef Type _TySelf; /**< @brief type of this class */
struct XYZ {
DECLARE_SELF(XYZ)
};
Ma to oczywisty problem z kopiowaniem i wklejaniem kodu do innej klasy i zapominaniem o zmianie XYZ, jak tutaj:
struct ABC {
DECLARE_SELF(XYZ) // !!
};
Moje pierwsze podejście nie było zbyt oryginalne - stworzenie funkcji takiej jak ta:
/**
* @brief namespace for checking the _TySelf type consistency
*/
namespace __self {
/**
* @brief compile-time assertion (_TySelf must be declared the same as the type of class)
*
* @tparam _TySelf is reported self type
* @tparam _TyDecltypeThis is type of <tt>*this</tt>
*/
template <class _TySelf, class _TyDecltypeThis>
class CSELF_TYPE_MUST_BE_THE_SAME_AS_CLASS_TYPE;
/**
* @brief compile-time assertion (specialization for assertion passing)
* @tparam _TySelf is reported self type (same as type of <tt>*this</tt>)
*/
template <class _TySelf>
class CSELF_TYPE_MUST_BE_THE_SAME_AS_CLASS_TYPE<_TySelf, _TySelf> {};
/**
* @brief static assertion helper type
* @tparam n_size is size of object being used as assertion message
* (if it's a incomplete type, compiler will display object name in error output)
*/
template <const size_t n_size>
class CStaticAssert {};
/**
* @brief helper function for self-check, this is used to derive type of this
* in absence of <tt>decltype()</tt> in older versions of C++
*
* @tparam _TyA is reported self type
* @tparam _TyB is type of <tt>*this</tt>
*/
template <class _TyA, class _TyB>
inline void __self_check_helper(_TyB *UNUSED(p_this))
{
typedef CStaticAssert<sizeof(CSELF_TYPE_MUST_BE_THE_SAME_AS_CLASS_TYPE<_TyA, _TyB>)> _TyAssert;
// make sure that the type reported as self and type of *this is the same
}
/**
* @def __SELF_CHECK
* @brief declares the body of __self_check() function
*/
#define __SELF_CHECK \
/** checks the consistency of _TySelf type (calling it has no effect) */ \
inline void __self_check() \
{ \
__self::__self_check_helper<_TySelf>(this); \
}
/**
* @def DECLARE_SELF
* @brief declares _TySelf type and adds code to make sure that it is indeed a correct one
* @param[in] Type is type of the enclosing class
*/
#define DECLARE_SELF(Type) \
typedef Type _TySelf; /**< @brief type of this class */ \
__SELF_CHECK
} // ~self
To trochę długie, ale proszę o wyrozumiałość. Ma to tę zaletę, że pracuje w C ++ 03 bez decltype
, ponieważ __self_check_helper
funkcja służy do wywnioskowania typu this
. Nie ma też static_assert
, ale sizeof()
zamiast tego stosuje się sztuczkę. Możesz go znacznie skrócić dla C ++ 0x. Teraz to nie zadziała w przypadku szablonów. Istnieje również niewielki problem z makrem, które nie oczekuje średnika na końcu, jeśli kompiluje się z pedantycznym, będzie narzekać na dodatkowy niepotrzebny średnik (lub pozostanie z dziwnie wyglądającym makrem nie kończącym się średnikiem w treści XYZ
i ABC
).
Sprawdzenie tego, Type
co jest przekazywane, DECLARE_SELF
nie jest opcją, ponieważ sprawdzałoby tylko XYZ
klasę (która jest w porządku), nieświadoma ABC
(która ma błąd). I wtedy do mnie dotarło. Bezkosztowe rozwiązanie do magazynowania bez dodatkowych kosztów, które współpracuje z szablonami:
namespace __self {
/**
* @brief compile-time assertion (_TySelf must be declared the same as the type of class)
* @tparam b_check is the asserted value
*/
template <bool b_check>
class CSELF_TYPE_MUST_BE_THE_SAME_AS_CLASS_TYPE2;
/**
* @brief compile-time assertion (specialization for assertion passing)
*/
template <>
class CSELF_TYPE_MUST_BE_THE_SAME_AS_CLASS_TYPE2<true> {};
/**
* @def DECLARE_SELF
* @brief declares _TySelf type and adds code to make sure that it is indeed a correct one
* @param[in] Type is type of the enclosing class
*/
#define DECLARE_SELF(Type) \
typedef Type _TySelf; /**< @brief type of this class */ \
__SELF_CHECK \
enum { __static_self_check_token = __LINE__ }; \
typedef __self::CStaticAssert<sizeof(CSELF_TYPE_MUST_BE_THE_SAME_AS_CLASS_TYPE2<int(__static_self_check_token) == int(_TySelf::__static_self_check_token)>)> __static_self_check
} // ~__self
To po prostu tworzy statyczne potwierdzenie na unikalnej wartości wyliczenia (lub przynajmniej unikalne w przypadku, gdy nie piszesz całego kodu w jednym wierszu), nie jest stosowana żadna sztuczka porównująca typy i działa jako asercja statyczna, nawet w szablonach . I jako bonus - ostatni średnik jest teraz wymagany :).
Chciałbym podziękować Yakkowi za dobrą inspirację. Nie napisałbym tego bez uprzedniego zobaczenia jego odpowiedzi.
Testowane z VS 2008 i g ++ 4.6.3. Rzeczywiście, z przykładem XYZ
i ABC
narzeka:
ipolok@ivs:~$ g++ self.cpp -c -o self.o
self.cpp:91:5: error: invalid application of âsizeofâ to incomplete type â__self::CSELF_TYPE_MUST_BE_THE_SAME_AS_CLASS_TYPE2<false>â
self.cpp:91:5: error: template argument 1 is invalid
self.cpp: In function âvoid __self::__self_check_helper(_TyB*) [with _TyA = XYZ, _TyB = ABC]â:
self.cpp:91:5: instantiated from here
self.cpp:58:87: error: invalid application of âsizeofâ to incomplete type â__self::CSELF_TYPE_MUST_BE_THE_SAME_AS_CLASS_TYPE<XYZ, ABC>â
Teraz, jeśli zrobimy ABC szablonem:
template <class X>
struct ABC {
DECLARE_SELF(XYZ); // line 92
};
int main(int argc, char **argv)
{
ABC<int> abc;
return 0;
}
Dostaniemy:
ipolok@ivs:~$ g++ self.cpp -c -o self.o
self.cpp: In instantiation of âABC<int>â:
self.cpp:97:18: instantiated from here
self.cpp:92:9: error: invalid application of âsizeofâ to incomplete type â__self::CSELF_TYPE_MUST_BE_THE_SAME_AS_CLASS_TYPE2<false>â
Wyzwolono tylko sprawdzenie numeru wiersza, ponieważ kontrola funkcji nie została skompilowana (zgodnie z oczekiwaniami).
W C ++ 0x (i bez złych znaków podkreślenia) wystarczyłoby:
namespace self_util {
/**
* @brief compile-time assertion (tokens in class and TySelf must match)
* @tparam b_check is the asserted value
*/
template <bool b_check>
class SELF_TYPE_MUST_BE_THE_SAME_AS_CLASS_TYPE;
/**
* @brief compile-time assertion (specialization for assertion passing)
*/
template <>
class SELF_TYPE_MUST_BE_THE_SAME_AS_CLASS_TYPE<true> {};
/**
* @brief static assertion helper type
* @tparam n_size is size of object being used as assertion message
* (if it's a incomplete type, compiler will display object name in error output)
*/
template <const size_t n_size>
class CStaticAssert {};
#define SELF_CHECK \
/** checks the consistency of TySelf type (calling it has no effect) */ \
void self_check() \
{ \
static_assert(std::is_same<TySelf, decltype(*this)>::value, "TySelf is not what it should be"); \
}
#define DECLARE_SELF(Type) \
typedef Type TySelf; /**< @brief type of this class */ \
SELF_CHECK \
enum { static_self_check_token = __LINE__ }; \
typedef self_util::CStaticAssert<sizeof(SELF_TYPE_MUST_BE_THE_SAME_AS_CLASS_TYPE<int(static_self_check_token) == int(TySelf::static_self_check_token)>)> static_self_check
} // ~self_util
Uważam, że bit CStaticAssert jest niestety nadal wymagany, ponieważ tworzy typ, który jest wpisany w treści szablonu (przypuszczam, że tego samego nie można zrobić static_assert
). Zaletą tego podejścia jest nadal zerowy koszt.
this_t
byłoby prawdopodobnie bardziej dopasowane do zwykłego nazewnictwa w C ++.