c++ - 通过来自另一个类的指针访问时包含 "random"值的类的数组属性

Question

我有一个类，它有一个字符数组作为属性。

class ClassA{
public:

  ClassA( char *fileName );

  char charArray[13][2];
};

ClassA构造函数从文件中读取 26 个值，并加载它们进入charArray如下。

ClassA::ClassA( fileName ){
   ifstream file;
   file.open( fileName );
   int contactOne, contactTwo;
   int pair=0;

   while( !file.eof() ){
     file >> contactOne;
     file >> contactTwo;
     if( !file.eof() ){
       charArray[pair][0] = (contactOne+65); // convert to Alpha UpperCase
       charArray[pair][1] = (contactTwo+65); // conv to Alpha UpperCase
       pair++;
     }
   }
}

ClassA在另一个类的构造函数中初始化，ClassB其中 ClassB包含指向 ClassA 的指针对象，允许 ClassA存储为 ClassB 的属性.

class ClassA; // forward derive
class ClassB{
  public:
  ClassB( char **argv );

  ClassA *a_class
};

ClassB 的构造函数:

ClassB::ClassB( argv ){
  ClassA a( argv[1] );
  a_class = &a;
}

在main ，我正在测试 ClassA 的关联在 ClassB .在 forloop I cout charArray 中的所有值通过以下方式。

int main( int argc, char **argv ){
  ClassB b_class( argv );

  for( int i=0; i < 13; i++ ){
     cout << b_class.a_class->charArray[i][0] << endl;
     cout << b_class.a_class->charArray[i][1] << endl;
  }
}

当我构建 ClassA 时在ClassB之外, cout << a.charArray[x][y];工作完美，从 charArray 输出各种大写字母符号.

事实上，即使我添加了 cout << a.charArray[x][y];对 ClassA 的声明构造函数和初始化 ClassA使用 ClassB构造函数，正确的值被加载到 charArray , 输出大写字母符号。

但是，当我执行 main 中的内容时，charArray 的输出很奇怪。数组的前几个值符合预期，但最后的值有些随机，通常随机放置空白值和问号。就好像指针以某种方式导致将值加载到 charArray 中一样。超出正确的范围，或者我以某种方式访问​​了错误的内存位置。

任何帮助将不胜感激。我正在访问 ClassA以一种奇怪的方式？是否有一种首选方法可以将其他类作为某个类的属性？

异常输出:

A
E
B
J
C
M
D
Z

?
?
?
y

常规输出:

A
E
B
J
C
M
D
Z
F
L
G
Y
H
X
I
V
K
W
N
R
O
Q
P
U
S
T

Answer 1

ClassA a( argv[1] ); 将 a 定义为将被销毁的局部变量，并且其内存在退出 ClassB< 时变为无效 构造函数，让 a_class 指向垃圾。

由于a_class 需要更长的生命周期，因此您必须new ClassA(argv[1]); 并确保它是delete d.我建议查看 std::unique_ptr .

但是...

为什么首先要这样做？相反，由于 ClassA 不依赖于 ClassB，因此可以正式定义 ClassA 或者在 ClassB 之前包含其头文件定义。然后您可以删除前向定义并:

class ClassB{
  public:
  ClassB( char **argv );

  ClassA a_class
};

ClassB::ClassB( argv ):a_class(argv[1]){
}

让整个指针困惑消失。

题外话:

while( !file.eof() ) 及其同类代码几乎不会按照您期望的方式工作。在这里阅读更多:Why is iostream::eof inside a loop condition considered wrong?

while( file >> contactOne >> contactTwo ){
    charArray[pair][0] = (contactOne+65); // convert to Alpha UpperCase
    charArray[pair][1] = (contactTwo+65); // conv to Alpha UpperCase
    pair++;
}

应该做你需要的。