Hello everyone!

I'm working on a app and trying to write code on the best way.

I have a View with 3 textfields and I want to join the three strings when an user press one button.

I create this func but I think there a more efficient way to do this.

Thanks!

func groupStrings(str1 : String, str2: String, str3: String) -> String {
        var arrStr : [String] = []
        if str1 != ""{
            arrStr.append(str1)
        }
        if str2 != "" {
            arrStr.append(str2)
        }
        if str3 != "" {
            arrStr.append(str3)
        }
        return arrStr.joined(separator: ", ")
    }

I have slightly different way. With this you do not have to put the String into an array in call.

func groupStrings(_ string : String...) -> String {
    string.filter { !$0.isEmpty }.joined(separator: " ")
}

let joinString = groupStrings(textInField1, textInField2, textInField3, textInField4)

print(joinString)

However if really want to make reusable then

func groupStrings(_ string : String..., separator: String = " ") -> String {
    string.filter { !$0.isEmpty }.joined(separator: separator)
}

let fullName = groupStrings(firstName, lastName)
let fullAddress = groupStrings(street, city, zip, separator: "\n")

print(fullName)
print(fullAddress)

David: This shows you the truth about Swift. There are many ways to accomplish a simple task!

I think @delaware's approach with filter is clever, but as he pointed out, it requires you to first put your strings into an array. This doesn't seem too objectionable, until @nigel proposed his response!

@Nigel gets my vote for the [Solved] tag. I only wish @nigel would have taken a paragraph or two to explain the String... notation in the function's parameter list.

Check this How to make a variadic function. Also new in Swift 5.4 Multiple variadic parameters in functions