class 的数组属性在通过另一个 class 的指针访问时包含 "random" 值

Array attribute of class containing "random" values when accessed via a pointer from another class

我有一个 class,它有一个字符数组作为属性。

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 在另一个 class、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 中,我正在测试 ClassAClassB 中的关联。 在 forloop 中,我按以下方式 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;
  }
}

当我在 ClassB 之外构建 ClassA 时,cout << a.charArray[x][y]; 完美运行,从 charArray.

输出各种大写字母符号

事实上,即使我将 cout << a.charArray[x][y]; 语句添加到 ClassA 构造函数并使用 ClassB 构造函数初始化 ClassA,正确的值也会被加载到 charArray,输出大写字母符号。

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

如有任何帮助,我们将不胜感激。我是否以一种奇怪的方式访问 ClassA?是否有一种首选方法可以将其他 classes 作为某些 class 的属性?

异常输出:

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

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

由于 a_class 需要更长的使用寿命,因此您必须 new ClassA(argv[1]); 并确保它是 deleted。我建议查看 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++;
}

应该做你需要的。