我正在和同事一起处理一个旧的大型代码库。该代码库使用了大量的
如何使这段代码更易读?继续使用对
std::shared_ptr
,而先前的开发人员喜欢使用较长的属性名称(例如m_first_username
)。我们的一些方法访问了许多这些属性,因此我们的代码往往非常冗长:
if (m_first_username->isSomethingOrOther() || m_second_username->isOtherOrSomething()...
为了使代码更易读,我的同事想要在本地范围内使用更多的std::shared_ptr &
:
const std::shared_ptr<...> &tmp = m_first_username->returnsASharedPtr()
tmp->isSomethingOrOther();
我不同意这种做法,因为它会增加共享指针的使用计数。如何使这段代码更易读?继续使用对
shared_ptr
的常量引用,使用std::weak_ptr
还是接受冗长的代码行?
tmp->isSomethingOrOther()
比m_first_username->isSomethingOrOther()
更易读?你能解释一下这个逻辑吗? - nwpm_first_username
看起来是一个合理的名称。引入一个名为tmp
的奇怪临时共享指针引用并不能提高可读性。 - user7860670auto &new_name = old_name;
。预期该操作将被完全优化掉。 - nwp