Moving logic to of instead of function

pkg/cluster/internal/create/create.go

@@ -86,8 +86,14 @@ func Cluster(logger log.Logger, p providers.Provider, opts *ClusterOptions) erro
 	}
 
 	// Check if the cluster name already exists
-	if err := alreadyExists(p, opts.Config.Name, opts.ForceDelete); err != nil {
-		return err
+	if err := alreadyExists(p, opts.Config.Name); err != nil {
+		if opts.ForceDelete {
+			// Delete current cluster container
+			_ = delete.Cluster(nil, p, opts.Config.Name, "")
+		} else {
+			return errors.Errorf("A cluster with the name %q already exists \n"+
+				"Please use a different cluster name or delete the current container with --force flag", opts.Config.Name)
+		}
 	}
 
 	// warn if cluster name might typically be too long
@@ -195,19 +201,14 @@ func Cluster(logger log.Logger, p providers.Provider, opts *ClusterOptions) erro
 }
 
 // alreadyExists returns an error if the cluster name already exists
-func alreadyExists(p providers.Provider, name string, forceDelete bool) error {
+// or if we had an error checking
+func alreadyExists(p providers.Provider, name string) error {
 	n, err := p.ListNodes(name)
 	if err != nil {
 		return err
 	}
-	if len(n) > 0 {
-		if forceDelete {
-			// Delete current cluster container
-			_ = delete.Cluster(nil, p, name, "")
-		} else {
-			return errors.Errorf("node(s) already exist for a cluster with the name %q \n"+
-				"Please use a different cluster name or delete the current container with --force flag", name)
-		}
+	if len(n) != 0 {
+		return errors.Errorf("node(s) already exist for a cluster with the name %q", name)
 	}
 	return nil
 }