，但...

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中，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，也会将正确的值加载到{{1}输出大写字母符号。

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

非常感谢任何帮助。我是否以奇怪的方式访问charArray？是否有其他类作为某些类的属性的首选方法？

不寻常的输出：

ClassA

常用输出：

A
E
B
J
C
M
D
Z

?
?
?
y

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++;
}

应该做你需要的。

通过来自另一个类

1 个答案:

，但...

非主题：