在我的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，所以我怀疑你程序中的代码不是你向我们展示的代码。