refactor: Add a helper to check rule match

Signed-off-by: JiangJiaWei1103 <waynechuang97@gmail.com>

Author: JiangJiaWei1103

ray-operator/controllers/ray/rayjob_controller.go

@@ -1217,13 +1217,7 @@ func (r *RayJobReconciler) handleDeletionRules(ctx context.Context, rayJob *rayv
 	// Categorize all applicable and incomplete rules into "overdue" or "pending".
 	for _, rule := range rayJob.Spec.DeletionStrategy.DeletionRules {
 		// Skip rules that don't match the current job status or job deployment status.
-		var match bool
-		if rule.Condition.JobStatus != nil {
-			match = *rule.Condition.JobStatus == rayJob.Status.JobStatus
-		} else if rule.Condition.JobDeploymentStatus != nil {
-			match = *rule.Condition.JobDeploymentStatus == rayJob.Status.JobDeploymentStatus
-		}
-		if !match {
+		if !isDeletionRuleMatched(rule, rayJob) {
 			continue
 		}
 
@@ -1388,6 +1382,17 @@ func (r *RayJobReconciler) executeDeletionPolicy(ctx context.Context, rayJob *ra
 	return ctrl.Result{}, nil
 }
 
+// isDeletionRuleMatched checks if the deletion rule matches the current job status or job deployment status.
+func isDeletionRuleMatched(rule rayv1.DeletionRule, rayJob *rayv1.RayJob) bool {
+	if rule.Condition.JobStatus != nil {
+		return *rule.Condition.JobStatus == rayJob.Status.JobStatus
+	}
+	if rule.Condition.JobDeploymentStatus != nil {
+		return *rule.Condition.JobDeploymentStatus == rayJob.Status.JobDeploymentStatus
+	}
+	return false
+}
+
 // isDeletionActionCompleted checks if the state corresponding to a deletion policy is already achieved.
 // This is crucial for making the reconciliation loop idempotent by checking the actual cluster state.
 func (r *RayJobReconciler) isDeletionActionCompleted(ctx context.Context, rayJob *rayv1.RayJob, policy rayv1.DeletionPolicyType) (bool, error) {