I fix the problem in the init(), and add the documentation for the init() and term()

Author: Hsu0227

Simulator/Connections/Neuro/ConnGrowth.cpp

@@ -46,7 +46,6 @@
 #include "ParseParamError.h"
 
 #ifdef USE_HDF5
-   //#include "Hdf5GrowthRecorder.h"
    #include "Hdf5Recorder.h"
 #endif
 

Simulator/Recorders/Hdf5Recorder.cpp

@@ -31,10 +31,9 @@ Hdf5Recorder::Hdf5Recorder()
    fileLogger_ = log4cplus::Logger::getInstance(LOG4CPLUS_TEXT("file"));
 
    resultOut_ = nullptr;
-
-   init();
 }
 
+// destructor
 Hdf5Recorder::~Hdf5Recorder()
 {
    term();
@@ -86,21 +85,24 @@ void Hdf5Recorder::init()
    }
 }
 
+// This method closes the HDF5 file and releases any associated resources
 void Hdf5Recorder::term()
 {
+   // checks if the file object `resultOut_` is not null, then attempts to close the file and delete the object
    if (resultOut_ != nullptr) {
       try {
          resultOut_->close();
          delete resultOut_;
          resultOut_ = nullptr;
       } catch (FileIException &error) {
+         // If an exception occurs during this process, it prints the error stack for debugging purposes
          cerr << "HDF5 File I/O Exception during termination: ";
          error.printErrorStack();
       }
    }
 }
 
-///  Create data spaces and data sets of the hdf5 for recording histories.
+//  Create data spaces and data sets of the hdf5 for recording histories.
 void Hdf5Recorder::initDataSet()
 {
 }

Simulator/Recorders/Hdf5Recorder.h

@@ -101,7 +101,6 @@ class Hdf5Recorder : public Recorder {
    Hdf5Recorder(const string &fileName)
    {
       resultFileName_ = fileName;
-      init();
    }
 
 private:

Testing/UnitTesting/Hdf5RecorderTests.cpp

@@ -12,23 +12,25 @@ TEST(Hdf5RecorderTest, CreateInstanceSuccess)
 }
 
 // Test case for init() and term()
-TEST(Hdf5RecorderTest, Hdf5InitAndTermTest) {
-    // Create an instance of Hdf5Recorder with a specific output file name
-    std::string outputFile = "../Testing/UnitTesting/TestOutput/Hdf5test_output_term.h5";
-    Hdf5Recorder recorder(outputFile);
-    
-    // Ensure the file has been created successfully by the constructor
-    FILE *f = fopen(outputFile.c_str(), "r");
-    ASSERT_TRUE(f != NULL);
-    fclose(f);
+TEST(Hdf5RecorderTest, Hdf5InitAndTermTest)
+{
+   // Create an instance of Hdf5Recorder with a specific output file name
+   std::string outputFile = "../Testing/UnitTesting/TestOutput/Hdf5test_output_term.h5";
+   Hdf5Recorder recorder(outputFile);
+   recorder.init();
+
+   // Ensure the file has been created successfully by the constructor
+   FILE *f = fopen(outputFile.c_str(), "r");
+   ASSERT_TRUE(f != NULL);
+   fclose(f);
 
-    // Call the term() method to close the HDF5 file
-    recorder.term();
+   // Call the term() method to close the HDF5 file
+   recorder.term();
 
-    // Check if the file can be reopened, indicating it was closed properly
-    f = fopen(outputFile.c_str(), "r");
-    bool fileExist = f != NULL;
-    fclose(f);
-    ASSERT_TRUE(fileExist);
+   // Check if the file can be reopened, indicating it was closed properly
+   f = fopen(outputFile.c_str(), "r");
+   bool fileExist = f != NULL;
+   fclose(f);
+   ASSERT_TRUE(fileExist);
 }
 #endif   // HDF5