关于字符的C++初学者问题

Question

我只是在尝试制作一个简单的井字游戏时遇到一些C++，并且遇到了一些问题。这是我的代码：

#include <iostream> 
using namespace std; 

class Square { 
public: 
    char getState() const; 
    void setState(char); 
    Square(); 
    ~Square(); 
private: 
    char * pState; 
}; 

class Board { 
public: 
    Board(); 
    ~Board(); 
    void printBoard() const; 
    Square getSquare(short x, short y) const; 
private: 
    Square board[3][3]; 
}; 

int main() { 
    Board board; 
    board.getSquare(1,2).setState('1'); 
    board.printBoard(); 
    return 0; 
} 

Square::Square() { 
    pState = new char; 
    *pState = ' '; 
} 
Square::~Square() { 
    delete pState; 
} 
char Square::getState() const { 
    return *pState; 
} 
void Square::setState(char set) { 
    *pState = set; 
} 

Board::~Board() { 

} 
Board::Board() { 

} 
void Board::printBoard() const { 
    for (int x = 0; x < 3; x++) { 
     cout << "|"; 
     for (int y = 0; y < 3; y++) { 
      cout << board[x][y].getState(); 
     } 
     cout << "|" << endl; 
    } 
} 
Square Board::getSquare(short x, short y) const { 
    return board[x][y]; 
} 

原谅我，如果它存在公然明显的问题，或者它愚蠢写的，这是我在C++中第一个程序：P，但问题是，当我尝试设置方1 ，2到字符'1'，它不会打印出来作为1，它会打印出一些我不认识的奇怪字符。

有谁能告诉我为什么？ :)

在此先感谢。

Answer 1

1. 您不需要使用new来创建变量实例。

2. 尝试将您的状态变量更改为char而不是char *（指向char）。

3. 通常，char *类型用于指示由nul字符终止的字符集合（数组）。

4. 另外，set是std命名空间中的数据类型。只需将名称更改为其他名称，例如new_value。

5. getState()方法返回副本的状态，而不是对状态的引用。这与Java和C++不同。尝试返回State&，这是C++行话一个国家实例的引用。

6. 你的程序是一个井字棋游戏有点大材小用;在古代那些使用char，而不是所谓的OO这个新的老式的东西阵列。

Answer 2

两个问题：

1）getSquare方法是const等返回const对象。

1）getSquare方法为const，因此不能返回非const的对象的非const引用。

2）从getSquare返回的对象是董事会广场的副本。

要解决：

1）从getSquare方法卸下const。

2）改变getSquare返回一个参考：Square & Board::getSquare(short x, short y)

Answer 3

Board::getSquare该方法返回Square对象的副本。被复制的Square对象的pState变量指向与原始Square对象相同的字符。当复制的对象被销毁时，它将删除pState变量指向的对象char。这会使中的Square对象中的char对象失效。当你去打印时，你正在打印一个无效的char对象。

正如其他人所说，pState变量应该可能是char而不是char*。这将使您在解决问题方面向前迈进一步。您仍然需要处理返回对象Square的引用，而不是Square对象的副本。

Answer 4

Square Board::getSquare(short x, short y) const { 
    return board[x][y]; 
} 

在这里，你returing广场实例的副本。由于没有拷贝构造函数，实例将被存储的值被复制。所以现在有2个Square实例，其状态指向相同的值。

但广场有析构函数。在析构函数中，状态指针被删除。但是剩下的副本现在拥有一个悬挂指针。

1. 方形实例A已创建。

2. 字符指针pA的被分配和设置为' '如，创建。

   +-----+ 
   | pA -----> ' ' 
   +-----+ 
   

3. 临时广场实例乙在getSquare从甲位复制。这意味着它的字符指针的pB点的同一位置pA的

   +-----+   
   | pA -----> ' ' 
   +-----+ ^
   +-----+  | 
   | pB --------' 
   +-----+ 
   

4. 广场实例乙是的setState到'1'，所以Pb含量被改变为'1'

   +-----+   
   | pA -----> '1' 
   +-----+ ^
   +-----+  | 
   | pB --------' 
   +-----+ *pB='1' 
   

5. 临时广场实例乙被破坏，因为，这是暂时的。

   +-----+   
   | pA -----> garbage 
   +-----+ ^
   + - - +  | 
   : pB --------' 
   + - - + delete pB 
   

6. 现在pA的点垃圾。

   +-----+   
   | pA -----> garbage 
   +-----+ 
   

---

你应该返回reference避免复制，

Square& Board::getSquare(short x, short y) { return board[x][y]; } 
//----^ "&" means reference. Similar to pointer, but not rebindable/nullable. 
//  Just think of it as a read-only pointer without needing a "*" 

const Square& Board::getSquare(short x, short y) const { return board[x][y]; } 
// Both mutable and const versions are needed. (Yes, code duplication.) 

和/或提供一个拷贝构造函数。

class Square { 
public: 
    ... 
    Square(const Square&); 

... 
Square::Square(const Square& other) { 
    pState = new char; 
    *pState = *other.pState; 
} 

---

你可以使用

class Square { 
public: 
    char getState() const; // { return state; } 
    void setState(char); // { state = input; } 
private: 
    char state; 
}; 

，以避免堆内存搞乱。 cout <<支持打印char。

请不要在Java中编写C++。