refactor(CLSCell): Optimizing code structure and formatting adjustment

Author: skywalker_cn

include/openmc/cell.h

@@ -424,12 +424,12 @@ class CLSCell : public CSGCell {
   // Set the translation vector for the filled universe
   void set_translation(const vector<double>& trans) { translation_ = trans; }
   // Using base class methods without modifying
+  using CSGCell::bounding_box;
+  using CSGCell::contains;
   using CSGCell::distance;
+  using CSGCell::is_simple;
   using CSGCell::surfaces;
-  using CSGCell::contains;
-  using CSGCell::bounding_box;
   using CSGCell::to_hdf5_inner;
-  using CSGCell::is_simple;
 
 private:
   Region region_;

src/cell.cpp

@@ -425,24 +425,26 @@ CLSCell::CLSCell()
 }
 
 CLSCell::CLSCell(pugi::xml_node cell_node)
-    : CSGCell(cell_node)  // Initialize base Cell properties (id, material, etc.)
-  {geom_type() = GeometryType::CLS;
-    // --- Parse CLS-Specific Parameters ---
-    auto dispersion_node = cell_node.child("dispersion");
-    
-    // Boolean flag to enable/disable CLS for this cell
-    bool is_dispersed = dispersion_node.attribute("enable").as_bool(false);
-    
-    // Particle parameters (conditional parsing)
-    if (is_dispersed) {
-      particle_radius_ = dispersion_node.attribute("particle_radius").as_double();
-      fill_ratio_ = dispersion_node.attribute("fill_ratio").as_double();
-      
-      validate_parameters();  // Ensure 0 < fill_ratio < 1 and radius > 0
-    }
+  : CSGCell(cell_node) // Initialize base Cell properties (id, material, etc.)
+{
+  geom_type() = GeometryType::CLS;
+  // --- Parse CLS-Specific Parameters ---
+  auto dispersion_node = cell_node.child("dispersion");
+
+  // Boolean flag to enable/disable CLS for this cell
+  bool is_dispersed = dispersion_node.attribute("enable").as_bool(false);
+
+  // Particle parameters (conditional parsing)
+  if (is_dispersed) {
+    particle_radius_ = dispersion_node.attribute("particle_radius").as_double();
+    fill_ratio_ = dispersion_node.attribute("fill_ratio").as_double();
+
+    validate_parameters(); // Ensure 0 < fill_ratio < 1 and radius > 0
   }
+}
 
-void CLSCell::validate_parameters() const {
+void CLSCell::validate_parameters() const
+{
   // Check that fill_ratio is between 0 and 1
   if (fill_ratio_ <= 0 || fill_ratio_ >= 1) {
     throw std::runtime_error("CLSCell: fill_ratio must be beteween (0, 1)");
@@ -452,20 +454,12 @@ void CLSCell::validate_parameters() const {
     throw std::runtime_error("CLSCell: particle_radius must be positive");
   }
   // Check that the cell is filled with a sphere universe
-  if (type_!= Fill::UNIVERSE){
-    throw std::runtime_error("CLSCell: cell must be filled with a sphere universe");
+  if (type_ != Fill::UNIVERSE) {
+    throw std::runtime_error(
+      "CLSCell: cell must be filled with a sphere universe");
   }
 }
 
-
-
-
-
-
-
-
-
-
 //==============================================================================
 // Region implementation
 //==============================================================================
@@ -1034,12 +1028,11 @@ void read_cells(pugi::xml_node node)
     bool is_dispersed = dispersion_node;
 
     if (is_dispersed) {
-      auto cell=make_unique<CLSCell>(cell_node);
+      auto cell = make_unique<CLSCell>(cell_node);
       model::cells.push_back(std::move(cell));
     } else {
       model::cells.push_back(make_unique<CSGCell>(cell_node));
     }
-
   }
 
   // Fill the cell map.