C++在函数中重用 .h 文件中的相同变量名（this.variable = 变量错误？）

发布于 2024-10-10 05:15:34 字数 395 浏览 1 评论 0原文

我的 .h 文件中有一个变量的名称，该函数具有相同的名称... 例如：

//computer.h  
class Computer{  
private:  
    string computerName;  
    int cores;  
}  

//computer.cpp  
Computer::Computer(string computerName, int cores)  
{  
    ...  
}

我想将 .cpp 函数中的值分配给 .h 文件变量。

每当我这样做时，它都不起作用。我做得正确吗？

this.computerName = computerName;  
this.cores=cores;

原文

I have a name of a variable in my .h file which a function has the same name...
for example:

//computer.h  
class Computer{  
private:  
    string computerName;  
    int cores;  
}  

//computer.cpp  
Computer::Computer(string computerName, int cores)  
{  
    ...  
}

I want to assign the values from the .cpp function to the .h file variables.

whenever i do this, it doesn't work. Am I doing it correctly??

this.computerName = computerName;  
this.cores=cores;

分享到QQ

分享到微博

如果你对这篇内容有疑问，欢迎到本站社区发帖提问参与讨论，获取更多帮助，或者扫码二维码加入 Web 技术交流群。

发布评论

需要登录才能够评论，你可以免费注册一个本站的账号。

演出会有结束 2024-10-17 05:15:34

在 C++ 中，this 是一个指针，因此它是：

this->computerName = computerName;
this->cores = cores;

In C++, this is a pointer, so it would be:

this->computerName = computerName;
this->cores = cores;

回复收藏 0 原文

陌路终见情 2024-10-17 05:15:34

使用初始化代替赋值

Computer::Computer(string computerName, int cores) 
  :computerName(computerName), cores(cores) 
{  
    ...  
}

是可行的。构造函数初始值设定项列表具有确切的目的。

Instead of assignment, use initialization

Computer::Computer(string computerName, int cores) 
  :computerName(computerName), cores(cores) 
{  
    ...  
}

Yes that works. Constructor initializer lists have that exact purpose.

回复收藏 0 原文

微暖i 2024-10-17 05:15:34

如果您发布了实际的错误消息而不是仅仅说“它不起作用”，那么很明显该问题与变量名称无关！

this 是一个指针，因此您应该使用 -> 运算符，而不是 .。

this->computerName = computerName;
this->cores = cores;

此外，建议通过引用 const 而不是通过值来接受字符串，并且您应该更喜欢初始化而不是赋值；稍后你会发现，对于某些事情（特别是常量），你实际上必须这样做：

class Computer {
private:
    std::string computerName;
    int cores;
}

Computer::Computer(const std::string& computerName, int cores)
   : computerName(computerName)
   , cores(cores)
{}

If you'd posted the actual error message rather than just saying "it doesn't work", it would become clear that the issue has nothing to do with the variable names!

this is a pointer, so you should use the -> operator, not ..

this->computerName = computerName;
this->cores = cores;

Additionally, it's advisable to accept strings by reference-to-const rather than by value, and you should really prefer initialisation over assignment; you'll find later on that for some things (notably, constants) you actually have to do this:

class Computer {
private:
    std::string computerName;
    int cores;
}

Computer::Computer(const std::string& computerName, int cores)
   : computerName(computerName)
   , cores(cores)
{}

回复收藏 0 原文