Swift - Filter array

Question

I'm having a bit of trouble with filtering an array.

I have this code:

var names = [Name]()
var filteredNames = [Name]()

 func searchBar(_ searchBar: UISearchBar, textDidChange searchText: String) {
    if searchBar.text == nil || searchBar.text == ""{

        inSearchMode = false
        collectionView.reloadData()
        view.endEditing(true)
    } else {

        inSearchMode = true

        let lower = searchBar.text!.lowercased()
        print(lower)

        filteredNames = names.filter({$0.name.range(of: lower) != nil})
        collectionView.reloadData()
    }
}

The problem is that it seems not to see letters correctly. I've printed on the console the name Array, the filetredNames array and the searchBar.text here's the result:

how is possible that the "Discus" value is not included when typing the d? it happens with all letters (eg. discus return zero result and so on)

Thank you

You need to do lowercased of your name when you do the range comparison. — paulvs
– paulvs, Commented Jul 26, 2017 at 14:40

paulvs · Accepted Answer · 2017-07-26 14:42:40Z

1

You need to lowercase both the search text as well as the name property when searching for strings using .range(of:.

answered Jul 26, 2017 at 14:42

paulvs

12.1k3 gold badges44 silver badges66 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

Marco Over a year ago

can't believe it! in the last 2 days have lost my mind on stupid errors... that's not the first! :) thx a lot you've fixed it in 2 mins! :)

paulvs Over a year ago

No worries @Marco! Feel free to accept if this fixed your problem.

torinpitchers · Accepted Answer · 2017-07-26 14:42:29Z

1

The Problem

You have changed the searchText to be lowercase however your datasource (names) contains uppercase letters.

The Fix

Change:

$0.name.range(of: lower)

To:

$0.name.lowercased().range(of: lower)

answered Jul 26, 2017 at 14:42

torinpitchers

1,2907 silver badges13 bronze badges

Comments

David Pasztor · Accepted Answer · 2017-07-26 14:42:49Z

1

The problem is that you are only searching for the lowercased version of the searchbar input.

You should use localizedCaseInsensitiveContains as the filter criteria.

names.filter{$0.localizedCaseInsensitiveContains(searchBar.text!)}

This way you don't have to handle upper/lowercase separately by hand, both will be handled by the function automatically.

answered Jul 26, 2017 at 14:42

David Pasztor

55.1k9 gold badges97 silver badges130 bronze badges

2 Comments

Marco Over a year ago

thx! can't believe that in the last 2 days have lost my mind on stupid errors like this! :)

David Pasztor Over a year ago

No worries, glad I could help :)

Collectives™ on Stack Overflow

Swift - Filter array

3 Answers 3

2 Comments

Comments

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

2 Comments

Comments

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related