mirror of
https://github.com/bspeice/speice.io
synced 2024-11-14 05:58:09 -05:00
Add some alternate forms for const T a
This commit is contained in:
parent
5e25faf5fd
commit
b2f7575ef5
@ -219,8 +219,46 @@ int main() {
|
|||||||
}
|
}
|
||||||
```
|
```
|
||||||
|
|
||||||
May be better off defining `const T` methods in one concept, `T` methods in another, and then having
|
Alternate alternate form:
|
||||||
one concept that `requires` the sub-concepts, but just trying to demonstrate what is possible.
|
|
||||||
|
```c++
|
||||||
|
template <typename T>
|
||||||
|
concept ConstMethod = requires {
|
||||||
|
requires requires (const T a) {
|
||||||
|
{ a.const_method() } -> std::same_as<std::uint64_t>;
|
||||||
|
};
|
||||||
|
|
||||||
|
requires requires (T a) {
|
||||||
|
{ a.nonconst_method() } -> std::same_as<std::uint64_t>;
|
||||||
|
{ a.unnecessary_const_method() } -> std::same_as<std::uint64_t>;
|
||||||
|
};
|
||||||
|
};
|
||||||
|
```
|
||||||
|
|
||||||
|
Third alternate form:
|
||||||
|
|
||||||
|
```c++
|
||||||
|
template<typename T>
|
||||||
|
concept ConstMethods = requires (const T a) {
|
||||||
|
{ a.const_method() } -> std::same_as<std::uint64_t>;
|
||||||
|
};
|
||||||
|
|
||||||
|
template<typename T>
|
||||||
|
concept NonConstMethods = requires (T a) {
|
||||||
|
{ a.nonconst_method() } -> std::same_as<std::uint64_t>;
|
||||||
|
{ a.unnecessary_const_method() } -> std::same_as<std::uint64_t>;
|
||||||
|
};
|
||||||
|
|
||||||
|
|
||||||
|
template <typename T>
|
||||||
|
concept ConstMethod = requires {
|
||||||
|
requires ConstMethods<T>;
|
||||||
|
requires NonConstMethods<T>;
|
||||||
|
};
|
||||||
|
```
|
||||||
|
|
||||||
|
...which goes a long way towards explaining why the "requires requires" form is necessary. Not sure
|
||||||
|
what the "best practices" form is, just trying to demonstrate what is possible.
|
||||||
|
|
||||||
Working with `const` parameters can be a bit weird because of implicit copies:
|
Working with `const` parameters can be a bit weird because of implicit copies:
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user