In a multithreaded scenario, I have a method like this:
bool WaitForChange( time_duration WaitTime ) const;
This method waits either until the state of the object has changed and returns true, or until the timeout times out (how do you say that?) and returns false.
My intuition is, that const
is to protect against unwanted side-effects of the method itself, so this is fine. But then again, some user might think that the state of the could not have changed, since the method is declared const
. Is that user stupid, or should I make the method non-const
in order to avoid confusion?
By declaring the method as const, you say "Calling this method doesn't change the state of the object." This is (hopefully) true. So make it const.
If anybody thinks, const-ness means "While this method is called, no one else can change the object state" than that person is wrong.
I vote for constness.
The method itself does not change anything, just waits...
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