存储返回值，然后使用vs直接使用C++

嗨，我正在写一些代码，我发现了一个奇怪的错误。函数convert_vector2d（& i_scale）将字符串转换为vector2d（从sf :: vector2f继承）。如果您检查接下来的几行代码，我将执行两次相同的操作。存储返回值，然后使用vs直接使用C++

代码：选择所有

float x = convert_Vector2D(&i_scale).x; 
float y = convert_Vector2D(&i_scale).y; 
object.SetScale((convert_Vector2D(&i_scale))); 
ss = object.GetScale(); 
object.SetScale(x , y); 
ss = object.GetScale();

我第一次调用setScale与convert_vector2d和SS = 1,1返回向量。然后我再次用x，y（存储的结果）调用object.setScale，当我调用object.getScale时，我得到ss = 1,2（这是预期的/正确的）。

我通过了convert函数，它通过两个函数调用返回1,2。

代码：选择为什么我得到不同的行为，所有

const Vector2D Map::convert_Vector2D(std::string * string_to_convert) 
{ 
    size_t foundit = 0; 
    Vector2D temp; 
    std::string one, two; 
    if((foundit = string_to_convert->find(',')) != std::string::npos && 
     string_to_convert->find_first_of(',') == string_to_convert->find_last_of(',')) // only one comma per line. 
    { 
     one = string_to_convert->substr(0, foundit); 
     two = string_to_convert->substr(foundit+1, string_to_convert->size()); // +1 to skip over the comma. 

     temp.x = (float)strtod(one.c_str(), NULL); 
     temp.y = (float)strtod(two.c_str(), NULL); 

     check_conversion_errors_vector2d(temp, string_to_convert); 
    } 
    else 
    { 
     Debugger::print("MapLoader: Error: more then one comma on line %d of file %s. Stopping reading of file.\n", 
      i_Current_Line, mMapName.c_str()); 
     i_QuitParsing = true; // TODO: maybe add return after this line? 
    } 

    return temp; 
}

任何想法？

void Drawable::SetScale(float ScaleX, float ScaleY) 
{ 
    SetScaleX(ScaleX); 
    SetScaleY(ScaleY); 
} 

void Drawable::SetScale(const Vector2f& Scale) 
{ 
    SetScaleX(Scale.x); 
    SetScaleY(Scale.y); 
} 

void Drawable::SetScaleX(float FactorX) 
{ 
    if (FactorX > 0) 
    { 
      myScale.x  = FactorX; 
      myNeedUpdate = true; 
      myInvNeedUpdate = true; 
    } 
} 


void Drawable::SetScaleY(float FactorY) 
{ 
    if (FactorY > 0) 
    { 
      myScale.y = FactorY; 
      myNeedUpdate = true; 
      myInvNeedUpdate = true; 
    } 
}

SFML拷贝构造函数和成员变量：

// = equals operator assignment 
Vector2D& operator=(const Vector2D &rhs) 
{ 
    if(this == &rhs) 
    { 
     return *this; 
    } 
    else 
    { 
     this->x = rhs.x; 
     this->y = rhs.y; 
     return *this; 
    } 
} 
// = equals operator assignment 
Vector2D& operator=(const sf::Vector2f &rhs) 
{ 
    this->x = rhs.x; 
    this->y = rhs.y; 
    return *this; 
} 

float x, y;

来源

2010-09-23 Ben Adamson

这两个'SetScale（）'函数是什么样的？ – 2010-09-23 22:36:10

已添加到原始帖子。 – 2010-09-23 22:53:35

你能向我们展示'Vector2f'和'Vector2D'的代码（特别是成员变量和任何拷贝构造函数）吗？我怀疑你可能有两个声明'x'和'y'的成员。 – Doug 2010-09-24 03:09:11

不要在栈上分配的Vector2D，做它用新的堆。您对函数外部的temp的引用是未定义的，可能是垃圾。

来源

2010-09-23 22:46:59 user318904

我返回值，通过返回值，我认为我没有引用变量temp存储在堆栈了，因为它被返回？ – 2010-09-23 22:59:04

@Ben：你正确使用它。我怀疑@user误读你的代码。 :) – 2010-09-23 23:16:24

非常外交乔纳森，我一直在Java编码太久我的脑电线越过... – user318904 2010-09-24 21:57:15

存储返回值，然后使用vs直接使用C++

回答

相关问题