团队换到c++17也有一段时间了,对于[[nodiscard]]attribute也是更多的看resharper的提示,加了下;
总体上觉得是一个类似const,表达下设计意图,但是对于其实用性,还是无感。
最近团队代码里出现了一个问题,倒是清晰地表达了[[nodiscard]]的用途:
代码:
f32 GetUnitTime(f32 time) const
{
const float duration = GetEndTime() - GetStartTime();
if( duration > 0.0f )
{
float unitTime = (time- GetStartTime())/duration;
Math::Clamp(unitTime, 0.0f, 1.0f);//这里程序员以为clamp是一个对引用修改,其实需要v = clamp(a,b,c);
return unitTime;
}
return 0.0f;
}
那么如果把clamp的函数加上[[nodiscard]]
Math::Clamp(unitTime, 0.0f, 1.0f);//这里就会报warning
stl里面像empty也是,也会给人以错觉,做了clear的事情,那么加上nodiscard也能很好的解决这个问题;
nodiscard比较烦的就是要打的字很多,加一个code template可以很好地解决;