Если бы вы опубликовали реальное сообщение об ошибке, а не просто сказали «оно не работает», станет ясно, что проблема не имеет никакого отношения к именам переменных!
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)
{}