返回参照临时C++

Question

SongPart mtm::SongStructure::getPart(int index) const { 
    assert(index >= 0 && index < num_of_parts); 
    return song_parts[index]; 
} 

const SongPart& mtm::Song::operator[](int index) const { 
    assert(index >= 0 && index < song_length); 
    return (song_format->getPart(index)); 
} 

我得到的第二个函数从返回值这样的警告：

返回参考临时[默认启用]

如何解决这一问题？而且我不能改变每个函数的返回值！

Answer 1

改变所述第一函数为：

const SongPart& mtm::SongStructure::getPart(int index) const { 
    assert(index >= 0 && index < num_of_parts); 
    return song_parts[index]; 
} 

的原因是调用由返回值song_format->getPart(index)，从而创建第二函数的堆栈上的本地。如果你返回一个参考吧，第二个函数返回后，轰....

Answer 2

你得到的警告，因为getPart返回复制的song_parts[index]。如果它返回参考到song_parts[index]，那么你的代码将是正确的。

所以，你需要的getPart返回类型更改为SongPart const&：

SongPart const & mtm::SongStructure::getPart(int index) const { 
    assert(index >= 0 && index < num_of_parts); 
    return song_parts[index]; 
} 

的const是必要的，因为函数是const成员函数。

为什么在operator[]中使用assert时，当您将呼叫转发到getPart时，仍然会发出断言？只需写下：

const SongPart& mtm::Song::operator[](int index) const { 
    //assert(index >= 0 && index < song_length); not needed! 
    return (song_format->getPart(index)); 
} 

当不需要时避免额外的边界检查。

Answer 3

如果您无法更改返回类型getPart()，那么您无法有效地调用它。让我们考虑如何在不调用getPart()的情况下访问数据。

解决方法1：调用一些其他功能：

const SongPart& mtm::SongStructure::getPartReference(int index) const { 
    assert(index >= 0 && index < num_of_parts); 
    return song_parts[index]; 
} 

const SongPart& mtm::Song::operator[](int index) const { 
    assert(index >= 0 && index < song_length); 
    return (song_format->getPartReference(index)); 
} 

解决方案2：直接从operator[]返回song_parts[index]：

const SongPart& mtm::Song::operator[](int index) const { 
    assert(index >= 0 && index < song_length); 
    return (song_format->song_parts[index]); 
}