I use boost::variant
a lot and am quite familiar with it. boost::variant
does not restrict the bounded types in any way, in particular, they may be references:
#include <boost/variant.hpp>
#include <cassert>
int main() {
int x = 3;
boost::variant<int&, char&> v(x); // v can hold references
boost::get<int>(v) = 4; // manipulate x through v
assert(x == 4);
}
I have a real use-case for using a variant of references as a view of some other data.
I was then surprised to find, that std::variant
does not allow references as bounded types, std::variant<int&, char&>
does not compile and it says here explicitly:
A variant is not permitted to hold references, arrays, or the type void.
I wonder why this is not allowed, I don't see a technical reason. I know that the implementations of std::variant
and boost::variant
are different, so maybe it has to do with that? Or did the authors think it is unsafe?
PS: I cannot really work around the limitation of std::variant
using std::reference_wrapper
, because the reference wrapper does not allow assignment from the base type.
#include <variant>
#include <cassert>
#include <functional>
int main() {
using int_ref = std::reference_wrapper<int>;
int x = 3;
std::variant<int_ref> v(std::ref(x)); // v can hold references
static_cast<int&>(std::get<int_ref>(v)) = 4; // manipulate x through v, extra cast needed
assert(x == 4);
}
Fundamentally, the reason that optional
and variant
don't allow reference types is that there's disagreement on what assignment (and, to a lesser extent, comparison) should do for such cases. optional
is easier than variant
to show in examples, so I'll stick with that:
int i = 4, j = 5;
std::optional<int&> o = i;
o = j; // (*)
The marked line can be interpreted to either:
o
, such that &*o == &j
. As a result of this line, the values of i
and j
themselves remain changed.o
, such &*o == &i
is still true but now i == 5
.Assign-through is the behavior you get by just pushing =
through to T
's =
, rebind is a more sound implementation and is what you really want (see also this question, as well as a Matt Calabrese talk on Reference Types).
A different way of explaining the difference between (1) and (2) is how we might implement both externally:
// rebind
o.emplace(j);
// assign through
if (o) {
*o = j;
} else {
o.emplace(j);
}
The Boost.Optional documentation provides this rationale:
Rebinding semantics for the assignment of initialized optional references has been chosen to provide consistency among initialization states even at the expense of lack of consistency with the semantics of bare C++ references. It is true that
optional<U>
strives to behave as much as possible as U does whenever it is initialized; but in the case whenU
isT&
, doing so would result in inconsistent behavior w.r.t to the lvalue initialization state.Imagine
optional<T&>
forwarding assignment to the referenced object (thus changing the referenced object value but not rebinding), and consider the following code:optional<int&> a = get(); int x = 1 ; int& rx = x ; optional<int&> b(rx); a = b ;
What does the assignment do?
If
a
is uninitialized, the answer is clear: it binds tox
(we now have another reference tox
). But what if a is already initialized? it would change the value of the referenced object (whatever that is); which is inconsistent with the other possible case.If
optional<T&>
would assign just likeT&
does, you would never be able to use Optional's assignment without explicitly handling the previous initialization state unless your code is capable of functioning whether after the assignment,a
aliases the same object asb
or not.That is, you would have to discriminate in order to be consistent.
If in your code rebinding to another object is not an option, then it is very likely that binding for the first time isn't either. In such case, assignment to an uninitialized
optional<T&>
shall be prohibited. It is quite possible that in such a scenario it is a precondition that the lvalue must be already initialized. If it isn't, then binding for the first time is OK while rebinding is not which is IMO very unlikely. In such a scenario, you can assign the value itself directly, as in:assert(!!opt); *opt=value;
Lack of agreement on what that line should do meant it was easier to just disallow references entirely, so that most of the value of optional
and variant
can at least make it for C++17 and start being useful. References could always be added later - or so the argument went.
If you love us? You can donate to us via Paypal or buy me a coffee so we can maintain and grow! Thank you!
Donate Us With