Как использовать Array.contains для опциональных типов?

Question

[norlin]

Добрый день!
Задача: выбрать уникальные строки из нескольких массивов, положить эти строки в новый массив, в котором первым элементом должен быть nil.

Я написал такой код:

// item.category = [String]()
var types:[String?] = [nil]
var typeNames = [String]()
        
for (item) in news! {
    for (type) in item.category {
        if (!typeNames.contains(type)) {
            types.append(type)
            typeNames.append(type)
        }
    }
}

Сильно смущает лишний массив typeNames, но .contains на массиве типа [String?] не хочет работать.
Можно ли всё это как-то оптимизировать?

Спасибо!

Answer 1

[norlin]

В итоге, отказался от запихивания nil в тот же массив, сделал всё через Set<String> + захардкоженый обработчик для nil.

Answer 2

[Александр Тихонов]

Если нужна уникальность, я бы воспользовался Set, вместо Array и старался бы избегать написания велосипедов.

var uniqueStringSet = Set<String>()
uniqueStringSet.unionInPlace(["Vasa", "Petya", "Gena"])
uniqueStringSet.unionInPlace(["Sasha", "Nikita", "Petya", "Serega", "Vasa"])

print(uniqueStringSet)

let array = Array<String>(uniqueStringSet)
var newArray = array.map{Optional($0)}
newArray.insert(nil, atIndex: 0)
print(newArray)

Comments

[norlin]

> и старался бы избегать написания велосипедов.

Спасибо, я Swift только изучаю, поэтому не знаю всех его возможностей. Про Set'ы даже не подумал.

[norlin]

Погодите-ка, но в вашем варианте всё равно 2 массива создаются. И это вдобавок к Set.

[Александр Тихонов]

Ну и что, это разве плохо ? Я полагаю, при создании array он просто увеличивает счетчик ссылок элементов в uniqueStringSet и не происходит "лишней" работы. Во втором случаем да, создается новый массив с копированием. Если у вас кол-во элементов <100, то можете не думать о таких мелочах. Я написал, как на мой взгляд кажется, читабельно и достаточно эффективно.