refactor(avoidance): add function to check if the object is moving (autowarefoundation#6243)

Signed-off-by: satoshi-ota <satoshi.ota928@gmail.com>

Author: satoshi-ota

planning/behavior_path_avoidance_module/src/scene.cpp

@@ -301,8 +301,7 @@ void AvoidanceModule::fillAvoidanceTargetObjects(
     data, parameters_, forward_detection_range + MARGIN, debug);
 
   for (const auto & object : object_outside_target_lane.objects) {
-    ObjectData other_object;
-    other_object.object = object;
+    ObjectData other_object = createObjectData(data, object);
     other_object.reason = "OutOfTargetArea";
     data.other_objects.push_back(other_object);
   }

planning/behavior_path_avoidance_module/src/utils.cpp

@@ -390,6 +390,14 @@ bool isVehicleTypeObject(const ObjectData & object)
   return true;
 }
 
+bool isMovingObject(
+  const ObjectData & object, const std::shared_ptr<AvoidanceParameters> & parameters)
+{
+  const auto object_type = utils::getHighestProbLabel(object.object.classification);
+  const auto object_parameter = parameters->object_parameters.at(object_type);
+  return object.move_time > object_parameter.moving_time_threshold;
+}
+
 bool isWithinCrosswalk(
   const ObjectData & object,
   const std::shared_ptr<const lanelet::routing::RoutingGraphContainer> & overall_graphs)
@@ -666,9 +674,7 @@ bool isSatisfiedWithCommonCondition(
   }
 
   // Step2. filtered stopped objects.
-  const auto object_type = utils::getHighestProbLabel(object.object.classification);
-  const auto object_parameter = parameters->object_parameters.at(object_type);
-  if (object.move_time > object_parameter.moving_time_threshold) {
+  if (filtering_utils::isMovingObject(object, parameters)) {
     object.reason = AvoidanceDebugFactor::MOVING_OBJECT;
     return false;
   }