爲什麼要存儲一個引用而不是兩次請求我的應用程序太慢？

Question

對象memory有簽名的方法

BinaryPattern const& getPattern(unsigned int index) const; 

我在使用這個下面的for循環：

for (unsigned int k = 0; k < memory->size(); k++) { 
    const BinaryPattern s = memory->getPattern(k); 
    w += s.at(i) * s.at(j); 
} 

這是非常緩慢的。出人意料的是，我發現下面的速度要快得多：

for (unsigned int k = 0; k < memory->size(); k++) { 
    w += memory->getPattern(k).at(i) * memory->getPattern(k).at(j); 
} 

「getPattern（）」沒有做任何的計算，它非常簡單，只是返回存儲在向量模式。

爲什麼當我將參考存儲在一個變量中時速度太慢了？我最初這樣做是爲了加快速度，因爲我預計兩次檢索參考速度會更慢。

Answer 1

這是因爲

const BinaryPattern s = memory->getPattern(k); 

使對象的副本，通過調用它的拷貝構造函數。既然你不想改變它，存儲參考，而不是：

const BinaryPattern& s = memory->getPattern(k); 
//    ^
//     note this 

（由於拍攝右值與const參考直到參考死亡延長了右值的一生中，這甚至工程應的getPattern()簽名永遠改爲返回副本，而不是一個參考。）

---

在一個優化的版本（你沒有比較調試版本，是不是？），編譯器可能能夠確定的是，兩次調用功能

memory->getPattern(k).at(i) * memory->getPattern(k).at(j) 

沒有任何副作用，因此保留對第一個呼叫獲得的對象的引用，並優化掉第二個呼叫 - 到達上面提出的代碼。

顯然，複製BinaryPattern對象的代價是不可忽略的。

Answer 2

因爲你在這裏創造一個副本

const BinaryPattern s = memory->getPattern(k); 

只是改變

const BinaryPattern &s = memory->getPattern(k); 

Answer 3

在行

const BinaryPattern s = memory->getPattern(k); 

創建的BinaryPattern一個新實例，並返回值getPattern被複制到它。

你可能是指

const BinaryPattern& s = memory->getPattern(k);