[已解决]关于类中指针初始化的问题

闪亮的马路

以下是一个大概的程序结构，我运行的时候一直报段错误，但是把初始化指针放在函数内部就没有问题，不知道是哪里的问题？


// point_cloud_yz.h文件
class PointCloudYZ
{
private:

public:
  pcl::PointCloud<pcl::PointXYZ>::Ptr cloud;   //Ptr是pcl定义的一个类型，会返回一个smart point



  PointCloudYZ();


  ~PointCloudYZ();


  int loadPCD();

};

#endif
#include "point_cloud_yz.h"

//point_cloud_yz.cpp文件
PointCloudYZ::PointCloudYZ()
{

  cloud = pcl::PointCloud<pcl::PointXYZ>::Ptr(new pcl::PointCloud<pcl::PointXYZ>);

}

int loadPCD()
{
  pcl::PointCloud<pcl::PointXYZ>::Ptr cloud_test (new pcl::PointCloud<pcl::PointXYZ>);
  std::cout << *cloud_test  << std::endl;   //这里是可以打印信息的
  std::cout << *cloud  << std::endl;  //这里只是测试，因为程序运行时，这里有问题，会出现段错误，应该是野指针，推断是构造函数初始化有问题
}

//
PointCloudYZ::~PointCloudYZ()
{
}


//main.cpp
#include "point_cloud_yz.h"
int main(int argc, char **argv)
{
  PointCloudYZ *test;
  test->loadPCD();
}

最佳答案

月排行榜 / 总排行榜

sfqxx

2023-4-12 19:20:53

在代码中，loadPCD函数定义是在类外部实现的，它没有指定所属的类，因此编译器将其视为全局函数。由于loadPCD函数没有访问类成员变量，因此在该函数中没有定义类指针this，故访问到的cloud指针是未初始化的野指针。

解决方法是在loadPCD函数前添加“PointCloudYZ::”以说明loadPCD函数所属的类是PointCloudYZ。修改后代码如下：

1. 
   
2. // point_cloud_yz.h文件
   
3. class PointCloudYZ
   
4. {
   
5. private:
   
6. 
   
7. public:
   
8.   pcl::PointCloud<pcl::PointXYZ>::Ptr cloud;   //Ptr是pcl定义的一个类型，会返回一个smart point
   
9. 
   
10. 
    
11. 
    
12.   PointCloudYZ();
    
13. 
    
14. 
    
15.   ~PointCloudYZ();
    
16. 
    
17. 
    
18.   int loadPCD();
    
19. 
    
20. };
    
21. 
    
22. #include "point_cloud_yz.h"
    
23. 
    
24. //point_cloud_yz.cpp文件
    
25. PointCloudYZ::PointCloudYZ()
    
26. {
    
27. 
    
28.   cloud = pcl::PointCloud<pcl::PointXYZ>::Ptr(new pcl::PointCloud<pcl::PointXYZ>);
    
29. 
    
30. }
    
31. 
    
32. int PointCloudYZ::loadPCD()  // 加上 PointCloudYZ:: 表示 loadPCD 函数属于 PointCloudYZ 类
    
33. {
    
34.   pcl::PointCloud<pcl::PointXYZ>::Ptr cloud_test (new pcl::PointCloud<pcl::PointXYZ>);
    
35.   std::cout << *cloud_test  << std::endl;   //这里是可以打印信息的
    
36.   std::cout << *cloud  << std::endl;  //这里只是测试，因为程序运行时，这里有问题，会出现段错误，应该是野指针，推断是构造函数初始化有问题
    
37. }
    
38. 
    
39. PointCloudYZ::~PointCloudYZ()
    
40. {
    
41. }
    
42. 
    
43. //main.cpp
    
44. #include "point_cloud_yz.h"
    
45. int main(int argc, char **argv)
    
46. {
    
47.   PointCloudYZ *test = new PointCloudYZ(); // 需要创建一个 PointCloudYZ 对象才能调用它的函数
    
48.   test->loadPCD();
    
49.   delete test; // 在程序结束前需要释放指针
    
50. }

复制代码

有用请设置最佳答案

跳转到最佳答案楼层

闪亮的马路

PointCloudYZ *test; 修改为PointCloudYZ *test = new  PointCloudYZ；是可行的，谁能解释一下？

永恒的蓝色梦想

闪亮的马路 发表于 2020-12-11 16:57
PointCloudYZ *test; 修改为PointCloudYZ *test = new  PointCloudYZ；是可行的，谁能解释一下？

构造函数什么问题没有，但是你声明了指针不分配空间就是大问题了

sfqxx

在代码中，loadPCD函数定义是在类外部实现的，它没有指定所属的类，因此编译器将其视为全局函数。由于loadPCD函数没有访问类成员变量，因此在该函数中没有定义类指针this，故访问到的cloud指针是未初始化的野指针。

解决方法是在loadPCD函数前添加“PointCloudYZ::”以说明loadPCD函数所属的类是PointCloudYZ。修改后代码如下：

1. 
   
2. // point_cloud_yz.h文件
   
3. class PointCloudYZ
   
4. {
   
5. private:
   
6. 
   
7. public:
   
8.   pcl::PointCloud<pcl::PointXYZ>::Ptr cloud;   //Ptr是pcl定义的一个类型，会返回一个smart point
   
9. 
   
10. 
    
11. 
    
12.   PointCloudYZ();
    
13. 
    
14. 
    
15.   ~PointCloudYZ();
    
16. 
    
17. 
    
18.   int loadPCD();
    
19. 
    
20. };
    
21. 
    
22. #include "point_cloud_yz.h"
    
23. 
    
24. //point_cloud_yz.cpp文件
    
25. PointCloudYZ::PointCloudYZ()
    
26. {
    
27. 
    
28.   cloud = pcl::PointCloud<pcl::PointXYZ>::Ptr(new pcl::PointCloud<pcl::PointXYZ>);
    
29. 
    
30. }
    
31. 
    
32. int PointCloudYZ::loadPCD()  // 加上 PointCloudYZ:: 表示 loadPCD 函数属于 PointCloudYZ 类
    
33. {
    
34.   pcl::PointCloud<pcl::PointXYZ>::Ptr cloud_test (new pcl::PointCloud<pcl::PointXYZ>);
    
35.   std::cout << *cloud_test  << std::endl;   //这里是可以打印信息的
    
36.   std::cout << *cloud  << std::endl;  //这里只是测试，因为程序运行时，这里有问题，会出现段错误，应该是野指针，推断是构造函数初始化有问题
    
37. }
    
38. 
    
39. PointCloudYZ::~PointCloudYZ()
    
40. {
    
41. }
    
42. 
    
43. //main.cpp
    
44. #include "point_cloud_yz.h"
    
45. int main(int argc, char **argv)
    
46. {
    
47.   PointCloudYZ *test = new PointCloudYZ(); // 需要创建一个 PointCloudYZ 对象才能调用它的函数
    
48.   test->loadPCD();
    
49.   delete test; // 在程序结束前需要释放指针
    
50. }

复制代码

有用请设置最佳答案