在我的C++方法gremlins

Question

我已經追溯到一個函數應該返回浮點值介於20和100左右，但有時（10的1次）返回比這更高的值。當我在方法的最後一行的表達，這樣存在的問題：

return snap(baseNumber, targets) + (octave * NOTES_PER_OCTAVE); 

如果我存儲在一個變量返回的值，然後再返回變量，問題消失：

float ret = snap(baseNumber, targets) + (octave * NOTES_PER_OCTAVE); 
return ret; 

下面是完整的方法：

static inline float octaveSnap(float number, std::vector<float>* targets){ 
    static const int NOTES_PER_OCTAVE = 12; 
    int octave = number/NOTES_PER_OCTAVE; 
    float baseNumber = number - (octave * NOTES_PER_OCTAVE); 
    float ret = snap(baseNumber, targets) + (octave * NOTES_PER_OCTAVE); 
    return ret; 
} 

和這裏的 '捕捉'：

// given a single value and a list of values (a scale), return the member of the list which is closest to the single value 
static inline float snap(float number, std::vector<float>* targets){ 
    float ret; 
    float leastDistance = -1; 
    for(int i = 0; i<targets->size(); i++){ 
     float distance = targets->at(i) - number; 
     if(distance < 0){ 
      distance = -distance; 
     } 
     if(leastDistance == -1){ 
      leastDistance = distance; 
     } 
     if(distance < leastDistance){ 
      leastDistance = distance; 
      ret = targets->at(i); 
     } 
    } 
    return ret; 
} 

我完全被這個困惑了。任何想法爲什麼第一個爆炸和第二個完美的作品？

Answer 1

snap()本地變量ret從不初始化，所以如果輸入矢量或者是零大小或者「found」元素是第一個，那麼您的返回值是未指定的。

嘗試修改快照是：

static inline float snap(float number, std::vector<float>* targets){ 
    float ret = 0; 
    float leastDistance = -1; 
    for(int i = 0; i<targets->size(); i++){ 
     float distance = targets->at(i) - number; 
     if(distance < 0){ 
      distance = -distance; 
     } 
     if(leastDistance == -1){ 
      leastDistance = distance; 
      ret = targets->at(i); 
     } 
     else if(distance < leastDistance){ 
      leastDistance = distance; 
      ret = targets->at(i); 
     } 
    } 
    return ret; 
} 

，看看是否能解決的事情。

編輯：我意識到這並不解決爲什麼添加一個臨時變量似乎修復了原來的問題。未初始化的ret將可能承擔堆棧上留下的任何值：這當然是未指定的，並且系統/平臺相關。但是，如果添加新的本地變量來存儲snap()的結果，則會移動堆棧，使ret具有不同的位置，最有可能是不同的未初始化值。返回結果仍然是「錯誤的」，但由於未初始化的值爲ret，它可能會出現「較少錯誤」。

Answer 2

我的心理調試的力量告訴我，當你使用的臨時變量的問題只出現走開，要麼你不小心做targets[<foo>]內snap或者你使用它正確，但很少流失結束，返回垃圾。

編輯點評：

我要詳細一點：targets是指針到vector所以用它[]將選擇幾個載體之一，從矢量不是元素。這就是說我無法理解你如何在這樣一個指針上調用.at，所以我懷疑你程序中的代碼不是你向我們展示的代碼。